Re: [PATCH 1/1] ARM: OMAP: fix build error in arch/arm/mach-omap2/timer.c

[Jon Hunter <jon-hunter@ti.com>]

On 11/27/2012 04:24 AM, Mugunthan V N wrote:
>   CC      arch/arm/mach-omap2/timer.o
> arch/arm/mach-omap2/timer.c: In function 'omap_get_timer_dt':
> arch/arm/mach-omap2/timer.c:195:3: error: implicit declaration of function 'prom_add_property'
> make[1]: *** [arch/arm/mach-omap2/timer.o] Error 1
> make: *** [arch/arm/mach-omap2] Error 2

Thanks for sending. You may wish to mentioned that this is seen on
today's linux-next (I am seeing the same problem).

> fix for build break as the functions "prom_*_property" is renamed to
> of_*_property by the below patch
> 
> commit 79d1c712958f94372482ad74578b00f44e744c12
> Author: Nathan Fontenot <nfont@linux.vnet.ibm.com>
> Date:   Tue Oct 2 16:58:46 2012 +0000
> 
>     powerpc+of: Rename the drivers/of prom_* functions to of_*
> 
>     Rename the prom_*_property routines of the generic OF code to of_*_property.
>     This brings them in line with the naming used by the rest of the OF code.
> 
> Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>
> ---
>  arch/arm/mach-omap2/timer.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c
> index 099e406..5fd5e23 100644
> --- a/arch/arm/mach-omap2/timer.c
> +++ b/arch/arm/mach-omap2/timer.c
> @@ -192,7 +192,7 @@ static struct device_node * __init omap_get_timer_dt(struct of_device_id *match,
>  			continue;
>  		}
>  
> -		prom_add_property(np, &device_disabled);
> +		of_add_property(np, &device_disabled);
>  		return np;
>  	}

I tested this on the AM335x device and it is working fine. So ...

Acked-by: Jon Hunter <jon-hunter@ti.com>
Tested-by: Jon Hunter <jon-hunter@ti.com>

Cheers
Jon