From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shawn Guo Subject: Re: [PATCH v4 5/8] pinctrl: freescale: imx: add ZERO_OFFSET_VALID flag Date: Thu, 24 Sep 2015 05:33:36 -0700 Message-ID: <20150924123335.GT3529@tiger> References: <1442593798-11501-1-git-send-email-aalonso@freescale.com> <1442593798-11501-5-git-send-email-aalonso@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail.kernel.org ([198.145.29.136]:59686 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755044AbbIXMdm (ORCPT ); Thu, 24 Sep 2015 08:33:42 -0400 Content-Disposition: inline In-Reply-To: <1442593798-11501-5-git-send-email-aalonso@freescale.com> Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Adrian Alonso Cc: linux-arm-kernel@lists.infradead.org, shawn.guo@linaro.org, linus.walleij@linaro.org, lznuaa@gmail.com, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, robh+dt@kernel.org, Anson.Huang@freescale.com, Frank.Li@freescale.com, yibin.gong@freescale.com, nitin.garg@freescale.com On Fri, Sep 18, 2015 at 11:29:55AM -0500, Adrian Alonso wrote: > - Add ZERO_OFFSET_VALID flag, on imx7d mux_conf reg > offset is zero for iomuxc-lspr controller > - Do default initialization on parse group function. > > Signed-off-by: Adrian Alonso > --- > Changes for V2: Resend > Changes for V3: Resend > Changes for V4: Simplify pin_id assigment when ZERO_OFFSET_VALID is set > > drivers/pinctrl/freescale/pinctrl-imx.c | 20 ++++++++++---------- > drivers/pinctrl/freescale/pinctrl-imx.h | 1 + > 2 files changed, 11 insertions(+), 10 deletions(-) > > diff --git a/drivers/pinctrl/freescale/pinctrl-imx.c b/drivers/pinctrl/freescale/pinctrl-imx.c > index b9c6deb..866d864 100644 > --- a/drivers/pinctrl/freescale/pinctrl-imx.c > +++ b/drivers/pinctrl/freescale/pinctrl-imx.c > @@ -437,7 +437,7 @@ static void imx_pinconf_dbg_show(struct pinctrl_dev *pctldev, > const struct imx_pin_reg *pin_reg = &info->pin_regs[pin_id]; > unsigned long config; > > - if (!pin_reg || pin_reg->conf_reg == -1) { > + if (pin_reg->conf_reg == -1) { > seq_printf(s, "N/A"); > return; > } > @@ -536,21 +536,26 @@ static int imx_pinctrl_parse_groups(struct device_node *np, > return -ENOMEM; > > for (i = 0; i < grp->npins; i++) { > - u32 mux_reg = be32_to_cpu(*list++); > + u32 mux_reg; > u32 conf_reg; > unsigned int pin_id; > struct imx_pin_reg *pin_reg; > struct imx_pin *pin = &grp->pins[i]; > > + mux_reg = be32_to_cpu(*list++); > + if (!(info->flags & ZERO_OFFSET_VALID) && !mux_reg) > + mux_reg = -1; > + > if (info->flags & SHARE_MUX_CONF_REG) { > conf_reg = mux_reg; > } else { > conf_reg = be32_to_cpu(*list++); > - if (!conf_reg) > + if (!(info->flags & ZERO_OFFSET_VALID) && !conf_reg) > conf_reg = -1; > } > > - pin_id = mux_reg ? mux_reg / 4 : conf_reg / 4; > + pin_id = (mux_reg != -1) ? mux_reg / 4 : conf_reg / 4; > + This is the only meaningful change to me. I do not understand why any other changes in this patch is necessary. > pin_reg = &info->pin_regs[pin_id]; > pin->pin = pin_id; > grp->pin_ids[i] = pin_id; > @@ -684,7 +689,7 @@ int imx_pinctrl_probe(struct platform_device *pdev, > { > struct imx_pinctrl *ipctl; > struct resource *res; > - int ret, i; > + int ret; > > if (!info || !info->pins || !info->npins) { > dev_err(&pdev->dev, "wrong pinctrl info\n"); > @@ -702,11 +707,6 @@ int imx_pinctrl_probe(struct platform_device *pdev, > if (!info->pin_regs) > return -ENOMEM; > > - for (i = 0; i < info->npins; i++) { > - info->pin_regs[i].mux_reg = -1; > - info->pin_regs[i].conf_reg = -1; > - } > - > res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > ipctl->base = devm_ioremap_resource(&pdev->dev, res); > if (IS_ERR(ipctl->base)) > diff --git a/drivers/pinctrl/freescale/pinctrl-imx.h b/drivers/pinctrl/freescale/pinctrl-imx.h > index 2a5fe72..2a592f6 100644 > --- a/drivers/pinctrl/freescale/pinctrl-imx.h > +++ b/drivers/pinctrl/freescale/pinctrl-imx.h > @@ -85,6 +85,7 @@ struct imx_pinctrl_soc_info { > }; > > #define SHARE_MUX_CONF_REG 0x1 > +#define ZERO_OFFSET_VALID 0x2 > > #define NO_MUX 0x0 > #define NO_PAD 0x0 > -- > 2.1.4 >