Johannes Berg <[email protected]> writes: > On Wed, 2016-09-28 at 11:19 -0400, Jes Sorensen wrote: >> >> I understand the intentions of this patch are all good, but you need >> to not post patches that include both staging and mainline drivers at >> the same time. In general make it a patchset and do one patch per >> driver. >> >> Ideally split up changes to generic code into their own patches too. > > 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. >> Last drivers/staging/rtl8723au is gone - so your patch is going to >> fail to apply anyway. > > It's there in my tree, for now, so I guess I'll see if it's still there > when I take this in :) Changes to staging needs to go in via staging, and rtl8723au is gone from the staging tree. Cheers, Jes
