linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / 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, other threads:[~2020-10-15 12:28 UTC | newest]

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).