From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752197AbcFZL6l (ORCPT ); Sun, 26 Jun 2016 07:58:41 -0400 Received: from mezzanine.sirena.org.uk ([106.187.55.193]:43068 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752069AbcFZL6k (ORCPT ); Sun, 26 Jun 2016 07:58:40 -0400 Date: Sun, 26 Jun 2016 12:58:15 +0100 From: Mark Brown To: Michal Suchanek Cc: Greg Kroah-Hartman , Ingo Molnar , Andrew Morton , Kees Cook , Thomas Gleixner , Dan Williams , Tejun Heo , "Paul E. McKenney" , Davidlohr Bueso , Andrey Ryabinin , Nikolay Aleksandrov , Dmitry Vyukov , Adrien Schildknecht , Linux Kernel Mailing List , linux-spi Message-ID: <20160626115815.GN28202@sirena.org.uk> References: <20160626011500.GF28202@sirena.org.uk> <20160626112133.GJ28202@sirena.org.uk> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="CeC2K4acttR/mmFn" Content-Disposition: inline In-Reply-To: X-Cookie: Beware of Bigfoot! User-Agent: Mutt/1.6.0 (2016-04-01) X-SA-Exim-Connect-IP: 2a01:348:6:8808:fab::3 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH 2/3] spi: of: allow instantiating slaves without a driver X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on mezzanine.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --CeC2K4acttR/mmFn Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Sun, Jun 26, 2016 at 01:35:46PM +0200, Michal Suchanek wrote: > On 26 June 2016 at 13:21, Mark Brown 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. --CeC2K4acttR/mmFn Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJXb8NWAAoJECTWi3JdVIfQ+WgH/1v7Y5uCQ5c9rCndAQfVY2js +oOtH8golvNuQpohNrC3vHhsaCaEXCeH8rQXmiAukjbLXOdUsUNLQE+PfhdtGvjV fEC6rK/kRNz1UGQek+f8NKYYDDEXOMGziRV7xyoxAQrHb46hw24OR14M828E+ZKF dxKICyCYJ7agGigkHaJOMig5n3ZAMXCLzrL3+wt/AIVwdECRyCxbG+6jb7f2H9a7 pl+7n9vB/6y6fUnKN8fzvAtCQ99qS0VcN1IYsyjc4fjPV8EOp0tQt0Y66cQrv1in TjsGFbhO3QXFf3c97slhcYVw55EcjAcKtdDyt90GMTNF2HHIuX7V/bEMKbgGiDA= =BNc/ -----END PGP SIGNATURE----- --CeC2K4acttR/mmFn--