On Thu, Dec 09, 2021 at 09:38:06PM +0300, Dmitry Osipenko wrote: > 09.12.2021 20:33, Thierry Reding пишет: > > From: Thierry Reding <treding@xxxxxxxxxx> > > > > OPP nodes do not have a "reg" property and therefore shouldn't have a > > unit-address. Instead, use a dash instead of the '@' and ',' characters > > to allow validation of the nodes against the DT schema. > > > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > > --- > > .../boot/dts/tegra124-peripherals-opp.dtsi | 142 ++++++++--------- > > .../boot/dts/tegra20-cpu-opp-microvolt.dtsi | 82 +++++----- > > arch/arm/boot/dts/tegra20-cpu-opp.dtsi | 82 +++++----- > > .../arm/boot/dts/tegra20-peripherals-opp.dtsi | 36 ++--- > > .../boot/dts/tegra30-cpu-opp-microvolt.dtsi | 144 +++++++++--------- > > arch/arm/boot/dts/tegra30-cpu-opp.dtsi | 144 +++++++++--------- > > .../arm/boot/dts/tegra30-peripherals-opp.dtsi | 130 ++++++++-------- > > 7 files changed, 382 insertions(+), 378 deletions(-) > > This patch is wrong, you haven't renamed the delete-node properties [1]. Yeah, I noticed that too as I was reworking the EMC timing nodes as you requested. > Please stop rewriting patches and use what already has been sent out and > tested, thanks. > > [1] > https://patchwork.ozlabs.org/project/linux-tegra/patch/20211130232347.950-35-digetx@xxxxxxxxx/ I've had versions of this in my tree for literally years now, so I haven't exactly been rewriting these. Rather this is an iteration of work that I had started over 18 months ago: http://patchwork.ozlabs.org/project/linux-tegra/patch/20200616135238.3001888-26-thierry.reding@xxxxxxxxx/ After that obviously things had to be changed again. The earliest version that you sent that I can find is from late October which is when I was already in the midst of this latest effort to get Tegra DTBs to validate. What's really been happening here is that we haven't been communicating and ended up duplicating work. Stop making this into something it isn't. Thierry
Attachment:
signature.asc
Description: PGP signature
