* [PATCH] squashfs: Fix compilation on big endian systems @ 2022-04-06 21:31 Pali Rohár 2022-04-07 7:54 ` Miquel Raynal ` (2 more replies) 0 siblings, 3 replies; 10+ messages in thread From: Pali Rohár @ 2022-04-06 21:31 UTC (permalink / raw) To: Joao Marcos Costa, Thomas Petazzoni, Miquel Raynal; +Cc: u-boot Signed-off-by: Pali Rohár <pali@kernel.org> --- fs/squashfs/sqfs.c | 3 +-- fs/squashfs/sqfs_dir.c | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c index 5d9c52af80ba..41cb811c1b32 100644 --- a/fs/squashfs/sqfs.c +++ b/fs/squashfs/sqfs.c @@ -11,8 +11,7 @@ #include <errno.h> #include <fs.h> #include <linux/types.h> -#include <linux/byteorder/little_endian.h> -#include <linux/byteorder/generic.h> +#include <asm/byteorder.h> #include <memalign.h> #include <stdlib.h> #include <string.h> diff --git a/fs/squashfs/sqfs_dir.c b/fs/squashfs/sqfs_dir.c index a265b98fe685..ed83c90682ff 100644 --- a/fs/squashfs/sqfs_dir.c +++ b/fs/squashfs/sqfs_dir.c @@ -7,8 +7,7 @@ #include <errno.h> #include <linux/types.h> -#include <linux/byteorder/little_endian.h> -#include <linux/byteorder/generic.h> +#include <asm/byteorder.h> #include <stdint.h> #include <stdio.h> #include <stdlib.h> -- 2.20.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-04-06 21:31 [PATCH] squashfs: Fix compilation on big endian systems Pali Rohár @ 2022-04-07 7:54 ` Miquel Raynal 2022-04-07 9:41 ` Pali Rohár 2022-05-10 14:33 ` Pali Rohár 2022-06-03 19:48 ` Tom Rini 2 siblings, 1 reply; 10+ messages in thread From: Miquel Raynal @ 2022-04-07 7:54 UTC (permalink / raw) To: Pali Rohár; +Cc: Joao Marcos Costa, Thomas Petazzoni, u-boot Hi Pali, pali@kernel.org wrote on Wed, 6 Apr 2022 23:31:53 +0200: Would you mind explaining a little bit how this change fixes it? It does not look straightforward to me. > Signed-off-by: Pali Rohár <pali@kernel.org> > --- > fs/squashfs/sqfs.c | 3 +-- > fs/squashfs/sqfs_dir.c | 3 +-- > 2 files changed, 2 insertions(+), 4 deletions(-) > > diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c > index 5d9c52af80ba..41cb811c1b32 100644 > --- a/fs/squashfs/sqfs.c > +++ b/fs/squashfs/sqfs.c > @@ -11,8 +11,7 @@ > #include <errno.h> > #include <fs.h> > #include <linux/types.h> > -#include <linux/byteorder/little_endian.h> > -#include <linux/byteorder/generic.h> > +#include <asm/byteorder.h> > #include <memalign.h> > #include <stdlib.h> > #include <string.h> > diff --git a/fs/squashfs/sqfs_dir.c b/fs/squashfs/sqfs_dir.c > index a265b98fe685..ed83c90682ff 100644 > --- a/fs/squashfs/sqfs_dir.c > +++ b/fs/squashfs/sqfs_dir.c > @@ -7,8 +7,7 @@ > > #include <errno.h> > #include <linux/types.h> > -#include <linux/byteorder/little_endian.h> > -#include <linux/byteorder/generic.h> > +#include <asm/byteorder.h> > #include <stdint.h> > #include <stdio.h> > #include <stdlib.h> Cheers, Miquèl ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-04-07 7:54 ` Miquel Raynal @ 2022-04-07 9:41 ` Pali Rohár 2022-04-07 9:54 ` Miquel Raynal 0 siblings, 1 reply; 10+ messages in thread From: Pali Rohár @ 2022-04-07 9:41 UTC (permalink / raw) To: Miquel Raynal; +Cc: Joao Marcos Costa, Thomas Petazzoni, u-boot On Thursday 07 April 2022 09:54:21 Miquel Raynal wrote: > Hi Pali, > > pali@kernel.org wrote on Wed, 6 Apr 2022 23:31:53 +0200: > > Would you mind explaining a little bit how this change fixes it? It > does not look straightforward to me. Yes! I though that it is straightforward this change... byteorder/little_endian.h defines cpu_to_le* macros for Little Endian systems and byteorder/big_endian.h for Big Endian systems. File asm/byteorder.h is then ARCH-specific and implements macros for the current architecture (by including the correct header file). So currently if you try to compile squashfs for big endian systems you get compile error: In file included from ./arch/powerpc/include/asm/byteorder.h:82, from include/linux/unaligned/access_ok.h:4, from ./arch/powerpc/include/asm/unaligned.h:9, from fs/squashfs/sqfs_filesystem.h:11, from fs/squashfs/sqfs_dir.c:16: include/linux/byteorder/big_endian.h:34: warning: "__cpu_to_le32" redefined #define __cpu_to_le32(x) ((__force __le32)__swab32((x))) In file included from fs/squashfs/sqfs_dir.c:10: include/linux/byteorder/little_endian.h:34: note: this is the location of the previous definition #define __cpu_to_le32(x) ((__force __le32)(__u32)(x)) Or: In file included from fs/squashfs/sqfs.c:14: include/linux/byteorder/little_endian.h:89:21: error: redefinition of ‘__be16_to_cpup’ static inline __u16 __be16_to_cpup(const __be16 *p) ^~~~~~~~~~~~~~ In file included from ./arch/powerpc/include/asm/byteorder.h:82, from include/linux/unaligned/access_ok.h:4, from ./arch/powerpc/include/asm/unaligned.h:9, from fs/squashfs/sqfs.c:10: include/linux/byteorder/big_endian.h:89:21: note: previous definition of ‘__be16_to_cpup’ was here static inline __u16 __be16_to_cpup(const __be16 *p) ^~~~~~~~~~~~~~ As some header files include correct asm/byteorder.h file and this squashfs includes additional little_endian.h. > > Signed-off-by: Pali Rohár <pali@kernel.org> > > --- > > fs/squashfs/sqfs.c | 3 +-- > > fs/squashfs/sqfs_dir.c | 3 +-- > > 2 files changed, 2 insertions(+), 4 deletions(-) > > > > diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c > > index 5d9c52af80ba..41cb811c1b32 100644 > > --- a/fs/squashfs/sqfs.c > > +++ b/fs/squashfs/sqfs.c > > @@ -11,8 +11,7 @@ > > #include <errno.h> > > #include <fs.h> > > #include <linux/types.h> > > -#include <linux/byteorder/little_endian.h> > > -#include <linux/byteorder/generic.h> > > +#include <asm/byteorder.h> > > #include <memalign.h> > > #include <stdlib.h> > > #include <string.h> > > diff --git a/fs/squashfs/sqfs_dir.c b/fs/squashfs/sqfs_dir.c > > index a265b98fe685..ed83c90682ff 100644 > > --- a/fs/squashfs/sqfs_dir.c > > +++ b/fs/squashfs/sqfs_dir.c > > @@ -7,8 +7,7 @@ > > > > #include <errno.h> > > #include <linux/types.h> > > -#include <linux/byteorder/little_endian.h> > > -#include <linux/byteorder/generic.h> > > +#include <asm/byteorder.h> > > #include <stdint.h> > > #include <stdio.h> > > #include <stdlib.h> > > Cheers, > Miquèl ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-04-07 9:41 ` Pali Rohár @ 2022-04-07 9:54 ` Miquel Raynal 2022-04-07 9:58 ` Pali Rohár 0 siblings, 1 reply; 10+ messages in thread From: Miquel Raynal @ 2022-04-07 9:54 UTC (permalink / raw) To: Pali Rohár; +Cc: Joao Marcos Costa, Thomas Petazzoni, u-boot Hi Pali, pali@kernel.org wrote on Thu, 7 Apr 2022 11:41:59 +0200: > On Thursday 07 April 2022 09:54:21 Miquel Raynal wrote: > > Hi Pali, > > > > pali@kernel.org wrote on Wed, 6 Apr 2022 23:31:53 +0200: > > > > Would you mind explaining a little bit how this change fixes it? It > > does not look straightforward to me. > > Yes! I though that it is straightforward this change... > byteorder/little_endian.h defines cpu_to_le* macros for Little Endian > systems and byteorder/big_endian.h for Big Endian systems. > > File asm/byteorder.h is then ARCH-specific and implements macros for the > current architecture (by including the correct header file). > > So currently if you try to compile squashfs for big endian systems you > get compile error: > > In file included from ./arch/powerpc/include/asm/byteorder.h:82, > from include/linux/unaligned/access_ok.h:4, > from ./arch/powerpc/include/asm/unaligned.h:9, > from fs/squashfs/sqfs_filesystem.h:11, > from fs/squashfs/sqfs_dir.c:16: > include/linux/byteorder/big_endian.h:34: warning: "__cpu_to_le32" redefined > #define __cpu_to_le32(x) ((__force __le32)__swab32((x))) > > In file included from fs/squashfs/sqfs_dir.c:10: > include/linux/byteorder/little_endian.h:34: note: this is the location of the previous definition > #define __cpu_to_le32(x) ((__force __le32)(__u32)(x)) > > Or: > > In file included from fs/squashfs/sqfs.c:14: > include/linux/byteorder/little_endian.h:89:21: error: redefinition of ‘__be16_to_cpup’ > static inline __u16 __be16_to_cpup(const __be16 *p) > ^~~~~~~~~~~~~~ > In file included from ./arch/powerpc/include/asm/byteorder.h:82, > from include/linux/unaligned/access_ok.h:4, > from ./arch/powerpc/include/asm/unaligned.h:9, > from fs/squashfs/sqfs.c:10: > include/linux/byteorder/big_endian.h:89:21: note: previous definition of ‘__be16_to_cpup’ was here > static inline __u16 __be16_to_cpup(const __be16 *p) > ^~~~~~~~~~~~~~ > > As some header files include correct asm/byteorder.h file and this > squashfs includes additional little_endian.h. Great, thanks for the thorough explanation. Based on what you said, wouldn't it be cleaner to just get rid of the little_endian.h include rather than also use the ARCH specific byteorder.h header? > > > > Signed-off-by: Pali Rohár <pali@kernel.org> > > > --- > > > fs/squashfs/sqfs.c | 3 +-- > > > fs/squashfs/sqfs_dir.c | 3 +-- > > > 2 files changed, 2 insertions(+), 4 deletions(-) > > > > > > diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c > > > index 5d9c52af80ba..41cb811c1b32 100644 > > > --- a/fs/squashfs/sqfs.c > > > +++ b/fs/squashfs/sqfs.c > > > @@ -11,8 +11,7 @@ > > > #include <errno.h> > > > #include <fs.h> > > > #include <linux/types.h> > > > -#include <linux/byteorder/little_endian.h> > > > -#include <linux/byteorder/generic.h> > > > +#include <asm/byteorder.h> > > > #include <memalign.h> > > > #include <stdlib.h> > > > #include <string.h> > > > diff --git a/fs/squashfs/sqfs_dir.c b/fs/squashfs/sqfs_dir.c > > > index a265b98fe685..ed83c90682ff 100644 > > > --- a/fs/squashfs/sqfs_dir.c > > > +++ b/fs/squashfs/sqfs_dir.c > > > @@ -7,8 +7,7 @@ > > > > > > #include <errno.h> > > > #include <linux/types.h> > > > -#include <linux/byteorder/little_endian.h> > > > -#include <linux/byteorder/generic.h> > > > +#include <asm/byteorder.h> > > > #include <stdint.h> > > > #include <stdio.h> > > > #include <stdlib.h> > > > > Cheers, > > Miquèl Thanks, Miquèl ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-04-07 9:54 ` Miquel Raynal @ 2022-04-07 9:58 ` Pali Rohár 2022-06-01 14:08 ` Tom Rini 0 siblings, 1 reply; 10+ messages in thread From: Pali Rohár @ 2022-04-07 9:58 UTC (permalink / raw) To: Miquel Raynal; +Cc: Joao Marcos Costa, Thomas Petazzoni, u-boot On Thursday 07 April 2022 11:54:55 Miquel Raynal wrote: > Hi Pali, > > pali@kernel.org wrote on Thu, 7 Apr 2022 11:41:59 +0200: > > > On Thursday 07 April 2022 09:54:21 Miquel Raynal wrote: > > > Hi Pali, > > > > > > pali@kernel.org wrote on Wed, 6 Apr 2022 23:31:53 +0200: > > > > > > Would you mind explaining a little bit how this change fixes it? It > > > does not look straightforward to me. > > > > Yes! I though that it is straightforward this change... > > byteorder/little_endian.h defines cpu_to_le* macros for Little Endian > > systems and byteorder/big_endian.h for Big Endian systems. > > > > File asm/byteorder.h is then ARCH-specific and implements macros for the > > current architecture (by including the correct header file). > > > > So currently if you try to compile squashfs for big endian systems you > > get compile error: > > > > In file included from ./arch/powerpc/include/asm/byteorder.h:82, > > from include/linux/unaligned/access_ok.h:4, > > from ./arch/powerpc/include/asm/unaligned.h:9, > > from fs/squashfs/sqfs_filesystem.h:11, > > from fs/squashfs/sqfs_dir.c:16: > > include/linux/byteorder/big_endian.h:34: warning: "__cpu_to_le32" redefined > > #define __cpu_to_le32(x) ((__force __le32)__swab32((x))) > > > > In file included from fs/squashfs/sqfs_dir.c:10: > > include/linux/byteorder/little_endian.h:34: note: this is the location of the previous definition > > #define __cpu_to_le32(x) ((__force __le32)(__u32)(x)) > > > > Or: > > > > In file included from fs/squashfs/sqfs.c:14: > > include/linux/byteorder/little_endian.h:89:21: error: redefinition of ‘__be16_to_cpup’ > > static inline __u16 __be16_to_cpup(const __be16 *p) > > ^~~~~~~~~~~~~~ > > In file included from ./arch/powerpc/include/asm/byteorder.h:82, > > from include/linux/unaligned/access_ok.h:4, > > from ./arch/powerpc/include/asm/unaligned.h:9, > > from fs/squashfs/sqfs.c:10: > > include/linux/byteorder/big_endian.h:89:21: note: previous definition of ‘__be16_to_cpup’ was here > > static inline __u16 __be16_to_cpup(const __be16 *p) > > ^~~~~~~~~~~~~~ > > > > As some header files include correct asm/byteorder.h file and this > > squashfs includes additional little_endian.h. > > Great, thanks for the thorough explanation. Based on what you said, > wouldn't it be cleaner to just get rid of the little_endian.h include > rather than also use the ARCH specific byteorder.h header? I think that this is not possible. squashfs code uses le16_to_cpu() macro and its correct definition is provided only by arch specific asm/byteorder.h. > > > > > > Signed-off-by: Pali Rohár <pali@kernel.org> > > > > --- > > > > fs/squashfs/sqfs.c | 3 +-- > > > > fs/squashfs/sqfs_dir.c | 3 +-- > > > > 2 files changed, 2 insertions(+), 4 deletions(-) > > > > > > > > diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c > > > > index 5d9c52af80ba..41cb811c1b32 100644 > > > > --- a/fs/squashfs/sqfs.c > > > > +++ b/fs/squashfs/sqfs.c > > > > @@ -11,8 +11,7 @@ > > > > #include <errno.h> > > > > #include <fs.h> > > > > #include <linux/types.h> > > > > -#include <linux/byteorder/little_endian.h> > > > > -#include <linux/byteorder/generic.h> > > > > +#include <asm/byteorder.h> > > > > #include <memalign.h> > > > > #include <stdlib.h> > > > > #include <string.h> > > > > diff --git a/fs/squashfs/sqfs_dir.c b/fs/squashfs/sqfs_dir.c > > > > index a265b98fe685..ed83c90682ff 100644 > > > > --- a/fs/squashfs/sqfs_dir.c > > > > +++ b/fs/squashfs/sqfs_dir.c > > > > @@ -7,8 +7,7 @@ > > > > > > > > #include <errno.h> > > > > #include <linux/types.h> > > > > -#include <linux/byteorder/little_endian.h> > > > > -#include <linux/byteorder/generic.h> > > > > +#include <asm/byteorder.h> > > > > #include <stdint.h> > > > > #include <stdio.h> > > > > #include <stdlib.h> > > > > > > Cheers, > > > Miquèl > > > Thanks, > Miquèl ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-04-07 9:58 ` Pali Rohár @ 2022-06-01 14:08 ` Tom Rini 2022-06-03 14:24 ` Miquel Raynal 0 siblings, 1 reply; 10+ messages in thread From: Tom Rini @ 2022-06-01 14:08 UTC (permalink / raw) To: Pali Rohár, Miquel Raynal Cc: wJoao Marcos Costa, Thomas Petazzoni, u-boot [-- Attachment #1: Type: text/plain, Size: 3081 bytes --] On Thu, Apr 07, 2022 at 11:58:55AM +0200, Pali Rohár wrote: > On Thursday 07 April 2022 11:54:55 Miquel Raynal wrote: > > Hi Pali, > > > > pali@kernel.org wrote on Thu, 7 Apr 2022 11:41:59 +0200: > > > > > On Thursday 07 April 2022 09:54:21 Miquel Raynal wrote: > > > > Hi Pali, > > > > > > > > pali@kernel.org wrote on Wed, 6 Apr 2022 23:31:53 +0200: > > > > > > > > Would you mind explaining a little bit how this change fixes it? It > > > > does not look straightforward to me. > > > > > > Yes! I though that it is straightforward this change... > > > byteorder/little_endian.h defines cpu_to_le* macros for Little Endian > > > systems and byteorder/big_endian.h for Big Endian systems. > > > > > > File asm/byteorder.h is then ARCH-specific and implements macros for the > > > current architecture (by including the correct header file). > > > > > > So currently if you try to compile squashfs for big endian systems you > > > get compile error: > > > > > > In file included from ./arch/powerpc/include/asm/byteorder.h:82, > > > from include/linux/unaligned/access_ok.h:4, > > > from ./arch/powerpc/include/asm/unaligned.h:9, > > > from fs/squashfs/sqfs_filesystem.h:11, > > > from fs/squashfs/sqfs_dir.c:16: > > > include/linux/byteorder/big_endian.h:34: warning: "__cpu_to_le32" redefined > > > #define __cpu_to_le32(x) ((__force __le32)__swab32((x))) > > > > > > In file included from fs/squashfs/sqfs_dir.c:10: > > > include/linux/byteorder/little_endian.h:34: note: this is the location of the previous definition > > > #define __cpu_to_le32(x) ((__force __le32)(__u32)(x)) > > > > > > Or: > > > > > > In file included from fs/squashfs/sqfs.c:14: > > > include/linux/byteorder/little_endian.h:89:21: error: redefinition of ‘__be16_to_cpup’ > > > static inline __u16 __be16_to_cpup(const __be16 *p) > > > ^~~~~~~~~~~~~~ > > > In file included from ./arch/powerpc/include/asm/byteorder.h:82, > > > from include/linux/unaligned/access_ok.h:4, > > > from ./arch/powerpc/include/asm/unaligned.h:9, > > > from fs/squashfs/sqfs.c:10: > > > include/linux/byteorder/big_endian.h:89:21: note: previous definition of ‘__be16_to_cpup’ was here > > > static inline __u16 __be16_to_cpup(const __be16 *p) > > > ^~~~~~~~~~~~~~ > > > > > > As some header files include correct asm/byteorder.h file and this > > > squashfs includes additional little_endian.h. > > > > Great, thanks for the thorough explanation. Based on what you said, > > wouldn't it be cleaner to just get rid of the little_endian.h include > > rather than also use the ARCH specific byteorder.h header? > > I think that this is not possible. squashfs code uses le16_to_cpu() > macro and its correct definition is provided only by arch specific > asm/byteorder.h. Miquel, is this sufficient explanation? Thanks. -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-06-01 14:08 ` Tom Rini @ 2022-06-03 14:24 ` Miquel Raynal 0 siblings, 0 replies; 10+ messages in thread From: Miquel Raynal @ 2022-06-03 14:24 UTC (permalink / raw) To: Tom Rini; +Cc: Pali Rohár, wJoao Marcos Costa, Thomas Petazzoni, u-boot Hi Tom, trini@konsulko.com wrote on Wed, 1 Jun 2022 10:08:20 -0400: > On Thu, Apr 07, 2022 at 11:58:55AM +0200, Pali Rohár wrote: > > On Thursday 07 April 2022 11:54:55 Miquel Raynal wrote: > > > Hi Pali, > > > > > > pali@kernel.org wrote on Thu, 7 Apr 2022 11:41:59 +0200: > > > > > > > On Thursday 07 April 2022 09:54:21 Miquel Raynal wrote: > > > > > Hi Pali, > > > > > > > > > > pali@kernel.org wrote on Wed, 6 Apr 2022 23:31:53 +0200: > > > > > > > > > > Would you mind explaining a little bit how this change fixes it? It > > > > > does not look straightforward to me. > > > > > > > > Yes! I though that it is straightforward this change... > > > > byteorder/little_endian.h defines cpu_to_le* macros for Little Endian > > > > systems and byteorder/big_endian.h for Big Endian systems. > > > > > > > > File asm/byteorder.h is then ARCH-specific and implements macros for the > > > > current architecture (by including the correct header file). > > > > > > > > So currently if you try to compile squashfs for big endian systems you > > > > get compile error: > > > > > > > > In file included from ./arch/powerpc/include/asm/byteorder.h:82, > > > > from include/linux/unaligned/access_ok.h:4, > > > > from ./arch/powerpc/include/asm/unaligned.h:9, > > > > from fs/squashfs/sqfs_filesystem.h:11, > > > > from fs/squashfs/sqfs_dir.c:16: > > > > include/linux/byteorder/big_endian.h:34: warning: "__cpu_to_le32" redefined > > > > #define __cpu_to_le32(x) ((__force __le32)__swab32((x))) > > > > > > > > In file included from fs/squashfs/sqfs_dir.c:10: > > > > include/linux/byteorder/little_endian.h:34: note: this is the location of the previous definition > > > > #define __cpu_to_le32(x) ((__force __le32)(__u32)(x)) > > > > > > > > Or: > > > > > > > > In file included from fs/squashfs/sqfs.c:14: > > > > include/linux/byteorder/little_endian.h:89:21: error: redefinition of ‘__be16_to_cpup’ > > > > static inline __u16 __be16_to_cpup(const __be16 *p) > > > > ^~~~~~~~~~~~~~ > > > > In file included from ./arch/powerpc/include/asm/byteorder.h:82, > > > > from include/linux/unaligned/access_ok.h:4, > > > > from ./arch/powerpc/include/asm/unaligned.h:9, > > > > from fs/squashfs/sqfs.c:10: > > > > include/linux/byteorder/big_endian.h:89:21: note: previous definition of ‘__be16_to_cpup’ was here > > > > static inline __u16 __be16_to_cpup(const __be16 *p) > > > > ^~~~~~~~~~~~~~ > > > > > > > > As some header files include correct asm/byteorder.h file and this > > > > squashfs includes additional little_endian.h. > > > > > > Great, thanks for the thorough explanation. Based on what you said, > > > wouldn't it be cleaner to just get rid of the little_endian.h include > > > rather than also use the ARCH specific byteorder.h header? > > > > I think that this is not possible. squashfs code uses le16_to_cpu() > > macro and its correct definition is provided only by arch specific > > asm/byteorder.h. > > Miquel, is this sufficient explanation? Thanks. Yes, sorry, it seems like in U-Boot including the asm header is the way to go, so: Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com> Thanks, Miquèl ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-04-06 21:31 [PATCH] squashfs: Fix compilation on big endian systems Pali Rohár 2022-04-07 7:54 ` Miquel Raynal @ 2022-05-10 14:33 ` Pali Rohár 2022-05-23 9:32 ` Pali Rohár 2022-06-03 19:48 ` Tom Rini 2 siblings, 1 reply; 10+ messages in thread From: Pali Rohár @ 2022-05-10 14:33 UTC (permalink / raw) To: Joao Marcos Costa, Thomas Petazzoni, Miquel Raynal; +Cc: u-boot On Wednesday 06 April 2022 23:31:53 Pali Rohár wrote: > Signed-off-by: Pali Rohár <pali@kernel.org> > --- > fs/squashfs/sqfs.c | 3 +-- > fs/squashfs/sqfs_dir.c | 3 +-- > 2 files changed, 2 insertions(+), 4 deletions(-) > > diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c > index 5d9c52af80ba..41cb811c1b32 100644 > --- a/fs/squashfs/sqfs.c > +++ b/fs/squashfs/sqfs.c > @@ -11,8 +11,7 @@ > #include <errno.h> > #include <fs.h> > #include <linux/types.h> > -#include <linux/byteorder/little_endian.h> > -#include <linux/byteorder/generic.h> > +#include <asm/byteorder.h> > #include <memalign.h> > #include <stdlib.h> > #include <string.h> > diff --git a/fs/squashfs/sqfs_dir.c b/fs/squashfs/sqfs_dir.c > index a265b98fe685..ed83c90682ff 100644 > --- a/fs/squashfs/sqfs_dir.c > +++ b/fs/squashfs/sqfs_dir.c > @@ -7,8 +7,7 @@ > > #include <errno.h> > #include <linux/types.h> > -#include <linux/byteorder/little_endian.h> > -#include <linux/byteorder/generic.h> > +#include <asm/byteorder.h> > #include <stdint.h> > #include <stdio.h> > #include <stdlib.h> > -- > 2.20.1 > PING? ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-05-10 14:33 ` Pali Rohár @ 2022-05-23 9:32 ` Pali Rohár 0 siblings, 0 replies; 10+ messages in thread From: Pali Rohár @ 2022-05-23 9:32 UTC (permalink / raw) To: Joao Marcos Costa, Thomas Petazzoni, Miquel Raynal; +Cc: u-boot On Tuesday 10 May 2022 16:33:56 Pali Rohár wrote: > On Wednesday 06 April 2022 23:31:53 Pali Rohár wrote: > > Signed-off-by: Pali Rohár <pali@kernel.org> > > --- > > fs/squashfs/sqfs.c | 3 +-- > > fs/squashfs/sqfs_dir.c | 3 +-- > > 2 files changed, 2 insertions(+), 4 deletions(-) > > > > diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c > > index 5d9c52af80ba..41cb811c1b32 100644 > > --- a/fs/squashfs/sqfs.c > > +++ b/fs/squashfs/sqfs.c > > @@ -11,8 +11,7 @@ > > #include <errno.h> > > #include <fs.h> > > #include <linux/types.h> > > -#include <linux/byteorder/little_endian.h> > > -#include <linux/byteorder/generic.h> > > +#include <asm/byteorder.h> > > #include <memalign.h> > > #include <stdlib.h> > > #include <string.h> > > diff --git a/fs/squashfs/sqfs_dir.c b/fs/squashfs/sqfs_dir.c > > index a265b98fe685..ed83c90682ff 100644 > > --- a/fs/squashfs/sqfs_dir.c > > +++ b/fs/squashfs/sqfs_dir.c > > @@ -7,8 +7,7 @@ > > > > #include <errno.h> > > #include <linux/types.h> > > -#include <linux/byteorder/little_endian.h> > > -#include <linux/byteorder/generic.h> > > +#include <asm/byteorder.h> > > #include <stdint.h> > > #include <stdio.h> > > #include <stdlib.h> > > -- > > 2.20.1 > > > > PING? PING? ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] squashfs: Fix compilation on big endian systems 2022-04-06 21:31 [PATCH] squashfs: Fix compilation on big endian systems Pali Rohár 2022-04-07 7:54 ` Miquel Raynal 2022-05-10 14:33 ` Pali Rohár @ 2022-06-03 19:48 ` Tom Rini 2 siblings, 0 replies; 10+ messages in thread From: Tom Rini @ 2022-06-03 19:48 UTC (permalink / raw) To: Pali Rohár Cc: Joao Marcos Costa, Thomas Petazzoni, Miquel Raynal, u-boot [-- Attachment #1: Type: text/plain, Size: 217 bytes --] On Wed, Apr 06, 2022 at 11:31:53PM +0200, Pali Rohár wrote: > Signed-off-by: Pali Rohár <pali@kernel.org> > Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com> Applied to u-boot/master, thanks! -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-06-03 19:49 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-04-06 21:31 [PATCH] squashfs: Fix compilation on big endian systems Pali Rohár 2022-04-07 7:54 ` Miquel Raynal 2022-04-07 9:41 ` Pali Rohár 2022-04-07 9:54 ` Miquel Raynal 2022-04-07 9:58 ` Pali Rohár 2022-06-01 14:08 ` Tom Rini 2022-06-03 14:24 ` Miquel Raynal 2022-05-10 14:33 ` Pali Rohár 2022-05-23 9:32 ` Pali Rohár 2022-06-03 19:48 ` Tom Rini
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.