[PATCH v4 2/6] at91: enable specific sam9x kernel config flags

[Claudiu.Beznea at microchip.com]

On 04.10.2021 01:52, Hauke Mehrtens wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the
> content is safe
> 
> On 9/20/21 11:27 AM, Claudiu Beznea wrote:
>> Enable specific sam9x kernel config flags.
>>
>> Signed-off-by: Claudiu Beznea <claudiu.beznea at microchip.com>
>> ---
>>   target/linux/at91/sam9x/config-default | 168 +++++++++++++++++++++++--
>>   1 file changed, 159 insertions(+), 9 deletions(-)
>>
>> diff --git a/target/linux/at91/sam9x/config-default
>> b/target/linux/at91/sam9x/config-default
>> index d8b5c31dc7e5..a9f66766b1f2 100644
>> --- a/target/linux/at91/sam9x/config-default
>> +++ b/target/linux/at91/sam9x/config-default
>> @@ -1,7 +1,6 @@
> .....
>>   CONFIG_INITRAMFS_SOURCE=""
>> +CONFIG_INPUT=y
>> +CONFIG_INPUT_KEYBOARD=y
> 
> Do you really need keyboard support in OpenWrt?
> OpenWrt uses some patches to hanlde buttons like a wifi on / off switch.

I'll double check it.

> 
>>   CONFIG_IRQCHIP=y
>>   CONFIG_IRQ_DOMAIN=y
>>   CONFIG_IRQ_FORCED_THREADING=y
>>   CONFIG_IRQ_WORK=y
>> +CONFIG_JBD2=y
>> +CONFIG_LEDS_GPIO=y
>> +CONFIG_LEDS_PWM=y
>> +CONFIG_LEDS_TRIGGER_CPU=y
>> +CONFIG_LEDS_TRIGGER_GPIO=y
> .....
>> +CONFIG_USB=y
>> +CONFIG_USB_ACM=y
>> +CONFIG_USB_AT91=y
>> +CONFIG_USB_ATMEL_USBA=y
>> +CONFIG_USB_COMMON=y
>> +CONFIG_USB_CONFIGFS=y
> 
> I would prefer if you put the USB gadget stuff into a optional openwrt
> kmod kernel package instead of compiling this into the kernel.

On sama5d2 target I have this selections in config-default:
CONFIG_USB=y
CONFIG_USB_ACM=y
CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
# CONFIG_USB_AT91 is not set
# CONFIG_USB_ATMEL_USBA is not set
# CONFIG_USB_AUDIO is not set
CONFIG_USB_COMMON=y
CONFIG_USB_EHCI_HCD=y
CONFIG_USB_EHCI_HCD_AT91=y
# CONFIG_USB_EHCI_HCD_PLATFORM is not set
# CONFIG_USB_EHCI_ROOT_HUB_TT is not set
# CONFIG_USB_ETH is not set
CONFIG_USB_GADGET=y
CONFIG_USB_HID=y
CONFIG_USB_OHCI_HCD=y
CONFIG_USB_OHCI_HCD_AT91=y
CONFIG_USB_OHCI_HCD_PLATFORM=y
# CONFIG_USB_PWC is not set
CONFIG_USB_SERIAL=y
# CONFIG_USB_SERIAL_CONSOLE is not set
CONFIG_USB_SERIAL_FTDI_SIO=y
CONFIG_USB_SERIAL_PL2303=y
CONFIG_USB_STORAGE=y
CONFIG_USB_SUPPORT=y

Would you prefer to do it like that also for sam9x config-default or would
you prefer to have all the usb stuff out of config-default and rely
strictly on openwrt kmod kernel package?

Thank you,
Claudiu Beznea

> 
>> +CONFIG_USB_CONFIGFS_ACM=y
>> +# CONFIG_USB_CONFIGFS_ECM is not set
>> +# CONFIG_USB_CONFIGFS_ECM_SUBSET is not set
>> +# CONFIG_USB_CONFIGFS_EEM is not set
>> +# CONFIG_USB_CONFIGFS_F_FS is not set
>> +# CONFIG_USB_CONFIGFS_F_HID is not set
>> +# CONFIG_USB_CONFIGFS_F_LB_SS is not set
>> +# CONFIG_USB_CONFIGFS_F_PRINTER is not set
>> +CONFIG_USB_CONFIGFS_MASS_STORAGE=y
>> +# CONFIG_USB_CONFIGFS_NCM is not set
>> +# CONFIG_USB_CONFIGFS_OBEX is not set
>> +# CONFIG_USB_CONFIGFS_RNDIS is not set
>> +# CONFIG_USB_CONFIGFS_SERIAL is not set
>> +CONFIG_USB_EHCI_HCD=y
>> +CONFIG_USB_EHCI_HCD_AT91=y
>> +# CONFIG_USB_EHCI_HCD_PLATFORM is not set
>> +CONFIG_USB_ETH=y
>> +# CONFIG_USB_ETH_EEM is not set
>> +CONFIG_USB_ETH_RNDIS=y
>> +CONFIG_USB_F_ACM=y
>> +CONFIG_USB_F_ECM=y
>> +CONFIG_USB_F_MASS_STORAGE=y
>> +CONFIG_USB_F_RNDIS=y
>> +CONFIG_USB_F_SUBSET=y
>> +CONFIG_USB_GADGET=y
>> +CONFIG_USB_LIBCOMPOSITE=y
>> +CONFIG_USB_OHCI_HCD=y
>> +CONFIG_USB_OHCI_HCD_AT91=y
>> +CONFIG_USB_OHCI_HCD_PLATFORM=y
>> +CONFIG_USB_SERIAL=y
>> +# CONFIG_USB_SERIAL_CONSOLE is not set
>> +CONFIG_USB_SERIAL_FTDI_SIO=y
> .....
>