[PATCH] ARM: OMAP4: Fix the init code to have OMAP4460 errata available in DT build
Santosh Shilimkar
santosh.shilimkar at ti.com
Tue Feb 26 02:43:14 EST 2013
On Monday 25 February 2013 08:32 PM, Jon Hunter wrote:
>
> On 02/25/2013 06:09 AM, Santosh Shilimkar wrote:
>> OMAP4460 ROM code bug needs the GIC distributor and local timer
>> bases to be available for the bug work around. In current code, dt
>> case these bases are not initialized leading to failure of the
>> errata work-around.
>>
>> Fix it by extracting the bases from dt blob and populating them.
>>
>> Reported-by: Sourav Poddar <sourav.poddar at ti.com>
>> Tested-by: Sourav Poddar <sourav.poddar at ti.com>
>> Signed-off-by: Santosh Shilimkar <santosh.shilimkar at ti.com>
>> ---
>> Posting this one seperatly but will add along with rest of
>> my fixes so that it doesn't get lost on the list.
>>
>> arch/arm/mach-omap2/omap4-common.c | 32 ++++++++++++++++++++++++++++++++
>> 1 file changed, 32 insertions(+)
>>
>> diff --git a/arch/arm/mach-omap2/omap4-common.c b/arch/arm/mach-omap2/omap4-common.c
>> index 6897ae2..d07740e 100644
>> --- a/arch/arm/mach-omap2/omap4-common.c
>> +++ b/arch/arm/mach-omap2/omap4-common.c
>> @@ -261,8 +261,40 @@ static struct of_device_id irq_match[] __initdata = {
>> { }
>> };
>>
>> +static struct of_device_id twd_match[] __initdata = {
>> + { .compatible = "arm,cortex-a9-twd-timer", },
>> + { }
>> +};
>> +
>> void __init omap_gic_of_init(void)
>> {
>> + struct device_node *np;
>> + const void *reg_prop;
>> + unsigned long start = 0, size = 0;
>> +
>> + /* Extract GIC distributor and TWD bases for OMAP4460 ROM Errata WA */
>> + if (!cpu_is_omap446x())
>> + goto skip_errata_init;
>> +
>> + np = of_find_matching_node(NULL, irq_match);
>> + if (np) {
>> + reg_prop = of_get_property(np, "reg", NULL);
>> + start = of_read_number(reg_prop, 1);
>> + size = of_read_number(reg_prop + 4, 1);
>
> Alternatively, you could replace the above 3 lines with ...
>
> gic_dist_base_addr = of_iomap(np, 0);
>
Nice !!
Will use the alternative. Thanks Jon for pointer.
Regards,
Santosh
More information about the linux-arm-kernel
mailing list