From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753460AbdASXmJ convert rfc822-to-8bit (ORCPT ); Thu, 19 Jan 2017 18:42:09 -0500 Received: from mx0b-0016f401.pphosted.com ([67.231.156.173]:36803 "EHLO mx0b-0016f401.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752500AbdASXmG (ORCPT ); Thu, 19 Jan 2017 18:42:06 -0500 From: Jon Pannell To: Andrew Lunn , Gregory CLEMENT , Bob Bernstein , Christine Qiu CC: Vivien Didelot , Florian Fainelli , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "David S. Miller" , Jason Cooper , "Sebastian Hesselbarth" , Thomas Petazzoni , "linux-arm-kernel@lists.infradead.org" , Nadav Haklai , Wilson Ding , Kostya Porotchkin , Joe Zhou Subject: RE: [EXT] Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin Thread-Topic: [EXT] Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin Thread-Index: AQHScp3uhC4puM68A02duz8g/jwpmqFA4VeA//+UZxA= Date: Thu, 19 Jan 2017 23:41:28 +0000 Message-ID: References: <20170119214934.27442-1-gregory.clement@free-electrons.com> <20170119220607.GA25907@lunn.ch> In-Reply-To: <20170119220607.GA25907@lunn.ch> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.93.176.43] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-01-19_12:,, signatures=0 X-Proofpoint-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 impostorscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1612050000 definitions=main-1701190285 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org + Bob + Christine Jon Pannell -----Original Message----- From: Andrew Lunn [mailto:andrew@lunn.ch] Sent: Thursday, January 19, 2017 2:06 PM To: Gregory CLEMENT Cc: Vivien Didelot ; Florian Fainelli ; netdev@vger.kernel.org; linux-kernel@vger.kernel.org; David S. Miller ; Jason Cooper ; Sebastian Hesselbarth ; Thomas Petazzoni ; linux-arm-kernel@lists.infradead.org; Nadav Haklai ; Wilson Ding ; Kostya Porotchkin ; Joe Zhou ; Jon Pannell Subject: [EXT] Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin External Email ---------------------------------------------------------------------- > While comparing the datasheet and the ops functions used, some > question came to me. They should not prevent applying this series, but > their answer would help me to have a better understanding of the dsa > subsystem. > > - Are the temperature related operation still useful with dsa2 ? No. I'm in the process of moving the code into the Marvell PHY driver, since the sensor is in the embedded PHYs. What ID does the embedded PHY use? The 6390 has a blank ID, where as older device have a real ID. > - Why the setup is done differently between the 6390 and the 6352 > families when the have exactly the same register? EDSA on 6390 works differently to 6352, meaning it breaks. So we need to run the 6390 with DSA tagging, not EDSA. Maybe this is the source of the differences? It should also be noted that the 6390 support is not yet complete. I have a few more patches in my tree to post. > - On the Port Controller 2, the bit PORT_CONTROL_2_MAP_DA is set for > 6352 and not for 6390 whereas the same bit exists in 6360 and the > description for this bit is the same for both datasheet. Humm, it does look like it is missing mv88e6xxx_6390_family(chip). > > - Register PORT_ATU_CONTROL and PORT_PRI_OVERRIDE are reset on 6352 > and not on 6390. While here again the registers description are > the same. And the same here. I've mostly been working on where the 6390 is different. Where it is the same i've mostly ignored it so far :-) There is also an ongoing effort to remove all these big if statements with a list of families. Andrew