On Friday 19 January 2018 12:10 AM, David Lechner wrote: > On 01/18/2018 09:27 AM, Sekhar Nori wrote: >> On Monday 08 January 2018 07:55 AM, David Lechner wrote: >>> This removes all of the clock init code from da8xx-dt.c. This includes >>> all of the OF_DEV_AUXDATA that was just used for looking up clocks. >>> >>> Note: You need to have clocks defined in your device tree or your system >>> won't boot after this patch. >> >> I am not sure we can do this then, as we cannot break DT compatibility. >> > > In the past, you have told me that you don't want the .dts changes and code > changes in the same patch. In this case, if you apply either one Thats still true. > separately, > it will break clocks. It does not matter which one is first. > > So either we have to squash [PATCH v5 44/44] ARM: dts: da850: Add clocks > into this patch or deal with the breakage. I am not so much concerned about temporary breakage in the middle of the series, but more about DT compatibility after the entire series is applied. Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
