[U-Boot] [PATCH v3] fdt: allow address translation in case of SPL_OF_TRANSLATE
Vikas Manocha
vikas.manocha at st.com
Wed Apr 5 16:12:02 UTC 2017
Hi Simon,
On 04/04/2017 08:27 PM, Simon Glass wrote:
> Hi Vikas,
>
> On 4 April 2017 at 15:59, Vikas Manocha <vikas.manocha at st.com> wrote:
>> Address translation is not working at present even if SPL_OF_TRANSLATE is
>> enabled which makes this configuration useless. This patch enables address
>> translation for SPL U-Boot when SPL_OF_TRANSLATE is selected.
>>
>> Signed-off-by: Vikas Manocha <vikas.manocha at st.com>
>> Reviewed-by: Simon Glass <sjg at chromium.org>
>> ---
>>
>> Changes in v3: changed "u-boot" to "U-Boot" in commit message.
>> Changes in v2: Added commit message
>>
>> lib/fdtdec.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/lib/fdtdec.c b/lib/fdtdec.c
>> index 81f47ef..a1c4d16 100644
>> --- a/lib/fdtdec.c
>> +++ b/lib/fdtdec.c
>> @@ -112,7 +112,7 @@ fdt_addr_t fdtdec_get_addr_size_fixed(const void *blob, int node,
>> return FDT_ADDR_T_NONE;
>> }
>>
>> -#if !defined(CONFIG_SPL_BUILD) && defined(CONFIG_OF_LIBFDT)
>> +#if defined(CONFIG_SPL_OF_TRANSLATE) || defined(CONFIG_OF_LIBFDT)
>> if (translate)
>> addr = fdt_translate_address(blob, node, prop_addr);
>> else
>> --
>> 1.9.1
>>
>
> Actually I'm a bit unsure about this patch. You already have a patch applied:
>
> 5efa1bf libfdt: use CONFIG_IS_ENABLED for OF_LIBFDT
This patch(5efa1bf) was recently applied & serves the same purpose.
We can ignore my patch.
Cheers,
Vikas
>
> Do you actually want this one else well? If so, please resend it as it
> conflicts.
>
> Regards,
> Simon
> .
>
More information about the U-Boot
mailing list