Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v4 0/3] Add support for mv88e6393x family of Marvell.
To:     Pavana Sharma <pavana.sharma@digi.com>, andrew@lunn.ch
Cc:     davem@davemloft.net, gregkh@linuxfoundation.org, kuba@kernel.org,
        linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
        vivien.didelot@gmail.com
References: <20201017193044.GO456889@lunn.ch>
 <cover.1603690201.git.pavana.sharma@digi.com>
From:   Florian Fainelli <f.fainelli@gmail.com>
Message-ID: <3746c90e-a3da-5484-2a85-a1bb6fa926a3@gmail.com>
Date:   Mon, 26 Oct 2020 06:43:15 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Firefox/78.0 Thunderbird/78.4.0
MIME-Version: 1.0
In-Reply-To: <cover.1603690201.git.pavana.sharma@digi.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 10/25/2020 10:52 PM, Pavana Sharma wrote:
> Hi,
> 
> Thanks for the review.
> Following is the updated patchset.
> 
> The 6393X family has MV88E6191X, MV88E6193X and MV88E6393X products listed in
> Gigabit Ethernet and Gigabit 10G+ Ethernet categories. There are no 6393 devices
> (without X) but there is 6191 device (without X)from a different family.
> The product id is listed with the 'X' in the name so I prefer to retain the
> product name 6393X in the driver whereas we can define the family name as
> 'MV88E6XXX_FAMILY_6393' without 'X'.
> 
> The patchset adds support for modes 5GBASE-R, 10GBASE-R and USXGMII on
> ports 0, 9 and 10.
> 
> Tested on MV88E6193X device.

Please subject your patches appropriately using "net: dsa: mv88e6xxx: " 
for DSA changes to the driver and "net: phy: " for PHY subsystem changes.
-- 
Florian