From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Tue, 30 Apr 2013 19:09:39 +0200
Subject: [U-Boot] [PATCH 2/3] ARM: Tegra: USB: Add driver support for
	Tegra30/Tegra114
In-Reply-To: <5FBF8E85CA34454794F0F7ECBA79798F37AC7648F8@HQMAIL04.nvidia.com>
References: <1367227275-7713-1-git-send-email-jilin@nvidia.com>
	<201304300146.45501.marex@denx.de>
	<5FBF8E85CA34454794F0F7ECBA79798F37AC7648F8@HQMAIL04.nvidia.com>
Message-ID: <201304301909.39852.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

Dear Tom Warren,

> Marek,
> 
> > -----Original Message-----
> > From: Marek Vasut [mailto:marex at denx.de]
> > Sent: Monday, April 29, 2013 4:47 PM
> > To: Jim Lin
> > Cc: u-boot at lists.denx.de; Tom Warren; Stephen Warren
> > Subject: Re: [PATCH 2/3] ARM: Tegra: USB: Add driver support for
> > Tegra30/Tegra114
> > 
> > Dear Jim Lin,
> > 
> > > Tegra30 and Tegra114 are compatible except 1. T30 takes 55 ms to
> > > finish Port Reset. T114 takes
> > > 
> > >    50 ms.
> > > 
> > > 2. PLL parameters
> > > 
> > > Tested on Tegra20 Harmony/Seaboard, Tegra30 Cardhu, and Tegra114
> > > Dalmore platforms. All works well.
> > > 
> > > Signed-off-by: Jim Lin <jilin@nvidia.com>
> > > ---
> > > 
> > >  arch/arm/include/asm/arch-tegra/clk_rst.h  |   10 +
> > >  arch/arm/include/asm/arch-tegra/usb.h      |  249 ------------------
> > >  arch/arm/include/asm/arch-tegra114/tegra.h |    1 +
> > >  arch/arm/include/asm/arch-tegra114/usb.h   |  287
> > 
> > +++++++++++++++++++++
> > 
> > >  arch/arm/include/asm/arch-tegra20/usb.h    |  279
> > 
> > +++++++++++++++++++++
> > 
> > >  arch/arm/include/asm/arch-tegra30/usb.h    |  294
> > 
> > ++++++++++++++++++++++
> > 
> > Do we now have three copies of the same stuff ?
> 
> When only T20 was supported (for USB), there was a common
> (arch-tegra/usb.h) header. That's been moved to arch-tegra20/usb.h, and
> (unfortunately) there are 2 new usb.h files due to the HW differences in
> the registers between T20 and T30/T114.  I don't see any easy way to have
> a common usb.h file for Tegra w/o adding ugly #ifdefs to the USB register
> space struct(s).

Thanks for clearing that up.

Best regards,
Marek Vasut