Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 3 Feb 2022 15:40:21 +0100
From:   Jonathan =?utf-8?Q?Neusch=C3=A4fer?= <j.neuschaefer@gmx.net>
To:     Andy Shevchenko <andy.shevchenko@gmail.com>
Cc:     Jonathan =?utf-8?Q?Neusch=C3=A4fer?= <j.neuschaefer@gmx.net>,
        "open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Joel Stanley <joel@jms.id.au>,
        kernel test robot <lkp@intel.com>,
        Avi Fishman <avifishman70@gmail.com>,
        Tomer Maimon <tmaimon77@gmail.com>,
        Tali Perry <tali.perry1@gmail.com>,
        Patrick Venture <venture@google.com>,
        Nancy Yuen <yuenn@google.com>,
        Benjamin Fair <benjaminfair@google.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        OpenBMC Maillist <openbmc@lists.ozlabs.org>
Subject: Re: [PATCH v2 1/2] pinctrl: nuvoton: npcm7xx: Use %zd printk format
 for ARRAY_SIZE()
Message-ID: <YfvpVVigOc4eEdR3@latitude>
References: <20220203133736.246397-1-j.neuschaefer@gmx.net>
 <20220203133736.246397-2-j.neuschaefer@gmx.net>
 <CAHp75VfrygdyjgQJ7iRnGL-CELCQ+6D30r5aWwitCTUJvGVf_g@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="eVcSzelghXg5He3N"
Content-Disposition: inline
In-Reply-To: <CAHp75VfrygdyjgQJ7iRnGL-CELCQ+6D30r5aWwitCTUJvGVf_g@mail.gmail.com>
Precedence: bulk


--eVcSzelghXg5He3N
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Feb 03, 2022 at 04:16:56PM +0200, Andy Shevchenko wrote:
> On Thu, Feb 3, 2022 at 3:37 PM Jonathan Neusch=C3=A4fer
> <j.neuschaefer@gmx.net> wrote:
> >
> > When compile-testing on 64-bit architectures, GCC complains about the
> > mismatch of types between the %d format specifier and value returned by
> > ARRAY_LENGTH(). Use %zd, which is correct everywhere.
> >
> > Reported-by: kernel test robot <lkp@intel.com>
> > Fixes: 3b588e43ee5c7 ("pinctrl: nuvoton: add NPCM7xx pinctrl and GPIO d=
river")
> > Signed-off-by: Jonathan Neusch=C3=A4fer <j.neuschaefer@gmx.net>
> > Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
>=20
> ...
>=20
> > -       dev_dbg(npcm->dev, "group size: %d\n", ARRAY_SIZE(npcm7xx_group=
s));
> > +       dev_dbg(npcm->dev, "group size: %zd\n", ARRAY_SIZE(npcm7xx_grou=
ps));
>=20
> Dunno how it's slipped away, but the proper specifier is %zu.

Right, I should have noticed that.

> I guess that's what I also mentioned in reply to LKP's report.

True,

>> Should be %zu


Thanks,
Jonathan

--eVcSzelghXg5He3N
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEvHAHGBBjQPVy+qvDCDBEmo7zX9sFAmH76TMACgkQCDBEmo7z
X9ut4A//aOQLQWn/GISMFzJ4/Pq/g4sc9jjB+rZZhAxaAYaJXKdOoIq++a00Ng0/
MUq7qOV18RJQiA/KYLud7LOmRNsbQXGNzDo1Z4TcplE/MjJr2VsJe+OuXO3fzvNN
oJmEtfn0k3cwbiAPTkF00izWfjMGHfou/0xRX+NDoJybCt9eVwj8SUUW8fPdhwTC
smiOUgLo39wHTlvP6WQiLSHu1RP53qe5aiqL11ZCmeSHMzH/vlQcG0jhBb4/46AR
rB1wnbMEFap3VoZtRs73al6Ze+wPdmdL6sUB4e8L+5d59LDi9ROe02DbamgMf0VB
C+sDv3GsemHp62J/id20G8eFit9C/xSyjVmng7uHMxWt75issT62gIxrfK2P81q+
1dTEH64apssen6L1w4NiPs0pxzKmQ3vzO5FhypsaqHv/Qgc+C/nEu7drwaJ4MtSs
nXZR+QU2vSKBxR5GK3ezz4UIuZKU8ZOYVlryQPm+wX4MBDumeH41RPqdSX3U/OZf
OmCnFxWxTOFxc2aMt1JnKSxqMMFqLtQ18EDrj0WlxMOiW+f2f4RmS4yNAOyU0h5J
ToH0Undp8fPD3fCsIJhI+io7VR13Jq0pJVqG0XbnEd2ljRcL9fRLaGf1ggwT3Ork
ZKST+SfL/krulH/yaPmmB4ty2/1mDvNgWMXlc1zbeNwWmNBjsQU=
=r2Xy
-----END PGP SIGNATURE-----

--eVcSzelghXg5He3N--