* [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO @ 2014-12-05 20:26 Sjoerd Simons 2014-12-09 3:01 ` Hyungwon Hwang 2014-12-22 12:22 ` Minkyu Kang 0 siblings, 2 replies; 7+ messages in thread From: Sjoerd Simons @ 2014-12-05 20:26 UTC (permalink / raw) To: u-boot Add samsung,vbus-gpio information for the XU3. This allows the usage of the EHCI controller on the XU3, which is connected to the SMSC LAN9514 chip (usb hub + network). Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk> --- Changes since v1: + Correct gpio number + Add USB configuration in the odroid XU3 default config Hyungwon could you add this one to your XU3 patchset if you send a next version (assuming it looks good)? For usb storage/network support this patch should be combined with the exynos configuration tweaks patch i submited earlier to the list: Exynos: Move down common USB configuration arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++ include/configs/odroid_xu3.h | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/arch/arm/dts/exynos5422-odroidxu3.dts b/arch/arm/dts/exynos5422-odroidxu3.dts index cff32a9..79a7acd 100644 --- a/arch/arm/dts/exynos5422-odroidxu3.dts +++ b/arch/arm/dts/exynos5422-odroidxu3.dts @@ -31,6 +31,10 @@ 0xb0000000 0xea00000>; }; + ehci at 12110000 { + samsung,vbus-gpio = <&gpio 0x66 0>; /* X26 */ + }; + serial at 12C20000 { status="okay"; }; diff --git a/include/configs/odroid_xu3.h b/include/configs/odroid_xu3.h index 88bb98d..aa0c142 100644 --- a/include/configs/odroid_xu3.h +++ b/include/configs/odroid_xu3.h @@ -47,6 +47,10 @@ #define CONFIG_DEFAULT_CONSOLE "console=ttySAC2,115200n8\0" +/* USB */ +#define CONFIG_USB_EHCI +#define CONFIG_USB_EHCI_EXYNOS + /* FIXME: MUST BE REMOVED AFTER TMU IS TURNED ON */ #undef CONFIG_EXYNOS_TMU #undef CONFIG_TMU_CMD_DTT -- 2.1.3 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO 2014-12-05 20:26 [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO Sjoerd Simons @ 2014-12-09 3:01 ` Hyungwon Hwang 2014-12-09 3:36 ` Suriyan Ramasami 2014-12-22 12:22 ` Minkyu Kang 1 sibling, 1 reply; 7+ messages in thread From: Hyungwon Hwang @ 2014-12-09 3:01 UTC (permalink / raw) To: u-boot Dear Sjoerd, On Fri, 05 Dec 2014 21:26:10 +0100 Sjoerd Simons <sjoerd.simons@collabora.co.uk> wrote: > Add samsung,vbus-gpio information for the XU3. This allows the usage > of the EHCI controller on the XU3, which is connected to the SMSC > LAN9514 chip (usb hub + network). > > Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk> > --- > Changes since v1: > + Correct gpio number > + Add USB configuration in the odroid XU3 default config > > Hyungwon could you add this one to your XU3 patchset if you send a > next version (assuming it looks good)? > Does it work only with this patch? I applied this patch on top of my patchset, and connected the ethernet cable to the device. But it seemed not working. Is there anything else that I should do for test? > For usb storage/network support this patch should be combined > with the exynos configuration tweaks patch i submited earlier to the > list: Exynos: Move down common USB configuration > > arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++ > include/configs/odroid_xu3.h | 4 ++++ > 2 files changed, 8 insertions(+) > > diff --git a/arch/arm/dts/exynos5422-odroidxu3.dts > b/arch/arm/dts/exynos5422-odroidxu3.dts index cff32a9..79a7acd 100644 > --- a/arch/arm/dts/exynos5422-odroidxu3.dts > +++ b/arch/arm/dts/exynos5422-odroidxu3.dts > @@ -31,6 +31,10 @@ > 0xb0000000 0xea00000>; > }; > > + ehci at 12110000 { > + samsung,vbus-gpio = <&gpio 0x66 0>; /* X26 */ > + }; > + > serial at 12C20000 { > status="okay"; > }; > diff --git a/include/configs/odroid_xu3.h > b/include/configs/odroid_xu3.h index 88bb98d..aa0c142 100644 > --- a/include/configs/odroid_xu3.h > +++ b/include/configs/odroid_xu3.h > @@ -47,6 +47,10 @@ > > #define > CONFIG_DEFAULT_CONSOLE "console=ttySAC2,115200n8\0" > +/* USB */ > +#define CONFIG_USB_EHCI > +#define CONFIG_USB_EHCI_EXYNOS > + > /* FIXME: MUST BE REMOVED AFTER TMU IS TURNED ON */ > #undef CONFIG_EXYNOS_TMU > #undef CONFIG_TMU_CMD_DTT Best regards, Hyungwon Hwang ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO 2014-12-09 3:01 ` Hyungwon Hwang @ 2014-12-09 3:36 ` Suriyan Ramasami 2014-12-09 4:10 ` Hyungwon Hwang 2014-12-09 4:25 ` Kevin Hilman 0 siblings, 2 replies; 7+ messages in thread From: Suriyan Ramasami @ 2014-12-09 3:36 UTC (permalink / raw) To: u-boot Hello Hyungwon Hwang, On Mon, Dec 8, 2014 at 7:01 PM, Hyungwon Hwang <human.hwang@samsung.com> wrote: > Dear Sjoerd, > > On Fri, 05 Dec 2014 21:26:10 +0100 > Sjoerd Simons <sjoerd.simons@collabora.co.uk> wrote: > >> Add samsung,vbus-gpio information for the XU3. This allows the usage >> of the EHCI controller on the XU3, which is connected to the SMSC >> LAN9514 chip (usb hub + network). >> >> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk> >> --- >> Changes since v1: >> + Correct gpio number >> + Add USB configuration in the odroid XU3 default config >> >> Hyungwon could you add this one to your XU3 patchset if you send a >> next version (assuming it looks good)? >> > > Does it work only with this patch? I applied this patch on top of my > patchset, and connected the ethernet cable to the device. But it > seemed not working. Is there anything else that I should do for test? > >> For usb storage/network support this patch should be combined >> with the exynos configuration tweaks patch i submited earlier to the >> list: Exynos: Move down common USB configuration >> >> arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++ >> include/configs/odroid_xu3.h | 4 ++++ >> 2 files changed, 8 insertions(+) >> >> diff --git a/arch/arm/dts/exynos5422-odroidxu3.dts >> b/arch/arm/dts/exynos5422-odroidxu3.dts index cff32a9..79a7acd 100644 >> --- a/arch/arm/dts/exynos5422-odroidxu3.dts >> +++ b/arch/arm/dts/exynos5422-odroidxu3.dts >> @@ -31,6 +31,10 @@ >> 0xb0000000 0xea00000>; >> }; >> >> + ehci at 12110000 { >> + samsung,vbus-gpio = <&gpio 0x66 0>; /* X26 */ >> + }; >> + >> serial at 12C20000 { >> status="okay"; >> }; >> diff --git a/include/configs/odroid_xu3.h >> b/include/configs/odroid_xu3.h index 88bb98d..aa0c142 100644 >> --- a/include/configs/odroid_xu3.h >> +++ b/include/configs/odroid_xu3.h >> @@ -47,6 +47,10 @@ >> >> #define >> CONFIG_DEFAULT_CONSOLE "console=ttySAC2,115200n8\0" >> +/* USB */ >> +#define CONFIG_USB_EHCI >> +#define CONFIG_USB_EHCI_EXYNOS >> + >> /* FIXME: MUST BE REMOVED AFTER TMU IS TURNED ON */ >> #undef CONFIG_EXYNOS_TMU >> #undef CONFIG_TMU_CMD_DTT > In odroid_xu3.h you might want to add the below as well (for LAN + USB storage) +/* Enable USB */ +#define CONFIG_CMD_USB +#define CONFIG_USB_EHCI +#define CONFIG_USB_EHCI_EXYNOS +#define CONFIG_USB_STORAGE +#define CONFIG_CMD_DHCP +#define CONFIG_USB_HOST_ETHER +#define CONFIG_USB_ETHER_SMSC95XX Thanks - Suriyan > Best regards, > Hyungwon Hwang > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO 2014-12-09 3:36 ` Suriyan Ramasami @ 2014-12-09 4:10 ` Hyungwon Hwang 2014-12-09 7:44 ` Sjoerd Simons 2014-12-09 4:25 ` Kevin Hilman 1 sibling, 1 reply; 7+ messages in thread From: Hyungwon Hwang @ 2014-12-09 4:10 UTC (permalink / raw) To: u-boot Dear all, On Mon, 08 Dec 2014 19:36:46 -0800 Suriyan Ramasami <suriyan.r@gmail.com> wrote: > Hello Hyungwon Hwang, > > On Mon, Dec 8, 2014 at 7:01 PM, Hyungwon Hwang > <human.hwang@samsung.com> wrote: > > Dear Sjoerd, > > > > On Fri, 05 Dec 2014 21:26:10 +0100 > > Sjoerd Simons <sjoerd.simons@collabora.co.uk> wrote: > > > >> Add samsung,vbus-gpio information for the XU3. This allows the > >> usage of the EHCI controller on the XU3, which is connected to the > >> SMSC LAN9514 chip (usb hub + network). > >> > >> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk> > >> --- > >> Changes since v1: > >> + Correct gpio number > >> + Add USB configuration in the odroid XU3 default config > >> > >> Hyungwon could you add this one to your XU3 patchset if you send a > >> next version (assuming it looks good)? > >> > > > > Does it work only with this patch? I applied this patch on top of my > > patchset, and connected the ethernet cable to the device. But it > > seemed not working. Is there anything else that I should do for > > test? > > > >> For usb storage/network support this patch should be combined > >> with the exynos configuration tweaks patch i submited earlier to > >> the list: Exynos: Move down common USB configuration > >> > >> arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++ > >> include/configs/odroid_xu3.h | 4 ++++ > >> 2 files changed, 8 insertions(+) > >> > >> diff --git a/arch/arm/dts/exynos5422-odroidxu3.dts > >> b/arch/arm/dts/exynos5422-odroidxu3.dts index cff32a9..79a7acd > >> 100644 --- a/arch/arm/dts/exynos5422-odroidxu3.dts > >> +++ b/arch/arm/dts/exynos5422-odroidxu3.dts > >> @@ -31,6 +31,10 @@ > >> 0xb0000000 0xea00000>; > >> }; > >> > >> + ehci at 12110000 { > >> + samsung,vbus-gpio = <&gpio 0x66 0>; /* X26 */ > >> + }; > >> + > >> serial at 12C20000 { > >> status="okay"; > >> }; > >> diff --git a/include/configs/odroid_xu3.h > >> b/include/configs/odroid_xu3.h index 88bb98d..aa0c142 100644 > >> --- a/include/configs/odroid_xu3.h > >> +++ b/include/configs/odroid_xu3.h > >> @@ -47,6 +47,10 @@ > >> > >> #define > >> CONFIG_DEFAULT_CONSOLE "console=ttySAC2,115200n8\0" > >> +/* USB */ > >> +#define CONFIG_USB_EHCI > >> +#define CONFIG_USB_EHCI_EXYNOS > >> + > >> /* FIXME: MUST BE REMOVED AFTER TMU IS TURNED ON */ > >> #undef CONFIG_EXYNOS_TMU > >> #undef CONFIG_TMU_CMD_DTT > > > > In odroid_xu3.h you might want to add the below as well (for LAN + > USB storage) > > +/* Enable USB */ > +#define CONFIG_CMD_USB > +#define CONFIG_USB_EHCI > +#define CONFIG_USB_EHCI_EXYNOS > +#define CONFIG_USB_STORAGE > +#define CONFIG_CMD_DHCP > +#define CONFIG_USB_HOST_ETHER > +#define CONFIG_USB_ETHER_SMSC95XX > Thanks for your reply. It works after above configs added. It seems all configs which Suriyan specified are needed. Even though DHCP can be useless depending on the network situation. Sjoerd, How about add above configs for odroid_xu3.h? Also, I think it is better to commit this patch separately with my patchset, because it took too long from when the first patch submitted, and I really want to make it merged with at least minimum functionalities. > Thanks > - Suriyan > > Best regards, > > Hyungwon Hwang > > _______________________________________________ > > U-Boot mailing list > > U-Boot at lists.denx.de > > http://lists.denx.de/mailman/listinfo/u-boot Best regards, Hyungwon Hwang ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO 2014-12-09 4:10 ` Hyungwon Hwang @ 2014-12-09 7:44 ` Sjoerd Simons 0 siblings, 0 replies; 7+ messages in thread From: Sjoerd Simons @ 2014-12-09 7:44 UTC (permalink / raw) To: u-boot Hey Hyungwon Hwang, On Tue, 2014-12-09 at 13:10 +0900, Hyungwon Hwang wrote: > Dear all, > > On Mon, 08 Dec 2014 19:36:46 -0800 > Suriyan Ramasami <suriyan.r@gmail.com> wrote: > > In odroid_xu3.h you might want to add the below as well (for LAN + > > USB storage) > > > > +/* Enable USB */ > > +#define CONFIG_CMD_USB > > +#define CONFIG_USB_EHCI > > +#define CONFIG_USB_EHCI_EXYNOS > > +#define CONFIG_USB_STORAGE > > +#define CONFIG_CMD_DHCP > > +#define CONFIG_USB_HOST_ETHER > > +#define CONFIG_USB_ETHER_SMSC95XX > > > > Thanks for your reply. It works after above configs added. > > It seems all configs which Suriyan specified are needed. Even though > DHCP can be useless depending on the network situation. Sjoerd, How > about add above configs for odroid_xu3.h? I sent a seperate patch with the title "Exynos: Move down common USB configuration", which adds the common configuration of USB storage/network support + common usb devices into exynos5-common.h so it can be shared among all exynos5 boards which i think is a better solution then adding non-board specific configuration to the XU3 config > Also, I think it is better to commit this patch separately with my > patchset, because it took too long from when the first patch > submitted, and I really want to make it merged with at least minimum > functionalities. That's fair enough. I don't mind either way. Thanks for testing! -- Sjoerd Simons <sjoerd.simons@collabora.co.uk> Collabora Ltd. -------------- next part -------------- A non-text attachment was scrubbed... Name: smime.p7s Type: application/x-pkcs7-signature Size: 6170 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141209/6089c4d2/attachment.bin> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO 2014-12-09 3:36 ` Suriyan Ramasami 2014-12-09 4:10 ` Hyungwon Hwang @ 2014-12-09 4:25 ` Kevin Hilman 1 sibling, 0 replies; 7+ messages in thread From: Kevin Hilman @ 2014-12-09 4:25 UTC (permalink / raw) To: u-boot Suriyan Ramasami <suriyan.r@gmail.com> writes: > Hello Hyungwon Hwang, > > On Mon, Dec 8, 2014 at 7:01 PM, Hyungwon Hwang <human.hwang@samsung.com> wrote: >> Dear Sjoerd, >> >> On Fri, 05 Dec 2014 21:26:10 +0100 >> Sjoerd Simons <sjoerd.simons@collabora.co.uk> wrote: >> >>> Add samsung,vbus-gpio information for the XU3. This allows the usage >>> of the EHCI controller on the XU3, which is connected to the SMSC >>> LAN9514 chip (usb hub + network). >>> >>> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk> >>> --- >>> Changes since v1: >>> + Correct gpio number >>> + Add USB configuration in the odroid XU3 default config >>> >>> Hyungwon could you add this one to your XU3 patchset if you send a >>> next version (assuming it looks good)? >>> >> >> Does it work only with this patch? I applied this patch on top of my >> patchset, and connected the ethernet cable to the device. But it >> seemed not working. Is there anything else that I should do for test? >> >>> For usb storage/network support this patch should be combined >>> with the exynos configuration tweaks patch i submited earlier to the >>> list: Exynos: Move down common USB configuration >>> >>> arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++ >>> include/configs/odroid_xu3.h | 4 ++++ >>> 2 files changed, 8 insertions(+) >>> >>> diff --git a/arch/arm/dts/exynos5422-odroidxu3.dts >>> b/arch/arm/dts/exynos5422-odroidxu3.dts index cff32a9..79a7acd 100644 >>> --- a/arch/arm/dts/exynos5422-odroidxu3.dts >>> +++ b/arch/arm/dts/exynos5422-odroidxu3.dts >>> @@ -31,6 +31,10 @@ >>> 0xb0000000 0xea00000>; >>> }; >>> >>> + ehci at 12110000 { >>> + samsung,vbus-gpio = <&gpio 0x66 0>; /* X26 */ >>> + }; >>> + >>> serial at 12C20000 { >>> status="okay"; >>> }; >>> diff --git a/include/configs/odroid_xu3.h >>> b/include/configs/odroid_xu3.h index 88bb98d..aa0c142 100644 >>> --- a/include/configs/odroid_xu3.h >>> +++ b/include/configs/odroid_xu3.h >>> @@ -47,6 +47,10 @@ >>> >>> #define >>> CONFIG_DEFAULT_CONSOLE "console=ttySAC2,115200n8\0" >>> +/* USB */ >>> +#define CONFIG_USB_EHCI >>> +#define CONFIG_USB_EHCI_EXYNOS >>> + >>> /* FIXME: MUST BE REMOVED AFTER TMU IS TURNED ON */ >>> #undef CONFIG_EXYNOS_TMU >>> #undef CONFIG_TMU_CMD_DTT >> > > In odroid_xu3.h you might want to add the below as well (for LAN + USB storage) > > +/* Enable USB */ > +#define CONFIG_CMD_USB > +#define CONFIG_USB_EHCI > +#define CONFIG_USB_EHCI_EXYNOS > +#define CONFIG_USB_STORAGE > +#define CONFIG_CMD_DHCP > +#define CONFIG_USB_HOST_ETHER > +#define CONFIG_USB_ETHER_SMSC95XX +1 I enabled these locally on v10 so that I could DHCP and TFTP boot. Kevin ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO 2014-12-05 20:26 [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO Sjoerd Simons 2014-12-09 3:01 ` Hyungwon Hwang @ 2014-12-22 12:22 ` Minkyu Kang 1 sibling, 0 replies; 7+ messages in thread From: Minkyu Kang @ 2014-12-22 12:22 UTC (permalink / raw) To: u-boot On 06/12/14 05:26, Sjoerd Simons wrote: > Add samsung,vbus-gpio information for the XU3. This allows the usage of > the EHCI controller on the XU3, which is connected to the SMSC LAN9514 > chip (usb hub + network). > > Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk> > --- > Changes since v1: > + Correct gpio number > + Add USB configuration in the odroid XU3 default config > > Hyungwon could you add this one to your XU3 patchset if you send a next > version (assuming it looks good)? > > For usb storage/network support this patch should be combined > with the exynos configuration tweaks patch i submited earlier to the list: > Exynos: Move down common USB configuration > > arch/arm/dts/exynos5422-odroidxu3.dts | 4 ++++ > include/configs/odroid_xu3.h | 4 ++++ > 2 files changed, 8 insertions(+) > > diff --git a/arch/arm/dts/exynos5422-odroidxu3.dts b/arch/arm/dts/exynos5422-odroidxu3.dts > index cff32a9..79a7acd 100644 > --- a/arch/arm/dts/exynos5422-odroidxu3.dts > +++ b/arch/arm/dts/exynos5422-odroidxu3.dts > @@ -31,6 +31,10 @@ > 0xb0000000 0xea00000>; > }; > > + ehci at 12110000 { > + samsung,vbus-gpio = <&gpio 0x66 0>; /* X26 */ > + }; > + > serial at 12C20000 { > status="okay"; > }; > diff --git a/include/configs/odroid_xu3.h b/include/configs/odroid_xu3.h > index 88bb98d..aa0c142 100644 > --- a/include/configs/odroid_xu3.h > +++ b/include/configs/odroid_xu3.h > @@ -47,6 +47,10 @@ > > #define CONFIG_DEFAULT_CONSOLE "console=ttySAC2,115200n8\0" > > +/* USB */ > +#define CONFIG_USB_EHCI > +#define CONFIG_USB_EHCI_EXYNOS > + > /* FIXME: MUST BE REMOVED AFTER TMU IS TURNED ON */ > #undef CONFIG_EXYNOS_TMU > #undef CONFIG_TMU_CMD_DTT > applied to u-boot-samsung. Thanks, Minkyu Kang. ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-12-22 12:22 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-12-05 20:26 [U-Boot] [PATCH v2] odroid-XU3: Add entry for DTS EHCI GPIO Sjoerd Simons 2014-12-09 3:01 ` Hyungwon Hwang 2014-12-09 3:36 ` Suriyan Ramasami 2014-12-09 4:10 ` Hyungwon Hwang 2014-12-09 7:44 ` Sjoerd Simons 2014-12-09 4:25 ` Kevin Hilman 2014-12-22 12:22 ` Minkyu Kang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox