Hi > > [ Upstream commit 3e8bcec0787d1a73703c915c31cb00a2fd18ccbf ] (snip) > This is a code refactoring. It is obviously not suitable for stable, > it's not even a warning fix or anything - just a refactoring. You've > marked it as a dependency of another patch which is doing the > refactoring in a specific driver which obviously shouldn't be being > backported either. > > To repeat what I said the other day in response to the other wildly > inappropriate backports (one was an entirely new driver...) I thought > that there was supposed to be some manual review of the patches being > included in stable prior to them being sent out. How on earth are > things like this getting as far as being even proposed for stable? I have noticed about this, but my guess was that because this each patch modifies driver file with wide range. Thus, other main/important patch can't be backported without this patch (to fsl/pxa) ? (I'm not 100% understanding stable rules...) Fortunatry, this patch-set adds (1) add new style, (2) switch to new style (3) remove old style. We can keep compatibility if (3) was not added. Best regards --- Kuninori Morimoto