From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH] clk: tegra: Fix wrong value written to PLLE_AUX Date: Sat, 17 May 2014 23:06:59 +0200 Message-ID: <20140517210657.GB1046@mithrandir> References: <1400248220-16025-1-git-send-email-ttynkkynen@nvidia.com> <20140516143110.GA28338@ulmo> <5376373E.1080401@wwwdotorg.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="wzJLGUyc3ArbnUjN" Return-path: Content-Disposition: inline In-Reply-To: <5376373E.1080401-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Warren Cc: Tuomas Tynkkynen , Mike Turquette , Peter De Schrijver , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org --wzJLGUyc3ArbnUjN Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, May 16, 2014 at 10:05:18AM -0600, Stephen Warren wrote: > On 05/16/2014 08:31 AM, Thierry Reding wrote: > > On Fri, May 16, 2014 at 04:50:20PM +0300, Tuomas Tynkkynen wrote: > >> The value written to PLLE_AUX was incorrect due to a wrong variable > >> being used. > >> > >> Signed-off-by: Tuomas Tynkkynen > >> Tested-by: Mikko Perttunen > >> --- > >> This fix is required for the (upcoming) SATA support. > >> drivers/clk/tegra/clk-pll.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >=20 > > Reviewed-by: Thierry Reding > > Tested-by: Thierry Reding > > Acked-by: Thierry Reding > >=20 > > Mike, Peter, it might be good to have this go into 3.16 as a > > prerequisite for the upcoming SATA driver (which presumably won't be > > ready until 3.17, but in that case it would be good to have this > > prerequisite merged already). >=20 > Probably even Cc: stable since it's a fix for a bug that I assume has > been in a kernel release or two? Yeah, I guess that makes sense. The bug isn't visible yet, but back- porting to stable might save somebody else from chasing this if they ever decide to backport PCIe, SATA or XUSB. Thierry --wzJLGUyc3ArbnUjN Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJTd89vAAoJEN0jrNd/PrOhghsP/0cLORjtzZ/FziFv1QJnRHY7 bcaCcu7Z2YSvjyRWR988jRESdj7QAz20tzCXunav8GheXVinHu+ZOCky83onm9HQ oZ+D5bxhXSXOC6ebFetb/lPlANCem8bYAWaASEGMYY84tuoz0AilaKjUe/DIg8hN pbEUwTuzgJZKPpCD1lty9ZA5nJPHQXVczulyyiaHWcF7WIDZ5exF8SrAV6edYFpo 2c6Mpl8QGzcLsdPlHlS3GAAV+MrsZVthGJbj+QQgU6K0oc5lAL7QHlNVNcy34k0R bsfz+w8oL7b0ySBltq8tDiW5RJ6WqnHQ8WkrHAb6eZa9bxmamrQlk24r5QFYddp0 lFYkqbdqxjOk02Zm1V0SAxa6YxZGv4oGAphn8YVVb982ROsNDJO1Ufrs9lP36fiK 7JZT7ib5B2sIJl0yiW3SiXlPsn0NDNdTLi9tY/0SlCddRwOUpOCFbxxAdqhPj2cY CsZ/04xqrD+kkthnEAmtN6uXgqfi6U8smQpR99gvKxYmoSoV7yXINAGfPg3E02oM pwmjVdZVpWo32kUD9h/b5MIdiuX2MTfiLCqK96Qy2jJO0g7tQxtaKLx4KX8usMAy HNZkBy8BHmFa/jDzWf2EyWM3PXUH+uL62IeEfajQO5UPkHYyVlNexJF0m2JADOzW fms2PTVrc1jjshWHSoOt =iaRZ -----END PGP SIGNATURE----- --wzJLGUyc3ArbnUjN--