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=-5.1 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 autolearn=ham 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 7C526C4BA0B for ; Wed, 26 Feb 2020 10:40:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 52F7A20714 for ; Wed, 26 Feb 2020 10:40:11 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="r/p4PjDp" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727934AbgBZKkK (ORCPT ); Wed, 26 Feb 2020 05:40:10 -0500 Received: from pandora.armlinux.org.uk ([78.32.30.218]:46082 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726936AbgBZKkK (ORCPT ); Wed, 26 Feb 2020 05:40:10 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=PHjOim15jd4o09NnZjH5LH9s+ZcSG6bZ/1c5E3qyvo4=; b=r/p4PjDpcixJ8lUnSiXLAl4AV M3/nlPwheegQWB6gz85+4JN8JbeNT+1nUpxDGwwTIaDpnbxhk/5Fo4H893Ls6FoWMiBxz1xGLG9zU Y4R3FIqxjy6zmwaDYotrgUjsLzwq0ZIG0/klV1V2Toz4/xVllt3hk+UDInPxvqY1x8Pp7iWePdElj G33ifII3WAfBYsu7CSq/IaGilQ8xfQ16ikE6nIARLjvi9laOHpUwg1SsCwTSLsyCiz1Oj4UTXJm42 n78Ijep4FE90dHV5Ow9NHKtdBZF2uAFagJmklL2ZqtgGNqDZMKCq+00wOYdK2Uu61a3/96O/gA99Q j23bm05mQ==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:57120) by pandora.armlinux.org.uk with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.90_1) (envelope-from ) id 1j6u6i-00070o-Gh; Wed, 26 Feb 2020 10:40:00 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.92) (envelope-from ) id 1j6u6d-0008LS-Mn; Wed, 26 Feb 2020 10:39:55 +0000 Date: Wed, 26 Feb 2020 10:39:55 +0000 From: Russell King - ARM Linux admin To: Vladimir Oltean Cc: Andrew Lunn , Linux Doc Mailing List , Thomas Petazzoni , Ioana Ciornei , linux-stm32@st-md-mailman.stormreply.com, Florian Fainelli , Jonathan Corbet , Michal Simek , Jose Abreu , Jakub Kicinski , Mark Lee , Sean Wang , Alexandre Torgue , Hauke Mehrtens , Radhey Shyam Pandey , "moderated list:ARM/Mediatek SoC support" , John Crispin , Matthias Brugger , Giuseppe Cavallaro , "moderated list:ARM/Mediatek SoC support" , netdev , Vivien Didelot , Maxime Coquelin , Felix Fietkau , "David S. Miller" , Heiner Kallweit Subject: Re: [PATCH net-next 2/8] net: dsa: propagate resolved link config via mac_link_up() Message-ID: <20200226103955.GY25745@shell.armlinux.org.uk> References: <20200225093703.GS25745@shell.armlinux.org.uk> <20200225232307.GU25745@shell.armlinux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200225232307.GU25745@shell.armlinux.org.uk> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Tue, Feb 25, 2020 at 11:23:07PM +0000, Russell King - ARM Linux admin wrote: > On Tue, Feb 25, 2020 at 11:09:35PM +0200, Vladimir Oltean wrote: > > Hi Russell, > > > > On Tue, 25 Feb 2020 at 11:39, Russell King wrote: > > > > > > Propagate the resolved link configuration down via DSA's > > > phylink_mac_link_up() operation to allow split PCS/MAC to work. > > > > > > Signed-off-by: Russell King > > > --- > > > drivers/net/dsa/b53/b53_common.c | 4 +++- > > > drivers/net/dsa/b53/b53_priv.h | 4 +++- > > > drivers/net/dsa/bcm_sf2.c | 4 +++- > > > drivers/net/dsa/lantiq_gswip.c | 4 +++- > > > drivers/net/dsa/mt7530.c | 4 +++- > > > drivers/net/dsa/mv88e6xxx/chip.c | 4 +++- > > > drivers/net/dsa/sja1105/sja1105_main.c | 4 +++- > > > include/net/dsa.h | 4 +++- > > > net/dsa/port.c | 3 ++- > > > 9 files changed, 26 insertions(+), 9 deletions(-) > > > > > > > It looks like you missed the felix_phylink_mac_link_up() conversion in > > this patch? (which also makes it fail to build, by the way, I'm > > supposed the Kbuild robot didn't already jump) > > Nonetheless, I've manually added the missing speed, duplex, tx_pause > > and rx_pause parameters, and it appears to work as before. > > Same for sja1105. > > Quite possibly; the patch was developed against 5.5 plus the phylink > changes, but applied to net-next. Hmm, it seems my coccinelle script > that detects .mac_link_up initialiser prototypes, but not the DSA > equivalent using the old prototype. > > Thanks for pointing it out. I've just sent v2 with ocelot and qca included... and some recipients for patch 2 bounced - the Cc line is now too long! As far as the kbuild robot goes, you are not the first to notice that kbuild seems to be flakey... I've noticed it for some time, as has Greg KH. It isn't just patches on mailing lists, it's git trees as well. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up 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=-5.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 49050C4BA0B for ; Wed, 26 Feb 2020 10:40:37 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 19F3820714 for ; Wed, 26 Feb 2020 10:40:36 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="QV23hT4w"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="r/p4PjDp" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 19F3820714 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=armlinux.org.uk Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=tta8dJpEYt0R49OOf+VpUf6MoMTroB5V4iOMzpbKRQ0=; b=QV23hT4wKsJCGE Km10ElY+MRXctH9d/66pxjSMlcFu8Vmb6yHUMXpUWra3bKE7bEf3jxfYCMxrYyxgLjy9M454pi2S4 3FNFh9MQIysCfugsAFoGJVCTzqffRx2HjlEyRiOHFRJHh4YomBxgUHRPfNaPU0ROmOCAEvDDCMstZ l4Rz3IPdJMjLY/6HX4pxbuzw0oK1JTKTPWnz3sJawlTkl1RqXaTbZZ4kk8W7a5i0HBG62S+eyRQaw l5ksPz5iEPlIhkPUhf47OZBZ4qLuFGWSItfVNLqRNYM24EsX/yDtxnbih4JOptuyJfBjEWWgE9JEq vM4FG508irMcP8yGwDuQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1j6u7C-0005ue-35; Wed, 26 Feb 2020 10:40:30 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:3201:214:fdff:fe10:1be6]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1j6u79-0005tw-2u; Wed, 26 Feb 2020 10:40:28 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=PHjOim15jd4o09NnZjH5LH9s+ZcSG6bZ/1c5E3qyvo4=; b=r/p4PjDpcixJ8lUnSiXLAl4AV M3/nlPwheegQWB6gz85+4JN8JbeNT+1nUpxDGwwTIaDpnbxhk/5Fo4H893Ls6FoWMiBxz1xGLG9zU Y4R3FIqxjy6zmwaDYotrgUjsLzwq0ZIG0/klV1V2Toz4/xVllt3hk+UDInPxvqY1x8Pp7iWePdElj G33ifII3WAfBYsu7CSq/IaGilQ8xfQ16ikE6nIARLjvi9laOHpUwg1SsCwTSLsyCiz1Oj4UTXJm42 n78Ijep4FE90dHV5Ow9NHKtdBZF2uAFagJmklL2ZqtgGNqDZMKCq+00wOYdK2Uu61a3/96O/gA99Q j23bm05mQ==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:57120) by pandora.armlinux.org.uk with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.90_1) (envelope-from ) id 1j6u6i-00070o-Gh; Wed, 26 Feb 2020 10:40:00 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.92) (envelope-from ) id 1j6u6d-0008LS-Mn; Wed, 26 Feb 2020 10:39:55 +0000 Date: Wed, 26 Feb 2020 10:39:55 +0000 From: Russell King - ARM Linux admin To: Vladimir Oltean Subject: Re: [PATCH net-next 2/8] net: dsa: propagate resolved link config via mac_link_up() Message-ID: <20200226103955.GY25745@shell.armlinux.org.uk> References: <20200225093703.GS25745@shell.armlinux.org.uk> <20200225232307.GU25745@shell.armlinux.org.uk> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200225232307.GU25745@shell.armlinux.org.uk> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200226_024027_129232_25E57FD1 X-CRM114-Status: GOOD ( 20.64 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Andrew Lunn , Linux Doc Mailing List , Thomas Petazzoni , Ioana Ciornei , linux-stm32@st-md-mailman.stormreply.com, Florian Fainelli , Jonathan Corbet , Michal Simek , Jose Abreu , Jakub Kicinski , Vivien Didelot , Sean Wang , Alexandre Torgue , Hauke Mehrtens , Radhey Shyam Pandey , "moderated list:ARM/Mediatek SoC support" , John Crispin , Matthias Brugger , Giuseppe Cavallaro , "moderated list:ARM/Mediatek SoC support" , netdev , Mark Lee , Maxime Coquelin , Heiner Kallweit , "David S. Miller" , Felix Fietkau Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Tue, Feb 25, 2020 at 11:23:07PM +0000, Russell King - ARM Linux admin wrote: > On Tue, Feb 25, 2020 at 11:09:35PM +0200, Vladimir Oltean wrote: > > Hi Russell, > > > > On Tue, 25 Feb 2020 at 11:39, Russell King wrote: > > > > > > Propagate the resolved link configuration down via DSA's > > > phylink_mac_link_up() operation to allow split PCS/MAC to work. > > > > > > Signed-off-by: Russell King > > > --- > > > drivers/net/dsa/b53/b53_common.c | 4 +++- > > > drivers/net/dsa/b53/b53_priv.h | 4 +++- > > > drivers/net/dsa/bcm_sf2.c | 4 +++- > > > drivers/net/dsa/lantiq_gswip.c | 4 +++- > > > drivers/net/dsa/mt7530.c | 4 +++- > > > drivers/net/dsa/mv88e6xxx/chip.c | 4 +++- > > > drivers/net/dsa/sja1105/sja1105_main.c | 4 +++- > > > include/net/dsa.h | 4 +++- > > > net/dsa/port.c | 3 ++- > > > 9 files changed, 26 insertions(+), 9 deletions(-) > > > > > > > It looks like you missed the felix_phylink_mac_link_up() conversion in > > this patch? (which also makes it fail to build, by the way, I'm > > supposed the Kbuild robot didn't already jump) > > Nonetheless, I've manually added the missing speed, duplex, tx_pause > > and rx_pause parameters, and it appears to work as before. > > Same for sja1105. > > Quite possibly; the patch was developed against 5.5 plus the phylink > changes, but applied to net-next. Hmm, it seems my coccinelle script > that detects .mac_link_up initialiser prototypes, but not the DSA > equivalent using the old prototype. > > Thanks for pointing it out. I've just sent v2 with ocelot and qca included... and some recipients for patch 2 bounced - the Cc line is now too long! As far as the kbuild robot goes, you are not the first to notice that kbuild seems to be flakey... I've noticed it for some time, as has Greg KH. It isn't just patches on mailing lists, it's git trees as well. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek 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=-5.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 31620C4BA0B for ; Wed, 26 Feb 2020 10:40:42 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 0192620714 for ; Wed, 26 Feb 2020 10:40:42 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="ltGFc33z"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="r/p4PjDp" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0192620714 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=armlinux.org.uk Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=GzZ5Opb5ll11puNtDFpPmJCM6sle+U74yK5P22myuCA=; b=ltGFc33zw8bzae 4biGP9KRCcfkxkKGIqkdVI1BS8AKCskyEd5gbMdgf+7pV++RYySKzmlPp6o/dOhuPz5+dFKBqhKND JNPwmcSKUjzMgOT+EUzhbcobMfSkjaR+Pz9WE5jmWHemnuehRrHQtNIsWrZXV8KEKMrX5IuD4ggL/ 6m6V1IMF0C0CA8z6de2N5SW81bpMymNwf/YdDH3/kdLuuthyLM43WY7gKEqRkAVq7LzcOFaSN2G4l 1rk5O/VUU8lUMEHnvmCkiSc/dPg34kgvmKgaDySC1TbfqeoCOMx7YXCq7YC66Jd1yK1ZzIiXhaxlJ Y6C1bkuxiicmgwOj3dwA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1j6u7D-0005vO-3N; Wed, 26 Feb 2020 10:40:31 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:3201:214:fdff:fe10:1be6]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1j6u79-0005tw-2u; Wed, 26 Feb 2020 10:40:28 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=PHjOim15jd4o09NnZjH5LH9s+ZcSG6bZ/1c5E3qyvo4=; b=r/p4PjDpcixJ8lUnSiXLAl4AV M3/nlPwheegQWB6gz85+4JN8JbeNT+1nUpxDGwwTIaDpnbxhk/5Fo4H893Ls6FoWMiBxz1xGLG9zU Y4R3FIqxjy6zmwaDYotrgUjsLzwq0ZIG0/klV1V2Toz4/xVllt3hk+UDInPxvqY1x8Pp7iWePdElj G33ifII3WAfBYsu7CSq/IaGilQ8xfQ16ikE6nIARLjvi9laOHpUwg1SsCwTSLsyCiz1Oj4UTXJm42 n78Ijep4FE90dHV5Ow9NHKtdBZF2uAFagJmklL2ZqtgGNqDZMKCq+00wOYdK2Uu61a3/96O/gA99Q j23bm05mQ==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:57120) by pandora.armlinux.org.uk with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.90_1) (envelope-from ) id 1j6u6i-00070o-Gh; Wed, 26 Feb 2020 10:40:00 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.92) (envelope-from ) id 1j6u6d-0008LS-Mn; Wed, 26 Feb 2020 10:39:55 +0000 Date: Wed, 26 Feb 2020 10:39:55 +0000 From: Russell King - ARM Linux admin To: Vladimir Oltean Subject: Re: [PATCH net-next 2/8] net: dsa: propagate resolved link config via mac_link_up() Message-ID: <20200226103955.GY25745@shell.armlinux.org.uk> References: <20200225093703.GS25745@shell.armlinux.org.uk> <20200225232307.GU25745@shell.armlinux.org.uk> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200225232307.GU25745@shell.armlinux.org.uk> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200226_024027_129232_25E57FD1 X-CRM114-Status: GOOD ( 20.64 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Andrew Lunn , Linux Doc Mailing List , Thomas Petazzoni , Ioana Ciornei , linux-stm32@st-md-mailman.stormreply.com, Florian Fainelli , Jonathan Corbet , Michal Simek , Jose Abreu , Jakub Kicinski , Vivien Didelot , Sean Wang , Alexandre Torgue , Hauke Mehrtens , Radhey Shyam Pandey , "moderated list:ARM/Mediatek SoC support" , John Crispin , Matthias Brugger , Giuseppe Cavallaro , "moderated list:ARM/Mediatek SoC support" , netdev , Mark Lee , Maxime Coquelin , Heiner Kallweit , "David S. Miller" , Felix Fietkau Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Feb 25, 2020 at 11:23:07PM +0000, Russell King - ARM Linux admin wrote: > On Tue, Feb 25, 2020 at 11:09:35PM +0200, Vladimir Oltean wrote: > > Hi Russell, > > > > On Tue, 25 Feb 2020 at 11:39, Russell King wrote: > > > > > > Propagate the resolved link configuration down via DSA's > > > phylink_mac_link_up() operation to allow split PCS/MAC to work. > > > > > > Signed-off-by: Russell King > > > --- > > > drivers/net/dsa/b53/b53_common.c | 4 +++- > > > drivers/net/dsa/b53/b53_priv.h | 4 +++- > > > drivers/net/dsa/bcm_sf2.c | 4 +++- > > > drivers/net/dsa/lantiq_gswip.c | 4 +++- > > > drivers/net/dsa/mt7530.c | 4 +++- > > > drivers/net/dsa/mv88e6xxx/chip.c | 4 +++- > > > drivers/net/dsa/sja1105/sja1105_main.c | 4 +++- > > > include/net/dsa.h | 4 +++- > > > net/dsa/port.c | 3 ++- > > > 9 files changed, 26 insertions(+), 9 deletions(-) > > > > > > > It looks like you missed the felix_phylink_mac_link_up() conversion in > > this patch? (which also makes it fail to build, by the way, I'm > > supposed the Kbuild robot didn't already jump) > > Nonetheless, I've manually added the missing speed, duplex, tx_pause > > and rx_pause parameters, and it appears to work as before. > > Same for sja1105. > > Quite possibly; the patch was developed against 5.5 plus the phylink > changes, but applied to net-next. Hmm, it seems my coccinelle script > that detects .mac_link_up initialiser prototypes, but not the DSA > equivalent using the old prototype. > > Thanks for pointing it out. I've just sent v2 with ocelot and qca included... and some recipients for patch 2 bounced - the Cc line is now too long! As far as the kbuild robot goes, you are not the first to notice that kbuild seems to be flakey... I've noticed it for some time, as has Greg KH. It isn't just patches on mailing lists, it's git trees as well. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel