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 04F02CF8875 for ; Thu, 20 Nov 2025 16:31:34 +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: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To: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-Owner; bh=YGrCewVHXvu+rFqlELM7WZye4F0fUw/JNM/JxlkEcks=; b=B0bZOtpYwCouXbjhznGaNbmD7U 9n5HGmUQT9OLSrBnjm24D+fdjcv1cgeEIiNutwVBKLXtshQiYUmegDs7YjiMFyNMORuSlon9G3zg+ jeFpf9t49NSJ1WQe7u9KDzrujeYhGbh3bgPRgI4+dbBkOKtpo+I3QPa6XSr2GT8ulFsGozaH/Dug9 fnMTZxnq/yeMQysIR2RgXUzil/zzALDHIk0LSE5XgZoiDkHz5vMuTsNd/77RXKKLJ5zAW+3lPXU+8 94wUgXDalk+6xX9bqRw8gBqKBWLktffP/EftBQTtaJb/s20Jup+KznatWfvqGvf+/AMhb4xscKgy+ hYhWNPrA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vM7Z4-000000071Hv-0Swd; Thu, 20 Nov 2025 16:31:22 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vM7Z2-000000071Hl-1VtG for linux-riscv@lists.infradead.org; Thu, 20 Nov 2025 16:31:20 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id 6EA5D60194; Thu, 20 Nov 2025 16:31:19 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A1E5BC4CEF1; Thu, 20 Nov 2025 16:31:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763656279; bh=+t2RxpZYsc6EBAkiIiAw8xa7Z0l4LUgdt6k1NQShhzY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=SHG3OFeLTB4Hj/eETcebnotdZAETCyejYgNdFRTUA1Z8nSG3U05azdEclaOz6Ssh6 WGlSzcJDRR7HjLoc9RiM/s7WxrbV8ZQmTzPP/nI5nW0RvCvF2imrMqN2t2TMNV//AA xhrUrqjY0CvoJkHJtTnNGJbtTmWBKSzsKc7fKM1PyLGKqzfyPd/ilqJayxqLipxuec RuHVValyQL2PlUbpDYtZrf9TALOC2xKsSS22gKw0fu7ud0tTUVY16Q6a784OjhOnTQ PQuzF8U11sO5gvcfKfX0jI7PpswnY9Grv0V8owCsIPlf5WO9hQzRre8hbwwJU/Srvp SmNMctiO7Ywrw== Date: Thu, 20 Nov 2025 16:31:11 +0000 From: Conor Dooley To: netdev@vger.kernel.org Cc: Conor Dooley , Valentina.FernandezAlanis@microchip.com, Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Daire McNamara , Paul Walmsley , Palmer Dabbelt , Albert Ou , Alexandre Ghiti , Nicolas Ferre , Claudiu Beznea , Richard Cochran , Samuel Holland , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, Dave Stevenson , Sean Anderson , Vineeth Karumanchi , Abin Joseph , neil.armstrong@linaro.org Subject: Re: [RFC net-next v1 0/7] highly rfc macb usrio/tsu patches Message-ID: <20251120-unreal-antitrust-e32900cf893b@spud> References: <20251120-jubilant-purposely-67ec45ce4e2f@spud> MIME-Version: 1.0 In-Reply-To: <20251120-jubilant-purposely-67ec45ce4e2f@spud> X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============3311202477777319013==" Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org --===============3311202477777319013== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="TId/bCOlj9KNVAAR" Content-Disposition: inline --TId/bCOlj9KNVAAR Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Nov 20, 2025 at 04:26:02PM +0000, Conor Dooley wrote: > From: Conor Dooley >=20 > Hey folks, >=20 > After doing some debugging of broken tsu/ptp support on mpfs, I've come > up with some very rfc patches that I'd like opinions on - particularly > because they impact a bunch of platforms that I have no access to at all > and have no idea how they work. The at91 platforms I can just ask > Nicolas about (and he already provided some info directly, so I'm not > super worried at least about the usrio portion there) but the others > my gut says are likely incorrect in the driver at the moment. >=20 > These patches *are* fairly opinionated and not necessarily technically > correct or w/e. The only thing I am confident in saying that they are is > more deliberate than what's being done at the moment. >=20 > At the very least, it'd be good of the soc vendor folks could check > their platforms and see if their usrio stuff actually lines up with what > the driver currently calls "macb_default_usrio". Ours didn't and it was > a nasty surprise. >=20 > Cheers, > Conor. >=20 > CC: Valentina.FernandezAlanis@microchip.com > CC: Andrew Lunn > CC: David S. Miller > CC: Eric Dumazet > CC: Jakub Kicinski > CC: Paolo Abeni > CC: Rob Herring > CC: Krzysztof Kozlowski > CC: Conor Dooley > CC: Daire McNamara > CC: Paul Walmsley > CC: Palmer Dabbelt > CC: Albert Ou > CC: Alexandre Ghiti > CC: Nicolas Ferre > CC: Claudiu Beznea > CC: Richard Cochran > CC: Samuel Holland > CC: netdev@vger.kernel.org > CC: devicetree@vger.kernel.org > CC: linux-kernel@vger.kernel.org > CC: linux-riscv@lists.infradead.org > CC: Neil Armstrong The perils of grabbing addresses from git blame.. +CC Neil @ linaro. > CC: Dave Stevenson > CC: Sean Anderson > CC: Vineeth Karumanchi > CC: Abin Joseph >=20 > Conor Dooley (7): > riscv: dts: microchip: add tsu clock to macb on mpfs > net: macb: warn on pclk use as a tsu_clk fallback > net: macb: rename macb_default_usrio to at91_default_usrio as not all > platforms have mii mode control in usrio > net: macb: np4 doesn't need a usrio pointer > dt-bindings: net: macb: add property indicating timer adjust mode > net: macb: afaict, the driver doesn't support tsu timer adjust mode > net: macb: add mpfs specific usrio configuration >=20 > .../devicetree/bindings/net/cdns,macb.yaml | 15 +++ > arch/riscv/boot/dts/microchip/Makefile.orig | 26 ++++ > arch/riscv/boot/dts/microchip/mpfs.dtsi | 8 +- > drivers/net/ethernet/cadence/macb.h | 3 + > drivers/net/ethernet/cadence/macb_main.c | 123 +++++++++++------- > 5 files changed, 125 insertions(+), 50 deletions(-) > create mode 100644 arch/riscv/boot/dts/microchip/Makefile.orig >=20 > --=20 > 2.51.0 >=20 --TId/bCOlj9KNVAAR Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCaR9CTwAKCRB4tDGHoIJi 0l9+AQCAjXs4acBGojRkX7id4pZPjNPBVHlGllV6ND5awkZO7AD/fgnp3TeWWyfS eZPQo7qlyaefIwAK1U7ee9plirXEHQw= =Lpsd -----END PGP SIGNATURE----- --TId/bCOlj9KNVAAR-- --===============3311202477777319013== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv --===============3311202477777319013==--