From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-lf0-x232.google.com ([2a00:1450:4010:c07::232]) by bombadil.infradead.org with esmtps (Exim 4.89 #1 (Red Hat Linux)) id 1ecSlG-0003rE-CV for barebox@lists.infradead.org; Fri, 19 Jan 2018 09:15:00 +0000 Received: by mail-lf0-x232.google.com with SMTP id m8so1241555lfc.6 for ; Fri, 19 Jan 2018 01:14:43 -0800 (PST) Date: Fri, 19 Jan 2018 12:29:01 +0300 From: Antony Pavlov Message-Id: <20180119122901.d58ed91f24057139b2d4239c@gmail.com> In-Reply-To: <20180117095314.rwdrslwpnkcwa6ey@pengutronix.de> References: <20180114212252.29682-1-antonynpavlov@gmail.com> <20180117095314.rwdrslwpnkcwa6ey@pengutronix.de> Mime-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH v2 0/7] sandbox: add gpio support with libftdi1 To: Sascha Hauer Cc: barebox@lists.infradead.org On Wed, 17 Jan 2018 10:53:14 +0100 Sascha Hauer wrote: > Hi Antony, > = > On Mon, Jan 15, 2018 at 12:22:45AM +0300, Antony Pavlov wrote: > > This patch series makes it possible to use FT2232H ACBUS[7:0] > > pins as gpio pins from sandbox barebox. > > = > > There are ready-to-use FT2232H-based boards: > > = > > * FT2232H-56Q Mini-Module (http://www.ftdichip.com/Products/Modules/D= evelopmentModules.htm#FT2232H-56_Mini) > > * FT2232 breakout board (http://dangerousprototypes.com/docs/FT2232_b= reakout_board) > > = > > The main goal of adding gpio functionality to sandbox barebox > > is using it for connecting real i2c and spi devices to sandbox barebox. > = > I applied the parseopt patches for now since they don't need any further > discussions. > = There are three common parseopt-related patches: move parseopt to lib/ include/parseopt.h: add guard macro parseopt: introduce parseopt_u16() and parseopt_str() Please confirm that you have got all three patches. > What puzzles me a bit is that the libftdi gpio controller is > instantiated from the device tree but configured through the > commandline. It probably has to be like this since we need the > device tree for using the gpios. Getting FTDI chip vendorID and deviceID from device tree is reasonable. However hardcoding FTDI chip serialnumber into device tree is not so good i= dea. In addition dropping cmdline FTDI options parsing makes parseopt_u16() and = parseopt_str() unused. I propose add FTDI chip vendorID, deviceID and serial options to device tree and keep parsing the same options from cmdline so once can override FTDI ch= ip serial number from cmdline. > Sascha > = > -- = > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- = Best regards, =A0 Antony Pavlov _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox