On 11/19/2018 10:47 AM, J, KEERTHY wrote:


On 11/19/2018 2:59 AM, Simon Glass wrote:
Hi Keerthy,

On Fri, 16 Nov 2018 at 17:09, Simon Glass <[email protected]> wrote:

On Tue, 13 Nov 2018 at 12:53, Simon Glass <[email protected]> wrote:

On 4 November 2018 at 22:47, Keerthy <[email protected]> wrote:
Currently the else part of ofnode_get_addr_size_index function
does not fetch addresses based on the index but rather just
returns the base address. Fix that.

Signed-off-by: Keerthy <[email protected]>
---
  drivers/core/ofnode.c | 13 ++++++++-----
  1 file changed, 8 insertions(+), 5 deletions(-)

Reviewed-by: Simon Glass <[email protected]>

This should really have a sandbox test, e.g. see
dm_test_fdt_remap_addr_flat,() for a test that relies on flagtree.

Reviewed-by: Simon Glass <[email protected]>

Unfortunately this patch causes build errors with sandbox. Can you
please take a look? I skip the second patch in this series too, since
it depends on this one.

Sure. Thanks for catching it.

Simon,

Sent a v3 with sandbox_defconfig warnings fixed. Please let me know if they are fine.

Thanks,
Keerthy


Regards,
Simon

_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to