Re: [linux-pm] [patch] pm: fix runtime powermanagement's /sys interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 06, 2006 at 09:59:21AM +0100, Pavel Machek wrote:
> On ??t 05-01-06 17:37:30, Patrick Mochel wrote:
> > +static ssize_t pm_possible_states_show(struct device * d,
> > +				       struct device_attribute * a,
> > +				       char * buffer)
> > +{
> > +	struct pci_dev * dev = to_pci_dev(d);
> > +	char * s = buffer;
> > +
> > +	s += sprintf(s, "d0");
> > +	if (dev->poss_states[PCI_D1])
> > +		s += sprintf(s, " d1");
> > +	if (dev->poss_states[PCI_D2])
> > +		s += sprintf(s, " d2");
> > +	if (dev->poss_states[PCI_D3hot])
> > +		s += sprintf(s, " d3");
> 
> Could we use same states as rest of code, i.e. "D2" instead of "d2"
> and "D3hot" instead of "d3"?

"D3hot" and "D3cold" most likely going to be removed and replaced with just
"D3" in my next set of changes.  Although this distinction is mentioned in
the spec, d3cold cannot actually be specified.  Rather, D3cold is entered when
the parent bridge is powered off.

Thanks,
Adam

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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux