From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Tue, 4 Nov 2014 09:13:10 +0100
Subject: [U-Boot] [PATCH 1/2] usb: ehci: rmobile: Add support R8A7793
In-Reply-To: <CABMQnV+79gq_gHxCCjd9iUXatOXz7cuc6raervF=_W25ORYG-A@mail.gmail.com>
References: <1415062602-29219-1-git-send-email-nobuhiro.iwamatsu.yj@renesas.com>
	<201411040602.21834.marex@denx.de>
	<CABMQnV+79gq_gHxCCjd9iUXatOXz7cuc6raervF=_W25ORYG-A@mail.gmail.com>
Message-ID: <201411040913.10677.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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