netdev - Re: [PATCH] dsa: mv88e6352/mv88e6xxx: Add support for Marvell 88E6320 and 88E6321

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20150709.143701.1788505384324235588.davem@davemloft.net>
Date:	Thu, 09 Jul 2015 14:37:01 -0700 (PDT)
From:	David Miller <davem@...emloft.net>
To:	andrew@...n.ch
Cc:	linux@...ck-us.net, netdev@...r.kernel.org, f.fainelli@...il.com,
	linux-kernel@...r.kernel.org, ioctl@...dex.ru
Subject: Re: [PATCH] dsa: mv88e6352/mv88e6xxx: Add support for Marvell
 88E6320 and 88E6321

From: Andrew Lunn <andrew@...n.ch>
Date: Wed, 8 Jul 2015 16:52:34 +0200

> On Tue, Jul 07, 2015 at 08:38:15PM -0700, Guenter Roeck wrote:
>> From: "Aleksey S. Kazantsev" <ioctl@...dex.ru>
>> 
>> MV88E6320 and MV88E6321 are largely compatible to MV886352,
>> but are members of a different chip family.
>> 
>> Signed-off-by: Aleksey S. Kazantsev <ioctl@...dex.ru>
>> Signed-off-by: Guenter Roeck <linux@...ck-us.net>
> 
> Reviewed-by: Andrew Lunn <andrew@...n.ch>
> 
> One thing we might want to consider, is moving the temperate sensor
> code into mv88e6xxx. It seems likely future devices will also have the
> same sensor, so moving it to the shared code would make sense. We can
> then keep all mv88e6xxx_XXXX_family() stuff in a single file.
> 
> I don't see this as a blocker for this patchset, but now might be the
> right time to do this.

Please do not quote an entire patch just to add a reviewed-by or
acked-by tag.   If you have to quote some of the patch to comment
on specific portions, only quote exactly the hunks of the patch
relevant to what you want to discuss.

It doesn't take much effort to properly clip all except the most
relevant quoted text in your reply, and doing so I consider a very
basic common courtesy to everyone else on this mailing list.  It's
especially a courtesy to me as I have to scroll through the threads
and patchwork entries for every single patch posted here.

Thanks in advance for your consideration.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html