[LEDE-DEV] [PATCH v2 1/8] apm821xx: add support for the apm821xx device target

[Felix Fietkau]

On 2016-07-13 21:23, Christian Lamparter wrote:
> From: Chris Blake <chrisrblake93 at gmail.com>
> 
> This adds a new target for PowerPC APM82181 and APM82161
> (464-based) boards, as well as adds support for the booke-wdt
> watchdog package.
> 
> Signed-off-by: Chris Blake <chrisrblake93 at gmail.com>
> ---
>  include/target.mk                                  |   1 +
>  package/kernel/linux/modules/other.mk              |   2 +-
>  target/linux/apm821xx/Makefile                     |  28 ++
>  .../linux/apm821xx/base-files/etc/board.d/01_leds  |  17 +
>  .../apm821xx/base-files/etc/board.d/02_network     |  19 +
>  target/linux/apm821xx/base-files/etc/diag.sh       |  33 ++
>  target/linux/apm821xx/base-files/etc/inittab       |   4 +
>  target/linux/apm821xx/base-files/lib/apm821xx.sh   |  36 ++
>  .../lib/preinit/03_preinit_do_apm821xx.sh          |   9 +
>  .../lib/preinit/05_set_preinit_iface_apm821xx      |   9 +
>  .../apm821xx/base-files/lib/upgrade/platform.sh    |  58 +++
>  target/linux/apm821xx/config-4.4                   | 327 +++++++++++++
>  target/linux/apm821xx/image/Makefile               |  22 +
>  ...to4xx-integrate-ppc4xx-rng-into-crypto4xx.patch | 507 +++++++++++++++++++++
>  .../300-fix-atheros-nics-on-apm82181.patch         |  51 +++
>  .../701-powerpc_ibm_apm82181_phyclk_fix.patch      |  56 +++
>  16 files changed, 1178 insertions(+), 1 deletion(-)
>  create mode 100644 target/linux/apm821xx/Makefile
>  create mode 100755 target/linux/apm821xx/base-files/etc/board.d/01_leds
>  create mode 100755 target/linux/apm821xx/base-files/etc/board.d/02_network
>  create mode 100755 target/linux/apm821xx/base-files/etc/diag.sh
>  create mode 100644 target/linux/apm821xx/base-files/etc/inittab
>  create mode 100755 target/linux/apm821xx/base-files/lib/apm821xx.sh
>  create mode 100644 target/linux/apm821xx/base-files/lib/preinit/03_preinit_do_apm821xx.sh
>  create mode 100644 target/linux/apm821xx/base-files/lib/preinit/05_set_preinit_iface_apm821xx
>  create mode 100755 target/linux/apm821xx/base-files/lib/upgrade/platform.sh
>  create mode 100644 target/linux/apm821xx/config-4.4
>  create mode 100644 target/linux/apm821xx/image/Makefile
>  create mode 100644 target/linux/apm821xx/patches-4.4/001-crypto4xx-integrate-ppc4xx-rng-into-crypto4xx.patch
>  create mode 100644 target/linux/apm821xx/patches-4.4/300-fix-atheros-nics-on-apm82181.patch
>  create mode 100644 target/linux/apm821xx/patches-4.4/701-powerpc_ibm_apm82181_phyclk_fix.patch
> 
> diff --git a/include/target.mk b/include/target.mk
> index b5153ff..7fb298a 100644
> --- a/include/target.mk
> +++ b/include/target.mk
> @@ -208,6 +208,7 @@ ifeq ($(DUMP),1)
>      CPU_CFLAGS_8540:=-mcpu=8540
>      CPU_CFLAGS_405:=-mcpu=405
>      CPU_CFLAGS_440:=-mcpu=440
> +    CPU_CFLAGS_464fp:=-mcpu=464fp
>    endif
>    ifeq ($(ARCH),sparc)
>      CPU_TYPE = sparc
> diff --git a/package/kernel/linux/modules/other.mk b/package/kernel/linux/modules/other.mk
> index a1fce48..ee1ee06 100644
> --- a/package/kernel/linux/modules/other.mk
> +++ b/package/kernel/linux/modules/other.mk
> @@ -517,7 +517,7 @@ $(eval $(call KernelPackage,wdt-orion))
>  define KernelPackage/booke-wdt
>    SUBMENU:=$(OTHER_MENU)
>    TITLE:=PowerPC Book-E Watchdog Timer
> -  DEPENDS:=@(TARGET_mpc85xx||TARGET_ppc40x||TARGET_ppc44x)
> +  DEPENDS:=@(TARGET_apm821xx||TARGET_mpc85xx||TARGET_ppc40x||TARGET_ppc44x)
>    KCONFIG:=CONFIG_BOOKE_WDT
>    FILES:=$(LINUX_DIR)/drivers/$(WATCHDOG_DIR)/booke_wdt.ko
>    AUTOLOAD:=$(call AutoLoad,50,booke_wdt,1)
Is there any reason to keep this as module? Unless there's a good reason
to do it this way, I would like to make this built-in in the kernel
config on all relevant targets and get rid of the kmod package.

- Felix