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

Groups > linux.kernel > #1316700

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

From Javier Martinez Canillas <javier@dowhile0.org>
Newsgroups linux.kernel
Subject Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator
Date 2016-01-25 14:30 +0100
Message-ID <qUPns-4s7-3@gated-at.bofh.it> (permalink)
References (2 earlier) <qUOhI-3K1-17@gated-at.bofh.it> <qUOhI-3K1-15@gated-at.bofh.it> <qUOB4-3Tm-27@gated-at.bofh.it> <qUPdM-4ol-23@gated-at.bofh.it> <qUPdN-4ol-47@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

On Mon, Jan 25, 2016 at 10:19 AM, Javier Martinez Canillas
<javier@dowhile0.org> wrote:

[snip]

>
> 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

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

Thread

[PATCH V2 1/2] dt-bindings: regulator: Add document for MT6323 regulator John Crispin <blogic@openwrt.org> - 2016-01-25 11:50 +0100
  [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator John Crispin <blogic@openwrt.org> - 2016-01-25 11:50 +0100
    Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 13:20 +0100
      Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 13:40 +0100
      Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator Mark Brown <broonie@kernel.org> - 2016-01-25 13:40 +0100
        Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 14:20 +0100
          Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 14:20 +0100
            Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 14:30 +0100
              Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 14:50 +0100
                Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 15:10 +0100
                Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 15:10 +0100
    Re: [PATCH V2 2/2] regulator: mt6323: Add support for MT6323 regulator Javier Martinez Canillas <javier@dowhile0.org> - 2016-01-25 13:20 +0100
  Re: [PATCH V2 1/2] dt-bindings: regulator: Add document for MT6323  regulator Mark Brown <broonie@kernel.org> - 2016-01-25 12:40 +0100
    Re: [PATCH V2 1/2] dt-bindings: regulator: Add document for MT6323  regulator John Crispin <blogic@openwrt.org> - 2016-01-25 13:10 +0100

csiph-web