From: Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> To: Linus Walleij <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>, Bryan Wu <cooloney-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Catalin Marinas <catalin.marinas-5wv7dgnIgG8@public.gmane.org>, Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>, Richard Purdie <rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.org>, Devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Benjamin Herrenschmidt <benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>, Grant Likely <grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>, Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>, Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>, Ian Campbell <ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>, Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> Subject: Re: [PATCH 1/5] MFD/OF: document MFD devices and handle simple-mfd Date: Tue, 3 Mar 2015 10:25:50 +0000 [thread overview] Message-ID: <20150303102550.GA32624@x1> (raw) In-Reply-To: <1425375148-4369-2-git-send-email-linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> On Tue, 03 Mar 2015, 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 <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > Cc: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> > Cc: Devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> > Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > Cc: Benjamin Herrenschmidt <benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> > Cc: Grant Likely <grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > Cc: Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org> > Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> > Cc: Ian Campbell <ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org> > Cc: Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> > Signed-off-by: Linus Walleij <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > --- > 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 I don't think you meant 'comprise' here. Perhaps 'represent' might fit better? > +spawning multiple child devices. Spawning devices is something software will do. Think less about what the actions the OS will take order to handle these IPs and more about what they are physically. I would prefer the description mentions that these chips are single pieces of silicon which contain >1 devices with non-unique/varying functionality. > +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 Please capitalise the expanded PMIC abbreviation. > + 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. s/registers/{hardware|devices}/ > +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 This is pretty Linux specific IMHO. Other OSes might not spawn seperate devices to handle MFD type h/w. The idea is fine, but please generify a little. > + "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@1000 { Should be 10000. > + compatible = "syscon", "simple-mfd"; Please address your tabbing issues. > + reg = <0x010000 0x1000>; > + > + led@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 */ -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: lee.jones@linaro.org (Lee Jones) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/5] MFD/OF: document MFD devices and handle simple-mfd Date: Tue, 3 Mar 2015 10:25:50 +0000 [thread overview] Message-ID: <20150303102550.GA32624@x1> (raw) In-Reply-To: <1425375148-4369-2-git-send-email-linus.walleij@linaro.org> On Tue, 03 Mar 2015, 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 <lee.jones@linaro.org> > Cc: Arnd Bergmann <arnd@arndb.de> > Cc: Devicetree <devicetree@vger.kernel.org> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> > Cc: Grant Likely <grant.likely@linaro.org> > Cc: Pawel Moll <pawel.moll@arm.com> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: Ian Campbell <ijc+devicetree@hellion.org.uk> > Cc: Kumar Gala <galak@codeaurora.org> > Signed-off-by: Linus Walleij <linus.walleij@linaro.org> > --- > 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 I don't think you meant 'comprise' here. Perhaps 'represent' might fit better? > +spawning multiple child devices. Spawning devices is something software will do. Think less about what the actions the OS will take order to handle these IPs and more about what they are physically. I would prefer the description mentions that these chips are single pieces of silicon which contain >1 devices with non-unique/varying functionality. > +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 Please capitalise the expanded PMIC abbreviation. > + 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. s/registers/{hardware|devices}/ > +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 This is pretty Linux specific IMHO. Other OSes might not spawn seperate devices to handle MFD type h/w. The idea is fine, but please generify a little. > + "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 { Should be 10000. > + compatible = "syscon", "simple-mfd"; Please address your tabbing issues. > + 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 */ -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
next prev parent reply other threads:[~2015-03-03 10:25 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-03-03 9:32 [PATCH 0/5] simple-mfd and syscon LEDs Linus Walleij [not found] ` <1425375148-4369-1-git-send-email-linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2015-03-03 9:32 ` [PATCH 1/5] MFD/OF: document MFD devices and handle simple-mfd Linus Walleij 2015-03-03 9:32 ` Linus Walleij [not found] ` <1425375148-4369-2-git-send-email-linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2015-03-03 10:25 ` Lee Jones [this message] 2015-03-03 10:25 ` Lee Jones 2015-03-09 14:29 ` Linus Walleij 2015-03-09 14:29 ` Linus Walleij 2015-03-03 17:47 ` Rob Herring 2015-03-03 17:47 ` Rob Herring 2015-03-03 17:56 ` Lee Jones 2015-03-03 17:56 ` Lee Jones 2015-03-09 14:35 ` Linus Walleij 2015-03-09 14:35 ` Linus Walleij 2015-03-12 9:14 ` Alexandre Belloni 2015-03-12 9:14 ` Alexandre Belloni [not found] ` <20150312091427.GA4329-m++hUPXGwpdeoWH0uzbU5w@public.gmane.org> 2015-03-17 10:28 ` Linus Walleij 2015-03-17 10:28 ` Linus Walleij [not found] ` <CACRpkdacQ-MR+Sziu-TQXWr+1UCtBphZ5eNSWbJ2ZpCnH+o9JA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2015-03-17 10:38 ` Linus Walleij 2015-03-17 10:38 ` Linus Walleij 2015-03-18 16:06 ` Alexandre Belloni 2015-03-18 16:06 ` Alexandre Belloni 2015-03-20 13:26 ` Antoine Tenart 2015-03-20 13:26 ` Antoine Tenart 2015-03-28 13:41 ` Grant Likely 2015-03-28 13:41 ` Grant Likely [not found] ` <20150328134157.000A3C40AE0-WNowdnHR2B42iJbIjFUEsiwD8/FfD2ys@public.gmane.org> 2015-04-07 9:03 ` Linus Walleij 2015-04-07 9:03 ` Linus Walleij 2015-04-03 18:34 ` Olof Johansson 2015-04-03 18:34 ` Olof Johansson [not found] ` <CAOesGMhz6WPweQFXfPhTO6Pz5L4Z1Si6YGiCRCiFD9B5qB9tVw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2015-04-07 7:16 ` Lee Jones 2015-04-07 7:16 ` Lee Jones 2015-04-07 9:00 ` Linus Walleij 2015-04-07 9:00 ` Linus Walleij 2015-03-03 9:32 ` [PATCH 2/5] ARM: dts: update syscons to use simple-mfd Linus Walleij 2015-03-03 9:32 ` [PATCH 3/5] leds: syscon: instantiate from platform device Linus Walleij 2015-03-17 10:36 ` Linus Walleij 2015-03-24 16:50 ` Bjorn Andersson 2015-03-03 9:32 ` [PATCH 4/5] arm64: juno: Add APB registers and LEDs using syscon Linus Walleij 2015-03-04 15:57 ` Liviu Dudau 2015-03-03 9:32 ` [PATCH 5/5] arm64: add LEDs and some trigger support to defconfig Linus Walleij
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20150303102550.GA32624@x1 \ --to=lee.jones-qsej5fyqhm4dnm+yrofe0a@public.gmane.org \ --cc=arnd-r2nGTMty4D4@public.gmane.org \ --cc=benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org \ --cc=catalin.marinas-5wv7dgnIgG8@public.gmane.org \ --cc=cooloney-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \ --cc=grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \ --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \ --cc=linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \ --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \ --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \ --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \ --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.org \ --cc=will.deacon-5wv7dgnIgG8@public.gmane.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.