From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Mon, 06 Jun 2016 17:59:30 +0200 Subject: [U-Boot] [PATCH] usb: dwc2_udc_otg: support 8-bit interface In-Reply-To: <1465228657-21027-1-git-send-email-srae@broadcom.com> References: <1465228657-21027-1-git-send-email-srae@broadcom.com> Message-ID: <57559DE2.6000205@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 06/06/2016 05:57 PM, Steve Rae wrote: > Define CONFIG_USB_GADGET_DWC2_PHY_8_BIT to allow the > physical interface to be 8-bit (rather than 16-bit). > > Signed-off-by: Steve Rae Can this config option be picked from DT rather than hard-coded by some new ad-hoc config option ? Also, new options should be Kconfig'd . Thanks > --- > > drivers/usb/gadget/dwc2_udc_otg.c | 4 ++++ > include/configs/bcm28155_ap.h | 1 + > 2 files changed, 5 insertions(+) > > diff --git a/drivers/usb/gadget/dwc2_udc_otg.c b/drivers/usb/gadget/dwc2_udc_otg.c > index cb20b00..a95c8d5 100644 > --- a/drivers/usb/gadget/dwc2_udc_otg.c > +++ b/drivers/usb/gadget/dwc2_udc_otg.c > @@ -415,7 +415,11 @@ static void reconfig_usbd(struct dwc2_udc *dev) > |0<<7 /* Ulpi DDR sel*/ > |0<<6 /* 0: high speed utmi+, 1: full speed serial*/ > |0<<4 /* 0: utmi+, 1:ulpi*/ > +#ifdef CONFIG_USB_GADGET_DWC2_PHY_8_BIT > + |0<<3 /* phy i/f 0:8bit, 1:16bit*/ > +#else > |1<<3 /* phy i/f 0:8bit, 1:16bit*/ > +#endif > |0x7<<0; /* HS/FS Timeout**/ > > if (dev->pdata->usb_gusbcfg) > diff --git a/include/configs/bcm28155_ap.h b/include/configs/bcm28155_ap.h > index 889e5db..e04cfbe 100644 > --- a/include/configs/bcm28155_ap.h > +++ b/include/configs/bcm28155_ap.h > @@ -134,6 +134,7 @@ > #define CONFIG_SYS_CACHELINE_SIZE 64 > #define CONFIG_FASTBOOT_BUF_SIZE (CONFIG_SYS_SDRAM_SIZE - SZ_1M) > #define CONFIG_FASTBOOT_BUF_ADDR CONFIG_SYS_SDRAM_BASE > +#define CONFIG_USB_GADGET_DWC2_PHY_8_BIT > #define CONFIG_USB_GADGET_BCM_UDC_OTG_PHY > #define CONFIG_USBID_ADDR 0x34052c46 > > -- Best regards, Marek Vasut