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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 40428EB64DD for ; Fri, 23 Jun 2023 05:57:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=FUlGQetoORnS1U39jTo8Jha9YPF3CEBaDWZcUGSt7j4=; b=VF6qM6DEGaJnQE 8L4qwIHyWeaFP/we5dYWpsQqqIswI8JqjXCwNaiLbHUVV1v1rGk8l8UC/zCkAg5AUI987B7SZSQwo T7z46VGfTr9a64pCtT2heIlXgFdKhyqFgfNFyB/IHJIo80eRw+VQrwXFTwcZPKlb7vEvLwW/L5ZV0 NeHz7N9vrNV3cGDENXvofb05a9F8euWci2H3VVTqDevssl6QHXneiNlrBJnCGNfSG1b1UeElRQLoj asQsZBPEI3lseqfOwe1bOfPc8LzNmlsZEgnyiDJm7ezZ7jrw0djJCfdtx0MSiX6DUQcTyQKAu4hSO LjmQ2xdlw7SuvWm4xZyg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qCZnM-002eJT-1L; Fri, 23 Jun 2023 05:57:20 +0000 Received: from mga11.intel.com ([192.55.52.93]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qCZnK-002eJ4-1a for linux-arm-kernel@lists.infradead.org; Fri, 23 Jun 2023 05:57:19 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1687499838; x=1719035838; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=Y/CKVqWgLHa+Q3CB7q4KwhnuffZiwcoExTpZS7D9MfY=; b=AcZUW7B6CKZXWZJinViGaRvbds73pNDjIKIhuvVyiSsSRYhFgoTmocPi kPZsKcNWDTNT/Pas41YP/93/gM2GS5zMawvgxt2224B1+v+e8aB5+dKLi M1rqLnei2pO+rS1Zk6Wa2ofderi+Bjmo1eIjVfuNvcns+pWiMUh2eOn7C GS3aaJdlENB7yPkzVU96jVffGWrS+kW7EpLxsWEQ+ddhIllF6YiqH9WAS nZlonafZTByU0lQRp1UFSNVyiXv8zDHD7cLAAZyFI/bb206XC4DOyaC+P WdROpJWZTZuVnoJDlJvVWOBqE24GO9cHsmmpmdWhD/R06oOBmroKnDNGA Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10749"; a="358187302" X-IronPort-AV: E=Sophos;i="6.01,151,1684825200"; d="scan'208";a="358187302" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Jun 2023 22:57:17 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10749"; a="828252631" X-IronPort-AV: E=Sophos;i="6.01,151,1684825200"; d="scan'208";a="828252631" Received: from choongyo-mobl.gar.corp.intel.com (HELO [10.215.253.116]) ([10.215.253.116]) by fmsmga002-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Jun 2023 22:57:07 -0700 Message-ID: <260499f9-4746-d06b-4fac-a01f29365704@linux.intel.com> Date: Fri, 23 Jun 2023 13:57:05 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Subject: Re: [PATCH net-next 3/6] net: phy: update in-band AN mode when changing interface by PHY driver Content-Language: en-US To: Simon Horman Cc: Rajneesh Bhardwaj , David E Box , Hans de Goede , Mark Gross , Jose Abreu , Andrew Lunn , Heiner Kallweit , Russell King , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , =?UTF-8?Q?Marek_Beh=c3=ban?= , Jean Delvare , Guenter Roeck , Giuseppe Cavallaro , Alexandre Torgue , Jose Abreu , Maxime Coquelin , Richard Cochran , Philipp Zabel , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend , Wong Vee Khee , Jon Hunter , Jesse Brandeburg , Revanth Kumar Uppala , Shenwei Wang , Andrey Konovalov , Jochen Henneberg , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, platform-driver-x86@vger.kernel.org, linux-hwmon@vger.kernel.org, bpf@vger.kernel.org, Voon Wei Feng , Tee Min , Michael Sit Wei Hong , Lai Peter Jun Ann References: <20230622041905.629430-1-yong.liang.choong@linux.intel.com> <20230622041905.629430-4-yong.liang.choong@linux.intel.com> From: Choong Yong Liang In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230622_225718_540161_5DC85525 X-CRM114-Status: GOOD ( 16.61 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Simon, I will update it in v2. Thank you. On 22/6/2023 10:43 pm, Simon Horman wrote: > On Thu, Jun 22, 2023 at 12:19:02PM +0800, Choong Yong Liang wrote: >> From: "Tan, Tee Min" >> >> Add cur_link_an_mode into phy_device struct for PHY drivers to >> communicate the in-band AN mode setting with phylink framework. >> >> As there is a mechanism in PHY drivers to switch the PHY interface >> between SGMII and 2500BaseX according to link speed. In this case, >> the in-band AN mode should be switching based on the PHY interface >> as well, if the PHY interface has been changed/updated by PHY driver. >> >> For e.g., disable in-band AN in 2500BaseX mode, or enable in-band AN >> back for SGMII mode (10/100/1000Mbps). >> >> Signed-off-by: Tan, Tee Min >> Signed-off-by: Choong Yong Liang > > ... > >> diff --git a/include/linux/phy.h b/include/linux/phy.h >> index 11c1e91563d4..c685b526e307 100644 >> --- a/include/linux/phy.h >> +++ b/include/linux/phy.h >> @@ -756,6 +756,8 @@ struct phy_device { >> /* MACsec management functions */ >> const struct macsec_ops *macsec_ops; >> #endif >> + /* For communicate the AN mode setting with phylink framework. */ >> + u8 cur_link_an_mode; >> }; > > Hi Choong Yong Liang, > > Please consider adding cur_link_an_mode to the kernel doc > for struct phy_device - which is above the definition of struct phy_device. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel