[PATCH V3 1/7] ARM: dts: Change i2s compatible string on exynos5250
t.figa at samsung.com
Wed Aug 7 07:17:42 EDT 2013
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 at kernel.org> 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.
More information about the linux-arm-kernel