mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Out-of-tree devfreq governor header
@ 2023-01-09 18:30 John Moon
  2023-01-09 18:42 ` John Moon
  0 siblings, 1 reply; 2+ messages in thread
From: John Moon @ 2023-01-09 18:30 UTC (permalink / raw)
  To: myungjoo.ham, kyungmin.park, cw00.choi
  Cc: linux-kernel, linux-arm-msml, quic_ccooper, mkrom, quic_tsoni

Hi all,

We have an out-of-tree devfreq governor which uses the "governor.h" 
header from "drivers/devfreq". The source code for the governor is 
public on CodeLinaro [1].

Currently, the driver directly includes the "governor.h" header even 
though it's not a public header. A comment at the top of "governor.h" 
says explicitly: "This header is for devfreq governors in drivers/devfreq/".

Could you please advise the best way to use this private devfreq header 
from an out-of-tree governor? Could we move the header into "include/linux"?

[1]: 
https://git.codelinaro.org/clo/la/platform/vendor/qcom/opensource/graphics-kernel/-/blob/gfx-kernel.lnx.1.0.r3-rel/governor_msm_adreno_tz.c#L20

Thanks,
John

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

* Re: Out-of-tree devfreq governor header
  2023-01-09 18:30 Out-of-tree devfreq governor header John Moon
@ 2023-01-09 18:42 ` John Moon
  0 siblings, 0 replies; 2+ messages in thread
From: John Moon @ 2023-01-09 18:42 UTC (permalink / raw)
  To: myungjoo.ham, kyungmin.park, cw00.choi
  Cc: linux-kernel, linux-arm-msm, ccooper, mkrom, quic_tsoni

Fixing CC list. Apologies!

On 1/9/2023 10:30 AM, John Moon wrote:
> Hi all,
> 
> We have an out-of-tree devfreq governor which uses the "governor.h" 
> header from "drivers/devfreq". The source code for the governor is 
> public on CodeLinaro [1].
> 
> Currently, the driver directly includes the "governor.h" header even 
> though it's not a public header. A comment at the top of "governor.h" 
> says explicitly: "This header is for devfreq governors in 
> drivers/devfreq/".
> 
> Could you please advise the best way to use this private devfreq header 
> from an out-of-tree governor? Could we move the header into 
> "include/linux"?
> 
> [1]: 
> https://git.codelinaro.org/clo/la/platform/vendor/qcom/opensource/graphics-kernel/-/blob/gfx-kernel.lnx.1.0.r3-rel/governor_msm_adreno_tz.c#L20
> 
> Thanks,
> John

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

end of thread, other threads:[~2023-01-09 18:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-09 18:30 Out-of-tree devfreq governor header John Moon
2023-01-09 18:42 ` John Moon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®