From mboxrd@z Thu Jan 1 00:00:00 1970 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Subject: Re: How to encode being an I2C slave in DT? Date: Wed, 6 May 2015 19:47:57 +0200 Message-ID: <20150506174757.GZ25193@pengutronix.de> References: <1427745615-5428-1-git-send-email-danindrey@mail.ru> <20150505105513.GA1841@katana> <20150506065928.GP25193@pengutronix.de> <3394114.re5da8pjTi@fb07-iapwap2> <20150506080951.GS25193@pengutronix.de> <554A39F1.9060507@wwwdotorg.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: <554A39F1.9060507-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Warren Cc: Marc Dietrich , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Wolfram Sang , Greg Kroah-Hartman , linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Andrey Danin , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-spec-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org Hello Stephen, On Wed, May 06, 2015 at 09:57:37AM -0600, Stephen Warren wrote: > On 05/06/2015 02:09 AM, Uwe Kleine-K=F6nig wrote: > >On Wed, May 06, 2015 at 09:53:55AM +0200, Marc Dietrich wrote: > >>Am Mittwoch, 6. Mai 2015, 08:59:28 schrieb Uwe Kleine-K=F6nig: > >>>On Tue, May 05, 2015 at 12:55:13PM +0200, Wolfram Sang wrote: > >>>>So what about adding a new property "i2c-slave-reg"? This does no= t only > >>>>prevent the confusion above, but also makes it very clear that th= is node > >>>>is an I2C slave without the need to encode that somehow in the > >>>>compatible property (although it probably should be described the= re as > >>>>well, still). > >>> > >>>I admit I didn't follow the discussions referenced in the footnote= s, but > >>>I wonder if the slave part should be added to the device tree at a= ll. > >>>AFAICT it could (and so should) be completely userspace-defined wh= ich > >>>slave driver is used on which address. I imagine that for most > >>>controllers the bus addresses to use can be chosen more or less fr= eely. > >>>So what am I missing? > >> > >>if you had read the footnotes you would know :-) Our usecase is con= nect an > >>embeedded controller via i2c to the host soc, similar to cros-ec, b= ut here the > >>ec is the i2c master. The ec connects keyboard, mouse, pwrmngt, and= other > >>stuff, for which the drivers are best implemented in kernel code AF= AIK. > > > >Right, the driver might sensibly be implemented in kernel space. But= I'd > >vote that you still need to do the binding of these drivers to your > >slave controller from userspace. Then there is no need to specify > >anything in your dtb. >=20 > I think the set of I2C slave devices that are implemented by the > Linux system can reasonably be considered part of the HW definition. > Most DT content to date has been a definition of the HW that's > available to SW, but at least in this case, this I2C slave device is > something that must be implemented (admittedly in conjunction with > SW) using the I2C slave HW on the main SoC, in order for the overall > HW to work as intended. I'm not convinced. Why must it be implemented? And is it too late if th= e setup is done by userspace? The devicetree description of a flash chip doesn't include the rootfs content either, although a working rootfs is critical for most operatin= g systems. The devicetree description of an ethernet adapter doesn't include its network setup although an ethernet adapter hardly makes any sense without an IP. =20 > BTW, I believe devicetree-spec-u79uwXL29TY76Z2rM5mHXA@public.gmane.org was created to > address subsystem-level DT schema questions like this. It's much > lower volume, so perhaps the thread would get noticed by the DT > maintainers if posted there (or perhaps just CC them)? I added them to Cc. Best regards Uwe --=20 Pengutronix e.K. | Uwe Kleine-K=F6nig = | Industrial Linux Solutions | http://www.pengutronix.de/= | From mboxrd@z Thu Jan 1 00:00:00 1970 From: u.kleine-koenig@pengutronix.de (Uwe =?iso-8859-1?Q?Kleine-K=F6nig?=) Date: Wed, 6 May 2015 19:47:57 +0200 Subject: How to encode being an I2C slave in DT? In-Reply-To: <554A39F1.9060507@wwwdotorg.org> References: <1427745615-5428-1-git-send-email-danindrey@mail.ru> <20150505105513.GA1841@katana> <20150506065928.GP25193@pengutronix.de> <3394114.re5da8pjTi@fb07-iapwap2> <20150506080951.GS25193@pengutronix.de> <554A39F1.9060507@wwwdotorg.org> Message-ID: <20150506174757.GZ25193@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello Stephen, On Wed, May 06, 2015 at 09:57:37AM -0600, Stephen Warren wrote: > On 05/06/2015 02:09 AM, Uwe Kleine-K?nig wrote: > >On Wed, May 06, 2015 at 09:53:55AM +0200, Marc Dietrich wrote: > >>Am Mittwoch, 6. Mai 2015, 08:59:28 schrieb Uwe Kleine-K?nig: > >>>On Tue, May 05, 2015 at 12:55:13PM +0200, Wolfram Sang wrote: > >>>>So what about adding a new property "i2c-slave-reg"? This does not only > >>>>prevent the confusion above, but also makes it very clear that this node > >>>>is an I2C slave without the need to encode that somehow in the > >>>>compatible property (although it probably should be described there as > >>>>well, still). > >>> > >>>I admit I didn't follow the discussions referenced in the footnotes, but > >>>I wonder if the slave part should be added to the device tree at all. > >>>AFAICT it could (and so should) be completely userspace-defined which > >>>slave driver is used on which address. I imagine that for most > >>>controllers the bus addresses to use can be chosen more or less freely. > >>>So what am I missing? > >> > >>if you had read the footnotes you would know :-) Our usecase is connect an > >>embeedded controller via i2c to the host soc, similar to cros-ec, but here the > >>ec is the i2c master. The ec connects keyboard, mouse, pwrmngt, and other > >>stuff, for which the drivers are best implemented in kernel code AFAIK. > > > >Right, the driver might sensibly be implemented in kernel space. But I'd > >vote that you still need to do the binding of these drivers to your > >slave controller from userspace. Then there is no need to specify > >anything in your dtb. > > I think the set of I2C slave devices that are implemented by the > Linux system can reasonably be considered part of the HW definition. > Most DT content to date has been a definition of the HW that's > available to SW, but at least in this case, this I2C slave device is > something that must be implemented (admittedly in conjunction with > SW) using the I2C slave HW on the main SoC, in order for the overall > HW to work as intended. I'm not convinced. Why must it be implemented? And is it too late if the setup is done by userspace? The devicetree description of a flash chip doesn't include the rootfs content either, although a working rootfs is critical for most operating systems. The devicetree description of an ethernet adapter doesn't include its network setup although an ethernet adapter hardly makes any sense without an IP. > BTW, I believe devicetree-spec at vger.kernel.org was created to > address subsystem-level DT schema questions like this. It's much > lower volume, so perhaps the thread would get noticed by the DT > maintainers if posted there (or perhaps just CC them)? I added them to Cc. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-K?nig | Industrial Linux Solutions | http://www.pengutronix.de/ |