Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver

From	Mark Brown <broonie@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver
Date	2016-06-26 14:00 +0200
Message-ID	<rOgTf-2tR-13@gated-at.bofh.it> (permalink)
References	(1 earlier) <rNAhj-Ks-7@gated-at.bofh.it> <rO6TU-4Bp-19@gated-at.bofh.it> <rO7ZD-5l3-7@gated-at.bofh.it> <rOgqe-2jQ-27@gated-at.bofh.it> <rOgzU-2mY-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Sun, Jun 26, 2016 at 01:35:46PM +0200, Michal Suchanek wrote:
> On 26 June 2016 at 13:21, Mark Brown <broonie@kernel.org> wrote:

> > You can just add the entire slave node in the overlay, it's not clear
> > that this buys us anything useful

> You have to target the master node and specify the CS in the overlay
> if you create the whole node. If you target the slave node you have
> the CS specified in the board devicetree and the overlay can apply to
> multiple boards without change.

No, there's a lot more to having overlays to board-neutral connectors...

> > and this needs to be joined up with the work going on to allow
> > expansion connector overlays to be loaded in the first place.

> The overlays work equally well before and after this patch. I don't
> see any problem with them other than part of the infrastructure
> missing in mainline kernel.

...you're missing all this stuff here - currently the active work on
this is being done by Patelis.  It's not at all clear to me that
referring to a slave rather than a master would be any easier in a board
neutral overlay.

Thread

[PATCH 2/3] spi: of: allow instantiating slaves without a driver Michal Suchanek <hramrach@gmail.com> - 2016-06-24 16:30 +0200
  Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Mark Brown <broonie@kernel.org> - 2016-06-26 03:20 +0200
    Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Michal Suchanek <hramrach@gmail.com> - 2016-06-26 04:30 +0200
      Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Mark Brown <broonie@kernel.org> - 2016-06-26 13:30 +0200
        Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Michal Suchanek <hramrach@gmail.com> - 2016-06-26 13:40 +0200
          Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Mark Brown <broonie@kernel.org> - 2016-06-26 14:00 +0200
            Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Mark Brown <broonie@kernel.org> - 2016-06-26 14:50 +0200
              Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Mark Brown <broonie@kernel.org> - 2016-06-26 15:00 +0200
                Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Michal Suchanek <hramrach@gmail.com> - 2016-06-26 17:30 +0200
                Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Mark Brown <broonie@kernel.org> - 2016-06-26 19:30 +0200
              Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Michal Suchanek <hramrach@gmail.com> - 2016-06-26 15:00 +0200
            Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver Michal Suchanek <hramrach@gmail.com> - 2016-06-26 14:50 +0200

csiph-web