Thread (35 messages) 35 messages, 8 authors, 2017-09-06

Re: [PATCH] DSA support for Micrel KSZ8895

From: Andrew Lunn <andrew@lunn.ch>
Date: 2017-08-27 13:59:18
Also in: lkml

On Sun, Aug 27, 2017 at 02:36:58PM +0200, Pavel Machek wrote:
Hi!

So I fought with the driver a bit more, and now I have something that
kind-of-works.

Thanks for keeping on working on this.
 
"great great hack" belows worries me.

Yeah, disabled code needs to be removed before merge.

No, tag_ksz part probably is not acceptable. Do you see solution
better than just copying it into tag_ksz1 file?

Any more comments, etc?
It would help with review if you split this up into multiple patches.
The change to the tagger should be one patch. The mdio emulation would
make a reasonable standalone patch etc.

I will do a more detailed review later.

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