Re: [PATCH 2/2] Revert "arm64: dts: ls1028a: add flextimer based pwm nodes"

[Michael Walle]

Am 2022-01-29 11:12, schrieb Michael Walle:
> Am 2022-01-29 06:42, schrieb Shawn Guo:
> > On Sat, Jan 15, 2022 at 10:09:07PM +0100, Michael Walle wrote:
> > > The changes to the device tree look very wrong. There are now two
> > > devices with the same base address: pwm0 and ftm_alarm0. Both are 
> > > using
> > > the Flex Timer Module. It seems like this should either be one driver 
> > > or
> > > and MFD driver. Either way, there should only be one node in the 
> > > device
> > > tree. Revert the offending changes to avoid getting a broken device 
> > > tree
> > > in circulation.
> >
> > Why not just fix the conflicting nodes?
>
> And how would you fix it? There are two conflicting drivers. Like
> I said, maybe it should be an MFD driver, see for example
> Documentation/devicetree/bindings/mfd/atmel-flexcom.txt, which also
> have a common block which you can switch between different modes. But
> that would mean a lot of work which I don't think will happen until
> the next merge window, let alone that this is not a bugfix.
>
> Therefore, the only sensible thing is to revert the latest changes,
> so that you don't have a broken device tree released with 5.17 and
> work on a proper support for the next release.
>
> And TBH, I'd expect that NXP will fix this.

I've just asked Rob and it is ok to have two nodes which have the
same reg on the same parent if one (or both) are disabled.

So you can drop this patch, but someone should really sort these
nodes by address.

I still think it would have been nicer the one proper driver for
the flextimer.

-michael