linux-kernel - Re: [PATCH v6 05/20] dt-bindings: memory: factorise LPDDR props into SDRAM props

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]

Message-ID: <20250910-flat-raptor-of-temperance-5e8c7c@kuoka>
Date: Wed, 10 Sep 2025 09:54:48 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Clément Le Goffic <legoffic.clement@...il.com>
Cc: Gatien Chevallier <gatien.chevallier@...s.st.com>, 
	Maxime Coquelin <mcoquelin.stm32@...il.com>, Alexandre Torgue <alexandre.torgue@...s.st.com>, 
	Michael Turquette <mturquette@...libre.com>, Stephen Boyd <sboyd@...nel.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Gabriel Fernandez <gabriel.fernandez@...s.st.com>, Julius Werner <jwerner@...omium.org>, 
	Will Deacon <will@...nel.org>, Mark Rutland <mark.rutland@....com>, 
	Philipp Zabel <p.zabel@...gutronix.de>, Jonathan Corbet <corbet@....net>, 
	linux-stm32@...md-mailman.stormreply.com, linux-arm-kernel@...ts.infradead.org, 
	linux-kernel@...r.kernel.org, linux-clk@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-perf-users@...r.kernel.org, linux-doc@...r.kernel.org, 
	Clément Le Goffic <clement.legoffic@...s.st.com>
Subject: Re: [PATCH v6 05/20] dt-bindings: memory: factorise LPDDR props into
 SDRAM props

On Tue, Sep 09, 2025 at 12:12:12PM +0200, Clément Le Goffic wrote:
> From: Clément Le Goffic <clement.legoffic@...s.st.com>
> 
> LPDDR and DDR bindings are SDRAM types and are likely to share the same
> properties (at least for density, io-width and reg).
> To avoid bindings duplication, factorise the properties.
> 
> The compatible description has been updated because the MR (Mode
> registers) used to get manufacturer ID and revision ID are not present
> in case of DDR.
> Those information should be in a SPD (Serial Presence Detect) EEPROM in
> case of DIMM module or are known in case of soldered memory chips as
> they are in the datasheet of the memory chips.
> 
> Signed-off-by: Clément Le Goffic <clement.legoffic@...s.st.com>

Is this email defunct now, that you add second SoB?

> Signed-off-by: Clément Le Goffic <legoffic.clement@...il.com>

I still see in this patchset around 5 different subsystems. Nothing in
the cover letter explains the dependencies, so grouping looks like
coincidence and you just make it difficult for maintainers for no
reason.

Please organize your patchsets per subsystems, see submitting patches
doc for more details.

Best regards,
Krzysztof