[v3] ARM: Add wpcm450_defconfig for Nuvoton WPCM450

This defconfig aims to offer a reasonable set of defaults for all
systems running on a Nuvoton WPCM450 chip.

Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
v3:
- Regenerate for 6.1-rc1 and enable a few Nuvoton-specific options as
  well as the PECI subsystem

v2:
- Regenerate for 5.19-rc1
---
 MAINTAINERS                        |   1 +
 arch/arm/configs/wpcm450_defconfig | 213 +++++++++++++++++++++++++++++
 2 files changed, 214 insertions(+)
 create mode 100644 arch/arm/configs/wpcm450_defconfig

--
2.35.1

On Tue, 1 Nov 2022 at 11:22, Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote:
>
> This defconfig aims to offer a reasonable set of defaults for all
> systems running on a Nuvoton WPCM450 chip.

I was going to ask if we could instead have a common nuvoton
defconfig, but this is an ARMv5 part so we can't also select the ARMv7
npcm750.

Reviewed-by: Joel Stanley <joel@jms.id.au>

>
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> ---
> v3:
> - Regenerate for 6.1-rc1 and enable a few Nuvoton-specific options as
>   well as the PECI subsystem
>
> v2:
> - Regenerate for 5.19-rc1
> ---
>  MAINTAINERS                        |   1 +
>  arch/arm/configs/wpcm450_defconfig | 213 +++++++++++++++++++++++++++++
>  2 files changed, 214 insertions(+)
>  create mode 100644 arch/arm/configs/wpcm450_defconfig
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index cf0f185023724..cac5a4ad4eb8a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2543,6 +2543,7 @@ S:        Maintained
>  W:     https://github.com/neuschaefer/wpcm450/wiki
>  F:     Documentation/devicetree/bindings/*/*wpcm*
>  F:     arch/arm/boot/dts/nuvoton-wpcm450*
> +F:     arch/arm/configs/wpcm450_defconfig
>  F:     arch/arm/mach-npcm/wpcm450.c
>  F:     drivers/*/*/*wpcm*
>  F:     drivers/*/*wpcm*
> diff --git a/arch/arm/configs/wpcm450_defconfig b/arch/arm/configs/wpcm450_defconfig
> new file mode 100644
> index 0000000000000..702f2bf909f93
> --- /dev/null
> +++ b/arch/arm/configs/wpcm450_defconfig
> @@ -0,0 +1,213 @@
> +CONFIG_SYSVIPC=y
> +CONFIG_NO_HZ_IDLE=y
> +CONFIG_HIGH_RES_TIMERS=y
> +CONFIG_PREEMPT=y
> +CONFIG_IKCONFIG=y
> +CONFIG_IKCONFIG_PROC=y
> +CONFIG_LOG_BUF_SHIFT=19
> +CONFIG_CGROUPS=y
> +CONFIG_BLK_DEV_INITRD=y
> +CONFIG_CC_OPTIMIZE_FOR_SIZE=y
> +CONFIG_PROFILING=y
> +# CONFIG_ARCH_MULTI_V7 is not set
> +CONFIG_ARCH_NPCM=y
> +CONFIG_ARCH_WPCM450=y
> +CONFIG_CPU_DCACHE_WRITETHROUGH=y
> +CONFIG_AEABI=y
> +CONFIG_UACCESS_WITH_MEMCPY=y
> +# CONFIG_ATAGS is not set
> +CONFIG_ARM_APPENDED_DTB=y

Do you use this? OpenBMC systems tend to just use FIT, so we don't
need this option.

> +CONFIG_KEXEC=y
> +CONFIG_CPU_FREQ=y

Does the wpcm do frequency scaling? If not you could disable this.

> +CONFIG_CPU_FREQ_STAT=y
> +CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND=y
> +CONFIG_CPU_IDLE=y
> +CONFIG_KPROBES=y
> +CONFIG_JUMP_LABEL=y
> +CONFIG_STRICT_KERNEL_RWX=y
> +CONFIG_MODULES=y
> +CONFIG_MODULE_UNLOAD=y
> +CONFIG_NET=y
> +CONFIG_PACKET=y
> +CONFIG_PACKET_DIAG=y
> +CONFIG_UNIX=y
> +CONFIG_UNIX_DIAG=y
> +CONFIG_INET=y
> +CONFIG_IP_MULTICAST=y
> +CONFIG_IP_PNP=y
> +CONFIG_IP_PNP_DHCP=y
> +CONFIG_IP_PNP_BOOTP=y
> +CONFIG_NET_DSA=y
> +CONFIG_NET_DSA_TAG_DSA=y
> +CONFIG_NET_DSA_TAG_EDSA=y
> +CONFIG_NET_DSA_TAG_TRAILER=y
> +CONFIG_NET_PKTGEN=m
> +# CONFIG_WIRELESS is not set
> +CONFIG_DEVTMPFS=y
> +CONFIG_DEVTMPFS_MOUNT=y
> +CONFIG_MTD=y
> +CONFIG_MTD_CMDLINE_PARTS=y
> +CONFIG_MTD_BLOCK=y
> +CONFIG_MTD_SPI_NOR=y
> +CONFIG_MTD_UBI=y
> +CONFIG_MTD_UBI_FASTMAP=y
> +CONFIG_MTD_UBI_BLOCK=y
> +CONFIG_BLK_DEV_LOOP=y
> +CONFIG_SRAM=y
> +CONFIG_EEPROM_AT24=y
> +CONFIG_SCSI=y
> +# CONFIG_SCSI_PROC_FS is not set
> +# CONFIG_SCSI_LOWLEVEL is not set
> +CONFIG_NETDEVICES=y
> +# CONFIG_NET_VENDOR_ALACRITECH is not set
> +# CONFIG_NET_VENDOR_AMAZON is not set
> +# CONFIG_NET_VENDOR_AQUANTIA is not set
> +# CONFIG_NET_VENDOR_ARC is not set
> +# CONFIG_NET_VENDOR_BROADCOM is not set
> +# CONFIG_NET_VENDOR_CADENCE is not set
> +# CONFIG_NET_VENDOR_CAVIUM is not set
> +# CONFIG_NET_VENDOR_CIRRUS is not set
> +# CONFIG_NET_VENDOR_CORTINA is not set
> +# CONFIG_NET_VENDOR_EZCHIP is not set
> +# CONFIG_NET_VENDOR_FARADAY is not set
> +# CONFIG_NET_VENDOR_GOOGLE is not set
> +# CONFIG_NET_VENDOR_HISILICON is not set
> +# CONFIG_NET_VENDOR_HUAWEI is not set
> +# CONFIG_NET_VENDOR_INTEL is not set
> +# CONFIG_NET_VENDOR_MARVELL is not set
> +# CONFIG_NET_VENDOR_MELLANOX is not set
> +# CONFIG_NET_VENDOR_MICREL is not set
> +# CONFIG_NET_VENDOR_MICROCHIP is not set
> +# CONFIG_NET_VENDOR_MICROSEMI is not set
> +# CONFIG_NET_VENDOR_NI is not set
> +# CONFIG_NET_VENDOR_NATSEMI is not set
> +# CONFIG_NET_VENDOR_NETRONOME is not set
> +# CONFIG_NET_VENDOR_PENSANDO is not set
> +# CONFIG_NET_VENDOR_QUALCOMM is not set
> +# CONFIG_NET_VENDOR_RENESAS is not set
> +# CONFIG_NET_VENDOR_ROCKER is not set
> +# CONFIG_NET_VENDOR_SAMSUNG is not set
> +# CONFIG_NET_VENDOR_SEEQ is not set
> +# CONFIG_NET_VENDOR_SOLARFLARE is not set
> +# CONFIG_NET_VENDOR_SMSC is not set
> +# CONFIG_NET_VENDOR_SOCIONEXT is not set
> +# CONFIG_NET_VENDOR_STMICRO is not set
> +# CONFIG_NET_VENDOR_SYNOPSYS is not set
> +# CONFIG_NET_VENDOR_VIA is not set
> +# CONFIG_NET_VENDOR_WIZNET is not set
> +# CONFIG_NET_VENDOR_XILINX is not set
> +CONFIG_REALTEK_PHY=y
> +# CONFIG_WLAN is not set
> +CONFIG_INPUT_FF_MEMLESS=y
> +CONFIG_INPUT_EVDEV=y
> +CONFIG_KEYBOARD_QT1070=m
> +CONFIG_KEYBOARD_GPIO=y
> +# CONFIG_INPUT_MOUSE is not set
> +CONFIG_VT_HW_CONSOLE_BINDING=y
> +CONFIG_LEGACY_PTY_COUNT=16
> +CONFIG_SERIAL_8250=y
> +CONFIG_SERIAL_8250_CONSOLE=y
> +CONFIG_SERIAL_8250_NR_UARTS=6
> +CONFIG_SERIAL_8250_RUNTIME_UARTS=6
> +CONFIG_SERIAL_8250_EXTENDED=y
> +CONFIG_SERIAL_8250_MANY_PORTS=y
> +CONFIG_SERIAL_OF_PLATFORM=y
> +CONFIG_NPCM7XX_KCS_IPMI_BMC=y
> +CONFIG_IPMI_KCS_BMC_CDEV_IPMI=y
> +CONFIG_IPMI_KCS_BMC_SERIO=y
> +CONFIG_HW_RANDOM=y
> +# CONFIG_HW_RANDOM_NPCM is not set
> +CONFIG_I2C=y
> +# CONFIG_I2C_COMPAT is not set
> +CONFIG_I2C_CHARDEV=y
> +CONFIG_I2C_MUX=y
> +CONFIG_I2C_NPCM=y
> +CONFIG_SPI=y
> +CONFIG_SPI_NPCM_PSPI=y
> +CONFIG_PINCTRL_SINGLE=y
> +CONFIG_PINCTRL_WPCM450=y
> +CONFIG_POWER_SUPPLY=y
> +CONFIG_SENSORS_NPCM7XX=y
> +# CONFIG_THERMAL is not set
> +CONFIG_WATCHDOG=y
> +CONFIG_NPCM7XX_WATCHDOG=y
> +CONFIG_MFD_SYSCON=y
> +CONFIG_REGULATOR=y
> +CONFIG_REGULATOR_FIXED_VOLTAGE=y
> +CONFIG_FB=y
> +CONFIG_FB_MODE_HELPERS=y
> +# CONFIG_HID is not set
> +CONFIG_USB_CHIPIDEA=y
> +CONFIG_USB_CHIPIDEA_UDC=y
> +CONFIG_USB_GADGET=y
> +CONFIG_USB_CONFIGFS=y
> +CONFIG_USB_CONFIGFS_SERIAL=y
> +CONFIG_USB_CONFIGFS_ACM=y
> +CONFIG_USB_CONFIGFS_EEM=y
> +CONFIG_USB_CONFIGFS_MASS_STORAGE=y
> +CONFIG_USB_CONFIGFS_F_FS=y
> +CONFIG_USB_CONFIGFS_F_HID=y
> +CONFIG_NEW_LEDS=y
> +CONFIG_LEDS_CLASS=y
> +CONFIG_LEDS_GPIO=y
> +CONFIG_LEDS_TRIGGERS=y
> +CONFIG_LEDS_TRIGGER_TIMER=y
> +CONFIG_LEDS_TRIGGER_HEARTBEAT=y
> +CONFIG_LEDS_TRIGGER_DEFAULT_ON=y
> +CONFIG_DMADEVICES=y
> +CONFIG_SYNC_FILE=y
> +# CONFIG_VIRTIO_MENU is not set
> +# CONFIG_VHOST_MENU is not set
> +CONFIG_STAGING=y
> +# CONFIG_IOMMU_SUPPORT is not set
> +CONFIG_PWM=y
> +CONFIG_GENERIC_PHY=y
> +CONFIG_PECI=y
> +CONFIG_PECI_CPU=y
> +CONFIG_MSDOS_FS=y
> +CONFIG_VFAT_FS=y
> +CONFIG_TMPFS=y
> +CONFIG_UBIFS_FS=y
> +CONFIG_SQUASHFS=y
> +CONFIG_SQUASHFS_XZ=y
> +CONFIG_SQUASHFS_ZSTD=y
> +# CONFIG_NETWORK_FILESYSTEMS is not set
> +CONFIG_NLS_CODEPAGE_437=y
> +CONFIG_NLS_CODEPAGE_850=y
> +CONFIG_NLS_ISO8859_1=y
> +CONFIG_NLS_ISO8859_2=y
> +CONFIG_NLS_UTF8=y
> +CONFIG_KEYS=y
> +CONFIG_HARDENED_USERCOPY=y
> +CONFIG_FORTIFY_SOURCE=y
> +CONFIG_CRYPTO_RSA=y
> +CONFIG_CRYPTO_AES=y
> +CONFIG_CRYPTO_CBC=m
> +CONFIG_CRYPTO_PCBC=m
> +CONFIG_CRYPTO_CCM=y
> +CONFIG_CRYPTO_GCM=y
> +CONFIG_CRYPTO_CMAC=y
> +CONFIG_CRYPTO_SHA256=y
> +CONFIG_ASYMMETRIC_KEY_TYPE=y
> +CONFIG_ASYMMETRIC_PUBLIC_KEY_SUBTYPE=y
> +CONFIG_X509_CERTIFICATE_PARSER=y
> +CONFIG_PKCS7_MESSAGE_PARSER=y
> +CONFIG_SYSTEM_TRUSTED_KEYRING=y
> +CONFIG_CRC_CCITT=y
> +CONFIG_CRC_ITU_T=m
> +CONFIG_LIBCRC32C=y
> +CONFIG_PRINTK_TIME=y
> +CONFIG_DEBUG_KERNEL=y
> +CONFIG_MAGIC_SYSRQ=y
> +CONFIG_DEBUG_FS=y
> +# CONFIG_SCHED_DEBUG is not set
> +# CONFIG_DEBUG_PREEMPT is not set
> +# CONFIG_FTRACE is not set
> +CONFIG_IO_STRICT_DEVMEM=y
> +CONFIG_DEBUG_USER=y
> +CONFIG_DEBUG_LL=y
> +CONFIG_DEBUG_LL_UART_8250=y
> +CONFIG_DEBUG_UART_PHYS=0xb8000000
> +CONFIG_DEBUG_UART_VIRT=0x0ff000000
> +CONFIG_DEBUG_UART_8250_WORD=y
> +CONFIG_EARLY_PRINTK=y
> --
> 2.35.1
>

On Wed, Nov 02, 2022 at 10:29:58PM +0000, Joel Stanley wrote:
> On Tue, 1 Nov 2022 at 11:22, Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote:
> >
> > This defconfig aims to offer a reasonable set of defaults for all
> > systems running on a Nuvoton WPCM450 chip.
> 
> I was going to ask if we could instead have a common nuvoton
> defconfig, but this is an ARMv5 part so we can't also select the ARMv7
> npcm750.

Correct.

> 
> Reviewed-by: Joel Stanley <joel@jms.id.au>
> 


> > +CONFIG_ARM_APPENDED_DTB=y
> 
> Do you use this? OpenBMC systems tend to just use FIT, so we don't
> need this option.

I use this, because there is no modern U-Boot port for WPCM450 yet
(as far as I'm aware).

> > +CONFIG_KEXEC=y

I also use kexec, it's useful during development, but I see that it's
not so useful for the BMC usecase.

> > +CONFIG_CPU_FREQ=y
> 
> Does the wpcm do frequency scaling? If not you could disable this.

No, as it currently stands the clock driver is read-only except for the
clock gates, and there is no cpufreq. I'll disable this option.

CPU_IDLE could plausibly be implemented on WPCM450, but currently isn't.


Thanks,
Jonathan