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=1.7 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_DBL_ABUSE_MALW autolearn=no 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 36262C2D0DB for ; Tue, 28 Jan 2020 18:26:07 +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 0E42720716 for ; Tue, 28 Jan 2020 18:26:07 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="FWy/DRYw"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=lunn.ch header.i=@lunn.ch header.b="VebU66qE" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0E42720716 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=lunn.ch 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=AbLscqdTTmVIGRXjQlZ6rVsdtuB0uCXwIo4ExDKqQdM=; b=FWy/DRYwMw+iky yNB7A34pDt7Ii1iRxhRDtJiknVKBryxyypoF/EnP6BpUR7jJqrE2vvZh45zwfKuzfFZP5DEnxB5S/ vSPuQJBAn0k8OtKYt3+wQRPmJh+8YjWkHLok5jdJManyND0ODaF3NZ0D4JDed9X0JR7cg9kch6XjT RfzCOUYGGI8TvwEhTpZ5byzlukZHCcpaK5RpQWLXDpX+25OUDuQIe5LIooZIpC16h5MGDIndTbi0h WAbRc0ZA1zEkzKaufsaTQF6RrTy6fXpIPYmcUvsmugvOg9Hm1H59jc+u82GEYC6zrMnPXgsgg6Rbu 8YW31UsEl5orWqTVg8ZA==; 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 1iwVYm-0007FB-78; Tue, 28 Jan 2020 18:26:00 +0000 Received: from vps0.lunn.ch ([185.16.172.187]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1iwVYj-0007EG-BI for linux-arm-kernel@lists.infradead.org; Tue, 28 Jan 2020 18:25:58 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Sender: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=9rzg8pOQc/eaFdW99Vz4O3jHl+7pHiM7u4M/N2zjXkU=; b=VebU66qEXJs9WAZC/uirqsZ7mX OeOrKJJw/KVKUhkMEVs/QGETVhl2qzSFUd3E/lV7hdBCotWajXxt6dvKTE9vVde4qW/MMN5mMOp3S ZA1c5MLXkU2rC0fe7ztoIqt4Ohv3tPUiuPC8w1qtDw5vdx11umzbQlCyqmRrmTHhgoUk=; Received: from andrew by vps0.lunn.ch with local (Exim 4.93) (envelope-from ) id 1iwVYU-0003W8-Ff; Tue, 28 Jan 2020 19:25:42 +0100 Date: Tue, 28 Jan 2020 19:25:42 +0100 From: Andrew Lunn To: Russell King - ARM Linux admin Subject: Re: [RFC net-next 6/8] net: phylink: Configure MAC/PCS when link is up without PHY Message-ID: <20200128182542.GB12180@lunn.ch> References: <9a2136885d9a892ff170be88fdffeda82c778a10.1580122909.git.Jose.Abreu@synopsys.com> <20200127112102.GT25745@shell.armlinux.org.uk> <20200127114600.GU25745@shell.armlinux.org.uk> <20200127140038.GD13647@lunn.ch> <20200127140834.GW25745@shell.armlinux.org.uk> <20200127145107.GE13647@lunn.ch> <20200128180802.GD25745@shell.armlinux.org.uk> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200128180802.GD25745@shell.armlinux.org.uk> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200128_102557_390903_2CDC2400 X-CRM114-Status: GOOD ( 18.32 ) 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: Jose Abreu , Joao Pinto , Alexandre Torgue , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "David S. Miller" , Florian Fainelli , Maxime Coquelin , Giuseppe Cavallaro , Jakub Kicinski , "linux-stm32@st-md-mailman.stormreply.com" , "linux-arm-kernel@lists.infradead.org" , Heiner Kallweit 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, Jan 28, 2020 at 06:08:03PM +0000, Russell King - ARM Linux admin wrote: > On Mon, Jan 27, 2020 at 03:51:07PM +0100, Andrew Lunn wrote: > > I've also had issues with the DSA links, also being configured to > > 10/Half. That seems to be related to having a phy-mode property in > > device tree. I need to add a fixed-link property to set the correct > > speed. Something is broken here, previously the fixed-link was only > > needed if the speed needed to be lower than the ports maximum. I think > > that is a separate issue i need to dig into, not part of the PCS to > > MAC transfer. > > I think I understand what is happening on this one more fully. > > When DSA initialises, the DSA and CPU ports are initially configured to > maximum speed via mv88e6xxx_setup_port(), called via mv88e6xxx_setup(), > the .setup method, dsa_switch_setup(), and dsa_tree_setup_switches(). > > dsa_tree_setup_switches() then moves on to calling dsa_port_setup(). > dsa_port_setup() calls dsa_port_link_register_of() for the DSA and CPU > ports, which calls into dsa_port_phylink_register(). > > That calls phylink_create(), and then attempts to attach a PHY using > phylink_of_phy_connect() - which itself is rather weird - since when > has a DSA or CPU port been allowed to have a PHY in its DT node? Hi Russell There are some boards which have back to back PHYs between the SoC and the Switch. Most designs rely on strapping the PHYs to just work, no software configuration needed. But then came along a board with a PHY which needed a kick to make it work :-( > That hasn't changed in phylink yet - so it's a bug that dates back > to the phylink integration into the DSA core, and is a regression > resulting from that. And i think i probably did not notice it because nearly all the boards i test with connect to the FEC, which is Fast Ethernet only, so needs fixed-link properties. Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel