From mboxrd@z Thu Jan 1 00:00:00 1970 From: aisheng.dong@freescale.com (Dong Aisheng) Date: Thu, 21 Jun 2012 10:13:59 +0800 Subject: [PATCH v2] pinctrl: pinctrl-imx: fix map setting problem if NO_PAD_CTL is set In-Reply-To: References: <1340187227-25343-1-git-send-email-jason77.wang@gmail.com> Message-ID: <20120621021358.GO10387@shlinux2.ap.freescale.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Jun 20, 2012 at 09:25:35PM +0200, Linus Walleij wrote: > On Wed, Jun 20, 2012 at 12:13 PM, Hui Wang wrote: > > > new_map is allocated according to map_num instead of grp->npins, > > if a pin or some pins of a group has NO_PAD_CTL property, the map_num > > and the grp->npin are different definitely. > > > > When we set mapping information to the new_map[], we should skip those > > pins with NO_PAD_CTL from index, otherwise it is possible the driver > > will aceesss to a unallocated region. > > > > Cc: Dong Aisheng > > Cc: Linus Walleij > > Cc: Shawn Guo > > Signed-off-by: Hui Wang > > --- > > In the v2, according to dong.aisheng's suggestion, change to use j as index. > > OK applied this v2 version. Is this a serious bug fix that is causing > regressions right now and needs to go into the -rc:s for v3.5 or can it wait > for v3.6? > Yes, it's a serious bug fix. Although still not triggered by current dts file but it did may block some legal using defined in binding doc. So can it go into rc? Regards Dong Aisheng