On 6/24/19 3:32 PM, Bin Meng wrote: > Hi Ramon, > > On Mon, Jun 24, 2019 at 8:22 PM Ramon Fried <[email protected]> wrote: >> >> On 6/24/19 8:03 AM, Bin Meng wrote: >> >> On Mon, Jun 24, 2019 at 12:03 PM Anup Patel <[email protected]> wrote: >> >> We should depend on __LITTLE_ENDIAN pre-defined compiler macro for >> little-endian system instead of U-Boot specific CONFIG_SYS_LITTLE_ENDIAN >> macro. >> >> Signed-off-by: Anup Patel <[email protected]> >> --- >> drivers/net/macb.c | 7 ++++--- >> 1 file changed, 4 insertions(+), 3 deletions(-) >> >> Reviewed-by: Bin Meng <[email protected]> >> >> Hi. >> I don't like this approach, each platform should configure it's >> endianess, this is stated in README in root folder. >> relying on a specific GCC preprocessor extension is limiting us only to use >> GCC. >> The RISCV issue with MACB can be easily resolved by defining the >> CONFIG_SYS_LITTLE_ENDIAN config. > OK, but a system wide CONFIG_SYS_LITTLE_ENDIAN may bring side effects > to other drivers, as not all devices are using the same endianness > even in the same system. Maybe we can do something by parsing some > property in device tree? > > Regards, > Bin
Hey Bin I grep'ed for all instances of CONFIG_SYS_LITTLE_ENDIAN and I don't see any place where something might brake. can you elaborate ? Thanks, Ramon. _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
