Groups > linux.kernel > #1720935

RE: [PATCH] DSA support for Micrel KSZ8895

From	<Woojung.Huh@microchip.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH] DSA support for Micrel KSZ8895
Date	2017-08-28 00:10 +0200
Message-ID	<ujdUK-5nR-9@gated-at.bofh.it> (permalink)
References	<uf1p8-4gv-31@gated-at.bofh.it> <uf7bb-83I-5@gated-at.bofh.it> <ujdUK-5nR-11@gated-at.bofh.it> <uj519-7Ez-21@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Pavel,

Thanks for update and sorry about email format (due to web-access version)
I'll do review when getting back to office later this week.

- Woojung
________________________________________
From: Pavel Machek [pavel@denx.de]
Sent: Sunday, August 27, 2017 8:36 AM
To: Woojung Huh - C21699; nathan.leigh.conrad@gmail.com
Cc: vivien.didelot@savoirfairelinux.com; f.fainelli@gmail.com; netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Tristram.Ha@micrel.com; andrew@lunn.ch; pavel@denx.de
Subject: [PATCH] DSA support for Micrel KSZ8895

Hi!

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

"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?

Help would be welcome.

Back to linux.kernel | Previous | Next | Find similar | Unroll thread

Thread

RE: [PATCH] DSA support for Micrel KSZ8895 <Woojung.Huh@microchip.com> - 2017-08-28 00:10 +0200

csiph-web