RE: [PATCH 1/9] ARM: OMAP2+: AM33XX: CM: Get rid of unncessary header inclusions

2013-01-20 Thread Bedia, Vaibhav
On Fri, Jan 18, 2013 at 18:01:01, Sergei Shtylyov wrote:
> On 18-01-2013 11:19, Vaibhav Bedia wrote:
> 
> > Some of the included header files are not needed so
> > remove them.
> 
> > Signed-off-by: Vaibhav Bedia 
> > Acked-by: Santosh Shilimkar 
> > ---
> > Change from RFC version:
> > No change
> 
> >   arch/arm/mach-omap2/cm33xx.h |7 +--
> >   1 files changed, 1 insertions(+), 6 deletions(-)
> 
> > diff --git a/arch/arm/mach-omap2/cm33xx.h b/arch/arm/mach-omap2/cm33xx.h
> > index 5fa0b62..8009e13 100644
> > --- a/arch/arm/mach-omap2/cm33xx.h
> > +++ b/arch/arm/mach-omap2/cm33xx.h
> > @@ -17,16 +17,11 @@
> >   #ifndef __ARCH_ARM_MACH_OMAP2_CM_33XX_H
> >   #define __ARCH_ARM_MACH_OMAP2_CM_33XX_H
> >
> > -#include 
> > -#include 
> > -#include 
> > -#include 
> > -
> >   #include "common.h"
> >
> >   #include "cm.h"
> >   #include "cm-regbits-33xx.h"
> > -#include "cm33xx.h"
> > +#include "iomap.h"
> 
> You don't comment on this addition in the changelog...

Hmm... ok. Will clarify this in the changelog for v2.

Regards,
Vaibhav
--
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/9] ARM: OMAP2+: AM33XX: CM: Get rid of unncessary header inclusions

2013-01-18 Thread Sergei Shtylyov

On 18-01-2013 11:19, Vaibhav Bedia wrote:


Some of the included header files are not needed so
remove them.



Signed-off-by: Vaibhav Bedia 
Acked-by: Santosh Shilimkar 
---
Change from RFC version:
No change



  arch/arm/mach-omap2/cm33xx.h |7 +--
  1 files changed, 1 insertions(+), 6 deletions(-)



diff --git a/arch/arm/mach-omap2/cm33xx.h b/arch/arm/mach-omap2/cm33xx.h
index 5fa0b62..8009e13 100644
--- a/arch/arm/mach-omap2/cm33xx.h
+++ b/arch/arm/mach-omap2/cm33xx.h
@@ -17,16 +17,11 @@
  #ifndef __ARCH_ARM_MACH_OMAP2_CM_33XX_H
  #define __ARCH_ARM_MACH_OMAP2_CM_33XX_H

-#include 
-#include 
-#include 
-#include 
-
  #include "common.h"

  #include "cm.h"
  #include "cm-regbits-33xx.h"
-#include "cm33xx.h"
+#include "iomap.h"


   You don't comment on this addition in the changelog...

WBR, Sergei

--
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