[PATCH 00/28] Twenty eight patches big Exynos cleanup

Fri Jun 14 22:30:52 EDT 2013

Tomasz Figa wrote:
> 
> Hi All,
> 
> As I promised, I am doing some spring (or rather summer) cleaning
> of Exynos- and Samsung- related code.
> 
> This first part consists mostly of removing dead code remaining after
> removal of ATAGS support for Exynos, but several patches cleans up
> other things found by the way.
> 
> See particular patches for more detailed description.
> 
> On Exynos4210-based Trats board:
> Tested-by: Tomasz Figa <t.figa at samsung.com>
> 
> Build tested every patch with s3c6400_defconfig, s5pv210_defconfig
> and exynos_defconfig.
> 
> Tomasz Figa (28):
>   ARM: EXYNOS: Remove unused board files
>   ARM: EXYNOS: Make ARCH_EXYNOS select USE_OF
>   ARM: SAMSUNG: Introduce SAMSUNG_ATAGS Kconfig entry
>   ARM: EXYNOS: common: Remove legacy PMU initialization code
>   ARM: EXYNOS: common: Remove legacy EINT initialization code
>   ARM: EXYNOS: common: Remove legacy UART initialization code
>   ARM: EXYNOS: Remove legacy dev- and setup- files
>   ARM: EXYNOS: common: Remove legacy interrupt initialization code
>   ARM: EXYNOS: common: Remove legacy timer initialization code
>   ARM: EXYNOS: common: Remove platform device initialization
>   ARM: EXYNOS: common: Remove legacy mapping of chip ID block
>   ARM: EXYNOS: firmware: Remove check for device tree presence
>   ARM: EXYNOS: pm_domains: Remove legacy power domain registration code
>   ARM: SAMSUNG: Introduce GPIO_SAMSUNG Kconfig entry
>   ARM: EXYNOS: Provide compatibility stubs for PM code in pm-core.h
>     header
>   ARM: SAMSUNG: Compile s5p-irq-pm only with ATAGS support
>   ARM: EXYNOS: Decouple ARCH_EXYNOS from PLAT_S5P
>   ARM: SAMSUNG: pm: Include most of mach/ headers conditionally
>   ARM: EXYNOS: Do not select legacy Kconfig symbols any more
>   ARM: EXYNOS: Remove setup-i2c0.c
>   ARM: EXYNOS: Remove mach/gpio.h
>   ARM: EXYNOS: Remove mach/regs-gpio.h header
>   ARM: SAMSUNG: Make legacy MFC support code depend on SAMSUNG_ATAGS
>   ARM: EXYNOS: Select SPARSE_IRQ for Exynos
>   ARM: EXYNOS: Remove mach/irqs.h header
>   ARM: EXYNOS: Remove unused base addresses from mach/map.h header
>   ARM: EXYNOS: Remove CONFIG_SOC_EXYNOS4412
>   ARM: EXYNOS: Remove mach/regs-usb-phy.h header
> 
>  arch/arm/Kconfig                                 |   15 +-
>  arch/arm/Makefile                                |    1 +
>  arch/arm/mach-exynos/Kconfig                     |   14 +-
>  arch/arm/mach-exynos/Makefile                    |    6 -
>  arch/arm/mach-exynos/common.c                    |  472 +-------
>  arch/arm/mach-exynos/dev-audio.c                 |  254 ----
>  arch/arm/mach-exynos/dev-uart.c                  |   55 -
>  arch/arm/mach-exynos/firmware.c                  |   22 +-
>  arch/arm/mach-exynos/include/mach/gpio.h         |  289 -----
>  arch/arm/mach-exynos/include/mach/irqs.h         |  476 --------
>  arch/arm/mach-exynos/include/mach/map.h          |  214 ----
>  arch/arm/mach-exynos/include/mach/pm-core.h      |    5 +
>  arch/arm/mach-exynos/include/mach/regs-gpio.h    |   40 -
>  arch/arm/mach-exynos/include/mach/regs-usb-phy.h |   74 --
>  arch/arm/mach-exynos/mach-armlex4210.c           |  207 ----
>  arch/arm/mach-exynos/mach-nuri.c                 | 1388
---------------------
> -
>  arch/arm/mach-exynos/mach-origen.c               |  823 -------------
>  arch/arm/mach-exynos/mach-smdk4x12.c             |  396 ------
>  arch/arm/mach-exynos/mach-smdkv310.c             |  444 -------
>  arch/arm/mach-exynos/mach-universal_c210.c       | 1159
------------------
>  arch/arm/mach-exynos/pm.c                        |    1 -
>  arch/arm/mach-exynos/pm_domains.c                |   94 +-
>  arch/arm/mach-exynos/setup-fimc.c                |   44 -
>  arch/arm/mach-exynos/setup-fimd0.c               |   43 -
>  arch/arm/mach-exynos/setup-i2c0.c                |   29 -
>  arch/arm/mach-exynos/setup-i2c1.c                |   23 -
>  arch/arm/mach-exynos/setup-i2c2.c                |   23 -
>  arch/arm/mach-exynos/setup-i2c3.c                |   23 -
>  arch/arm/mach-exynos/setup-i2c4.c                |   23 -
>  arch/arm/mach-exynos/setup-i2c5.c                |   23 -
>  arch/arm/mach-exynos/setup-i2c6.c                |   23 -
>  arch/arm/mach-exynos/setup-i2c7.c                |   23 -
>  arch/arm/mach-exynos/setup-keypad.c              |   36 -
>  arch/arm/mach-exynos/setup-sdhci-gpio.c          |  152 ---
>  arch/arm/mach-exynos/setup-spi.c                 |   45 -
>  arch/arm/mach-exynos/setup-usb-phy.c             |  223 ----
>  arch/arm/plat-samsung/Kconfig                    |   40 +-
>  arch/arm/plat-samsung/Makefile                   |   11 +-
>  arch/arm/plat-samsung/include/plat/cpu.h         |    6 +-
>  arch/arm/plat-samsung/include/plat/pm.h          |    5 +
>  arch/arm/plat-samsung/init.c                     |    8 +-
>  arch/arm/plat-samsung/pm-gpio.c                  |    5 +-
>  arch/arm/plat-samsung/pm.c                       |    8 +-
>  arch/arm/plat-samsung/s5p-dev-mfc.c              |   11 +-
>  drivers/cpufreq/Kconfig.arm                      |    2 +-
>  drivers/gpio/Kconfig                             |    7 +
>  drivers/gpio/Makefile                            |    2 +-
>  drivers/media/platform/exynos4-is/Kconfig        |    2 +-
>  drivers/thermal/exynos_thermal.c                 |    2 +-
>  drivers/tty/serial/samsung.c                     |    3 +-
>  50 files changed, 115 insertions(+), 7179 deletions(-)
>  delete mode 100644 arch/arm/mach-exynos/dev-audio.c
>  delete mode 100644 arch/arm/mach-exynos/dev-uart.c
>  delete mode 100644 arch/arm/mach-exynos/include/mach/gpio.h
>  delete mode 100644 arch/arm/mach-exynos/include/mach/irqs.h
>  delete mode 100644 arch/arm/mach-exynos/include/mach/regs-gpio.h
>  delete mode 100644 arch/arm/mach-exynos/include/mach/regs-usb-phy.h
>  delete mode 100644 arch/arm/mach-exynos/mach-armlex4210.c
>  delete mode 100644 arch/arm/mach-exynos/mach-nuri.c
>  delete mode 100644 arch/arm/mach-exynos/mach-origen.c
>  delete mode 100644 arch/arm/mach-exynos/mach-smdk4x12.c
>  delete mode 100644 arch/arm/mach-exynos/mach-smdkv310.c
>  delete mode 100644 arch/arm/mach-exynos/mach-universal_c210.c
>  delete mode 100644 arch/arm/mach-exynos/setup-fimc.c
>  delete mode 100644 arch/arm/mach-exynos/setup-fimd0.c
>  delete mode 100644 arch/arm/mach-exynos/setup-i2c0.c
>  delete mode 100644 arch/arm/mach-exynos/setup-i2c1.c
>  delete mode 100644 arch/arm/mach-exynos/setup-i2c2.c
>  delete mode 100644 arch/arm/mach-exynos/setup-i2c3.c
>  delete mode 100644 arch/arm/mach-exynos/setup-i2c4.c
>  delete mode 100644 arch/arm/mach-exynos/setup-i2c5.c
>  delete mode 100644 arch/arm/mach-exynos/setup-i2c6.c
>  delete mode 100644 arch/arm/mach-exynos/setup-i2c7.c
>  delete mode 100644 arch/arm/mach-exynos/setup-keypad.c
>  delete mode 100644 arch/arm/mach-exynos/setup-sdhci-gpio.c
>  delete mode 100644 arch/arm/mach-exynos/setup-spi.c
>  delete mode 100644 arch/arm/mach-exynos/setup-usb-phy.c
> 
> --
> 1.8.2.1

Basically, looks good. I'm looking at this series then let me take.

BTW, just note, happens build error with s5pc100_defconfig....

arch/arm/plat-samsung/pm-gpio.c: In function 'samsung_gpio_pm_1bit_save':
arch/arm/plat-samsung/pm-gpio.c:33:6: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:34:6: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c: In function 'samsung_gpio_pm_1bit_resume':
arch/arm/plat-samsung/pm-gpio.c:42:22: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:43:22: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c: At top level:
arch/arm/plat-samsung/pm-gpio.c:63:24: error: expected identifier or '('
before 'void'
arch/arm/plat-samsung/pm-gpio.c:63:24: error: expected ')' before numeric
constant
arch/arm/plat-samsung/pm-gpio.c: In function 'samsung_gpio_pm_2bit_save':
arch/arm/plat-samsung/pm-gpio.c:70:6: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:71:6: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:72:6: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c: In function 'samsung_gpio_pm_2bit_resume':
arch/arm/plat-samsung/pm-gpio.c:129:22: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:130:22: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:136:19: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c: At top level:
arch/arm/plat-samsung/pm-gpio.c:190:24: error: expected identifier or '('
before 'void'
arch/arm/plat-samsung/pm-gpio.c:190:24: error: expected ')' before numeric
constant
arch/arm/plat-samsung/pm-gpio.c: In function 'samsung_gpio_pm_4bit_save':
arch/arm/plat-samsung/pm-gpio.c:199:6: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:200:6: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:201:6: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:204:7: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c: In function 'samsung_gpio_pm_4bit_con':
arch/arm/plat-samsung/pm-gpio.c:250:22: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c: In function 'samsung_gpio_pm_4bit_resume':
arch/arm/plat-samsung/pm-gpio.c:266:22: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:281:19: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:282:19: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:284:20: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:286:19: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:287:19: error: 'struct samsung_gpio_chip'
has no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c: At top level:
arch/arm/plat-samsung/pm-gpio.c:302:24: error: expected identifier or '('
before 'void'
arch/arm/plat-samsung/pm-gpio.c:302:24: error: expected ')' before numeric
constant
arch/arm/plat-samsung/pm-gpio.c: In function 'samsung_pm_save_gpios':
arch/arm/plat-samsung/pm-gpio.c:342:3: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:342:3: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:342:3: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c:342:3: error: 'struct samsung_gpio_chip' has
no member named 'pm_save'
arch/arm/plat-samsung/pm-gpio.c: At top level:
arch/arm/plat-samsung/pm-gpio.c:31:13: warning: 'samsung_gpio_pm_1bit_save'
defined but not used [-Wunused-function]
arch/arm/plat-samsung/pm-gpio.c:37:13: warning:
'samsung_gpio_pm_1bit_resume' defined but not used [-Wunused-function]
arch/arm/plat-samsung/pm-gpio.c:68:13: warning: 'samsung_gpio_pm_2bit_save'
defined but not used [-Wunused-function]
arch/arm/plat-samsung/pm-gpio.c:124:13: warning:
'samsung_gpio_pm_2bit_resume' defined but not used [-Wunused-function]
arch/arm/plat-samsung/pm-gpio.c:197:13: warning: 'samsung_gpio_pm_4bit_save'
defined but not used [-Wunused-function]
arch/arm/plat-samsung/pm-gpio.c:261:13: warning:
'samsung_gpio_pm_4bit_resume' defined but not used [-Wunused-function]
make[2]: *** [arch/arm/plat-samsung/pm-gpio.o] Error 1
make[2]: *** Waiting for unfinished jobs....

- Kukjin