From: Arnd Bergmann <arnd@arndb.de>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
Mika Westerberg <mika.westerberg@linux.intel.com>,
Mark Brown <broonie@kernel.org>,
Michal Simek <michal.simek@xilinx.com>,
dev <dev@lists.96boards.org>, Rob Herring <robh+dt@kernel.org>,
John Stultz <john.stultz@linaro.org>,
Andy Shevchenko <andy@infradead.org>,
Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
Frank Rowand <frowand.list@gmail.com>,
Linux ARM <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 0/5] RFC: Mezzanine handling for 96boards
Date: Tue, 26 Jun 2018 10:08:00 +0200 [thread overview]
Message-ID: <CAK8P3a1V3BGY_RbPbOaj_sw==52s3AC6iBD8js599hzZqWNr0g@mail.gmail.com> (raw)
In-Reply-To: <CACRpkdZY0+q=VKrNhk8w-NSn0zbjCqdgdyApBTs+tVRBWm8ReA@mail.gmail.com>
On Tue, Jun 26, 2018 at 10:02 AM, Linus Walleij
<linus.walleij@linaro.org> wrote:
> On Mon, Jun 18, 2018 at 2:21 PM Arnd Bergmann <arnd@arndb.de> wrote:
>
>> - I really like the idea of having C code deal with the mezzanine
>> connector itself, acting as an intermediate to tie a number of
>> boards to a number of add-on cards, this seems much simpler than
>> trying to do everything with overlays or one of the other more
>> generic mechanisms.
>
> This is my point of view as well.
>
>> - I don't like the idea of having the bus driver contain a list of possible
>> add-ons, this seems to go against our usual driver model. What
>> I think we want instead is to make the connector itself a proper
>> bus_type, to allow drivers to register against it as loadable modules,
>> and devices (maybe limited to one device) being created as probed
>> from DT or some other method as you describe.
>
> OK I think I can do that. It will take some thinking and I would
> guess I should also move it to drivers/platform/*
I'd probably leave it in drivers/bus/, though this might be one of the
rare cases that drivers/platform makes sense for as well.
> My idea here would be to have a connector bus that some random
> add-on (such as a 96board mezzanine) would probe to, so we
> make it generic for any connector plug.
The question here is whether we want one 'struct bus_type' instance
shared by all connector types, or one for each type. I was actually
leaning towards the latter here.
>> - You export symbols in the mezzanine_* namespace, which I think
>> is a bit too generic and should perhaps contain something related
>> to 96boards in its name to make it less ambiguous. I suspect we
>> would add a number of further connectors for hats, capes, lures etc,
>> which could all be described as mezzanines.
>
> Michal also brings up the (discoverable!) FMC connectors Xilinx are
> using. So the idea would be to create something generic to help
> probe and populate them all whether partly discoverable or not.
If a bus is fully discoverable, I think it's better to use a separate
bus_type that does not use DT based probing for its children.
However, many buses (even USB and PCI) turn out to be only
mostly discoverable, so we end up having to come up with a way
to represent child devices in DT as well.
Arnd
next prev parent reply other threads:[~2018-06-26 8:08 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-18 7:45 [PATCH 0/5] RFC: Mezzanine handling for 96boards Linus Walleij
2018-06-18 7:45 ` [PATCH 1/5] RFC: gpio: Add API to explicitly name a consumer Linus Walleij
2018-06-18 7:45 ` [PATCH 2/5] RFC: eeprom: at24: Allow passing gpiodesc from pdata Linus Walleij
2018-06-20 0:45 ` Andy Shevchenko
2018-06-18 7:45 ` [PATCH 3/5] RFC: spi: Make of_find_spi_device_by_node() available Linus Walleij
2018-06-18 7:45 ` [PATCH 4/5] RFC: bus: 96boards Low-Speed Connector Linus Walleij
2018-06-19 11:19 ` [Dev] " Daniel Thompson
2018-06-18 7:45 ` [PATCH 5/5] RFC: ARM64: dts: Add Low-Speed Connector to ZCU100 Linus Walleij
2018-06-19 14:55 ` Rob Herring
2018-06-22 13:22 ` Michal Simek
2018-06-18 12:21 ` [PATCH 0/5] RFC: Mezzanine handling for 96boards Arnd Bergmann
2018-06-18 13:22 ` Ard Biesheuvel
2018-06-18 14:15 ` Arnd Bergmann
2018-06-22 13:36 ` Michal Simek
2018-06-19 15:14 ` [Dev] " Yang Zhang
2018-06-19 15:25 ` Ard Biesheuvel
2018-06-19 15:26 ` Yang Zhang
2018-06-19 15:30 ` Ard Biesheuvel
2018-06-19 15:32 ` Yang Zhang
2018-06-22 13:08 ` Michal Simek
2018-06-26 9:10 ` Linus Walleij
2018-06-26 8:09 ` Linus Walleij
2018-06-26 8:19 ` Ard Biesheuvel
2018-06-26 8:36 ` Linus Walleij
2018-06-26 8:02 ` Linus Walleij
2018-06-26 8:08 ` Arnd Bergmann [this message]
2018-06-26 12:12 ` Mark Brown
2018-07-16 15:07 ` Rob Herring
2018-06-19 11:10 ` [Dev] " Daniel Thompson
2018-06-19 11:56 ` Mark Brown
2018-06-19 15:52 ` Rob Herring
2018-06-21 13:02 ` Frank Rowand
2018-06-26 9:03 ` Linus Walleij
2018-06-26 9:00 ` Linus Walleij
2018-06-27 15:23 ` [Dev] " Rob Herring
2018-06-28 6:10 ` Michal Simek
2018-06-21 12:58 ` Frank Rowand
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='CAK8P3a1V3BGY_RbPbOaj_sw==52s3AC6iBD8js599hzZqWNr0g@mail.gmail.com' \
--to=arnd@arndb.de \
--cc=andy@infradead.org \
--cc=broonie@kernel.org \
--cc=dev@lists.96boards.org \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=john.stultz@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=manivannan.sadhasivam@linaro.org \
--cc=mark.rutland@arm.com \
--cc=michal.simek@xilinx.com \
--cc=mika.westerberg@linux.intel.com \
--cc=robh+dt@kernel.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).