Johannes Berg <johannes@xxxxxxxxxxxxxxxx> writes: > On Wed, 2016-09-28 at 11:39 -0400, Jes Sorensen wrote: > >> > No Jes, you're wrong this time - this is changing internal API so >> > it does have to touch all users thereof. >> >> Even in this case, change the individual components in individual >> patches and post them as a set. > > No, still wrong - it has to be committed as a single patch so it > doesn't break bisect. > >> Changes to staging needs to go in via staging, and rtl8723au is gone >> from the staging tree. >> > > I've previously taken API change patches that touch staging, if people > feel so inclined, and I don't think Greg will mind. I'm going to keep > doing that unless Dave tells me he won't pull from me when I do it :) I'll still argue this could be handled better through gradual migration rather than one large patch that touches too many places, but if you are willing to take it, I am not going to fight you over it :) Jes
