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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F1601C77B61 for ; Tue, 25 Apr 2023 08:30:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233484AbjDYIa3 (ORCPT ); Tue, 25 Apr 2023 04:30:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49842 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233659AbjDYIaE (ORCPT ); Tue, 25 Apr 2023 04:30:04 -0400 Received: from fudo.makrotopia.org (fudo.makrotopia.org [IPv6:2a07:2ec0:3002::71]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F0D5040E0; Tue, 25 Apr 2023 01:30:01 -0700 (PDT) Received: from local by fudo.makrotopia.org with esmtpsa (TLS1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.96) (envelope-from ) id 1prE3Z-0001vV-1h; Tue, 25 Apr 2023 10:29:49 +0200 Date: Tue, 25 Apr 2023 09:27:56 +0100 From: Daniel Golle To: Jakub Kicinski Cc: Andrew Lunn , Heiner Kallweit , netdev@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Russell King , "David S. Miller" , Eric Dumazet , Paolo Abeni , Qingfang Deng , SkyLake Huang , Matthias Brugger , AngeloGioacchino Del Regno Subject: Re: [PATCH v2] net: phy: add driver for MediaTek SoC built-in GE PHYs Message-ID: References: <20230424183755.3fac65b0@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230424183755.3fac65b0@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Apr 24, 2023 at 06:37:55PM -0700, Jakub Kicinski wrote: > On Sat, 22 Apr 2023 13:36:58 +0100 Daniel Golle wrote: > > Some of MediaTek's Filogic SoCs come with built-in gigabit Ethernet > > PHYs which require calibration data from the SoC's efuse. > > Despite the similar design the driver doesn't share any code with the > > existing mediatek-ge.c, so add support for these PHYs by introducing a > > new driver for only MediaTek's ARM64 SoCs. > > Andrew, Heiner, how do you feel about this driver? > > Daniel, is it high prio for you to get it into 6.4 or 6.5 is okay? No rush, as this is mostly relevant for OpenWrt the thing which matters most to me is having it in the next LTS kernel, so 6.5 should still be fine I suppose. > > I'm trying to get a read on whether we should merge it because it's > kind of on the goal line for this merge window. > > If nobody feels strongly we can do this the traditional way - I'll > just complain about RCT, hence kicking it back out into the field :) I understand Andrew's concerns and will resubmit for the next merge window in 2 weeks, and without the LED part for now until PHY LED infrastructure is more ready (ie. able to offload 'netdev' trigger to hardware)