lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Mon, 25 Jul 2016 16:18:18 +0200
From:	Thierry Reding <thierry.reding@...il.com>
To:	Ralf Ramsauer <ralf@...ses-pyramidenbau.de>
Cc:	Arnd Bergmann <arnd@...db.de>, arm@...nel.org,
	Stephen Warren <swarren@...dotorg.org>,
	Alexandre Courbot <gnurou@...il.com>,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-tegra@...r.kernel.org
Subject: Re: [RESEND PATCH] ARM: tegra: fix erroneous address in dts

On Sun, Jul 24, 2016 at 10:38:38PM +0200, Ralf Ramsauer wrote:
> On 07/21/2016 02:28 PM, Arnd Bergmann wrote:
> > On Monday, July 18, 2016 11:58:02 AM CEST Thierry Reding wrote:
> >> On Mon, Jul 18, 2016 at 11:46:48AM +0200, Ralf Ramsauer wrote:
> >>> c90bb7b enabled the high speed UARTs of the Jetson TK1. Due to a merge
> >>> quirk, wrong addresses were introduced. Fix it and use the correct
> >>> addresses.
> >>>
> >>> Thierry let me know, that there is another patch (b5896f67ab3c in
> >>> linux-next) in preparation which removes all the '0,' prefixes of unit
> >>> addresses on Tegra124 and is planned to go upstream in 4.8, so
> >>> this patch will get reverted then.
> >>>
> >>> But for the moment, this patch is necessary to fix current misbehaviour.
> >>>
> >>> Fixes: c90bb7b9b9 ("ARM: tegra: Add high speed UARTs to Jetson TK1 device tree")
> >>> Signed-off-by: Ralf Ramsauer <ralf@...ses-pyramidenbau.de>
> >>> Acked-by: Thierry Reding <thierry.reding@...il.com>
> >>> Cc: linux-tegra@...r.kernel.org
> >>> ---
> >>>  arch/arm/boot/dts/tegra124-jetson-tk1.dts | 4 ++--
> >>>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> ARM-SoC maintainers,
> >>
> >> Just to clarify, this is targetted at v4.7 because the offending patch
> >> was merged in v4.7-rc1. If at all possible it'd be great to get this in,
> >> even if this is terribly last minute.
> >>
> > 
> > I've applied it to the fixes branch, but as Linus is currently offline
> > and hasn't pulled the previous set of fixes, I suspect it's not
> > going to make it.
> Yep, didn't make it. I'll resend the fix with stable in Cc tomorrow.
> Keep you also in Cc.
> 
> Thierry, this fix is only required for 4.7.x and gets superfluous when
> b5896f67ab3c is applied. So I guess there is no need to fix mainline, right?

Agreed.

Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ