From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: [PATCH v2 2/7] ARM: tegra: update device trees for USB binding rework Date: Wed, 03 Apr 2013 13:33:19 -0600 Message-ID: <515C83FF.10004@wwwdotorg.org> References: <1364978502-22887-1-git-send-email-vbyravarasu@nvidia.com> <1364978502-22887-3-git-send-email-vbyravarasu@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1364978502-22887-3-git-send-email-vbyravarasu-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Venu Byravarasu Cc: gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org, balbi-l0cyMroinI0@public.gmane.org, stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org On 04/03/2013 02:41 AM, Venu Byravarasu wrote: > This patch updates all Tegra board files so that they contain all the > properties required by the updated USB DT binding. Note that this patch > only adds the new properties and does not yet remove the old properties, > in order to maintain bisectability. The old properties will be removed > once the driver has been updated to assume the new bindings. > diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi b/arch/arm/boot/dts/tegra20-colibri-512.dtsi > index cb73e62..af5a7ae 100644 > --- a/arch/arm/boot/dts/tegra20-colibri-512.dtsi > +++ b/arch/arm/boot/dts/tegra20-colibri-512.dtsi > @@ -443,6 +443,10 @@ > nvidia,phy-reset-gpio = <&gpio 169 0>; /* gpio PV1 */ Another problem here: The code pulses this GPIO low to reset the PHY, so in other words, the GPIO is active low. Can you please prepare a separate patch to fix this; you need to change all existing instances from: nvidia,phy-reset-gpio = <&gpio 169 0>; /* gpio PV1 */ to: nvidia,phy-reset-gpio = <&gpio 169 1>; /* gpio PV1, active low */ Insert that new patch before this patch in the series, and then fix this patch so that any new copies of that property have the correct content from the start. Note: I pointed out this problem in my review of V1, but V2 didn't include a fix for it:-(