diff mbox series

ARM: imx_v6_v7_defconfig: Fix the USB driver option selection

Message ID 20240408131540.1558782-1-festevam@gmail.com (mailing list archive)
State New, archived
Headers show
Series ARM: imx_v6_v7_defconfig: Fix the USB driver option selection | expand

Commit Message

Fabio Estevam April 8, 2024, 1:15 p.m. UTC
From: Fabio Estevam <festevam@denx.de>

The correct symbol is CONFIG_USB_ONBOARD_DEV, so fix it accordingly.

Tested on a imx6q-udoo board. USB hub is now detected as well as
the Wifi chip.

usb 1-1: New USB device found, idVendor=0424, idProduct=2514, bcdDevice= b.b3
usb 1-1: New USB device strings: Mfr=0, Product=0, SerialNumber=0
hub 1-1:1.0: USB hub found
hub 1-1:1.0: 4 ports detected
...

usb 1-1.3: New USB device found, idVendor=148f, idProduct=5370, bcdDevice= 1.01
usb 1-1.3: New USB device strings: Mfr=1, Product=2, SerialNumber=3
usb 1-1.3: Product: 802.11 n WLAN
usb 1-1.3: Manufacturer: Ralink
usb 1-1.3: SerialNumber: 1.0

Fixes: 0bb36055c049 ("ARM: imx_v6_v7_defconfig: Select CONFIG_USB_ONBOARD_HUB")
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/configs/imx_v6_v7_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Fabio Estevam April 8, 2024, 1:32 p.m. UTC | #1
On Mon, Apr 8, 2024 at 10:15 AM Fabio Estevam <festevam@gmail.com> wrote:
>
> From: Fabio Estevam <festevam@denx.de>
>
> The correct symbol is CONFIG_USB_ONBOARD_DEV, so fix it accordingly.
>
> Tested on a imx6q-udoo board. USB hub is now detected as well as
> the Wifi chip.
>
> usb 1-1: New USB device found, idVendor=0424, idProduct=2514, bcdDevice= b.b3
> usb 1-1: New USB device strings: Mfr=0, Product=0, SerialNumber=0
> hub 1-1:1.0: USB hub found
> hub 1-1:1.0: 4 ports detected
> ...
>
> usb 1-1.3: New USB device found, idVendor=148f, idProduct=5370, bcdDevice= 1.01
> usb 1-1.3: New USB device strings: Mfr=1, Product=2, SerialNumber=3
> usb 1-1.3: Product: 802.11 n WLAN
> usb 1-1.3: Manufacturer: Ralink
> usb 1-1.3: SerialNumber: 1.0
>
> Fixes: 0bb36055c049 ("ARM: imx_v6_v7_defconfig: Select CONFIG_USB_ONBOARD_HUB")
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>  arch/arm/configs/imx_v6_v7_defconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
> index b06128da94e5..cf2480dce285 100644
> --- a/arch/arm/configs/imx_v6_v7_defconfig
> +++ b/arch/arm/configs/imx_v6_v7_defconfig
> @@ -336,7 +336,7 @@ CONFIG_USB_SERIAL_FTDI_SIO=m
>  CONFIG_USB_SERIAL_OPTION=m
>  CONFIG_USB_TEST=m
>  CONFIG_USB_EHSET_TEST_FIXTURE=m
> -CONFIG_USB_ONBOARD_HUB=y
> +CONFIG_USB_ONBOARD_DEV=y

Ah, in linux-next there is the following change that did the rename:
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?h=next-20240408&id=31e7f6c015d9eb35e77ae9868801c53ab0ff19ac

I will send a v2 that points to this commit in the Fixes tag.
diff mbox series

Patch

diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
index b06128da94e5..cf2480dce285 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm/configs/imx_v6_v7_defconfig
@@ -336,7 +336,7 @@  CONFIG_USB_SERIAL_FTDI_SIO=m
 CONFIG_USB_SERIAL_OPTION=m
 CONFIG_USB_TEST=m
 CONFIG_USB_EHSET_TEST_FIXTURE=m
-CONFIG_USB_ONBOARD_HUB=y
+CONFIG_USB_ONBOARD_DEV=y
 CONFIG_NOP_USB_XCEIV=y
 CONFIG_USB_MXS_PHY=y
 CONFIG_USB_GADGET=y