From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Date: Wed, 25 Sep 2013 04:51:53 +0000 Subject: Re: [PATCH 1/4] pinctrl: sh-pfc: r8a7790: add pin definitions for the I2C3 interface Message-Id: <20130925045152.GD24243@verge.net.au> List-Id: References: <1378742636-11215-1-git-send-email-g.liakhovetski@gmx.de> <1378742636-11215-2-git-send-email-g.liakhovetski@gmx.de> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Linus Walleij Cc: Guennadi Liakhovetski , Laurent Pinchart , "linux-kernel@vger.kernel.org" , Magnus Damm , "linux-sh@vger.kernel.org" , Guennadi Liakhovetski On Mon, Sep 23, 2013 at 10:50:07AM +0200, Linus Walleij wrote: > On Tue, Sep 17, 2013 at 7:10 PM, Guennadi Liakhovetski > wrote: > > On Tue, 17 Sep 2013, Linus Walleij wrote: > > >> > +/* R8A7790 has 6 banks with 32 GPIOs in each = 192 GPIOs */ > >> > +#define ROW_GROUP_A(r) ('Z' - 'A' + 1 + (r)) > >> > +#define PIN_NUMBER(r, c) (((r) - 'A') * 16 + (c) + 200) > >> > +#define PIN_A_NUMBER(r, c) PIN_NUMBER(ROW_GROUP_A(r), c) > >> > >> You add these #defines but do not use them. > > > > ehm, actually I do: > > Argh I was just wrong, forget about this. > > Woudl really like Laurent's ACK on this before I apply it though. Laurent, ping.