Re: [RESEND PATCH 3/6] ARM: dts: imx7s: add CAAM device node

From: Bryan O'Donoghue
Date: Wed Jan 24 2018 - 17:56:51 EST


On 24/01/18 22:48, Bryan O'Donoghue wrote:
The clock configuration has changed to just one CAAM-specific clock in
addition to the ahb clock. This also means that additional
modifications to the CAAM driver are necessary or it will complain that
it doesn't find all clocks.

Sure - but, those clock changes aren't merged to upstream just yet, so we won't those changes to this DTS addition yet.

Thanks for the review.

---
bod

Ah Rui has pointed out to me for i.mx7 there are two clocks on on i.mx6 there are four clocks.

The bit that confused me there was the clock names on the 4.1 tree you pointed to was clock-names = "caam_ipg", "caam_aclk"; and for 4.15 those should be "ipg" and "aclk" by the looks of it.

So yes.

We will add the second job-ring and subtract the two extraneous clocks.

---
bod