From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 36E93C43219 for ; Tue, 30 Apr 2019 12:04:21 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id F096E2075E for ; Tue, 30 Apr 2019 12:04:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1556625861; bh=AA/ezfwnN75aWAfPNZjA1OF7FxN3hdmIpnqpwiSVqM4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:List-ID:From; b=BJGk6kqDkPey9JmTRauiR+yWXT1L50vfJIKtFFrkVcqC5SYH5pS51MmgsnwMlhqPq QmrPdejXy/8ywk/c4y2SiFu1jOFlq6p6DUR0qUBfSNfFp4ga868+G2i1zfCXb/V8b5 xEjCavD66Q1ynyxGtw3zq6ScgfyjJ8R//BMi/EVE= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728708AbfD3Lmy (ORCPT ); Tue, 30 Apr 2019 07:42:54 -0400 Received: from mail.kernel.org ([198.145.29.99]:52506 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729621AbfD3Lmx (ORCPT ); Tue, 30 Apr 2019 07:42:53 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 4101121734; Tue, 30 Apr 2019 11:42:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1556624572; bh=AA/ezfwnN75aWAfPNZjA1OF7FxN3hdmIpnqpwiSVqM4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=jOXp1/VFOinEpSI8Ntez9+kSICCG7VVYfrN6SGA5Mcec4/CidIdAqrNRCFjC4jP4/ fCOYmjC3H/Vm/5XHoBH7Ea83fRL3jqxPHr39dInJXdkUespqVX+oHIFCYNfYbjnS/e e3a3uHs/nSGyA9BrOtSRN3V2C4gBQrTgQ7edhmM8= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Amit Cohen , Ido Schimmel , Jiri Pirko , "David S. Miller" Subject: [PATCH 4.14 45/53] mlxsw: spectrum: Fix autoneg status in ethtool Date: Tue, 30 Apr 2019 13:38:52 +0200 Message-Id: <20190430113558.669947083@linuxfoundation.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190430113549.400132183@linuxfoundation.org> References: <20190430113549.400132183@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Amit Cohen [ Upstream commit 151f0dddbbfe4c35c9c5b64873115aafd436af9d ] If link is down and autoneg is set to on/off, the status in ethtool does not change. The reason is when the link is down the function returns with zero before changing autoneg value. Move the checking of link state (up/down) to be performed after setting autoneg value, in order to be sure that autoneg will change in any case. Fixes: 56ade8fe3fe1 ("mlxsw: spectrum: Add initial support for Spectrum ASIC") Signed-off-by: Amit Cohen Signed-off-by: Ido Schimmel Acked-by: Jiri Pirko Signed-off-by: David S. Miller Signed-off-by: Greg Kroah-Hartman --- drivers/net/ethernet/mellanox/mlxsw/spectrum.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c +++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c @@ -2521,11 +2521,11 @@ mlxsw_sp_port_set_link_ksettings(struct if (err) return err; + mlxsw_sp_port->link.autoneg = autoneg; + if (!netif_running(dev)) return 0; - mlxsw_sp_port->link.autoneg = autoneg; - mlxsw_sp_port_admin_status_set(mlxsw_sp_port, false); mlxsw_sp_port_admin_status_set(mlxsw_sp_port, true);