Re: [PATCH 04/25] ARM: tegra: Rename top-level clocks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Dec 09, 2021 at 09:27:01PM +0300, Dmitry Osipenko wrote:
> 09.12.2021 20:33, Thierry Reding пишет:
> > From: Thierry Reding <treding@xxxxxxxxxx>
> > 
> > Clocks defined at the top level in device tree are no longer part of a
> > simple bus and therefore don't have a reg property. Nodes without a reg
> > property shouldn't have a unit-address either, so drop the unit address
> > from the node names. To ensure nodes aren't duplicated (in which case
> > they would end up merged in the final DTB), append the name of the clock
> > to the node name.
> > 
> > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> > ---
> >  arch/arm/boot/dts/tegra114-dalmore.dts                    | 2 +-
> >  arch/arm/boot/dts/tegra114-roth.dts                       | 2 +-
> >  arch/arm/boot/dts/tegra114-tn7.dts                        | 2 +-
> >  arch/arm/boot/dts/tegra124-jetson-tk1.dts                 | 2 +-
> >  arch/arm/boot/dts/tegra124-nyan.dtsi                      | 2 +-
> >  arch/arm/boot/dts/tegra124-venice2.dts                    | 2 +-
> >  arch/arm/boot/dts/tegra20-acer-a500-picasso.dts           | 4 ++--
> >  arch/arm/boot/dts/tegra20-harmony.dts                     | 2 +-
> >  arch/arm/boot/dts/tegra20-paz00.dts                       | 2 +-
> >  arch/arm/boot/dts/tegra20-seaboard.dts                    | 2 +-
> >  arch/arm/boot/dts/tegra20-tamonten.dtsi                   | 2 +-
> >  arch/arm/boot/dts/tegra20-trimslice.dts                   | 2 +-
> >  arch/arm/boot/dts/tegra20-ventana.dts                     | 2 +-
> >  arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi | 2 +-
> >  arch/arm/boot/dts/tegra30-beaver.dts                      | 2 +-
> >  arch/arm/boot/dts/tegra30-cardhu.dtsi                     | 2 +-
> >  16 files changed, 17 insertions(+), 17 deletions(-)
> 
> This and the next patch duplicate the preexisting patch [1] that you saw
> and skipped previously. It looks odd that you redoing it on your own
> now. This is not okay to me unless you talked to David and he is aware
> about it.

I had completely forgotten about it. I'll substitute David's authorship
for mine, but I'd prefer to keep the two changes in separate patches.

Thierry

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux