Re: [GIT PULL] fix to use static declarations for samsung stuff

2012-03-07 Thread Kukjin Kim

On 02/10/12 04:55, Kukjin Kim wrote:

Hi Russell, Arnd and Olof,

Please pull following branch which is including fix to use static
declarations for samsung stuff.
  git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
next/cleanup-use-static

I'm not sure it can be sent to upstream for v3.3 but if so, would be better.


Hi Arnd, Olof,

I can't see this series in arm-soc yet. If missed, please pull from:
 http://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git 
next/cleanup-use-static


As a note, I rebased it on top of arm-soc/depends/rmk/for-armsoc because 
this touches same points(dma declaration part).


If any problems, please kindly let me know.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim kgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

The following changes since commit a5f17d1f4c2831b9b9bf8b1a537cdbac995d6e13:

  Merge tag 'v3.3-rc2' into depends/rmk/for-armsoc (2012-02-07 15:05:20 
-0800)


are available in the git repository at:


http://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git 
next/cleanup-use-static


Kukjin Kim (8):
  ARM: EXYNOS: use static declaration when it is not used in other 
files
  ARM: S3C24XX: use static declaration when it is not used in other 
files
  ARM: S3C64XX: use static declaration when it is not used in other 
files
  ARM: S5P64X0: use static declaration when it is not used in other 
files
  ARM: S5PC100: use static declaration when it is not used in other 
files
  ARM: S5PV210: use static declaration when it is not used in other 
files
  ARM: SAMSUNG: use static declaration when it is not used in other 
files

  Merge branch 'topic/cleanup-use-static' into next/cleanup-use-static

 arch/arm/mach-exynos/clock-exynos4210.c|2 +-
 arch/arm/mach-exynos/clock-exynos4212.c|2 +-
 arch/arm/mach-exynos/clock.c   |2 +-
 arch/arm/mach-exynos/common.c  |2 +-
 arch/arm/mach-exynos/dma.c |   16 +-
 arch/arm/mach-exynos/mach-origen.c |2 +-
 arch/arm/mach-exynos/mach-universal_c210.c |2 +-
 arch/arm/mach-s3c2410/mach-h1940.c |   12 
 arch/arm/mach-s3c2416/clock.c  |6 
 arch/arm/mach-s3c2416/mach-smdk2416.c  |8 +++---
 arch/arm/mach-s3c2440/mach-gta02.c |4 +-
 arch/arm/mach-s3c2440/mach-rx1950.c|   12 
 arch/arm/mach-s3c64xx/common.c |2 +-
 arch/arm/mach-s3c64xx/common.h |2 -
 arch/arm/mach-s3c64xx/irq-pm.c |2 +-
 arch/arm/mach-s5p64x0/clock.c  |   11 ---
 arch/arm/mach-s5p64x0/dma.c|   12 
 arch/arm/mach-s5p64x0/include/mach/s5p64x0-clock.h |7 -
 arch/arm/mach-s5pc100/clock.c  |   28 
++--

 arch/arm/mach-s5pc100/dma.c|   16 +-
 arch/arm/mach-s5pv210/dma.c|   16 +-
 arch/arm/mach-s5pv210/mach-goni.c  |2 +-
 arch/arm/mach-s5pv210/mach-smdkv210.c  |2 +-
 arch/arm/plat-s3c24xx/s3c2443-clock.c  |2 +-
 arch/arm/plat-s5p/irq-eint.c   |2 +-
 arch/arm/plat-s5p/irq-gpioint.c|2 +-
 arch/arm/plat-samsung/devs.c   |   13 +
 arch/arm/plat-samsung/dma-ops.c|2 +-
 28 files changed, 83 insertions(+), 108 deletions(-)


As a note, conflicts have been occurred when merge
arm-soc/depends/rmk/for-armsoc.
Following is my preferred resolution.

If any problems, please kindly let me know.

diff --cc arch/arm/mach-exynos/dma.c
index 8dec3db,91370de..000
--- a/arch/arm/mach-exynos/dma.c
+++ b/arch/arm/mach-exynos/dma.c
@@@ -74,23 -74,10 +74,10 @@@ static struct dma_pl330_platdata exynos
.peri_id = pdma0_peri,
   };

- static struct amba_device exynos4_device_pdma0 = {
-   .dev = {
-   .init_name = dma-pl330.0,
-   .dma_mask =dma_dmamask,
-   .coherent_dma_mask = DMA_BIT_MASK(32),
-   .platform_data =exynos4_pdma0_pdata,
-   },
-   .res = {
-   .start = EXYNOS4_PA_PDMA0,
-   .end = EXYNOS4_PA_PDMA0 + SZ_4K,
-   .flags = IORESOURCE_MEM,
-   },
-   .irq = {IRQ_PDMA0, NO_IRQ},
-   .periphid = 0x00041330,
- };
  -AMBA_AHB_DEVICE(exynos4_pdma0, dma-pl330.0, 0x00041330,
EXYNOS4_PA_PDMA0,
  - {IRQ_PDMA0},exynos4_pdma0_pdata);
++static AMBA_AHB_DEVICE(exynos4_pdma0, dma-pl330.0, 0x00041330,
++  EXYNOS4_PA_PDMA0, {IRQ_PDMA0},exynos4_pdma0_pdata);

  -u8 pdma1_peri[] = {
  +static u8 pdma1_peri[] = {
DMACH_PCM0_RX,
DMACH_PCM0_TX,
DMACH_PCM1_RX,
@@@ -123,21 -110,8 +110,8 @@@ static struct 

Re: [GIT PULL] fix to use static declarations for samsung stuff

2012-03-07 Thread Arnd Bergmann
On Wednesday 07 March 2012, Kukjin Kim wrote:
 im wrote:
  Hi Russell, Arnd and Olof,
 
  Please pull following branch which is including fix to use static
  declarations for samsung stuff.
git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
  next/cleanup-use-static
 
  I'm not sure it can be sent to upstream for v3.3 but if so, would be better.
 
 Hi Arnd, Olof,
 
 I can't see this series in arm-soc yet. If missed, please pull from:
   http://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git 
 next/cleanup-use-static
 
 As a note, I rebased it on top of arm-soc/depends/rmk/for-armsoc because 
 this touches same points(dma declaration part).
 
 If any problems, please kindly let me know.
 

Sorry, I must have missed this request, I can't remember seeing it before.
The patches all look good, Olof will take care of getting them in.

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


Re: [GIT PULL] fix to use static declarations for samsung stuff

2012-03-07 Thread Olof Johansson
On Wed, Mar 7, 2012 at 4:15 AM, Arnd Bergmann a...@arndb.de wrote:
 On Wednesday 07 March 2012, Kukjin Kim wrote:
 im wrote:
  Hi Russell, Arnd and Olof,
 
  Please pull following branch which is including fix to use static
  declarations for samsung stuff.
    git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
  next/cleanup-use-static
 
  I'm not sure it can be sent to upstream for v3.3 but if so, would be 
  better.
 
 Hi Arnd, Olof,

 I can't see this series in arm-soc yet. If missed, please pull from:
   http://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
 next/cleanup-use-static

 As a note, I rebased it on top of arm-soc/depends/rmk/for-armsoc because
 this touches same points(dma declaration part).

 If any problems, please kindly let me know.


 Sorry, I must have missed this request, I can't remember seeing it before.
 The patches all look good, Olof will take care of getting them in.

Yep, pulled now. Apologies for missing it earlier.


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


[GIT PULL] fix to use static declarations for samsung stuff

2012-02-10 Thread Kukjin Kim
Hi Russell, Arnd and Olof,

Please pull following branch which is including fix to use static
declarations for samsung stuff.
 git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
next/cleanup-use-static

I'm not sure it can be sent to upstream for v3.3 but if so, would be better.

As a note, conflicts have been occurred when merge
arm-soc/depends/rmk/for-armsoc.
Following is my preferred resolution.

If any problems, please kindly let me know.

diff --cc arch/arm/mach-exynos/dma.c
index 8dec3db,91370de..000
--- a/arch/arm/mach-exynos/dma.c
+++ b/arch/arm/mach-exynos/dma.c
@@@ -74,23 -74,10 +74,10 @@@ static struct dma_pl330_platdata exynos
.peri_id = pdma0_peri,
  };
  
- static struct amba_device exynos4_device_pdma0 = {
-   .dev = {
-   .init_name = dma-pl330.0,
-   .dma_mask = dma_dmamask,
-   .coherent_dma_mask = DMA_BIT_MASK(32),
-   .platform_data = exynos4_pdma0_pdata,
-   },
-   .res = {
-   .start = EXYNOS4_PA_PDMA0,
-   .end = EXYNOS4_PA_PDMA0 + SZ_4K,
-   .flags = IORESOURCE_MEM,
-   },
-   .irq = {IRQ_PDMA0, NO_IRQ},
-   .periphid = 0x00041330,
- };
 -AMBA_AHB_DEVICE(exynos4_pdma0, dma-pl330.0, 0x00041330,
EXYNOS4_PA_PDMA0,
 -  {IRQ_PDMA0}, exynos4_pdma0_pdata);
++static AMBA_AHB_DEVICE(exynos4_pdma0, dma-pl330.0, 0x00041330,
++  EXYNOS4_PA_PDMA0, {IRQ_PDMA0}, exynos4_pdma0_pdata);
  
 -u8 pdma1_peri[] = {
 +static u8 pdma1_peri[] = {
DMACH_PCM0_RX,
DMACH_PCM0_TX,
DMACH_PCM1_RX,
@@@ -123,21 -110,8 +110,8 @@@ static struct dma_pl330_platdata exynos
.peri_id = pdma1_peri,
  };
  
- static struct amba_device exynos4_device_pdma1 = {
-   .dev = {
-   .init_name = dma-pl330.1,
-   .dma_mask = dma_dmamask,
-   .coherent_dma_mask = DMA_BIT_MASK(32),
-   .platform_data = exynos4_pdma1_pdata,
-   },
-   .res = {
-   .start = EXYNOS4_PA_PDMA1,
-   .end = EXYNOS4_PA_PDMA1 + SZ_4K,
-   .flags = IORESOURCE_MEM,
-   },
-   .irq = {IRQ_PDMA1, NO_IRQ},
-   .periphid = 0x00041330,
- };
 -AMBA_AHB_DEVICE(exynos4_pdma1,  dma-pl330.1, 0x00041330,
EXYNOS4_PA_PDMA1,
 -  {IRQ_PDMA1}, exynos4_pdma1_pdata);
++static AMBA_AHB_DEVICE(exynos4_pdma1,  dma-pl330.1, 0x00041330,
++  EXYNOS4_PA_PDMA1, {IRQ_PDMA1}, exynos4_pdma1_pdata);
  
  static int __init exynos4_dma_init(void)
  {
diff --cc arch/arm/mach-s5p64x0/dma.c
index 8fce1161,f7f68ad..000
--- a/arch/arm/mach-s5p64x0/dma.c
+++ b/arch/arm/mach-s5p64x0/dma.c
@@@ -108,20 -108,8 +108,8 @@@ static struct dma_pl330_platdata s5p645
.peri_id = s5p6450_pdma_peri,
  };
  
- static struct amba_device s5p64x0_device_pdma = {
-   .dev = {
-   .init_name = dma-pl330,
-   .dma_mask = dma_dmamask,
-   .coherent_dma_mask = DMA_BIT_MASK(32),
-   },
-   .res = {
-   .start = S5P64X0_PA_PDMA,
-   .end = S5P64X0_PA_PDMA + SZ_4K,
-   .flags = IORESOURCE_MEM,
-   },
-   .irq = {IRQ_DMA0, NO_IRQ},
-   .periphid = 0x00041330,
- };
 -AMBA_AHB_DEVICE(s5p64x0_pdma, dma-pl330, 0x00041330, S5P64X0_PA_PDMA,
 -  {IRQ_DMA0}, NULL);
++static AMBA_AHB_DEVICE(s5p64x0_pdma, dma-pl330, 0x00041330,
++  S5P64X0_PA_PDMA, {IRQ_DMA0}, NULL);
  
  static int __init s5p64x0_dma_init(void)
  {
diff --cc arch/arm/mach-s5pc100/dma.c
index 395e46b,96b1ab3..000
--- a/arch/arm/mach-s5pc100/dma.c
+++ b/arch/arm/mach-s5pc100/dma.c
@@@ -73,23 -73,10 +73,10 @@@ static struct dma_pl330_platdata s5pc10
.peri_id = pdma0_peri,
  };
  
- static struct amba_device s5pc100_device_pdma0 = {
-   .dev = {
-   .init_name = dma-pl330.0,
-   .dma_mask = dma_dmamask,
-   .coherent_dma_mask = DMA_BIT_MASK(32),
-   .platform_data = s5pc100_pdma0_pdata,
-   },
-   .res = {
-   .start = S5PC100_PA_PDMA0,
-   .end = S5PC100_PA_PDMA0 + SZ_4K,
-   .flags = IORESOURCE_MEM,
-   },
-   .irq = {IRQ_PDMA0, NO_IRQ},
-   .periphid = 0x00041330,
- };
 -AMBA_AHB_DEVICE(s5pc100_pdma0,  dma-pl330.0, 0x00041330,
S5PC100_PA_PDMA0,
 -  {IRQ_PDMA0}, s5pc100_pdma0_pdata);
++static AMBA_AHB_DEVICE(s5pc100_pdma0,  dma-pl330.0, 0x00041330,
++  S5PC100_PA_PDMA0, {IRQ_PDMA0}, s5pc100_pdma0_pdata);
  
 -u8 pdma1_peri[] = {
 +static u8 pdma1_peri[] = {
DMACH_UART0_RX,
DMACH_UART0_TX,
DMACH_UART1_RX,
@@@ -127,21 -114,8 +114,8 @@@ static struct dma_pl330_platdata s5pc10
.peri_id = pdma1_peri,
  };
  
- static struct amba_device s5pc100_device_pdma1 = {
-   .dev = {
-   .init_name = dma-pl330.1,
-   .dma_mask = dma_dmamask,
-   .coherent_dma_mask = DMA_BIT_MASK(32),
-   .platform_data = s5pc100_pdma1_pdata,
-   },
-   .res = {
-