On Wednesday 07 of August 2013 16:02:16 Padma Venkat wrote: > Hi Mark, > > On Wed, Aug 7, 2013 at 3:40 PM, Mark Brown <broonie@xxxxxxxxxx> wrote: > > On Wed, Aug 07, 2013 at 02:40:10PM +0530, Padmavathi Venna wrote: > >> This patch removes quirks from i2s node and change the i2s > >> compatible names. > > > > This needs to go along with the driver change otherwise we break > > bisection. > > some of the patches at dts side are dependent on this patch. So I > separated it into another set. Should I re-post this patch along with > other driver side patches? I fully agree with Mark. There are 2 ways of solving this: a) Do things in steps that don't break anything, i.e.: 1) add new bindings to the driver, 2) update DTS files, 3) remove old bindings from the driver. b) Do all the things above atomically, in one patch. Personally, I'm a fan of method a), but if it requires much effort then method b) can be used as a fallback. Best regards, Tomasz -- 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
