From mboxrd@z Thu Jan 1 00:00:00 1970 From: antoine.tenart@free-electrons.com (Antoine Tenart) Date: Fri, 20 Mar 2015 14:26:46 +0100 Subject: [PATCH 1/5] MFD/OF: document MFD devices and handle simple-mfd In-Reply-To: <1425375148-4369-2-git-send-email-linus.walleij@linaro.org> References: <1425375148-4369-1-git-send-email-linus.walleij@linaro.org> <1425375148-4369-2-git-send-email-linus.walleij@linaro.org> Message-ID: <20150320132646.GH4802@kwain> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Mar 03, 2015 at 10:32:24AM +0100, Linus Walleij wrote: > This defines a new compatible option for MFD devices "simple-mfd" that will > make the OF core spawn child devices for all subnodes of that MFD device. > It is optional but handy for things like syscon and possibly other > simpler MFD devices. > > Since there was no file to put the documentation in, I took this opportunity > to make a small writeup on MFD devices and add the compatible definition > there. > > Suggested-by: Lee Jones > Cc: Arnd Bergmann > Cc: Devicetree > Cc: Rob Herring > Cc: Benjamin Herrenschmidt > Cc: Grant Likely > Cc: Pawel Moll > Cc: Mark Rutland > Cc: Ian Campbell > Cc: Kumar Gala > Signed-off-by: Linus Walleij Since I'm also using this, Acked-by: Antoine Tenart > --- > I make the patch to the OF core in this one, it makes much more sense since > it's a oneliner > > Lee: this is a (tested!) implementation of your suggestion for simple-mfd. > If you can eventually ACK this from the MFD side, I think it should be > funneled through the ARM SoC tree. > > Grant/Rob: if either of you can ACK the change to the OF core likewise it can > be taken through ARM SoC. > > DT binings maintainers: there is some background discussion on this here: > http://marc.info/?l=linux-arm-kernel&m=142486676603889&w=2 > http://marc.info/?l=devicetree&m=142166313621469&w=2 > --- > Documentation/devicetree/bindings/mfd/mfd.txt | 40 +++++++++++++++++++++++++++ > drivers/of/platform.c | 1 + > 2 files changed, 41 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/mfd.txt > > diff --git a/Documentation/devicetree/bindings/mfd/mfd.txt b/Documentation/devicetree/bindings/mfd/mfd.txt > new file mode 100644 > index 000000000000..cc057438abe8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/mfd.txt > @@ -0,0 +1,40 @@ > +Multi-Function Devices (MFD) > + > +These devices comprise a nexus for heterogeneous hardware blocks spawning > +multiple child devices. > + > +A typical MFD can be: > + > +- A mixed signal ASIC on an external bus, sometimes a PMIC (power management > + integrated circuit) that is manufactured in a lower technology node (rough > + silicon) that handles analog drivers for things like audio amplifiers, LED > + drivers, level shifters, PHY (physical interfaces to things like USB or > + ethernet), regulators etc. > + > +- A range of memory registers containing "miscellaneous system registers" also > + known as a system controller "syscon" or any other memory range containing a > + mix of unrelated registers. > + > +Optional properties: > + > +- compatible : "simple-mfd" - this signifies that the operating system should > + spawn child devices for all the subnodes of the MFD device akin to how > + "simple-bus" inidicates when to spawn children for a simple memory-mapped > + bus. For more complex devices, when the nexus driver has to probe registers > + to figure out what children exist etc, this should not be used. In the latter > + case the child devices will be instantiated by the operating system. > + > +Example: > + > +foo at 1000 { > + compatible = "syscon", "simple-mfd"; > + reg = <0x010000 0x1000>; > + > + led at 08.0 { > + compatible = "register-bit-led"; > + offset = <0x08>; > + mask = <0x01>; > + label = "myled"; > + default-state = "on"; > + }; > +}; > diff --git a/drivers/of/platform.c b/drivers/of/platform.c > index b189733a1539..969ba4e1460c 100644 > --- a/drivers/of/platform.c > +++ b/drivers/of/platform.c > @@ -26,6 +26,7 @@ > > const struct of_device_id of_default_bus_match_table[] = { > { .compatible = "simple-bus", }, > + { .compatible = "simple-mfd", }, > #ifdef CONFIG_ARM_AMBA > { .compatible = "arm,amba-bus", }, > #endif /* CONFIG_ARM_AMBA */ > -- > 1.9.3 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Antoine T?nart, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Antoine Tenart Subject: Re: [PATCH 1/5] MFD/OF: document MFD devices and handle simple-mfd Date: Fri, 20 Mar 2015 14:26:46 +0100 Message-ID: <20150320132646.GH4802@kwain> References: <1425375148-4369-1-git-send-email-linus.walleij@linaro.org> <1425375148-4369-2-git-send-email-linus.walleij@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <1425375148-4369-2-git-send-email-linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Linus Walleij Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Arnd Bergmann , Lee Jones , Bryan Wu , Mark Rutland , Devicetree , Pawel Moll , Ian Campbell , Catalin Marinas , Kumar Gala , Will Deacon , Rob Herring , Richard Purdie , Benjamin Herrenschmidt , Grant Likely List-Id: devicetree@vger.kernel.org On Tue, Mar 03, 2015 at 10:32:24AM +0100, Linus Walleij wrote: > This defines a new compatible option for MFD devices "simple-mfd" tha= t will > make the OF core spawn child devices for all subnodes of that MFD dev= ice. > It is optional but handy for things like syscon and possibly other > simpler MFD devices. >=20 > Since there was no file to put the documentation in, I took this oppo= rtunity > to make a small writeup on MFD devices and add the compatible definit= ion > there. >=20 > Suggested-by: Lee Jones > Cc: Arnd Bergmann > Cc: Devicetree > Cc: Rob Herring > Cc: Benjamin Herrenschmidt > Cc: Grant Likely > Cc: Pawel Moll > Cc: Mark Rutland > Cc: Ian Campbell > Cc: Kumar Gala > Signed-off-by: Linus Walleij Since I'm also using this, Acked-by: Antoine Tenart > --- > I make the patch to the OF core in this one, it makes much more sense= since > it's a oneliner >=20 > Lee: this is a (tested!) implementation of your suggestion for simple= -mfd. > If you can eventually ACK this from the MFD side, I think it should b= e > funneled through the ARM SoC tree. >=20 > Grant/Rob: if either of you can ACK the change to the OF core likewis= e it can > be taken through ARM SoC. >=20 > DT binings maintainers: there is some background discussion on this h= ere: > http://marc.info/?l=3Dlinux-arm-kernel&m=3D142486676603889&w=3D2 > http://marc.info/?l=3Ddevicetree&m=3D142166313621469&w=3D2 > --- > Documentation/devicetree/bindings/mfd/mfd.txt | 40 +++++++++++++++++= ++++++++++ > drivers/of/platform.c | 1 + > 2 files changed, 41 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/mfd.txt >=20 > diff --git a/Documentation/devicetree/bindings/mfd/mfd.txt b/Document= ation/devicetree/bindings/mfd/mfd.txt > new file mode 100644 > index 000000000000..cc057438abe8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/mfd.txt > @@ -0,0 +1,40 @@ > +Multi-Function Devices (MFD) > + > +These devices comprise a nexus for heterogeneous hardware blocks spa= wning > +multiple child devices. > + > +A typical MFD can be: > + > +- A mixed signal ASIC on an external bus, sometimes a PMIC (power ma= nagement > + integrated circuit) that is manufactured in a lower technology nod= e (rough > + silicon) that handles analog drivers for things like audio amplifi= ers, LED > + drivers, level shifters, PHY (physical interfaces to things like U= SB or > + ethernet), regulators etc. > + > +- A range of memory registers containing "miscellaneous system regis= ters" also > + known as a system controller "syscon" or any other memory range co= ntaining a > + mix of unrelated registers. > + > +Optional properties: > + > +- compatible : "simple-mfd" - this signifies that the operating syst= em should > + spawn child devices for all the subnodes of the MFD device akin to= how > + "simple-bus" inidicates when to spawn children for a simple memory= -mapped > + bus. For more complex devices, when the nexus driver has to probe = registers > + to figure out what children exist etc, this should not be used. In= the latter > + case the child devices will be instantiated by the operating syste= m. > + > +Example: > + > +foo@1000 { > + compatible =3D "syscon", "simple-mfd"; > + reg =3D <0x010000 0x1000>; > + > + led@08.0 { > + compatible =3D "register-bit-led"; > + offset =3D <0x08>; > + mask =3D <0x01>; > + label =3D "myled"; > + default-state =3D "on"; > + }; > +}; > diff --git a/drivers/of/platform.c b/drivers/of/platform.c > index b189733a1539..969ba4e1460c 100644 > --- a/drivers/of/platform.c > +++ b/drivers/of/platform.c > @@ -26,6 +26,7 @@ > =20 > const struct of_device_id of_default_bus_match_table[] =3D { > { .compatible =3D "simple-bus", }, > + { .compatible =3D "simple-mfd", }, > #ifdef CONFIG_ARM_AMBA > { .compatible =3D "arm,amba-bus", }, > #endif /* CONFIG_ARM_AMBA */ > --=20 > 1.9.3 >=20 >=20 > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --=20 Antoine T=E9nart, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html