[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