From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Fri, 20 Feb 2015 16:54:08 +0100
Subject: [U-Boot] [PATCH 11/14] usb: UniPhier: add UniPhier on-chip xHCI
	host driver support
In-Reply-To: <20150220220454.DDC8.AA925319@jp.panasonic.com>
References: <1424156430-23499-12-git-send-email-yamada.m@jp.panasonic.com>
 <201502172051.48111.marex@denx.de>
 <20150220220454.DDC8.AA925319@jp.panasonic.com>
Message-ID: <201502201654.08192.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 Friday, February 20, 2015 at 02:04:54 PM, Masahiro Yamada wrote:
> Hi Marek,

Hi!

> On Tue, 17 Feb 2015 20:51:48 +0100
> 
> Marek Vasut <marex@denx.de> wrote:
> > On Tuesday, February 17, 2015 at 08:00:27 AM, Masahiro Yamada wrote:
> > > Support xHCI host driver used on Panasonic UniPhier platform.
> > > 
> > > Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> > > ---
> > > Hi Marek,
> > > 
> > > I want apply this patch onto u-boot-uniphier/master
> > > to avoid conflicts.
> > > 
> > > If you are OK with it, could you issue your Acked-by tag, please?
> > 
> > [...]
> > 
> > > +static int get_uniphier_xhci_base(int index, struct xhci_hccr **base)
> > > +{
> > > +	int offset;
> > > +
> > > +	for (offset = fdt_node_offset_by_compatible(FDT, 0, COMPAT);
> > > +	     offset >= 0;
> > > +	     offset = fdt_node_offset_by_compatible(FDT, offset, COMPAT)) {
> > > +		if (index == 0) {
> > > +			*base = (struct xhci_hccr *)
> > > +					fdtdec_get_addr(FDT, offset, "reg");
> > 
> > Hi!
> > 
> > does it make sense to check if the value returned by fdtdec_get_addr() is
> > valid here?
> 
> Yes.
> I knew it, but was lazy.
> Finally, I fixed it in v2.

Thank you ! :)

> BTW, why did we invent FDT_ADDR_T_NONE?
> 
> We had already had the suitable error code, FDT_ERR_NOTFOUND in libfdt.h
> (or Linux error code in include/asm-generic/errno.h)
> 
> I do not think fdtdec functions are well-implemented.

That's for Simon to answer.

Best regards,
Marek Vasut