From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756309AbcIMRKB (ORCPT ); Tue, 13 Sep 2016 13:10:01 -0400 Received: from mail-pf0-f193.google.com ([209.85.192.193]:36324 "EHLO mail-pf0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751222AbcIMRJ7 (ORCPT ); Tue, 13 Sep 2016 13:09:59 -0400 Subject: Re: [PATCH 3/3] net-next: dsa: add new driver for qca8xxx family To: Andrew Lunn , John Crispin References: <1473669337-21221-1-git-send-email-john@phrozen.org> <1473669337-21221-4-git-send-email-john@phrozen.org> <20160913004047.GB3585@lunn.ch> <7f9c1463-1cc0-e230-e7fe-dbb434427251@phrozen.org> <20160913155926.GP11400@lunn.ch> Cc: "David S. Miller" , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, qsdk-review@qca.qualcomm.com From: Florian Fainelli Message-ID: <371ed906-5ce8-0553-1400-11f0abf4b489@gmail.com> Date: Tue, 13 Sep 2016 10:09:56 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20160913155926.GP11400@lunn.ch> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/13/2016 08:59 AM, Andrew Lunn wrote: >> Hi Andrew, >> >> this function does indeed duplicate the functionality of >> phy_ethtool_get_eee() with the small difference, that e->eee_active is >> also set which phy_ethtool_get_eee() does not set. >> >> dsa_slave_get_eee() will call phy_ethtool_get_eee() right after the >> get_eee() op has been called. would it be ok to move the code setting >> eee_active to phy_ethtool_get_eee(). Humm, AFAIR, the reason why eee_active is set outside of phy_ethtool_set_eee() is because this is a MAC + PHY thing, both need to agree and support that, and so while the PHY may be configured to have EEE advertised and enabled, you also need to take care of the MAC portion and enable EEE in there as well. Is not there such a thing for the qca8k switch where the PHY needs to be configured through the standard phylib calls, but the switch's transmitter/receiver also needs to have EEE enabled? -- Florian