Thread (8 messages) flat view 8 messages, 2 authors, 2017-08-30

[RESEND PATCH 2/3] regulator: Add support for stm32-vrefbuf

From: broonie@kernel.org (Mark Brown)
Date: 2017-08-30 15:03:02
Also in: linux-devicetree, lkml

On Wed, Aug 30, 2017 at 11:11:24AM +0200, Fabrice Gasnier wrote:
On 08/29/2017 08:57 PM, Mark Brown wrote:
quoted
On Mon, Aug 28, 2017 at 02:58:52PM +0200, Fabrice Gasnier wrote:
quoted
quoted
+static int __init stm32_vrefbuf_init(void)
+{
+	return platform_driver_register(&stm32_vrefbuf_driver);
+}
+subsys_initcall(stm32_vrefbuf_init);
quoted
Why is this at subsys_initcall()?
Several consumers depend on it when it's being used, among which: STM32
internal ADC and DAC, but also external components. Purpose is to ensure
it's ready before these drivers gets probed, instead of being deferred.
Is it ok to keep it ?
No, that's not OK - just let deferred probe handle it.  The same thing
applies to all regulator usage.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170830/d47026aa/attachment.sig>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help