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 0A1F8C7EE25 for ; Fri, 12 May 2023 09:02:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240376AbjELJCg (ORCPT ); Fri, 12 May 2023 05:02:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38880 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240351AbjELJCb (ORCPT ); Fri, 12 May 2023 05:02:31 -0400 Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [IPv6:2001:4d48:ad52:32c8:5054:ff:fe00:142]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DAFB5100FD; Fri, 12 May 2023 02:02:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: 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-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=Mggnu95n/RU6DeRgS9ZUs2b/avyHGbZvVWNqilcNijI=; b=MDA2vJcmlGYWbAYVWlbKScMTRY vF42WVOhOikWSi9jWbuFHQ1JtHvU+8G0TGE1XfkmCbjSEXOLHOVHrIJ0VnEFagkOI1H6/No9N+j4O bWS5qIDeN9+7jVQqNBdeLLEQV9cJTPdrn4oc3QMb1vAo1q3WAOb/RI8dV//oqci/88ItYS0mRLrbH +M7Lf+sPQWMHsM3p8dndw1zrdtj6CtRmEQiaKmgMd2CouTyywvldb1/ivBWaaqPCMiFY/9Knc1vbG PprqPQeiksn1ahH0l/mpmcEBqq2Fmy6SrZeBdJJkTGwCi3hUQyxwK6WBxkmKQ1Fn44mlmFlEfHbeg WP/aYyOQ==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:35578) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1pxOfA-00081X-H1; Fri, 12 May 2023 10:02:08 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1pxOf7-0004vy-PP; Fri, 12 May 2023 10:02:05 +0100 Date: Fri, 12 May 2023 10:02:05 +0100 From: "Russell King (Oracle)" To: Yan Wang Cc: andrew@lunn.ch, hkallweit1@gmail.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5] net: mdiobus: Add a function to deassert reset Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: Russell King (Oracle) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 12, 2023 at 03:08:53PM +0800, Yan Wang wrote: > + gpiod_set_value_cansleep(reset, gpiod_is_active_low(reset)); > + fsleep(reset_assert_delay); > + gpiod_set_value_cansleep(reset, !gpiod_is_active_low(reset)); Andrew, one of the phylib maintainers and thus is responsible for code in the area you are touching. Andrew has complained about the above which asserts and then deasserts reset on two occasions now, explained why it is wrong, but still the code persists in doing this. I am going to add my voice as another phylib maintainer to this and say NO to this code, for the exact same reasons that Andrew has given. You now have two people responsible for the code in question telling you that this is the wrong approach. Until this is addressed in some way, it is pointless you posting another version of this patch. Thanks. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!