Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
On 07/03/2016 09:37 PM, Rajesh Bhagat wrote: > > > Will take care in v3. > Did you send v3 patch set? York ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
> -Original Message- > From: Lukasz Majewski [mailto:l.majew...@samsung.com] > Sent: Thursday, June 23, 2016 8:23 PM > To: Rajat Srivastava> Cc: u-boot@lists.denx.de; s...@chromium.org; ma...@denx.de; > albert.u.b...@aribaud.net; prabha...@freescale.com; york sun > ; Mingkai Hu ; Rajesh Bhagat > ; michal.si...@xilinx.com; felipe.ba...@linux.intel.com > Subject: Re: [PATCH v2 2/4] usb: dwc3: Add helper functions to enable > snooping and > burst settings > > Hi Rajat, > > > Adds helper functions to enable snooping and outstanding burst beat > > settings. > > > > Signed-off-by: Rajat Srivastava > > Signed-off-by: Rajesh Bhagat > > --- > > Changes in v2: > > - Removes SoC specific flags and added helper functions > > > > drivers/usb/dwc3/core.c | 45 > > + > drivers/usb/dwc3/core.h > > | 7 +++ 2 files changed, 52 insertions(+) > > > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index > > 85cc96a..0b3c596 100644 > > --- a/drivers/usb/dwc3/core.c > > +++ b/drivers/usb/dwc3/core.c > > @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) > > > > #define DWC3_ALIGN_MASK(16 - 1) > > > > +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, > > +int breq_limit) > > +{ > > + struct dwc3 *dwc; > > + u32 reg; > > + > > + list_for_each_entry(dwc, _list, list) { > > + if (dwc->index != index) > > + continue; > > + > > + /* > > +* Change burst beat and outstanding pipelined > > +* transfers requests > > +*/ > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > > + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > > + > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); > > + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << > > 8); > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); > > + break; > > + } > > +} > > + > > +void dwc3_core_set_snooping(int index, bool snoop) { > > + struct dwc3 *dwc; > > + u32 reg; > > + > > + list_for_each_entry(dwc, _list, list) { > > + if (dwc->index != index) > > + continue; > > + > > + /* Enable/Disable snooping */ > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > > + if (snoop) > > + reg = reg | DWC3_SNOOP_ENABLE; Hello Lukasz, > > You can replace the above line with reg |= DWC3_SNOOP_ENABLE > Will take care in v3. > > + else > > + reg = reg & ~DWC3_SNOOP_ENABLE; > > The same here - req &= ~DWC3_SNOOP_ENABLE > Will take care in v3. > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > > + break; > > + } > > +} > > + > > /** > > * dwc3_uboot_init - dwc3 core uboot initialization code > > * @dwc3_dev: struct dwc3_device containing initialization data diff > > --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index > > 72d2fcd..455e7fa 100644 > > --- a/drivers/usb/dwc3/core.h > > +++ b/drivers/usb/dwc3/core.h > > @@ -593,6 +593,13 @@ struct dwc3_hwparams { > > /* HWPARAMS7 */ > > #define DWC3_RAM1_DEPTH(n) ((n) & 0x) > > > > +/* GSBUSCFG0 */ > > +#define DWC3_SNOOP_ENABLE (0x) > > +#define DWC3_INCR_BTYPE_MASK (0xff) > > + > > +/* GSBUSCFG1 */ > > +#define DWC3_BREQ_LIMIT_MASK (0xf00) > > + > > struct dwc3_request { > > struct usb_request request; > > struct list_headlist; > > Best Regards, Rajesh Bhagat > > -- > Best regards, > > Lukasz Majewski > > Samsung R Institute Poland (SRPOL) | Linux Platform Group ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
> -Original Message- > From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass > Sent: Friday, June 17, 2016 9:22 AM > To: Rajesh Bhagat> Cc: Rajat Srivastava ; U-Boot Mailing List b...@lists.denx.de>; Lukasz Majewski ; Marek Vašut > ; Albert ARIBAUD ; Prabhakar > Kushwaha ; york sun ; Mingkai Hu > ; Michal Simek ; > felipe.ba...@linux.intel.com > Subject: Re: [PATCH v2 2/4] usb: dwc3: Add helper functions to enable > snooping and > burst settings > > Hi Rajesh, > > On 13 June 2016 at 22:09, Rajesh Bhagat wrote: > > > > > >> -Original Message- > >> From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass > >> Sent: Friday, June 10, 2016 6:05 AM > >> To: Rajat Srivastava > >> Cc: U-Boot Mailing List ; Lukasz Majewski > >> ; Marek Vašut ; Albert ARIBAUD > >> ; Prabhakar Kushwaha > >> ; york sun ; Mingkai Hu > >> ; Rajesh Bhagat ; Michal > >> Simek ; felipe.ba...@linux.intel.com > >> Subject: Re: [PATCH v2 2/4] usb: dwc3: Add helper functions to enable > >> snooping and burst settings > >> > >> Hi, > >> > >> On 6 June 2016 at 03:16, Rajat Srivastava wrote: > >> > Adds helper functions to enable snooping and outstanding burst beat > >> > settings. > >> > > >> > Signed-off-by: Rajat Srivastava > >> > Signed-off-by: Rajesh Bhagat > >> > --- > >> > Changes in v2: > >> > - Removes SoC specific flags and added helper functions > >> > > >> > drivers/usb/dwc3/core.c | 45 > >> > + > >> > drivers/usb/dwc3/core.h | 7 +++ > >> > 2 files changed, 52 insertions(+) > >> > > >> > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c > >> > index > >> > 85cc96a..0b3c596 100644 > >> > --- a/drivers/usb/dwc3/core.c > >> > +++ b/drivers/usb/dwc3/core.c > >> > @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 > >> > *dwc) > >> > > >> > #define DWC3_ALIGN_MASK(16 - 1) > >> > > >> > +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, > >> > +int breq_limit) { > >> > + struct dwc3 *dwc; > >> > + u32 reg; > >> > + > >> > + list_for_each_entry(dwc, _list, list) { > > > > Hello Simon, > > > >> > >> Ick - can this be converted to use driver model? > >> > > > > We have not moved to use driver model yet :( . Is it possible to pass > > these register settings by some other mechanism ? > Hello Simon, > Not that I know of - is there any reason not to convert to driver model now? > Reason is, the dwc3 driver itself as not moved to CONFIG_DM_USB, and seems to be in progress by patch "driver model bring-up of dwc3 usb peripheral" (sent by Mugunthan V N). Best Regards, Rajesh Bhagat > > > > Best Regards, > > Rajesh Bhagat > > > >> > + if (dwc->index != index) > >> > + continue; > >> > + > >> > + /* > >> > +* Change burst beat and outstanding pipelined > >> > +* transfers requests > >> > +*/ > >> > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > >> > + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; > >> > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > >> > + > >> > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); > >> > + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << 8); > >> > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); > >> > + break; > >> > + } > >> > +} > >> > + > >> > +void dwc3_core_set_snooping(int index, bool snoop) { > >> > + struct dwc3 *dwc; > >> > + u32 reg; > >> > + > >> > + list_for_each_entry(dwc, _list, list) { > >> > + if (dwc->index != index) > >> > + continue; > >> > + > >> > + /* Enable/Disable snooping */ > >> > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > >> > + if (snoop) > >> > + reg = reg | DWC3_SNOOP_ENABLE; > >> > + else > >> > + reg = reg & ~DWC3_SNOOP_ENABLE; > >> > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > >> > + break; > >> > + } > >> > +} > >> > + > >> > /** > >> > * dwc3_uboot_init - dwc3 core uboot initialization code > >> > * @dwc3_dev: struct dwc3_device containing initialization data > >> > diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h >
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
Hi Rajat, > Adds helper functions to enable snooping and outstanding burst beat > settings. > > Signed-off-by: Rajat Srivastava> Signed-off-by: Rajesh Bhagat > --- > Changes in v2: > - Removes SoC specific flags and added helper functions > > drivers/usb/dwc3/core.c | 45 > + drivers/usb/dwc3/core.h > | 7 +++ 2 files changed, 52 insertions(+) > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c > index 85cc96a..0b3c596 100644 > --- a/drivers/usb/dwc3/core.c > +++ b/drivers/usb/dwc3/core.c > @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) > > #define DWC3_ALIGN_MASK (16 - 1) > > +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, > + int breq_limit) > +{ > + struct dwc3 *dwc; > + u32 reg; > + > + list_for_each_entry(dwc, _list, list) { > + if (dwc->index != index) > + continue; > + > + /* > + * Change burst beat and outstanding pipelined > + * transfers requests > + */ > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > + > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); > + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << > 8); > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); > + break; > + } > +} > + > +void dwc3_core_set_snooping(int index, bool snoop) > +{ > + struct dwc3 *dwc; > + u32 reg; > + > + list_for_each_entry(dwc, _list, list) { > + if (dwc->index != index) > + continue; > + > + /* Enable/Disable snooping */ > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > + if (snoop) > + reg = reg | DWC3_SNOOP_ENABLE; You can replace the above line with reg |= DWC3_SNOOP_ENABLE > + else > + reg = reg & ~DWC3_SNOOP_ENABLE; The same here - req &= ~DWC3_SNOOP_ENABLE > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > + break; > + } > +} > + > /** > * dwc3_uboot_init - dwc3 core uboot initialization code > * @dwc3_dev: struct dwc3_device containing initialization data > diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h > index 72d2fcd..455e7fa 100644 > --- a/drivers/usb/dwc3/core.h > +++ b/drivers/usb/dwc3/core.h > @@ -593,6 +593,13 @@ struct dwc3_hwparams { > /* HWPARAMS7 */ > #define DWC3_RAM1_DEPTH(n) ((n) & 0x) > > +/* GSBUSCFG0 */ > +#define DWC3_SNOOP_ENABLE(0x) > +#define DWC3_INCR_BTYPE_MASK (0xff) > + > +/* GSBUSCFG1 */ > +#define DWC3_BREQ_LIMIT_MASK (0xf00) > + > struct dwc3_request { > struct usb_request request; > struct list_headlist; -- Best regards, Lukasz Majewski Samsung R Institute Poland (SRPOL) | Linux Platform Group ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
Hi Rajesh, On 13 June 2016 at 22:09, Rajesh Bhagatwrote: > > >> -Original Message- >> From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass >> Sent: Friday, June 10, 2016 6:05 AM >> To: Rajat Srivastava >> Cc: U-Boot Mailing List ; Lukasz Majewski >> ; Marek Vašut ; Albert ARIBAUD >> ; Prabhakar Kushwaha ; >> york sun ; Mingkai Hu ; Rajesh Bhagat >> ; Michal Simek ; >> felipe.ba...@linux.intel.com >> Subject: Re: [PATCH v2 2/4] usb: dwc3: Add helper functions to enable >> snooping and >> burst settings >> >> Hi, >> >> On 6 June 2016 at 03:16, Rajat Srivastava wrote: >> > Adds helper functions to enable snooping and outstanding burst beat >> > settings. >> > >> > Signed-off-by: Rajat Srivastava >> > Signed-off-by: Rajesh Bhagat >> > --- >> > Changes in v2: >> > - Removes SoC specific flags and added helper functions >> > >> > drivers/usb/dwc3/core.c | 45 >> > + >> > drivers/usb/dwc3/core.h | 7 +++ >> > 2 files changed, 52 insertions(+) >> > >> > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index >> > 85cc96a..0b3c596 100644 >> > --- a/drivers/usb/dwc3/core.c >> > +++ b/drivers/usb/dwc3/core.c >> > @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) >> > >> > #define DWC3_ALIGN_MASK(16 - 1) >> > >> > +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, >> > +int breq_limit) { >> > + struct dwc3 *dwc; >> > + u32 reg; >> > + >> > + list_for_each_entry(dwc, _list, list) { > > Hello Simon, > >> >> Ick - can this be converted to use driver model? >> > > We have not moved to use driver model yet :( . Is it possible to pass these > register settings by some other mechanism ? Not that I know of - is there any reason not to convert to driver model now? > > Best Regards, > Rajesh Bhagat > >> > + if (dwc->index != index) >> > + continue; >> > + >> > + /* >> > +* Change burst beat and outstanding pipelined >> > +* transfers requests >> > +*/ >> > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); >> > + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; >> > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); >> > + >> > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); >> > + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << 8); >> > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); >> > + break; >> > + } >> > +} >> > + >> > +void dwc3_core_set_snooping(int index, bool snoop) { >> > + struct dwc3 *dwc; >> > + u32 reg; >> > + >> > + list_for_each_entry(dwc, _list, list) { >> > + if (dwc->index != index) >> > + continue; >> > + >> > + /* Enable/Disable snooping */ >> > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); >> > + if (snoop) >> > + reg = reg | DWC3_SNOOP_ENABLE; >> > + else >> > + reg = reg & ~DWC3_SNOOP_ENABLE; >> > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); >> > + break; >> > + } >> > +} >> > + >> > /** >> > * dwc3_uboot_init - dwc3 core uboot initialization code >> > * @dwc3_dev: struct dwc3_device containing initialization data diff >> > --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index >> > 72d2fcd..455e7fa 100644 >> > --- a/drivers/usb/dwc3/core.h >> > +++ b/drivers/usb/dwc3/core.h >> > @@ -593,6 +593,13 @@ struct dwc3_hwparams { >> > /* HWPARAMS7 */ >> > #define DWC3_RAM1_DEPTH(n) ((n) & 0x) >> > >> > +/* GSBUSCFG0 */ >> > +#define DWC3_SNOOP_ENABLE (0x) >> > +#define DWC3_INCR_BTYPE_MASK (0xff) >> > + >> > +/* GSBUSCFG1 */ >> > +#define DWC3_BREQ_LIMIT_MASK (0xf00) >> > + >> > struct dwc3_request { >> > struct usb_request request; >> > struct list_headlist; >> > -- >> > 2.6.2.198.g614a2ac >> > >> >> Regards, >> Simon Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
> -Original Message- > From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass > Sent: Friday, June 10, 2016 6:05 AM > To: Rajat Srivastava> Cc: U-Boot Mailing List ; Lukasz Majewski > ; Marek Vašut ; Albert ARIBAUD > ; Prabhakar Kushwaha ; > york sun ; Mingkai Hu ; Rajesh Bhagat > ; Michal Simek ; > felipe.ba...@linux.intel.com > Subject: Re: [PATCH v2 2/4] usb: dwc3: Add helper functions to enable > snooping and > burst settings > > Hi, > > On 6 June 2016 at 03:16, Rajat Srivastava wrote: > > Adds helper functions to enable snooping and outstanding burst beat > > settings. > > > > Signed-off-by: Rajat Srivastava > > Signed-off-by: Rajesh Bhagat > > --- > > Changes in v2: > > - Removes SoC specific flags and added helper functions > > > > drivers/usb/dwc3/core.c | 45 > > + > > drivers/usb/dwc3/core.h | 7 +++ > > 2 files changed, 52 insertions(+) > > > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index > > 85cc96a..0b3c596 100644 > > --- a/drivers/usb/dwc3/core.c > > +++ b/drivers/usb/dwc3/core.c > > @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) > > > > #define DWC3_ALIGN_MASK(16 - 1) > > > > +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, > > +int breq_limit) { > > + struct dwc3 *dwc; > > + u32 reg; > > + > > + list_for_each_entry(dwc, _list, list) { Hello Simon, > > Ick - can this be converted to use driver model? > We have not moved to use driver model yet :( . Is it possible to pass these register settings by some other mechanism ? Best Regards, Rajesh Bhagat > > + if (dwc->index != index) > > + continue; > > + > > + /* > > +* Change burst beat and outstanding pipelined > > +* transfers requests > > +*/ > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > > + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > > + > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); > > + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << 8); > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); > > + break; > > + } > > +} > > + > > +void dwc3_core_set_snooping(int index, bool snoop) { > > + struct dwc3 *dwc; > > + u32 reg; > > + > > + list_for_each_entry(dwc, _list, list) { > > + if (dwc->index != index) > > + continue; > > + > > + /* Enable/Disable snooping */ > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > > + if (snoop) > > + reg = reg | DWC3_SNOOP_ENABLE; > > + else > > + reg = reg & ~DWC3_SNOOP_ENABLE; > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > > + break; > > + } > > +} > > + > > /** > > * dwc3_uboot_init - dwc3 core uboot initialization code > > * @dwc3_dev: struct dwc3_device containing initialization data diff > > --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index > > 72d2fcd..455e7fa 100644 > > --- a/drivers/usb/dwc3/core.h > > +++ b/drivers/usb/dwc3/core.h > > @@ -593,6 +593,13 @@ struct dwc3_hwparams { > > /* HWPARAMS7 */ > > #define DWC3_RAM1_DEPTH(n) ((n) & 0x) > > > > +/* GSBUSCFG0 */ > > +#define DWC3_SNOOP_ENABLE (0x) > > +#define DWC3_INCR_BTYPE_MASK (0xff) > > + > > +/* GSBUSCFG1 */ > > +#define DWC3_BREQ_LIMIT_MASK (0xf00) > > + > > struct dwc3_request { > > struct usb_request request; > > struct list_headlist; > > -- > > 2.6.2.198.g614a2ac > > > > Regards, > Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
Hi, On 6 June 2016 at 03:16, Rajat Srivastavawrote: > Adds helper functions to enable snooping and outstanding burst beat > settings. > > Signed-off-by: Rajat Srivastava > Signed-off-by: Rajesh Bhagat > --- > Changes in v2: > - Removes SoC specific flags and added helper functions > > drivers/usb/dwc3/core.c | 45 + > drivers/usb/dwc3/core.h | 7 +++ > 2 files changed, 52 insertions(+) > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c > index 85cc96a..0b3c596 100644 > --- a/drivers/usb/dwc3/core.c > +++ b/drivers/usb/dwc3/core.c > @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) > > #define DWC3_ALIGN_MASK(16 - 1) > > +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, > +int breq_limit) > +{ > + struct dwc3 *dwc; > + u32 reg; > + > + list_for_each_entry(dwc, _list, list) { Ick - can this be converted to use driver model? > + if (dwc->index != index) > + continue; > + > + /* > +* Change burst beat and outstanding pipelined > +* transfers requests > +*/ > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > + > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); > + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << 8); > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); > + break; > + } > +} > + > +void dwc3_core_set_snooping(int index, bool snoop) > +{ > + struct dwc3 *dwc; > + u32 reg; > + > + list_for_each_entry(dwc, _list, list) { > + if (dwc->index != index) > + continue; > + > + /* Enable/Disable snooping */ > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > + if (snoop) > + reg = reg | DWC3_SNOOP_ENABLE; > + else > + reg = reg & ~DWC3_SNOOP_ENABLE; > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > + break; > + } > +} > + > /** > * dwc3_uboot_init - dwc3 core uboot initialization code > * @dwc3_dev: struct dwc3_device containing initialization data > diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h > index 72d2fcd..455e7fa 100644 > --- a/drivers/usb/dwc3/core.h > +++ b/drivers/usb/dwc3/core.h > @@ -593,6 +593,13 @@ struct dwc3_hwparams { > /* HWPARAMS7 */ > #define DWC3_RAM1_DEPTH(n) ((n) & 0x) > > +/* GSBUSCFG0 */ > +#define DWC3_SNOOP_ENABLE (0x) > +#define DWC3_INCR_BTYPE_MASK (0xff) > + > +/* GSBUSCFG1 */ > +#define DWC3_BREQ_LIMIT_MASK (0xf00) > + > struct dwc3_request { > struct usb_request request; > struct list_headlist; > -- > 2.6.2.198.g614a2ac > Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
On 06/08/2016 11:44 AM, Rajesh Bhagat wrote: > > >> -Original Message- >> From: Marek Vasut [mailto:ma...@denx.de] >> Sent: Monday, June 06, 2016 6:24 PM >> To: Rajat Srivastava; u-boot@lists.denx.de >> Cc: l.majew...@samsung.com; s...@chromium.org; albert.u.b...@aribaud.net; >> prabha...@freescale.com; york sun ; Mingkai Hu >> ; Rajesh Bhagat ; >> michal.si...@xilinx.com; felipe.ba...@linux.intel.com >> Subject: Re: [PATCH v2 2/4] usb: dwc3: Add helper functions to enable >> snooping and >> burst settings >> >> On 06/06/2016 11:16 AM, Rajat Srivastava wrote: >>> Adds helper functions to enable snooping and outstanding burst beat >>> settings. >>> >>> Signed-off-by: Rajat Srivastava >>> Signed-off-by: Rajesh Bhagat >>> --- >>> Changes in v2: >>> - Removes SoC specific flags and added helper functions >>> >>> drivers/usb/dwc3/core.c | 45 >>> + >>> drivers/usb/dwc3/core.h | 7 +++ >>> 2 files changed, 52 insertions(+) >>> >>> diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index >>> 85cc96a..0b3c596 100644 >>> --- a/drivers/usb/dwc3/core.c >>> +++ b/drivers/usb/dwc3/core.c >>> @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) >>> >>> #define DWC3_ALIGN_MASK(16 - 1) >>> >>> +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, >>> +int breq_limit) >>> +{ >>> + struct dwc3 *dwc; >>> + u32 reg; >>> + >>> + list_for_each_entry(dwc, _list, list) { >> > > Hello Marek, > >> Why is this iterating over a list of all controllers ? > > This function has to be called from Soc specific code to pass some register > settings. And dwc pointer > would not be available there. Hence, index is passed to determine dwc > pointer. > >> Should this be enabled on per-controller basis by some DT prop ? >> > > We are using dwc3 driver with non DT support. That's real unfortunate. > Best Regards, > Rajesh Bhagat > > >>> + if (dwc->index != index) >>> + continue; >>> + >>> + /* >>> +* Change burst beat and outstanding pipelined >>> +* transfers requests >>> +*/ >>> + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); >>> + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; >>> + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); >>> + >>> + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); >>> + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << 8); >>> + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); >>> + break; >>> + } >>> +} >>> + >>> +void dwc3_core_set_snooping(int index, bool snoop) { >>> + struct dwc3 *dwc; >>> + u32 reg; >>> + >>> + list_for_each_entry(dwc, _list, list) { >>> + if (dwc->index != index) >>> + continue; >>> + >>> + /* Enable/Disable snooping */ >>> + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); >>> + if (snoop) >>> + reg = reg | DWC3_SNOOP_ENABLE; >>> + else >>> + reg = reg & ~DWC3_SNOOP_ENABLE; >>> + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); >>> + break; >>> + } >>> +} >>> + >>> /** >>> * dwc3_uboot_init - dwc3 core uboot initialization code >>> * @dwc3_dev: struct dwc3_device containing initialization data diff >>> --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index >>> 72d2fcd..455e7fa 100644 >>> --- a/drivers/usb/dwc3/core.h >>> +++ b/drivers/usb/dwc3/core.h >>> @@ -593,6 +593,13 @@ struct dwc3_hwparams { >>> /* HWPARAMS7 */ >>> #define DWC3_RAM1_DEPTH(n) ((n) & 0x) >>> >>> +/* GSBUSCFG0 */ >>> +#define DWC3_SNOOP_ENABLE (0x) >>> +#define DWC3_INCR_BTYPE_MASK (0xff) >>> + >>> +/* GSBUSCFG1 */ >>> +#define DWC3_BREQ_LIMIT_MASK (0xf00) >>> + >>> struct dwc3_request { >>> struct usb_request request; >>> struct list_headlist; >>> >> >> >> -- >> Best regards, >> Marek Vasut -- Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
> -Original Message- > From: Marek Vasut [mailto:ma...@denx.de] > Sent: Monday, June 06, 2016 6:24 PM > To: Rajat Srivastava; u-boot@lists.denx.de > Cc: l.majew...@samsung.com; s...@chromium.org; albert.u.b...@aribaud.net; > prabha...@freescale.com; york sun ; Mingkai Hu > ; Rajesh Bhagat ; > michal.si...@xilinx.com; felipe.ba...@linux.intel.com > Subject: Re: [PATCH v2 2/4] usb: dwc3: Add helper functions to enable > snooping and > burst settings > > On 06/06/2016 11:16 AM, Rajat Srivastava wrote: > > Adds helper functions to enable snooping and outstanding burst beat > > settings. > > > > Signed-off-by: Rajat Srivastava > > Signed-off-by: Rajesh Bhagat > > --- > > Changes in v2: > > - Removes SoC specific flags and added helper functions > > > > drivers/usb/dwc3/core.c | 45 > > + > > drivers/usb/dwc3/core.h | 7 +++ > > 2 files changed, 52 insertions(+) > > > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index > > 85cc96a..0b3c596 100644 > > --- a/drivers/usb/dwc3/core.c > > +++ b/drivers/usb/dwc3/core.c > > @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) > > > > #define DWC3_ALIGN_MASK(16 - 1) > > > > +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, > > +int breq_limit) > > +{ > > + struct dwc3 *dwc; > > + u32 reg; > > + > > + list_for_each_entry(dwc, _list, list) { > Hello Marek, > Why is this iterating over a list of all controllers ? This function has to be called from Soc specific code to pass some register settings. And dwc pointer would not be available there. Hence, index is passed to determine dwc pointer. > Should this be enabled on per-controller basis by some DT prop ? > We are using dwc3 driver with non DT support. Best Regards, Rajesh Bhagat > > + if (dwc->index != index) > > + continue; > > + > > + /* > > +* Change burst beat and outstanding pipelined > > +* transfers requests > > +*/ > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > > + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > > + > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); > > + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << 8); > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); > > + break; > > + } > > +} > > + > > +void dwc3_core_set_snooping(int index, bool snoop) { > > + struct dwc3 *dwc; > > + u32 reg; > > + > > + list_for_each_entry(dwc, _list, list) { > > + if (dwc->index != index) > > + continue; > > + > > + /* Enable/Disable snooping */ > > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > > + if (snoop) > > + reg = reg | DWC3_SNOOP_ENABLE; > > + else > > + reg = reg & ~DWC3_SNOOP_ENABLE; > > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > > + break; > > + } > > +} > > + > > /** > > * dwc3_uboot_init - dwc3 core uboot initialization code > > * @dwc3_dev: struct dwc3_device containing initialization data diff > > --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index > > 72d2fcd..455e7fa 100644 > > --- a/drivers/usb/dwc3/core.h > > +++ b/drivers/usb/dwc3/core.h > > @@ -593,6 +593,13 @@ struct dwc3_hwparams { > > /* HWPARAMS7 */ > > #define DWC3_RAM1_DEPTH(n) ((n) & 0x) > > > > +/* GSBUSCFG0 */ > > +#define DWC3_SNOOP_ENABLE (0x) > > +#define DWC3_INCR_BTYPE_MASK (0xff) > > + > > +/* GSBUSCFG1 */ > > +#define DWC3_BREQ_LIMIT_MASK (0xf00) > > + > > struct dwc3_request { > > struct usb_request request; > > struct list_headlist; > > > > > -- > Best regards, > Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
On 06/06/2016 11:16 AM, Rajat Srivastava wrote: > Adds helper functions to enable snooping and outstanding burst beat > settings. > > Signed-off-by: Rajat Srivastava> Signed-off-by: Rajesh Bhagat > --- > Changes in v2: > - Removes SoC specific flags and added helper functions > > drivers/usb/dwc3/core.c | 45 + > drivers/usb/dwc3/core.h | 7 +++ > 2 files changed, 52 insertions(+) > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c > index 85cc96a..0b3c596 100644 > --- a/drivers/usb/dwc3/core.c > +++ b/drivers/usb/dwc3/core.c > @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) > > #define DWC3_ALIGN_MASK (16 - 1) > > +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, > + int breq_limit) > +{ > + struct dwc3 *dwc; > + u32 reg; > + > + list_for_each_entry(dwc, _list, list) { Why is this iterating over a list of all controllers ? Should this be enabled on per-controller basis by some DT prop ? > + if (dwc->index != index) > + continue; > + > + /* > + * Change burst beat and outstanding pipelined > + * transfers requests > + */ > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > + > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); > + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << 8); > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); > + break; > + } > +} > + > +void dwc3_core_set_snooping(int index, bool snoop) > +{ > + struct dwc3 *dwc; > + u32 reg; > + > + list_for_each_entry(dwc, _list, list) { > + if (dwc->index != index) > + continue; > + > + /* Enable/Disable snooping */ > + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); > + if (snoop) > + reg = reg | DWC3_SNOOP_ENABLE; > + else > + reg = reg & ~DWC3_SNOOP_ENABLE; > + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); > + break; > + } > +} > + > /** > * dwc3_uboot_init - dwc3 core uboot initialization code > * @dwc3_dev: struct dwc3_device containing initialization data > diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h > index 72d2fcd..455e7fa 100644 > --- a/drivers/usb/dwc3/core.h > +++ b/drivers/usb/dwc3/core.h > @@ -593,6 +593,13 @@ struct dwc3_hwparams { > /* HWPARAMS7 */ > #define DWC3_RAM1_DEPTH(n) ((n) & 0x) > > +/* GSBUSCFG0 */ > +#define DWC3_SNOOP_ENABLE(0x) > +#define DWC3_INCR_BTYPE_MASK (0xff) > + > +/* GSBUSCFG1 */ > +#define DWC3_BREQ_LIMIT_MASK (0xf00) > + > struct dwc3_request { > struct usb_request request; > struct list_headlist; > -- Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH v2 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings
Adds helper functions to enable snooping and outstanding burst beat settings. Signed-off-by: Rajat SrivastavaSigned-off-by: Rajesh Bhagat --- Changes in v2: - Removes SoC specific flags and added helper functions drivers/usb/dwc3/core.c | 45 + drivers/usb/dwc3/core.h | 7 +++ 2 files changed, 52 insertions(+) diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index 85cc96a..0b3c596 100644 --- a/drivers/usb/dwc3/core.c +++ b/drivers/usb/dwc3/core.c @@ -599,6 +599,51 @@ static void dwc3_core_exit_mode(struct dwc3 *dwc) #define DWC3_ALIGN_MASK(16 - 1) +void dwc3_core_incr_burst_enable(int index, int btype_incr_val, +int breq_limit) +{ + struct dwc3 *dwc; + u32 reg; + + list_for_each_entry(dwc, _list, list) { + if (dwc->index != index) + continue; + + /* +* Change burst beat and outstanding pipelined +* transfers requests +*/ + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); + reg = (reg & ~DWC3_INCR_BTYPE_MASK) | btype_incr_val; + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); + + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG1); + reg = (reg & ~DWC3_BREQ_LIMIT_MASK) | (breq_limit << 8); + dwc3_writel(dwc->regs, DWC3_GSBUSCFG1, reg); + break; + } +} + +void dwc3_core_set_snooping(int index, bool snoop) +{ + struct dwc3 *dwc; + u32 reg; + + list_for_each_entry(dwc, _list, list) { + if (dwc->index != index) + continue; + + /* Enable/Disable snooping */ + reg = dwc3_readl(dwc->regs, DWC3_GSBUSCFG0); + if (snoop) + reg = reg | DWC3_SNOOP_ENABLE; + else + reg = reg & ~DWC3_SNOOP_ENABLE; + dwc3_writel(dwc->regs, DWC3_GSBUSCFG0, reg); + break; + } +} + /** * dwc3_uboot_init - dwc3 core uboot initialization code * @dwc3_dev: struct dwc3_device containing initialization data diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index 72d2fcd..455e7fa 100644 --- a/drivers/usb/dwc3/core.h +++ b/drivers/usb/dwc3/core.h @@ -593,6 +593,13 @@ struct dwc3_hwparams { /* HWPARAMS7 */ #define DWC3_RAM1_DEPTH(n) ((n) & 0x) +/* GSBUSCFG0 */ +#define DWC3_SNOOP_ENABLE (0x) +#define DWC3_INCR_BTYPE_MASK (0xff) + +/* GSBUSCFG1 */ +#define DWC3_BREQ_LIMIT_MASK (0xf00) + struct dwc3_request { struct usb_request request; struct list_headlist; -- 2.6.2.198.g614a2ac ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot