On Tue, Jul 26, 2016 at 2:14 PM, Robert Foss <robert.foss@xxxxxxxxxxxxx> wrote: ... > Thanks for the feedback (for this patch and the other ones)! > I'm preparing a v2 and will submit it withing a day or two. Excellent! very welcome and thanks again for picking this up. ... >> FTR, current drivers/net/usb/ax88179_178a.c uses "in_pm" as well. > > Ah! Would you like that to be noted in the commit message? No - I generally do not muck with commit messages that I didn't write. It's part of the "finger print" of a patch (so one can trace origin across distro's and kernel versions). My comment was intended for reviewers and could be included in the "comments after the commit message". (ie below the "---" marker in the patch email). ... >> BTW, I have two additional changes for AX88772x support sitting in my >> "needs more work" queue (for quite a while already): >> https://chromium-review.googlesource.com/#/c/229620/ >> "asix: autoneg will set WRITE_MEDIUM reg" >> >> https://chromium-review.googlesource.com/#/c/231162/ >> "net: asix: see 802.3 spec for phy reset" >> >> I would certainly approve if _anyone_ picked these up, tested them, >> and then submitted them to netdev. > > Unfortunately I am without appropriate hardware at the moment. That's a risky thing to point out for two reasons: 1) I can send you more HW than you could possibly ever use. :) 2) Reviewers of this patch series are already wondering how you verified the AX88772x patches work on current kernel.org (or netdev-next) branches. Please email me with your shipping address (OFFLIST!) and I'll try to send you some Asix HW to test with. I'm located in "Silicon Valley" (USA) and can easily ship in the US. Other countries is a bit harder and in some cases not possible. cheers, grant -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
