* [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793
@ 2014-11-04 0:56 Nobuhiro Iwamatsu
2014-11-04 0:56 ` [U-Boot] [PATCH 2/2] usb: ehci: rmobile: Collect up address data Nobuhiro Iwamatsu
2014-11-04 5:02 ` [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793 Marek Vasut
0 siblings, 2 replies; 5+ messages in thread
From: Nobuhiro Iwamatsu @ 2014-11-04 0:56 UTC (permalink / raw)
To: u-boot
R8A7793 has same IP of USB controller as R8A7791 and R8A7794 of rmobile
ARM SoCs. This adds support R8A7793 to EHCI HCD of rmobile.
Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
---
drivers/usb/host/ehci-rmobile.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/usb/host/ehci-rmobile.c b/drivers/usb/host/ehci-rmobile.c
index 0d1a726..a89f04b 100644
--- a/drivers/usb/host/ehci-rmobile.c
+++ b/drivers/usb/host/ehci-rmobile.c
@@ -27,6 +27,11 @@ static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
0xEE080000, /* USB0 (EHCI) */
0xEE0C0000, /* USB1 */
};
+#elif defined(CONFIG_R8A7793)
+static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
+ 0xEE080000, /* USB0 (EHCI) */
+ 0xEE0C0000, /* USB1 */
+};
#elif defined(CONFIG_R8A7794)
static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
0xEE080000, /* USB0 (EHCI) */
--
2.1.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH 2/2] usb: ehci: rmobile: Collect up address data
2014-11-04 0:56 [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793 Nobuhiro Iwamatsu
@ 2014-11-04 0:56 ` Nobuhiro Iwamatsu
2014-11-04 5:02 ` [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793 Marek Vasut
1 sibling, 0 replies; 5+ messages in thread
From: Nobuhiro Iwamatsu @ 2014-11-04 0:56 UTC (permalink / raw)
To: u-boot
R8A7791, R8A7793 and R8A7794 have same IP of USB controller.
This collect up address data of each SoCs.
Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
---
drivers/usb/host/ehci-rmobile.c | 13 ++-----------
1 file changed, 2 insertions(+), 11 deletions(-)
diff --git a/drivers/usb/host/ehci-rmobile.c b/drivers/usb/host/ehci-rmobile.c
index a89f04b..b433087 100644
--- a/drivers/usb/host/ehci-rmobile.c
+++ b/drivers/usb/host/ehci-rmobile.c
@@ -22,17 +22,8 @@ static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
0xEE0A0000, /* USB1 */
0xEE0C0000, /* USB2 */
};
-#elif defined(CONFIG_R8A7791)
-static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
- 0xEE080000, /* USB0 (EHCI) */
- 0xEE0C0000, /* USB1 */
-};
-#elif defined(CONFIG_R8A7793)
-static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
- 0xEE080000, /* USB0 (EHCI) */
- 0xEE0C0000, /* USB1 */
-};
-#elif defined(CONFIG_R8A7794)
+#elif defined(CONFIG_R8A7791) || defined(CONFIG_R8A7793) || \
+ defined(CONFIG_R8A7794)
static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
0xEE080000, /* USB0 (EHCI) */
0xEE0C0000, /* USB1 */
--
2.1.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793
2014-11-04 0:56 [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793 Nobuhiro Iwamatsu
2014-11-04 0:56 ` [U-Boot] [PATCH 2/2] usb: ehci: rmobile: Collect up address data Nobuhiro Iwamatsu
@ 2014-11-04 5:02 ` Marek Vasut
2014-11-04 7:10 ` Nobuhiro Iwamatsu
1 sibling, 1 reply; 5+ messages in thread
From: Marek Vasut @ 2014-11-04 5:02 UTC (permalink / raw)
To: u-boot
On Tuesday, November 04, 2014 at 01:56:41 AM, Nobuhiro Iwamatsu wrote:
Hi!
> R8A7793 has same IP of USB controller as R8A7791 and R8A7794 of rmobile
> ARM SoCs. This adds support R8A7793 to EHCI HCD of rmobile.
>
> Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
> ---
> drivers/usb/host/ehci-rmobile.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/drivers/usb/host/ehci-rmobile.c
> b/drivers/usb/host/ehci-rmobile.c index 0d1a726..a89f04b 100644
> --- a/drivers/usb/host/ehci-rmobile.c
> +++ b/drivers/usb/host/ehci-rmobile.c
> @@ -27,6 +27,11 @@ static u32
> usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = { 0xEE080000, /* USB0
> (EHCI) */
> 0xEE0C0000, /* USB1 */
> };
> +#elif defined(CONFIG_R8A7793)
> +static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
> + 0xEE080000, /* USB0 (EHCI) */
> + 0xEE0C0000, /* USB1 */
> +};
I see you combined those three arrays in the next patch, thanks!
> #elif defined(CONFIG_R8A7794)
> static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
> 0xEE080000, /* USB0 (EHCI) */
btw. just an idea, but you don't need to explicitly use
usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT], you can
just use usb_base_address[] in the array definition. Then,
in the code, you can just use "ARRAY_SIZE(usb_base_address) - 1"
instead of CONFIG_USB_MAX_CONTROLLER_COUNT . This rids you of
one annoying macro in that driver ;-) If you want to implement
this, then please do so, but in a separate patch.
I'll pick these two patches now.
Thanks!
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793
2014-11-04 5:02 ` [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793 Marek Vasut
@ 2014-11-04 7:10 ` Nobuhiro Iwamatsu
2014-11-04 8:13 ` Marek Vasut
0 siblings, 1 reply; 5+ messages in thread
From: Nobuhiro Iwamatsu @ 2014-11-04 7:10 UTC (permalink / raw)
To: u-boot
Hi!
2014-11-04 14:02 GMT+09:00 Marek Vasut <marex@denx.de>:
> On Tuesday, November 04, 2014 at 01:56:41 AM, Nobuhiro Iwamatsu wrote:
>
> Hi!
>
>> R8A7793 has same IP of USB controller as R8A7791 and R8A7794 of rmobile
>> ARM SoCs. This adds support R8A7793 to EHCI HCD of rmobile.
>>
>> Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
>> ---
>> drivers/usb/host/ehci-rmobile.c | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/drivers/usb/host/ehci-rmobile.c
>> b/drivers/usb/host/ehci-rmobile.c index 0d1a726..a89f04b 100644
>> --- a/drivers/usb/host/ehci-rmobile.c
>> +++ b/drivers/usb/host/ehci-rmobile.c
>> @@ -27,6 +27,11 @@ static u32
>> usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = { 0xEE080000, /* USB0
>> (EHCI) */
>> 0xEE0C0000, /* USB1 */
>> };
>> +#elif defined(CONFIG_R8A7793)
>> +static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
>> + 0xEE080000, /* USB0 (EHCI) */
>> + 0xEE0C0000, /* USB1 */
>> +};
>
> I see you combined those three arrays in the next patch, thanks!
>
>> #elif defined(CONFIG_R8A7794)
>> static u32 usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT] = {
>> 0xEE080000, /* USB0 (EHCI) */
>
> btw. just an idea, but you don't need to explicitly use
> usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT], you can
> just use usb_base_address[] in the array definition. Then,
> in the code, you can just use "ARRAY_SIZE(usb_base_address) - 1"
> instead of CONFIG_USB_MAX_CONTROLLER_COUNT . This rids you of
> one annoying macro in that driver ;-) If you want to implement
> this, then please do so, but in a separate patch.
Indeed. I will create a patch which fix your point.
>
> I'll pick these two patches now.
Thanks!
>
> Thanks!
>
> Best regards,
> Marek Vasut
Best regards,
Nobuhiro
--
Nobuhiro Iwamatsu
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793
2014-11-04 7:10 ` Nobuhiro Iwamatsu
@ 2014-11-04 8:13 ` Marek Vasut
0 siblings, 0 replies; 5+ messages in thread
From: Marek Vasut @ 2014-11-04 8:13 UTC (permalink / raw)
To: u-boot
On Tuesday, November 04, 2014 at 08:10:30 AM, Nobuhiro Iwamatsu wrote:
> Hi!
Hi!
[...]
> > btw. just an idea, but you don't need to explicitly use
> > usb_base_address[CONFIG_USB_MAX_CONTROLLER_COUNT], you can
> > just use usb_base_address[] in the array definition. Then,
> > in the code, you can just use "ARRAY_SIZE(usb_base_address) - 1"
> > instead of CONFIG_USB_MAX_CONTROLLER_COUNT . This rids you of
> > one annoying macro in that driver ;-) If you want to implement
> > this, then please do so, but in a separate patch.
>
> Indeed. I will create a patch which fix your point.
>
> > I'll pick these two patches now.
>
> Thanks!
Thank you, I will queue it for the next PR, since I sent this one out
already.
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-11-04 8:13 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-04 0:56 [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793 Nobuhiro Iwamatsu
2014-11-04 0:56 ` [U-Boot] [PATCH 2/2] usb: ehci: rmobile: Collect up address data Nobuhiro Iwamatsu
2014-11-04 5:02 ` [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793 Marek Vasut
2014-11-04 7:10 ` Nobuhiro Iwamatsu
2014-11-04 8:13 ` Marek Vasut
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.