Thread (17 messages) flat view 17 messages, 4 authors, 2017-01-25

Re: [PATCH net-next 0/5] External MDIO support for mv88e6xxx

From: David Miller <davem@davemloft.net>
Date: 2017-01-24 21:07:51

From: Vivien Didelot <redacted>
Date: Tue, 24 Jan 2017 16:05:08 -0500
Hi David,

David Miller [off-list ref] writes:
quoted
Series applied, thanks Andrew.

There seems to be a bit of inconsistency wrt. tab vs. space for
indentation in the binding files, just FYI.  I had to make some small
adjustments to this patch series in order to keep GIT from warning
when I applied this stuff.
Andrew just replied to my comment an hour ago, and I still have concerns
about the patchset. I didn't like the list_head of busses, and the 6352
family don't support the external mode. That needed to be addressed.

Why such an hurry to merge patches?
Because my backlog sucks otherwise?

I can revert, or you can ask Andrew to post follow-on fixups to
satisfy your concerns.  Just because I applied it doesn't make it
the end of the world.

And maybe I should apply patches aggressively, as this lights a fire
under everyone's butt and people review and fix things more quickly.

:-)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help