Re: [PATCH v3 2/3] configs: imx93-phyboard-segin: Add USB support
Hi Fabio, hi Marek, On Sun, 2024-03-24 at 15:00 -0300, Fabio Estevam wrote: > On Thu, Mar 21, 2024 at 4:17 PM Marek Vasut wrote: > > > $ git grep -i usb.*phytec configs > > configs/phycore_pcl063_defconfig:CONFIG_USB_GADGET_MANUFACTURER="Phytec" > > configs/phycore_pcl063_ull_defconfig:CONFIG_USB_GADGET_MANUFACTURER="Phytec" > > > > It would be good to be consistent. > > > > Also, what is the vendor/product number those two boards use ? > > They both use: > > CONFIG_USB_GADGET_VENDOR_NUM=0x01b67 > CONFIG_USB_GADGET_PRODUCT_NUM=0x4fff > > configs/phycore-imx8mp_defconfig has: > > CONFIG_USB_GADGET_MANUFACTURER="FSL" > CONFIG_USB_GADGET_VENDOR_NUM=0x0525 > CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 > > I agree we should make this consistent. > > To not block this series, I applied it, but it would be great if > > Phytec could submit > a separate series making it consistent across their boards. We'll take care of that. Thank you for the feedback. Yannic > > Thanks
Re: [PATCH v3 2/3] configs: imx93-phyboard-segin: Add USB support
On 3/24/24 7:00 PM, Fabio Estevam wrote: On Thu, Mar 21, 2024 at 4:17 PM Marek Vasut wrote: $ git grep -i usb.*phytec configs configs/phycore_pcl063_defconfig:CONFIG_USB_GADGET_MANUFACTURER="Phytec" configs/phycore_pcl063_ull_defconfig:CONFIG_USB_GADGET_MANUFACTURER="Phytec" It would be good to be consistent. Also, what is the vendor/product number those two boards use ? They both use: CONFIG_USB_GADGET_VENDOR_NUM=0x01b67 USB VID is 16bit number, the nubmer above is 20bit , drop the leading zero while at it.
Re: [PATCH v3 2/3] configs: imx93-phyboard-segin: Add USB support
On Thu, Mar 21, 2024 at 4:17 PM Marek Vasut wrote: > $ git grep -i usb.*phytec configs > configs/phycore_pcl063_defconfig:CONFIG_USB_GADGET_MANUFACTURER="Phytec" > configs/phycore_pcl063_ull_defconfig:CONFIG_USB_GADGET_MANUFACTURER="Phytec" > > It would be good to be consistent. > > Also, what is the vendor/product number those two boards use ? They both use: CONFIG_USB_GADGET_VENDOR_NUM=0x01b67 CONFIG_USB_GADGET_PRODUCT_NUM=0x4fff configs/phycore-imx8mp_defconfig has: CONFIG_USB_GADGET_MANUFACTURER="FSL" CONFIG_USB_GADGET_VENDOR_NUM=0x0525 CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 I agree we should make this consistent. To not block this series, I applied it, but it would be great if Phytec could submit a separate series making it consistent across their boards. Thanks
Re: [PATCH v3 2/3] configs: imx93-phyboard-segin: Add USB support
On 3/21/24 3:45 PM, Mathieu Othacehe wrote: Add USB support by enabling `usb` command and required USB drivers. Signed-off-by: Mathieu Othacehe --- configs/imx93-phyboard-segin_defconfig | 8 1 file changed, 8 insertions(+) diff --git a/configs/imx93-phyboard-segin_defconfig b/configs/imx93-phyboard-segin_defconfig index 24f9bd553aa..0345e7deeab 100644 --- a/configs/imx93-phyboard-segin_defconfig +++ b/configs/imx93-phyboard-segin_defconfig @@ -67,6 +67,7 @@ CONFIG_CMD_GPIO=y CONFIG_CMD_I2C=y CONFIG_CMD_MMC=y CONFIG_CMD_POWEROFF=y +CONFIG_CMD_USB=y CONFIG_CMD_SNTP=y CONFIG_CMD_CACHE=y CONFIG_CMD_EFIDEBUG=y @@ -132,6 +133,13 @@ CONFIG_SPL_SYSRESET=y CONFIG_SYSRESET_WATCHDOG=y CONFIG_DM_THERMAL=y CONFIG_IMX_TMU=y +CONFIG_USB=y +CONFIG_USB_EHCI_HCD=y +CONFIG_USB_GADGET=y +CONFIG_USB_GADGET_MANUFACTURER="PHYTEC" $ git grep -i usb.*phytec configs configs/phycore_pcl063_defconfig:CONFIG_USB_GADGET_MANUFACTURER="Phytec" configs/phycore_pcl063_ull_defconfig:CONFIG_USB_GADGET_MANUFACTURER="Phytec" It would be good to be consistent. Also, what is the vendor/product number those two boards use ? Also, it would be good to fix the extra leading zero in those board boards vendor number, this, in both configs: CONFIG_USB_GADGET_VENDOR_NUM=0x01b67
[PATCH v3 2/3] configs: imx93-phyboard-segin: Add USB support
Add USB support by enabling `usb` command and required USB drivers. Signed-off-by: Mathieu Othacehe --- configs/imx93-phyboard-segin_defconfig | 8 1 file changed, 8 insertions(+) diff --git a/configs/imx93-phyboard-segin_defconfig b/configs/imx93-phyboard-segin_defconfig index 24f9bd553aa..0345e7deeab 100644 --- a/configs/imx93-phyboard-segin_defconfig +++ b/configs/imx93-phyboard-segin_defconfig @@ -67,6 +67,7 @@ CONFIG_CMD_GPIO=y CONFIG_CMD_I2C=y CONFIG_CMD_MMC=y CONFIG_CMD_POWEROFF=y +CONFIG_CMD_USB=y CONFIG_CMD_SNTP=y CONFIG_CMD_CACHE=y CONFIG_CMD_EFIDEBUG=y @@ -132,6 +133,13 @@ CONFIG_SPL_SYSRESET=y CONFIG_SYSRESET_WATCHDOG=y CONFIG_DM_THERMAL=y CONFIG_IMX_TMU=y +CONFIG_USB=y +CONFIG_USB_EHCI_HCD=y +CONFIG_USB_GADGET=y +CONFIG_USB_GADGET_MANUFACTURER="PHYTEC" +CONFIG_USB_GADGET_VENDOR_NUM=0x1fc9 +CONFIG_USB_GADGET_PRODUCT_NUM=0x0152 +CONFIG_CI_UDC=y CONFIG_ULP_WATCHDOG=y CONFIG_LZO=y CONFIG_BZIP2=y -- 2.41.0