Message ID | 20171207152044.14572-1-jakob.unterwurzacher@theobroma-systems.com |
---|---|
State | Accepted |
Commit | b32b1bd10b86d01a26056f50de89f768e5183ed1 |
Delegated to: | Philipp Tomsich |
Headers | show |
Series | [U-Boot] rockchip: rk3399-puma: preserve leading zeros in serial# | expand |
> On 7 Dec 2017, at 16:20, Jakob Unterwurzacher <jakob.unterwurzacher@theobroma-systems.com> wrote: > > Linux preserves leading zeros in /proc/cpuinfo, so we > should as well. > > Otherwise we have the situation that > /sys/firmware/devicetree/base/serial-number > and /proc/cpuinfo disagree in Linux. > > Signed-off-by: Jakob Unterwurzacher <jakob.unterwurzacher@theobroma-systems.com> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Linux preserves leading zeros in /proc/cpuinfo, so we > should as well. > > Otherwise we have the situation that > /sys/firmware/devicetree/base/serial-number > and /proc/cpuinfo disagree in Linux. > > Signed-off-by: Jakob Unterwurzacher <jakob.unterwurzacher@theobroma-systems.com> > Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com> > --- > > board/theobroma-systems/puma_rk3399/puma-rk3399.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Linux preserves leading zeros in /proc/cpuinfo, so we > should as well. > > Otherwise we have the situation that > /sys/firmware/devicetree/base/serial-number > and /proc/cpuinfo disagree in Linux. > > Signed-off-by: Jakob Unterwurzacher <jakob.unterwurzacher@theobroma-systems.com> > Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com> > Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com> > --- > > board/theobroma-systems/puma_rk3399/puma-rk3399.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Applied to u-boot-rockchip, thanks!
diff --git a/board/theobroma-systems/puma_rk3399/puma-rk3399.c b/board/theobroma-systems/puma_rk3399/puma-rk3399.c index 26c4535f1b..737206ae0f 100644 --- a/board/theobroma-systems/puma_rk3399/puma-rk3399.c +++ b/board/theobroma-systems/puma_rk3399/puma-rk3399.c @@ -138,7 +138,7 @@ static void setup_serial(void) serialno = crc32_no_comp(0, low, 8); serialno |= (u64)crc32_no_comp(serialno, high, 8) << 32; - snprintf(serialno_str, sizeof(serialno_str), "%llx", serialno); + snprintf(serialno_str, sizeof(serialno_str), "%016llx", serialno); env_set("cpuid#", cpuid_str); env_set("serial#", serialno_str);
Linux preserves leading zeros in /proc/cpuinfo, so we should as well. Otherwise we have the situation that /sys/firmware/devicetree/base/serial-number and /proc/cpuinfo disagree in Linux. Signed-off-by: Jakob Unterwurzacher <jakob.unterwurzacher@theobroma-systems.com> --- board/theobroma-systems/puma_rk3399/puma-rk3399.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)