Re: [PATCH v2] hexagon: modify ffs() and fls() to return int
On Mon, Jul 23, 2018 at 04:27:47PM -0700, Randy Dunlap wrote: > On 07/23/2018 03:50 PM, Richard Kuo wrote: > > On Sun, Jul 22, 2018 at 04:03:58PM -0700, Randy Dunlap wrote: > >> From: Randy Dunlap > >> > >> Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a > >> printk format build warning. This is due to hexagon's ffs() being > >> coded as returning long instead of int. > >> > >> Fix the printk format warning by changing all of hexagon's ffs() and > >> fls() functions to return int instead of long. The variables that > >> they return are already int instead of long. This return type > >> matches the return type in . > >> > >> ../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim': > >> ../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects > >> argument of type 'unsigned int', but argument 2 has type 'long int' > >> [-Wformat] > >> > >> There are no ffs() or fls() allmodconfig build errors after making this > >> change. > >> > >> Signed-off-by: Randy Dunlap > >> Cc: Richard Kuo > >> Cc: linux-hexa...@vger.kernel.org > >> Cc: Geert Uytterhoeven > >> --- > >> v2: > >> add hexagon contacts, drop erroneous sh contacts; [thanks, Geert] > >> only change return type for ffs() and fls() [thanks, Geert] > >> [drop the changes for ffz(), __ffs(), and __fls()] > >> > >> arch/hexagon/include/asm/bitops.h |4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > >> > > > > > > Acked-by: Richard Kuo > > > > Hi Richard, > > You are listed as the arch/hexagon/ maintainer. Can you please merge these > patches? > > thanks, > -- > ~Randy Yes, I can queue it up and take it through my tree. Thanks, Richard Kuo -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH v2] hexagon: modify ffs() and fls() to return int
On Mon, Jul 23, 2018 at 04:27:47PM -0700, Randy Dunlap wrote: > On 07/23/2018 03:50 PM, Richard Kuo wrote: > > On Sun, Jul 22, 2018 at 04:03:58PM -0700, Randy Dunlap wrote: > >> From: Randy Dunlap > >> > >> Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a > >> printk format build warning. This is due to hexagon's ffs() being > >> coded as returning long instead of int. > >> > >> Fix the printk format warning by changing all of hexagon's ffs() and > >> fls() functions to return int instead of long. The variables that > >> they return are already int instead of long. This return type > >> matches the return type in . > >> > >> ../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim': > >> ../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects > >> argument of type 'unsigned int', but argument 2 has type 'long int' > >> [-Wformat] > >> > >> There are no ffs() or fls() allmodconfig build errors after making this > >> change. > >> > >> Signed-off-by: Randy Dunlap > >> Cc: Richard Kuo > >> Cc: linux-hexa...@vger.kernel.org > >> Cc: Geert Uytterhoeven > >> --- > >> v2: > >> add hexagon contacts, drop erroneous sh contacts; [thanks, Geert] > >> only change return type for ffs() and fls() [thanks, Geert] > >> [drop the changes for ffz(), __ffs(), and __fls()] > >> > >> arch/hexagon/include/asm/bitops.h |4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > >> > > > > > > Acked-by: Richard Kuo > > > > Hi Richard, > > You are listed as the arch/hexagon/ maintainer. Can you please merge these > patches? > > thanks, > -- > ~Randy Yes, I can queue it up and take it through my tree. Thanks, Richard Kuo -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH v2] hexagon: modify ffs() and fls() to return int
On 07/23/2018 03:50 PM, Richard Kuo wrote: > On Sun, Jul 22, 2018 at 04:03:58PM -0700, Randy Dunlap wrote: >> From: Randy Dunlap >> >> Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a >> printk format build warning. This is due to hexagon's ffs() being >> coded as returning long instead of int. >> >> Fix the printk format warning by changing all of hexagon's ffs() and >> fls() functions to return int instead of long. The variables that >> they return are already int instead of long. This return type >> matches the return type in . >> >> ../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim': >> ../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects >> argument of type 'unsigned int', but argument 2 has type 'long int' >> [-Wformat] >> >> There are no ffs() or fls() allmodconfig build errors after making this >> change. >> >> Signed-off-by: Randy Dunlap >> Cc: Richard Kuo >> Cc: linux-hexa...@vger.kernel.org >> Cc: Geert Uytterhoeven >> --- >> v2: >> add hexagon contacts, drop erroneous sh contacts; [thanks, Geert] >> only change return type for ffs() and fls() [thanks, Geert] >> [drop the changes for ffz(), __ffs(), and __fls()] >> >> arch/hexagon/include/asm/bitops.h |4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> > > > Acked-by: Richard Kuo > Hi Richard, You are listed as the arch/hexagon/ maintainer. Can you please merge these patches? thanks, -- ~Randy
Re: [PATCH v2] hexagon: modify ffs() and fls() to return int
On 07/23/2018 03:50 PM, Richard Kuo wrote: > On Sun, Jul 22, 2018 at 04:03:58PM -0700, Randy Dunlap wrote: >> From: Randy Dunlap >> >> Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a >> printk format build warning. This is due to hexagon's ffs() being >> coded as returning long instead of int. >> >> Fix the printk format warning by changing all of hexagon's ffs() and >> fls() functions to return int instead of long. The variables that >> they return are already int instead of long. This return type >> matches the return type in . >> >> ../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim': >> ../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects >> argument of type 'unsigned int', but argument 2 has type 'long int' >> [-Wformat] >> >> There are no ffs() or fls() allmodconfig build errors after making this >> change. >> >> Signed-off-by: Randy Dunlap >> Cc: Richard Kuo >> Cc: linux-hexa...@vger.kernel.org >> Cc: Geert Uytterhoeven >> --- >> v2: >> add hexagon contacts, drop erroneous sh contacts; [thanks, Geert] >> only change return type for ffs() and fls() [thanks, Geert] >> [drop the changes for ffz(), __ffs(), and __fls()] >> >> arch/hexagon/include/asm/bitops.h |4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> > > > Acked-by: Richard Kuo > Hi Richard, You are listed as the arch/hexagon/ maintainer. Can you please merge these patches? thanks, -- ~Randy
Re: [PATCH v2] hexagon: modify ffs() and fls() to return int
On Sun, Jul 22, 2018 at 04:03:58PM -0700, Randy Dunlap wrote: > From: Randy Dunlap > > Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a > printk format build warning. This is due to hexagon's ffs() being > coded as returning long instead of int. > > Fix the printk format warning by changing all of hexagon's ffs() and > fls() functions to return int instead of long. The variables that > they return are already int instead of long. This return type > matches the return type in . > > ../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim': > ../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects > argument of type 'unsigned int', but argument 2 has type 'long int' [-Wformat] > > There are no ffs() or fls() allmodconfig build errors after making this > change. > > Signed-off-by: Randy Dunlap > Cc: Richard Kuo > Cc: linux-hexa...@vger.kernel.org > Cc: Geert Uytterhoeven > --- > v2: > add hexagon contacts, drop erroneous sh contacts; [thanks, Geert] > only change return type for ffs() and fls() [thanks, Geert] > [drop the changes for ffz(), __ffs(), and __fls()] > > arch/hexagon/include/asm/bitops.h |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Acked-by: Richard Kuo -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH v2] hexagon: modify ffs() and fls() to return int
On Sun, Jul 22, 2018 at 04:03:58PM -0700, Randy Dunlap wrote: > From: Randy Dunlap > > Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a > printk format build warning. This is due to hexagon's ffs() being > coded as returning long instead of int. > > Fix the printk format warning by changing all of hexagon's ffs() and > fls() functions to return int instead of long. The variables that > they return are already int instead of long. This return type > matches the return type in . > > ../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim': > ../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects > argument of type 'unsigned int', but argument 2 has type 'long int' [-Wformat] > > There are no ffs() or fls() allmodconfig build errors after making this > change. > > Signed-off-by: Randy Dunlap > Cc: Richard Kuo > Cc: linux-hexa...@vger.kernel.org > Cc: Geert Uytterhoeven > --- > v2: > add hexagon contacts, drop erroneous sh contacts; [thanks, Geert] > only change return type for ffs() and fls() [thanks, Geert] > [drop the changes for ffz(), __ffs(), and __fls()] > > arch/hexagon/include/asm/bitops.h |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Acked-by: Richard Kuo -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH v2] hexagon: modify ffs() and fls() to return int
On Mon, Jul 23, 2018 at 1:04 AM Randy Dunlap wrote:
> From: Randy Dunlap
>
> Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a
> printk format build warning. This is due to hexagon's ffs() being
> coded as returning long instead of int.
>
> Fix the printk format warning by changing all of hexagon's ffs() and
> fls() functions to return int instead of long. The variables that
> they return are already int instead of long. This return type
> matches the return type in .
>
> ../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim':
> ../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects
> argument of type 'unsigned int', but argument 2 has type 'long int' [-Wformat]
>
> There are no ffs() or fls() allmodconfig build errors after making this
> change.
>
> Signed-off-by: Randy Dunlap
> Cc: Richard Kuo
> Cc: linux-hexa...@vger.kernel.org
> Cc: Geert Uytterhoeven
Reviewed-by: Geert Uytterhoeven
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Re: [PATCH v2] hexagon: modify ffs() and fls() to return int
On Mon, Jul 23, 2018 at 1:04 AM Randy Dunlap wrote:
> From: Randy Dunlap
>
> Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a
> printk format build warning. This is due to hexagon's ffs() being
> coded as returning long instead of int.
>
> Fix the printk format warning by changing all of hexagon's ffs() and
> fls() functions to return int instead of long. The variables that
> they return are already int instead of long. This return type
> matches the return type in .
>
> ../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim':
> ../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects
> argument of type 'unsigned int', but argument 2 has type 'long int' [-Wformat]
>
> There are no ffs() or fls() allmodconfig build errors after making this
> change.
>
> Signed-off-by: Randy Dunlap
> Cc: Richard Kuo
> Cc: linux-hexa...@vger.kernel.org
> Cc: Geert Uytterhoeven
Reviewed-by: Geert Uytterhoeven
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
[PATCH v2] hexagon: modify ffs() and fls() to return int
From: Randy Dunlap
Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a
printk format build warning. This is due to hexagon's ffs() being
coded as returning long instead of int.
Fix the printk format warning by changing all of hexagon's ffs() and
fls() functions to return int instead of long. The variables that
they return are already int instead of long. This return type
matches the return type in .
../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim':
../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects argument
of type 'unsigned int', but argument 2 has type 'long int' [-Wformat]
There are no ffs() or fls() allmodconfig build errors after making this
change.
Signed-off-by: Randy Dunlap
Cc: Richard Kuo
Cc: linux-hexa...@vger.kernel.org
Cc: Geert Uytterhoeven
---
v2:
add hexagon contacts, drop erroneous sh contacts; [thanks, Geert]
only change return type for ffs() and fls() [thanks, Geert]
[drop the changes for ffz(), __ffs(), and __fls()]
arch/hexagon/include/asm/bitops.h |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--- linux-next-20180717.orig/arch/hexagon/include/asm/bitops.h
+++ linux-next-20180717/arch/hexagon/include/asm/bitops.h
@@ -211,7 +211,7 @@ static inline long ffz(int x)
* This is defined the same way as ffs.
* Note fls(0) = 0, fls(1) = 1, fls(0x8000) = 32.
*/
-static inline long fls(int x)
+static inline int fls(int x)
{
int r;
@@ -232,7 +232,7 @@ static inline long fls(int x)
* the libc and compiler builtin ffs routines, therefore
* differs in spirit from the above ffz (man ffs).
*/
-static inline long ffs(int x)
+static inline int ffs(int x)
{
int r;
[PATCH v2] hexagon: modify ffs() and fls() to return int
From: Randy Dunlap
Building drivers/mtd/nand/raw/nandsim.c on arch/hexagon/ produces a
printk format build warning. This is due to hexagon's ffs() being
coded as returning long instead of int.
Fix the printk format warning by changing all of hexagon's ffs() and
fls() functions to return int instead of long. The variables that
they return are already int instead of long. This return type
matches the return type in .
../drivers/mtd/nand/raw/nandsim.c: In function 'init_nandsim':
../drivers/mtd/nand/raw/nandsim.c:760:2: warning: format '%u' expects argument
of type 'unsigned int', but argument 2 has type 'long int' [-Wformat]
There are no ffs() or fls() allmodconfig build errors after making this
change.
Signed-off-by: Randy Dunlap
Cc: Richard Kuo
Cc: linux-hexa...@vger.kernel.org
Cc: Geert Uytterhoeven
---
v2:
add hexagon contacts, drop erroneous sh contacts; [thanks, Geert]
only change return type for ffs() and fls() [thanks, Geert]
[drop the changes for ffz(), __ffs(), and __fls()]
arch/hexagon/include/asm/bitops.h |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--- linux-next-20180717.orig/arch/hexagon/include/asm/bitops.h
+++ linux-next-20180717/arch/hexagon/include/asm/bitops.h
@@ -211,7 +211,7 @@ static inline long ffz(int x)
* This is defined the same way as ffs.
* Note fls(0) = 0, fls(1) = 1, fls(0x8000) = 32.
*/
-static inline long fls(int x)
+static inline int fls(int x)
{
int r;
@@ -232,7 +232,7 @@ static inline long fls(int x)
* the libc and compiler builtin ffs routines, therefore
* differs in spirit from the above ffz (man ffs).
*/
-static inline long ffs(int x)
+static inline int ffs(int x)
{
int r;
