[PATCH] cpufreq: imx6: rework getting clocks and regulators
s.hauer at pengutronix.de
Tue Oct 20 05:47:31 PDT 2015
On Tue, Oct 20, 2015 at 02:26:58PM +0200, Lucas Stach wrote:
> Am Dienstag, den 20.10.2015, 13:56 +0200 schrieb Sascha Hauer:
> > Use devm_* functions for getting clocks and regulators. This simplfifies
> > the error path. Also return the error from the failing call instead of
> > collating all regulator and clock errors to -ENOENT. With this the
> > driver can properly handle -EPROBE_DEFER. Handling deferred probing
> > becomes necessary once the SoC internal regulators get supplies. While
> > the SoC internal regulators are registered early in architecture code,
> > the supplies may be regular I2C devices which may be probed after the
> > cpufreq driver.
> > Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
> This has come up several times before, maybe we should get a comment in
> the code.
Indeed, yes. This would help not wasting developing time.
Anyway, the driver needs to handle -EPROBE_DEFER properly, if not with
devm_ then without it.
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the linux-arm-kernel