Thread (33 messages) 33 messages, 6 authors, 2021-07-30

Re: [PATCH 6/9] arm64: dts: qcom: sc7280: Update reserved memory map

From: Bjorn Andersson <hidden>
Date: 2021-07-30 18:25:06
Also in: linux-arm-kernel, linux-arm-msm, linux-remoteproc, lkml

On Wed 30 Jun 15:02 CDT 2021, Sibi Sankar wrote:
On 2021-06-28 23:41, Matthias Kaehlcke wrote:
quoted
On Fri, Jun 25, 2021 at 01:17:35AM +0530, Sibi Sankar wrote:
quoted
Subject: arm64: dts: qcom: sc7280: Update reserved memory map
That's very vague. Also personally I'm not a fan of patches that touch
SoC and board files with a commit message that only mentions the SoC, as
is frequently done for IDP boards. Why not split this in (at least) two,
one for adding the missing memory regions to the SoC, and one for the
IDP.
sure will split this up.
quoted
quoted
Add missing regions and remove unused regions from the reserved memory
map, as described in version 1.
What is this 'version 1'?
lol, it's the memory map version number
and it's not entirely internal to qc so
we have been mentioning them in commit
messages from older SoCs. I'll just drop
it when I re-spin the series since it
doesn't add much value.
Every now and then we run into issues with the reserved-memory layout,
where knowing were the numbers comes from is useful information to have
in order to characterize the issue and come up with a fix.

So including information about where those numbers came from is useful,
even if it's referencing a version of a document that's not public.

Regards,
Bjorn
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help