* [PATCH] evb_rk3399: add usb ohci definations @ 2021-05-26 9:33 Artem Lapkin 2021-06-01 12:15 ` Kever Yang 0 siblings, 1 reply; 5+ messages in thread From: Artem Lapkin @ 2021-05-26 9:33 UTC (permalink / raw) To: kever.yang; +Cc: sjg, philipp.tomsich, jagan, sunil, u-boot, art, nick, gouwa Problem: not possible to use CONFIG_USB_OHCI_HCD=y and CONFIG_USB_OHCI_GENERIC=y options without CONFIG_USB_OHCI_NEW and CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS definations Add missed definations. Signed-off-by: Artem Lapkin <art@khadas.com> --- include/configs/evb_rk3399.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/configs/evb_rk3399.h b/include/configs/evb_rk3399.h index b7e85037..492b7b4d 100644 --- a/include/configs/evb_rk3399.h +++ b/include/configs/evb_rk3399.h @@ -15,4 +15,7 @@ #define SDRAM_BANK_SIZE (2UL << 30) +#define CONFIG_USB_OHCI_NEW +#define CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS 2 + #endif -- 2.25.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] evb_rk3399: add usb ohci definations 2021-05-26 9:33 [PATCH] evb_rk3399: add usb ohci definations Artem Lapkin @ 2021-06-01 12:15 ` Kever Yang 2021-06-02 2:33 ` Art Nikpal 0 siblings, 1 reply; 5+ messages in thread From: Kever Yang @ 2021-06-01 12:15 UTC (permalink / raw) To: Artem Lapkin; +Cc: sjg, jagan, sunil, u-boot, art, nick, gouwa Hi Artem, On 2021/5/26 下午5:33, Artem Lapkin wrote: > Problem: not possible to use CONFIG_USB_OHCI_HCD=y and > CONFIG_USB_OHCI_GENERIC=y options without CONFIG_USB_OHCI_NEW and > CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS definations > > Add missed definations. But , evb_rk3399 board does not enable CONFIG_USB_OHCI_HCD and CONFIG_USB_OHCI_GENERIC. Thanks, - Kever > > Signed-off-by: Artem Lapkin <art@khadas.com> > --- > include/configs/evb_rk3399.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/include/configs/evb_rk3399.h b/include/configs/evb_rk3399.h > index b7e85037..492b7b4d 100644 > --- a/include/configs/evb_rk3399.h > +++ b/include/configs/evb_rk3399.h > @@ -15,4 +15,7 @@ > > #define SDRAM_BANK_SIZE (2UL << 30) > > +#define CONFIG_USB_OHCI_NEW > +#define CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS 2 > + > #endif ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] evb_rk3399: add usb ohci definations 2021-06-01 12:15 ` Kever Yang @ 2021-06-02 2:33 ` Art Nikpal 2021-06-02 2:44 ` Art Nikpal 2021-06-02 7:38 ` Peter Robinson 0 siblings, 2 replies; 5+ messages in thread From: Art Nikpal @ 2021-06-02 2:33 UTC (permalink / raw) To: Kever Yang; +Cc: sjg, Jagan Teki, sunil, u-boot, Artem Lapkin, nick, Gouwa Wang > But , evb_rk3399 board does not enable CONFIG_USB_OHCI_HCD and CONFIG_USB_OHCI_GENERIC. yes i know it ! for example some other rk3399 boards have it because (usb 1.1 didnt work without ohci on usb2.0 port and not possible to use usb kbd for this port ) grep CONFIG_USB_OHCI_GENERIC configs/*3399* configs/pinebook-pro-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y configs/rock960-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y configs/rockpro64-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y many other boards which used TARGET_EVB_RK3399=y must have same problem (with usb1.1) and i think other boards must have possibilities to use CONFIG_USB_OHCI_HCD and CONFIG_USB_OHCI_GENERIC. grep EVB configs/*3399* configs/evb-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/firefly-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/khadas-edge-captain-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/khadas-edge-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/khadas-edge-v-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/leez-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/nanopc-t4-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/nanopi-m4-2gb-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/nanopi-m4b-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/nanopi-m4-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/nanopi-neo4-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/nanopi-r4s-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/orangepi-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/rock-pi-4c-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/rock-pi-4-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y configs/rock-pi-n10-rk3399pro_defconfig:CONFIG_TARGET_EVB_RK3399=y On Tue, Jun 1, 2021 at 8:15 PM Kever Yang <kever.yang@rock-chips.com> wrote: > > Hi Artem, > > On 2021/5/26 下午5:33, Artem Lapkin wrote: > > Problem: not possible to use CONFIG_USB_OHCI_HCD=y and > > CONFIG_USB_OHCI_GENERIC=y options without CONFIG_USB_OHCI_NEW and > > CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS definations > > > > Add missed definations. > > But , evb_rk3399 board does not enable CONFIG_USB_OHCI_HCD and > CONFIG_USB_OHCI_GENERIC. > > > Thanks, > > - Kever > > > > > Signed-off-by: Artem Lapkin <art@khadas.com> > > --- > > include/configs/evb_rk3399.h | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/include/configs/evb_rk3399.h b/include/configs/evb_rk3399.h > > index b7e85037..492b7b4d 100644 > > --- a/include/configs/evb_rk3399.h > > +++ b/include/configs/evb_rk3399.h > > @@ -15,4 +15,7 @@ > > > > #define SDRAM_BANK_SIZE (2UL << 30) > > > > +#define CONFIG_USB_OHCI_NEW > > +#define CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS 2 > > + > > #endif > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] evb_rk3399: add usb ohci definations 2021-06-02 2:33 ` Art Nikpal @ 2021-06-02 2:44 ` Art Nikpal 2021-06-02 7:38 ` Peter Robinson 1 sibling, 0 replies; 5+ messages in thread From: Art Nikpal @ 2021-06-02 2:44 UTC (permalink / raw) To: Kever Yang; +Cc: sjg, Jagan Teki, sunil, u-boot, Artem Lapkin, nick, Gouwa Wang workable example for CONFIG_USB_OHCI_HCD and CONFIG_USB_OHCI_GENERIC (usb kbd on usb2.0 port) kedge# usb tree USB device tree: 1 Hub (480 Mb/s, 0mA) u-boot EHCI Host Controller 1 Hub (12 Mb/s, 0mA) | U-Boot Root Hub | +-2 Human Interface (1.5 Mb/s, 100mA) Dell KB216 Wired Keyboard 1 Hub (480 Mb/s, 0mA) u-boot EHCI Host Controller 1 Hub (12 Mb/s, 0mA) U-Boot Root Hub 1 Hub (5 Gb/s, 0mA) U-Boot XHCI Host Controller and for usbkbd connected to usb3 port kedge# usb tree USB device tree: 1 Hub (480 Mb/s, 0mA) u-boot EHCI Host Controller 1 Hub (12 Mb/s, 0mA) U-Boot Root Hub 1 Hub (480 Mb/s, 0mA) u-boot EHCI Host Controller 1 Hub (12 Mb/s, 0mA) U-Boot Root Hub 1 Hub (5 Gb/s, 0mA) | U-Boot XHCI Host Controller | +-2 Human Interface (1.5 Mb/s, 100mA) Dell KB216 Wired Keyboard PS: usb2.0 port cant recognize any usb1.1 devices without OHCI On Wed, Jun 2, 2021 at 10:33 AM Art Nikpal <email2tema@gmail.com> wrote: > > > But , evb_rk3399 board does not enable CONFIG_USB_OHCI_HCD and CONFIG_USB_OHCI_GENERIC. > > yes i know it ! > > for example some other rk3399 boards have it because (usb 1.1 didnt > work without ohci on usb2.0 port and not possible to use usb kbd for > this port ) > > grep CONFIG_USB_OHCI_GENERIC configs/*3399* > configs/pinebook-pro-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y > configs/rock960-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y > configs/rockpro64-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y > > many other boards which used TARGET_EVB_RK3399=y must have same > problem (with usb1.1) > > and i think other boards must have possibilities to use > CONFIG_USB_OHCI_HCD and CONFIG_USB_OHCI_GENERIC. > > grep EVB configs/*3399* > > configs/evb-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/firefly-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/khadas-edge-captain-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/khadas-edge-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/khadas-edge-v-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/leez-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/nanopc-t4-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/nanopi-m4-2gb-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/nanopi-m4b-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/nanopi-m4-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/nanopi-neo4-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/nanopi-r4s-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/orangepi-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/rock-pi-4c-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/rock-pi-4-rk3399_defconfig:CONFIG_TARGET_EVB_RK3399=y > configs/rock-pi-n10-rk3399pro_defconfig:CONFIG_TARGET_EVB_RK3399=y > > On Tue, Jun 1, 2021 at 8:15 PM Kever Yang <kever.yang@rock-chips.com> wrote: > > > > Hi Artem, > > > > On 2021/5/26 下午5:33, Artem Lapkin wrote: > > > Problem: not possible to use CONFIG_USB_OHCI_HCD=y and > > > CONFIG_USB_OHCI_GENERIC=y options without CONFIG_USB_OHCI_NEW and > > > CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS definations > > > > > > Add missed definations. > > > > But , evb_rk3399 board does not enable CONFIG_USB_OHCI_HCD and > > CONFIG_USB_OHCI_GENERIC. > > > > > > Thanks, > > > > - Kever > > > > > > > > Signed-off-by: Artem Lapkin <art@khadas.com> > > > --- > > > include/configs/evb_rk3399.h | 3 +++ > > > 1 file changed, 3 insertions(+) > > > > > > diff --git a/include/configs/evb_rk3399.h b/include/configs/evb_rk3399.h > > > index b7e85037..492b7b4d 100644 > > > --- a/include/configs/evb_rk3399.h > > > +++ b/include/configs/evb_rk3399.h > > > @@ -15,4 +15,7 @@ > > > > > > #define SDRAM_BANK_SIZE (2UL << 30) > > > > > > +#define CONFIG_USB_OHCI_NEW > > > +#define CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS 2 > > > + > > > #endif > > > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] evb_rk3399: add usb ohci definations 2021-06-02 2:33 ` Art Nikpal 2021-06-02 2:44 ` Art Nikpal @ 2021-06-02 7:38 ` Peter Robinson 1 sibling, 0 replies; 5+ messages in thread From: Peter Robinson @ 2021-06-02 7:38 UTC (permalink / raw) To: Art Nikpal Cc: Kever Yang, Simon Glass, Jagan Teki, sunil, u-boot, Artem Lapkin, nick, Gouwa Wang On Wed, Jun 2, 2021 at 3:33 AM Art Nikpal <email2tema@gmail.com> wrote: > > > But , evb_rk3399 board does not enable CONFIG_USB_OHCI_HCD and CONFIG_USB_OHCI_GENERIC. > > yes i know it ! > > for example some other rk3399 boards have it because (usb 1.1 didnt > work without ohci on usb2.0 port and not possible to use usb kbd for > this port ) > > grep CONFIG_USB_OHCI_GENERIC configs/*3399* > configs/pinebook-pro-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y > configs/rock960-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y > configs/rockpro64-rk3399_defconfig:CONFIG_USB_OHCI_GENERIC=y > > many other boards which used TARGET_EVB_RK3399=y must have same > problem (with usb1.1) The 3 boards you mention above define their own target and don't use TARGET_EVB_RK3399 > and i think other boards must have possibilities to use > CONFIG_USB_OHCI_HCD and CONFIG_USB_OHCI_GENERIC. So I think Kever's point was that without defining the following two options in the appropriate config files your patch has no effect because the OHCI support isn't actually built for the boards. I suggest a v2 patch which does that. CONFIG_USB_OHCI_HCD=y CONFIG_USB_OHCI_GENERIC=y Regards, Peter ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-06-02 7:39 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-05-26 9:33 [PATCH] evb_rk3399: add usb ohci definations Artem Lapkin 2021-06-01 12:15 ` Kever Yang 2021-06-02 2:33 ` Art Nikpal 2021-06-02 2:44 ` Art Nikpal 2021-06-02 7:38 ` Peter Robinson
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.