[U-Boot] [PATCH 30/31] 8xx, km8xx: update in dts node "flash", the reg value correct.

Signed-off-by: Heiko Schocher hs@denx.de --- board/keymile/km8xx/km8xx.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/board/keymile/km8xx/km8xx.c b/board/keymile/km8xx/km8xx.c index 3f0e65f..a117845 100644 --- a/board/keymile/km8xx/km8xx.c +++ b/board/keymile/km8xx/km8xx.c @@ -173,6 +173,7 @@ void ft_blob_update (void *blob, bd_t *bd) ulong brg_data[1] = {0}; ulong memory_data[2] = {0}; ulong flash_data[4] = {0}; + ulong flash_reg[3] = {0};
memory_data[0] = cpu_to_be32 (bd->bi_memstart); memory_data[1] = cpu_to_be32 (bd->bi_memsize); @@ -184,6 +185,10 @@ void ft_blob_update (void *blob, bd_t *bd) fdt_set_node_and_value (blob, "/localbus", "ranges", flash_data, sizeof (flash_data));
+ flash_reg[2] = cpu_to_be32 (bd->bi_flashsize); + fdt_set_node_and_value (blob, "/localbus/flash@0,0", "reg", flash_reg, + sizeof (flash_reg)); + /* BRG */ brg_data[0] = cpu_to_be32 (bd->bi_busfreq); fdt_set_node_and_value (blob, "/soc/cpm", "brg-frequency", brg_data,

Dear Heiko Schocher,
please chose a better Subject.
In message 4980282A.5010001@denx.de you wrote:
ulong flash_reg[3] = {0};
memory_data[0] = cpu_to_be32 (bd->bi_memstart); memory_data[1] = cpu_to_be32 (bd->bi_memsize);
@@ -184,6 +185,10 @@ void ft_blob_update (void *blob, bd_t *bd) fdt_set_node_and_value (blob, "/localbus", "ranges", flash_data, sizeof (flash_data));
- flash_reg[2] = cpu_to_be32 (bd->bi_flashsize);
- fdt_set_node_and_value (blob, "/localbus/flash@0,0", "reg", flash_reg,
sizeof (flash_reg));
What exactly are you doing here?
This patch by itself looks pretty fishy to me. Maybe you split your patches in a sub-optimal way?
Best regards,
Wolfgang Denk
participants (2)
-
Heiko Schocher
-
Wolfgang Denk