From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] pinctrl: single: Remove invalid message Date: Sun, 17 Dec 2017 08:07:40 -0800 Message-ID: <20171217160740.GJ3875@atomide.com> References: <20171214165115.27150-1-tony@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from muru.com ([72.249.23.125]:60720 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932188AbdLQQHo (ORCPT ); Sun, 17 Dec 2017 11:07:44 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Haojian Zhuang Cc: Linus Walleij , Grygorii Strashko , Nishanth Menon , linux-gpio@vger.kernel.org, linux-omap@vger.kernel.org * Haojian Zhuang [171217 05:43]: > On 2017/12/15 0:51, Tony Lindgren wrote: > > Pinctrl single should just show how many pins were found, the physical > > address is already in the dev information. So let's remove the wrong > > information that claims to show the physical address but really prints > > a virtual address that is now hashed. > > > > Signed-off-by: Tony Lindgren > > --- > > drivers/pinctrl/pinctrl-single.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c > > --- a/drivers/pinctrl/pinctrl-single.c > > +++ b/drivers/pinctrl/pinctrl-single.c > > @@ -1774,8 +1774,7 @@ static int pcs_probe(struct platform_device *pdev) > > dev_warn(pcs->dev, "initialized with no interrupts\n"); > > } > > - dev_info(pcs->dev, "%i pins at pa %p size %u\n", > > - pcs->desc.npins, pcs->base, pcs->size); > > + dev_info(pcs->dev, "%i pins, size %u\n", pcs->desc.npins, pcs->size); > > return pinctrl_enable(pcs->pctl); > > > Hi Tony, > > If there's only one pinctrl region in SoC, it's fine to remove the pa part. > But there're always multiple pinctrl regions in Hisilicon SoCs. It causes > multiple pinctrl register bases are used. Will it cause debug information > harder to read when pa parts are removed? I doubt as for years now we've been printing out the va instead of pa and so far nobody noticed :) We still get the real device pa with the dev_info, size and number of pins. Regards, Tony