Linux-ACPI Archive on lore.kernel.org
 help / color / Atom feed
* Re: DPTF_POWER and DPTF_PCH_FIVR
       [not found] ` <20201015095851.GB11838@zn.tnic>
@ 2020-10-15 11:31   ` Rafael J. Wysocki
  2020-10-15 12:06     ` Borislav Petkov
  0 siblings, 1 reply; 3+ messages in thread
From: Rafael J. Wysocki @ 2020-10-15 11:31 UTC (permalink / raw)
  To: Borislav Petkov, Srinivas Pandruvada; +Cc: lkml, rafael, Linux ACPI

On 10/15/2020 11:58 AM, Borislav Petkov wrote:
> On Thu, Oct 15, 2020 at 11:53:11AM +0200, Borislav Petkov wrote:
>> Dudes,
>>
>> what are those new symbols in Kconfig and why do they wanna get enabled
>> on my box when doing oldconfig? Depends on X86? Really, that widespread?

Well, they depend on ACPI too and it doesn't look like there is anything 
more fine grained that could be used here.

Also I'm not sure if adding ACPI_DPTF (bool, disabled by default and 
depending on x86) would help a lot.


>> The help text doesn't say why do I need this...
> And that thing in sysfs:
>
> "/sys/bus/platform/drivers/DPTF Platform Power"
>
> has spaces in the name. What's going on?!
>
Obviously an oversight on my part, sorry.

I think that changing it to "dptf_power" (and analogously for the FIVR 
participant driver) should work.  Srinivas?



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: DPTF_POWER and DPTF_PCH_FIVR
  2020-10-15 11:31   ` DPTF_POWER and DPTF_PCH_FIVR Rafael J. Wysocki
@ 2020-10-15 12:06     ` Borislav Petkov
  2020-10-15 12:28       ` Rafael J. Wysocki
  0 siblings, 1 reply; 3+ messages in thread
From: Borislav Petkov @ 2020-10-15 12:06 UTC (permalink / raw)
  To: Rafael J. Wysocki; +Cc: Srinivas Pandruvada, lkml, rafael, Linux ACPI

On Thu, Oct 15, 2020 at 01:31:55PM +0200, Rafael J. Wysocki wrote:
> Well, they depend on ACPI too and it doesn't look like there is anything
> more fine grained that could be used here.
> 
> Also I'm not sure if adding ACPI_DPTF (bool, disabled by default and
> depending on x86) would help a lot.

So you're saying this is going to be present in current and future
hardware so might as well enable it?

> Obviously an oversight on my part, sorry.
> 
> I think that changing it to "dptf_power" (and analogously for the FIVR
> participant driver) should work.  Srinivas?

Makes sense.

Thx.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: DPTF_POWER and DPTF_PCH_FIVR
  2020-10-15 12:06     ` Borislav Petkov
@ 2020-10-15 12:28       ` Rafael J. Wysocki
  0 siblings, 0 replies; 3+ messages in thread
From: Rafael J. Wysocki @ 2020-10-15 12:28 UTC (permalink / raw)
  To: Borislav Petkov
  Cc: Rafael J. Wysocki, Srinivas Pandruvada, lkml, Rafael J. Wysocki,
	Linux ACPI

On Thu, Oct 15, 2020 at 2:06 PM Borislav Petkov <bp@alien8.de> wrote:
>
> On Thu, Oct 15, 2020 at 01:31:55PM +0200, Rafael J. Wysocki wrote:
> > Well, they depend on ACPI too and it doesn't look like there is anything
> > more fine grained that could be used here.
> >
> > Also I'm not sure if adding ACPI_DPTF (bool, disabled by default and
> > depending on x86) would help a lot.
>
> So you're saying this is going to be present in current and future
> hardware so might as well enable it?

Yes.  [Only on clients AFAICS, but still.]

In the meantime I have concluded that having a submenu for it would
make sense, though.  At least from the clarity POV.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20201015095311.GA11838@zn.tnic>
     [not found] ` <20201015095851.GB11838@zn.tnic>
2020-10-15 11:31   ` DPTF_POWER and DPTF_PCH_FIVR Rafael J. Wysocki
2020-10-15 12:06     ` Borislav Petkov
2020-10-15 12:28       ` Rafael J. Wysocki

Linux-ACPI Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-acpi/0 linux-acpi/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-acpi linux-acpi/ https://lore.kernel.org/linux-acpi \
		linux-acpi@vger.kernel.org
	public-inbox-index linux-acpi

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-acpi


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git