From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 24F59257D for ; Sun, 13 Aug 2023 19:05:04 +0000 (UTC) Received: from pidgin.makrotopia.org (pidgin.makrotopia.org [185.142.180.65]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 72246DA; Sun, 13 Aug 2023 12:04:59 -0700 (PDT) Received: from local by pidgin.makrotopia.org with esmtpsa (TLS1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.96) (envelope-from ) id 1qVGOJ-0000QV-0X; Sun, 13 Aug 2023 19:04:43 +0000 Date: Sun, 13 Aug 2023 20:04:29 +0100 From: Daniel Golle To: Andrew Lunn Cc: Qingfang Deng , SkyLake Huang , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Matthias Brugger , AngeloGioacchino Del Regno , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH net-next v2] net: phy: mediatek-ge-soc: support PHY LEDs Message-ID: References: <32e534441225c62e3bf9384b797d9beda7475053.1691943605.git.daniel@makrotopia.org> <8f20d427-91cc-4fbc-b263-dfc76df855f9@lunn.ch> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8f20d427-91cc-4fbc-b263-dfc76df855f9@lunn.ch> X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net On Sun, Aug 13, 2023 at 08:57:01PM +0200, Andrew Lunn wrote: > On Sun, Aug 13, 2023 at 05:24:55PM +0100, Daniel Golle wrote: > > Implement netdev trigger and primitive bliking offloading as well as > > simple set_brigthness function for both PHY LEDs of the in-SoC PHYs > > found in MT7981 and MT7988. > > > > For MT7988, read boottrap register and apply LED polarities accordingly > > Should this be bootstrap? With an S? boottrap appears quite often in > the code, so maybe the datasheet does say boottrap? Yes, datasheet and vendor code refer to boottrap register which reflects the state of the pins used for bootStrapping the SoC... > > Andrew