On Tue, Jun 18, 2019 at 05:12:35PM +0200, Geert Uytterhoeven wrote: > Hi Magnus, > > On Tue, Jun 18, 2019 at 5:06 PM Magnus Damm <magnus.damm@xxxxxxxxx> wrote: > > On Tue, Jun 18, 2019 at 11:08 PM Geert Uytterhoeven > > <geert@xxxxxxxxxxxxxx> wrote: > > > On Mon, Jun 17, 2019 at 11:16 AM Magnus Damm <magnus.damm@xxxxxxxxx> wrote: > > > > From: Magnus Damm <damm+renesas@xxxxxxxxxxxxx> > > > > > > > > Update the r8a7740 to use the CMT1 DT compat string documented in: > > > > [PATCH 2/8] dt-bindings: timer: renesas, cmt: Update CMT1 on sh73a0 and r8a7740 > > > > > > > > The "renesas,cmt-48" portion is left as-is to avoid breakage until the CMT > > > > driver has been updated to make use of the new DT compat strings. > > > > > > Hence wouldn't it be better to update the driver first, and the DTS in > > > the next release, so you can remove the "renesas,cmt-48" portion with > > > the single DTS update? > > > > Changing the DTS once sounds nice indeed. I guess my current series > > are optimized for easy merge of DT Binding docs and DTS. The driver > > changes are considered slow path. > > > > Regarding the driver itself, I was under the impression that > > introducing new DT compat strings is often disconnected from removing > > old DT compat strings. Do you agree? > > Yes, you have to add a safety period before removing them. > > > This is how I understand your proposed order: > > > > Step 1: > > - Update DT binding document to include new compat strings, remove > > deprecated compat strings > > - Add new DT compat string matching code to the driver > > - Mark old DT compat strings in driver as deprecated > > > > Step 2: (Any time after step 1 is complete) > > - Convert DTS to use DT new compat strings > > > > Step 3: (After N releases or years) > > - Remove deprecated DT compat string matching code in driver > > Exactly. > Thanks! Likewise, thanks. I am marking patches 1/3 and 2/3 as deferred pending the completion of step 1.
