lkml.org 
[lkml]   [2005]   [Nov]   [16]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC][PATCH 2/6] PCI PM: capability probing and setup
FromAdam Belay <>
DateWed, 16 Nov 2005 02:21:01 -0500
On Tue, 2005-11-15 at 22:21 -0800, Greg KH wrote:
> On Tue, Nov 15, 2005 at 10:31:17PM -0500, Adam Belay wrote:
> > +int pci_setup_device_pm(struct pci_dev *dev)
> 
> Care to give kernel doc for this new function?

Absolutely.  I was planing to do this but must have forgotten.

> > +	unsigned char	state_mask;	/* a mask of supported power states */
> > +	unsigned char	pme_mask;	/* a mask of power states that allow #PME */ 
> 
> Trailing space, use quilt it strips this :)

Sorry about that :)

> 
> > +	struct pci_dev_pm *pm;		/* power management information */
> 
> Why make this a pointer and not just part of this structure?  Don't all
> pci devices need this?

Actually, not every PCI device supports the PCI PM spec.  There are many
devices, even in modern systems, that can only be in D0.  I was thinking
we could save some memory and allocate this structure when PCI PM is
detected.  Would that be ok?

Thanks,
Adam


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-11-16 08:14    [from the cache]
©2003-2008