Re: [PATCH 22/25] ARM: tegra: Move I2C clock frequency to bus nodes

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

 



On Thu, Dec 09, 2021 at 09:57:08PM +0300, Dmitry Osipenko wrote:
> 09.12.2021 20:33, Thierry Reding пишет:
> > From: Thierry Reding <treding@xxxxxxxxxx>
> > 
> > The "clock-frequency" property for an I2C controller needs to be
> > specified at the bus level.
> > 
> > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> > ---
> >  arch/arm/boot/dts/tegra30-asus-nexus7-grouper.dtsi | 3 ++-
> >  arch/arm/boot/dts/tegra30-asus-nexus7-tilapia.dtsi | 4 ++--
> >  2 files changed, 4 insertions(+), 3 deletions(-)
> 
> The clock-frequency is specified in the parent-common N7 DTSI, NAK.
> 
> There is already patch from David that removes the clock-frequency from
> NFC node [1], please use it.
> 
> Please never again rewrite the preexisting patches, this makes you look odd.

Again, please don't make this into something it isn't.

> I suggest to collect *all* patches (not only your own) into single
> for-next branch, resolving all conflicts, making sure that you don't
> duplicate the effort and letting it all to be tested before it will hit
> the bus.

You know, if you guys communicated better over existing channels or
actually let me know of what's going on then a lot of this could've been
avoided. I don't have visibility into what you're all up to, so I can't
take that into account.

If I happen to start working on something that someone else is also
doing in their corner, that's by accident, not because I have bad
intentions or because "I want to do everything myself", or whatever
other reason you keep coming up with.

Stop insinuating things that aren't true.

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