Thread (14 messages) 14 messages, 3 authors, 2016-01-25

[PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator

From: John Crispin <hidden>
Date: 2016-01-25 13:46:15
Also in: linux-mediatek, lkml


On 25/01/2016 14:25, Javier Martinez Canillas wrote:
On Mon, Jan 25, 2016 at 10:19 AM, Javier Martinez Canillas
[off-list ref] wrote:

[snip]
quoted
In fact, the kernel is currently not matching the compatible, it is
only matching because you provided a .of_compatible is provided in the
mfd_cell.
Sorry my English was a bit off in this paragraph...

I tried to say that OF does not traverse MFD sub-devices and lookups a
device driver that matches the compatible automatically since a MFD
device is not a bus. Currently it is only trying to match a compatible
string because the mfd_cell has a .of_compatible set so an of_node is
assigned on mfd_add_device().

But it is failing to match because no OF device table is provided and
the platform bus match callback is falling back to the driver .name to
match so the compatible is not really used as Mark said.

Best regards,
Javier
Hi,

just so i am sure to have understood properly. i just need to drop the
compatible string from [1/2] and resend. if this is the case i will fix
the mt6397 binding doc while at it.

	John
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help