Julian Calaby [off-list ref] writes:
Hi Kalle,
On Mon, Feb 1, 2016 at 7:21 PM, Kalle Valo [off-list ref] wrote:
quoted
Sudip Mukherjee [off-list ref] writes:
quoted
Sure, I am starting that way. I checked in patchwork and I do not see
any checkpatch related patch pending (except staging, which Greg will
handle). I think you must have cleared all of them.
They are in deferred state. The search functionality in patchwork is not
that intuitive and they are not easy to find so here's a direct link:
https://patchwork.kernel.org/project/linux-wireless/list/?state=10&order=date
I'm currently going through that list and producing a bundle of
"applyable" patches.
Nice.
My criteria is:
1. The change is sane.
2. It's either obviously correct, I can review it, or someone else has
reviewed or acked it.
3. No changes other than rebasing and fixing commit messages are
required to apply it.
BTW, 'git am -s -3' is the best way to apply a patch. The three way
merge is awesome (if the submitter has sent the patch correctly).
Some of these patches need work on their commit messages, some are
complicated enough that I feel I should be providing review notes so
someone else can double check my review, and all of them should be
rebased and compile tested. Also, some are controversial, so I'll be
segregating them from the main set.
How would you like me to communicate this list to you? I'm happy to
provide branches you can pull from or I could just post updated
versions to the list and give reviewed-by tags to those that don't
need more work.
Every patch will get an email on linux-wireless regardless.
I guess posting the patches to linux-wireless is the easiest for
everyone? I have a script which automatically takes patches from
patchwork so that's very easy for me. But remember to use Signed-off-by
instead of Reviewed-by as you are resending the patches.
Thanks you, your help here is very much appreciated.
--
Kalle Valo