From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mika Westerberg Subject: Re: [PATCH] pinctrl: intel: Do pin translation in other GPIO operations as well Date: Wed, 19 Sep 2018 11:40:02 +0300 Message-ID: <20180919084002.GR14465@lahna.fi.intel.com> References: <20180918153621.71984-1-mika.westerberg@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Rajat Jain Cc: Andy Shevchenko , Linus Walleij , casey.g.bowman@intel.com, "Atwood, Matthew S" , linux-gpio@vger.kernel.org, Linux Kernel Mailing List List-Id: linux-gpio@vger.kernel.org On Tue, Sep 18, 2018 at 03:14:44PM -0700, Rajat Jain wrote: > Also consider fixing the checkpatch warning: > > Errors: > * checkpatch.pl errors/warnings > > WARNING: Prefer 'unsigned int' to bare use of 'unsigned' > #48: FILE: drivers/pinctrl/intel/pinctrl-intel.c:764: > +static int intel_gpio_to_pin(struct intel_pinctrl *pctrl, > unsigned offset, Right, it is currently the "convention" used in Intel pinctrl drivers so I did not want to change that single entry. We should eventually convert the whole set of Intel pinctrl drivers to use unsigned int instead of unsigned.