Hi Stephen, On Thu, Feb 2, 2012 at 1:55 PM, Stephen Warren <swar...@nvidia.com> wrote: > On 01/24/2012 04:21 PM, Simon Glass wrote: >> This adds a property to indicate a port which can switch between host and >> device >> mode. > ... >> diff --git a/doc/device-tree-bindings/usb/tegra-usb.txt >> b/doc/device-tree-bindings/usb/tegra-usb.txt > ... >> +Optional properties: > ... >> + - nvidia,has-legacy-mode : boolean indicates whether this controller can >> + operate in legacy mode (as APX 2500 / 2600). In legacy mode some >> + registers are accessed through the APB_MISC base address instead of >> + the USB controller. > > I was hoping to represent this as a different compatible value, in the > same way as one of Tegra's I2C controllers is, eg. nvidia,tegra20-ehci > and nvidia,tegra20-ehci-legacy. > > However, I suppose representing this using a property is OK too.
I prefer this because this is a legacy issue, and I feel that creating a different compatible node when no one actually uses legacy mode is an unnecessary complexity. > > Either way, can you please also push this patch to the Linux kernel (and > the associated changes to tegra*.dtsi) in order to run it past the other > Tegra maintainers before committing to it within U-Boot. OK I have done this. > > Thanks. > > -- > nvpublic Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
