Re: [PATCH v2 14/15] ARM: configs: Add Milbeaut M10V defconfig

[Arnd Bergmann]

On Fri, Feb 8, 2019 at 1:30 PM Sugaya Taichi
<sugaya.taichi@xxxxxxxxxxxxx> wrote:
>
> This patch adds the minimal defconfig for the Milbeaut M10V.
>
> Signed-off-by: Sugaya Taichi <sugaya.taichi@xxxxxxxxxxxxx>
> ---
>  arch/arm/configs/milbeaut_m10v_defconfig | 366 +++++++++++++++++++++++++++++++
>  1 file changed, 366 insertions(+)


I think it would be good to reduce this a little bit. Please also add any
options you need for booting to multi_v7_defconfig. Device drivers you
don't need for booting but for additional features should be loadable
modules there.

> +CONFIG_KALLSYMS_ALL=y
> +CONFIG_EMBEDDED=y

CONFIG_EMBEDDED should not normally be set on device
configurations. Despite the name it's not something for general
embedded computing, but it's rather an option that lets you
turn other option that one should normally not have to change.

> +CONFIG_PROFILING=y
> +CONFIG_OPROFILE=m

Oprofile is rarely used these days, maybe change the default to
PERF_EVENTS=y?

> +CONFIG_HAVE_ARM_ARCH_TIMER=y
> +CONFIG_VMSPLIT_2G=y
> +CONFIG_PREEMPT=y
> +CONFIG_THUMB2_KERNEL=y

I think this will be the first armv8-a platform with thumb2 by default,
and it's probably a good thing to start testing that more.

> +CONFIG_HIGHMEM=y

It's unusual to have both VMSPLIT_2G and HIGHMEM.

> +CONFIG_ARM_APPENDED_DTB=y

If you are able to fix the boot loaders, this should be turned off

> +CONFIG_NL80211_TESTMODE=y
> +CONFIG_CFG80211_DEVELOPER_WARNINGS=y
> +CONFIG_CFG80211_REG_DEBUG=y

That seems like more debugging that you should need.

> +# CONFIG_I2C is not set
> +# CONFIG_I2C_BOARDINFO is not set
> +# CONFIG_I2C_COMPAT is not set
> +# CONFIG_I2C_CHARDEV is not set
> +# CONFIG_I2C_MUX is not set
> +# CONFIG_I2C_HELPER_AUTO is not set
> +# CONFIG_I2C_ALGOBIT is not set
> +# CONFIG_I2C_SNI_M10V is not set

If I2C is disabled, you shoulnd't need any of the later
options here.

> +CONFIG_FB=y
> +# CONFIG_VGA_ARB is not set

Since you don't have a framebuffer driver, and likely won't
add one (new drivers should use drivers/gpu/drm), I'd drop
this.

> +# CONFIG_USB is not set
> +# CONFIG_USB_ANNOUNCE_NEW_DEVICES is not set
> +# CONFIG_USB_MON is not set
> +# CONFIG_USB_XHCI_HCD is not set
> +# CONFIG_USB_EHCI_HCD is not set
> +# CONFIG_USB_EHCI_ROOT_HUB_TT is not set
...
> +# CONFIG_MMC is not set
> +# CONFIG_MMC_SDHCI is not set
> +# CONFIG_MMC_SDHCI_PLTFM is not set
> +# CONFIG_MMC_SDHCI_F_SDH30 is not set
> +# CONFIG_MMC_SDHCI_F_EMMC50 is not set

Also for USB and MMC: if you don't have the subsystem, you
should not need to list the other options.

> +CONFIG_NEW_LEDS=y
> +CONFIG_LEDS_CLASS=y
> +CONFIG_LEDS_GPIO=y
> +CONFIG_LEDS_TRIGGER_TIMER=y
> +CONFIG_LEDS_TRIGGER_HEARTBEAT=y
> +CONFIG_RTC_CLASS=y
> +# CONFIG_RTC_HCTOSYS is not set
> +# CONFIG_RTC_SYSTOHC is not set
> +CONFIG_RTC_DEBUG=y
> +CONFIG_RTC_DRV_RX8025=y
> +CONFIG_DMADEVICES=y
> +CONFIG_MB86S7X_HDMAC=y
> +CONFIG_MB8AC0300_XDMAC=y
> +CONFIG_UIO=y
> +CONFIG_UIO_SNI=y
> +# CONFIG_IOMMU_SUPPORT is not set
> +#CONFIG_PWM=y is not set
> +# CONFIG_PWM_SYSFS is not set
> +#CONFIG_PWM_M10V=y is not set
> +CONFIG_RESET_CONTROLLER=y
> +CONFIG_EXFAT=y

CONFIG_EXFAT does not exist in mainline kernels.

> +CONFIG_IPCU_FS=y
> +# CONFIG_TEST_IPCU_FS is not set

same for IPCU_FS, and probably a lot of other options.

        Arnd