Re: [PATCH] dts: Revert at91 switch to new PMC clock bindings

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

 



On Tue, Feb 05, 2019 at 12:59:27PM +0100, Sascha Hauer wrote:
> Hi Ladis,
> 
> On Mon, Feb 04, 2019 at 11:29:37AM +0100, Ladislav Michl wrote:
> > Recent DTS update brought in switch to the new PMC clock
> > bindings, however we do not support that yet. Revert this
> > change until PMC clock bindings support is implemented.
> > 
> > Fixes: 33fdc89d4cbd ("dts: update to v5.0-rc1")
> > Signed-off-by: Ladislav Michl <ladis@xxxxxxxxxxxxxx>
> > ---
> >  Disclaimer: Runtime tested only on at91sam9g20.
> > 
> >  dts/src/arm/at91sam9260.dtsi       | 308 ++++++++++++++++++++++++---
> >  dts/src/arm/at91sam9261.dtsi       | 287 +++++++++++++++++++++++--
> >  dts/src/arm/at91sam9263.dtsi       | 315 +++++++++++++++++++++++++---
> >  dts/src/arm/at91sam9g15.dtsi       |   4 -
> >  dts/src/arm/at91sam9g20.dtsi       |  23 +-
> >  dts/src/arm/at91sam9g25.dtsi       |   4 -
> >  dts/src/arm/at91sam9g25ek.dts      |   4 +-
> >  dts/src/arm/at91sam9g35.dtsi       |   4 -
> >  dts/src/arm/at91sam9rl.dtsi        | 239 +++++++++++++++++++--
> >  dts/src/arm/at91sam9x25.dtsi       |   4 -
> >  dts/src/arm/at91sam9x35.dtsi       |   4 -
> >  dts/src/arm/at91sam9x5.dtsi        | 326 ++++++++++++++++++++++++++---
> >  dts/src/arm/at91sam9x5_can.dtsi    |  18 +-
> >  dts/src/arm/at91sam9x5_isi.dtsi    |  11 +-
> >  dts/src/arm/at91sam9x5_lcd.dtsi    |  19 +-
> >  dts/src/arm/at91sam9x5_macb0.dtsi  |  11 +-
> >  dts/src/arm/at91sam9x5_macb1.dtsi  |  11 +-
> >  dts/src/arm/at91sam9x5_usart3.dtsi |  11 +-
> >  18 files changed, 1434 insertions(+), 169 deletions(-)
> 
> dts/ contains a 1:1 copy from the upstream devicetree-rebasing
> repository. I can't apply any patches to this directory.
> 
> You would have to merge the changes to the changes to the corresponding
> files in arch/arm/dts. I'm not sure how feasible this is in this case
> since not only the clock nodes are changed, but also the references to
> the clocks.
> 
> For sure it should be enough to fix the trees you actually can test and
> care about.

My board is not upstreamed as it does not make sense to do it so. It will
never hit the market. Also upstream barebox does not use DT on any
at91sam9g20 SoC, so it is safe. However there seem to be devices using
AT91SAM9X5, SAMA5D3 and SAM9263 which depends on DT clocks and those
are now broken. That said, I can live with one more offtree patch until
new PMC clock binding is implemented (reverting change in arch/arm/dts
is a bit painfull).

On the other hand, was the DT update to v5.0-rc1 and later to v5.0-rc2
motivated by any need or it is just an update to be in sync? If that
is the latter I'd just revert it for barebox-2019.02.0 :)

Best regards,
	ladis

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux