From: Russell King - ARM Linux admin <linux@armlinux.org.uk>
To: Marek Szyprowski <m.szyprowski@samsung.com>,
Geert Uytterhoeven <geert+renesas@glider.be>
Cc: "Nicolas Pitre" <nico@fluxnic.net>,
"Arnd Bergmann" <arnd@arndb.de>,
"Eric Miao" <eric.miao@nvidia.com>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Chris Brandt" <chris.brandt@renesas.com>,
"Linux ARM" <linux-arm-kernel@lists.infradead.org>,
Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ARM: boot: Fix ATAGs with appended DTB
Date: Wed, 26 Feb 2020 17:49:05 +0000 [thread overview]
Message-ID: <20200226174905.GE25745@shell.armlinux.org.uk> (raw)
In-Reply-To: <e249c123-8d00-4aa3-34b8-f82d52428966@samsung.com>
On Wed, Feb 26, 2020 at 07:35:14AM +0100, Marek Szyprowski wrote:
> Hi Geert,
>
> On 25.02.2020 15:47, Geert Uytterhoeven wrote:
> > At early boot, register r8 may contain an ATAGs or DTB pointer.
> > When an appended DTB is found, its address is stored in r8, for
> > extraction of the RAM base address later.
> >
> > However, if r8 contained an ATAGs pointer before, that pointer will be
> > lost, and the provided ATAGs is no longer folded into the provided DTB.
> >
> > Fix this by leaving r8 untouched.
> >
> > Fixes: 137e522593918be2 ("ARM: 8960/1: boot: Obtain start of physical memory from DTB")
> > Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
The original commit hasn't been submitted, so it can be fixed before it
hits mainline if you want. Let me know what you want to do. Thanks.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up
WARNING: multiple messages have this Message-ID (diff)
From: Russell King - ARM Linux admin <linux@armlinux.org.uk>
To: Marek Szyprowski <m.szyprowski@samsung.com>,
Geert Uytterhoeven <geert+renesas@glider.be>
Cc: "Arnd Bergmann" <arnd@arndb.de>,
"Nicolas Pitre" <nico@fluxnic.net>,
linux-kernel@vger.kernel.org,
Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
"Chris Brandt" <chris.brandt@renesas.com>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Eric Miao" <eric.miao@nvidia.com>,
"Linux ARM" <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] ARM: boot: Fix ATAGs with appended DTB
Date: Wed, 26 Feb 2020 17:49:05 +0000 [thread overview]
Message-ID: <20200226174905.GE25745@shell.armlinux.org.uk> (raw)
In-Reply-To: <e249c123-8d00-4aa3-34b8-f82d52428966@samsung.com>
On Wed, Feb 26, 2020 at 07:35:14AM +0100, Marek Szyprowski wrote:
> Hi Geert,
>
> On 25.02.2020 15:47, Geert Uytterhoeven wrote:
> > At early boot, register r8 may contain an ATAGs or DTB pointer.
> > When an appended DTB is found, its address is stored in r8, for
> > extraction of the RAM base address later.
> >
> > However, if r8 contained an ATAGs pointer before, that pointer will be
> > lost, and the provided ATAGs is no longer folded into the provided DTB.
> >
> > Fix this by leaving r8 untouched.
> >
> > Fixes: 137e522593918be2 ("ARM: 8960/1: boot: Obtain start of physical memory from DTB")
> > Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
The original commit hasn't been submitted, so it can be fixed before it
hits mainline if you want. Let me know what you want to do. Thanks.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-02-26 17:49 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20200225144815eucas1p1229ceb0d017b46cbbe2409639a7c1f83@eucas1p1.samsung.com>
2020-02-25 14:47 ` [PATCH] ARM: boot: Fix ATAGs with appended DTB Geert Uytterhoeven
2020-02-25 14:47 ` Geert Uytterhoeven
2020-02-26 6:35 ` Marek Szyprowski
2020-02-26 6:35 ` Marek Szyprowski
2020-02-26 17:49 ` Russell King - ARM Linux admin [this message]
2020-02-26 17:49 ` Russell King - ARM Linux admin
2020-02-26 17:56 ` Geert Uytterhoeven
2020-02-26 17:56 ` Geert Uytterhoeven
2020-02-26 17:57 ` Russell King - ARM Linux admin
2020-02-26 17:57 ` Russell King - ARM Linux admin
2020-02-26 20:48 ` Geert Uytterhoeven
2020-02-26 20:48 ` Geert Uytterhoeven
2020-03-12 12:23 ` Marek Szyprowski
2020-03-12 12:23 ` Marek Szyprowski
2020-03-12 12:28 ` Russell King - ARM Linux admin
2020-03-12 12:28 ` Russell King - ARM Linux admin
2020-03-12 12:29 ` Geert Uytterhoeven
2020-03-12 12:29 ` Geert Uytterhoeven
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200226174905.GE25745@shell.armlinux.org.uk \
--to=linux@armlinux.org.uk \
--cc=arnd@arndb.de \
--cc=chris.brandt@renesas.com \
--cc=eric.miao@nvidia.com \
--cc=geert+renesas@glider.be \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=nico@fluxnic.net \
--cc=u.kleine-koenig@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.