Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1465677

Re: [PATCH v2 1/4] nvmem: rockchip-efuse: Change initcall to subsys

Show all headers | View raw

Am Donnerstag, 18. August 2016, 13:28:58 CEST schrieb Kevin Hilman:
> Finlye Xiao <finley.xiao@rock-chips.com> writes:
> > From: Finley Xiao <finley.xiao@rock-chips.com>
> > 
> > We will register a cpufreq notifier for adjusting opp's voltage, and it
> > need to fetch cpu's leakage from efuse in the notifier_call. so the efuse
> > driver should probe before cpufreq driver.
> > 
> > Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
> 
> Why can't this be handled with deferred probling?  initcall ordering is
> a can of worms.

I think the issue is less between efuse and avs driver, but more between avs 
driver and cpufreq. The avs driver aims to modify the opp table and thus wants 
to do that / register the notifier before cpufreq starts.

And as there is no direct connection between cpufreq and the avs driver, 
making cpufreq defer probing is probably not really easy.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/4] PM / AVS: add Rockchip cpu avs Finlye Xiao <finley.xiao@rock-chips.com> - 2016-08-18 11:00 +0200
  [PATCH v2 2/4] of: introduce of_property_read_s32_index Finlye Xiao <finley.xiao@rock-chips.com> - 2016-08-18 11:10 +0200
  [PATCH v2 3/4] dt-bindings: add binding document for Rockchip cpu avs Finlye Xiao <finley.xiao@rock-chips.com> - 2016-08-18 11:10 +0200
  [PATCH v2 1/4] nvmem: rockchip-efuse: Change initcall to subsys Finlye Xiao <finley.xiao@rock-chips.com> - 2016-08-18 11:10 +0200
    Re: [PATCH v2 1/4] nvmem: rockchip-efuse: Change initcall to subsys Heiko Stuebner <heiko@sntech.de> - 2016-08-19 03:00 +0200
      Re: [PATCH v2 1/4] nvmem: rockchip-efuse: Change initcall to subsys Kevin Hilman <khilman@baylibre.com> - 2016-08-19 18:20 +0200
    Re: [PATCH v2 1/4] nvmem: rockchip-efuse: Change initcall to subsys Kevin Hilman <khilman@baylibre.com> - 2016-08-19 05:20 +0200

csiph-web