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

2012-12-03 Thread Vaibhav Hiremath


On 11/27/2012 10:22 PM, Jon Hunter wrote:
 
 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 nf...@linux.vnet.ibm.com
 Date:   Tue Oct 2 16:58:46 2012 +

 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 mugunthan...@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-hun...@ti.com
 Tested-by: Jon Hunter jon-hun...@ti.com
 

This patch still not made its way to linux-next, so still build is
broken for all OMAP platforms.

Thanks,
Vaibhav

 Cheers
 Jon
 --
 To unsubscribe from this list: send the line unsubscribe linux-omap in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html
 
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2012-11-27 Thread Mugunthan V N
  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

fix for build break as the functions prom_*_property is renamed to
of_*_property by the below patch

commit 79d1c712958f94372482ad74578b00f44e744c12
Author: Nathan Fontenot nf...@linux.vnet.ibm.com
Date:   Tue Oct 2 16:58:46 2012 +

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 mugunthan...@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;
}
 
-- 
1.7.0.4

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2012-11-27 Thread Jon Hunter

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 nf...@linux.vnet.ibm.com
 Date:   Tue Oct 2 16:58:46 2012 +
 
 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 mugunthan...@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-hun...@ti.com
Tested-by: Jon Hunter jon-hun...@ti.com

Cheers
Jon
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html