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 C1BDFC3064D for ; Tue, 25 Jun 2024 06:17:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: Content-Transfer-Encoding:Content-Type:In-Reply-To:From:References: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=lUfaaGjmkEsv8TO+/zIvt/jIaH/4czGbfBiNdDpBPnU=; b=Yj6T0iD2NTolax OUVlwBBp1k1n9klqj5/aXLlJlV21iLqHkalyjsoYZUBFcZOuQegTK2Abdiqp2Nb6wcYz+jRc7+kkB H4Ni8ELJINnQNROOJUNg9ZdXKgARdW4r36XFrr7rZxYUzg0Jhc3M6dwX04ZmXZ9EVgpGIlM2EbEiE lzAYWRBxIAEn+NQ7Ayl+MnmZ1ABRuiYhu0vHq4ZNXBaBNbFODWCYdMoY9o1ETZPL56nTKmdejSqxI 0wZ872fxGt6oxU57Cny30Fn8I3urpFHfsZ0tP23vZKL8TrCVIlazVVg//m8RqshhAhin0mzSpPawM o4V40vakD00/hfjRWGpw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sLzUY-00000001mFP-0SwN; Tue, 25 Jun 2024 06:17:22 +0000 Received: from relay4-d.mail.gandi.net ([217.70.183.196]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sLzUS-00000001mDS-1dAP; Tue, 25 Jun 2024 06:17:18 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id 1EFA3E0007; Tue, 25 Jun 2024 06:17:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arinc9.com; s=gm1; t=1719296234; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=lUfaaGjmkEsv8TO+/zIvt/jIaH/4czGbfBiNdDpBPnU=; b=gottfuXyiDMseBTnGspopuXIw9529/9Sg8tVXkB/CfX+qkFNzuNLD3y0yeBtuhKxNBAkNe ZK5m650lLIhx+QPEZIDBbbwDMsqsxTiecOt3ksKVFyPts+2z8TiWsCmNQYVWFzymGIRTaX ahbVGJDQ3Z0cpMZg7en5CRA6laI8pgztW5RsyUdeskY58RAjMeGiQiKEYyXhwLFPen/f7k T6LiwaW6m4tAC7Xm2gazxlRsD3SV+CqggSC1Nt/R3OC2k1RNfvPbuPZtBUOY8WtllyfGri 7Zkj2qEE5eYrmT8yZ1HWrJwS53nlyKa5iXgjb9IGSyDoZrV83rr0CioXdd9y1A== Message-ID: <7a2ea06b-ae4e-4374-82c2-4de4184e06c3@arinc9.com> Date: Tue, 25 Jun 2024 09:17:09 +0300 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] arm64: dts: mt7622: fix switch probe on bananapi-r64 To: Linux regressions mailing list , Paolo Abeni , AngeloGioacchino Del Regno References: <20240516204847.171029-1-linux@fw-web.de> <5AEE5668-0C8E-4EE4-A398-66CB99DF5650@public-files.de> <43aacd9d-b851-4100-8ccc-878ac6ae10f8@leemhuis.info> <698cf562-1ca9-4aa3-be7e-a1474b612c5b@leemhuis.info> <0cba095c-3d55-416a-a7ad-b359129731cf@arinc9.com> <714da201-654b-4183-8e5e-8ff0b64fe621@leemhuis.info> <2cac4cf68304e81abffbd9ff0387ee100323c2b7.camel@redhat.com> <1807a142-1534-4fa4-ad4b-d1c03af014c2@arinc9.com> <58d8ddea-71cc-427a-94cc-a95f6bce61d2@collabora.com> <16e9c06e-9908-455d-a387-614fefe5bcf8@arinc9.com> <5e87d31c-b059-4f9a-93f7-dc87465ed14a@collabora.com> <4416ef22-78cc-4ce5-b61d-69ff0903811e@arinc9.com> <750a60a6-4585-4bd2-97be-cf944e51fbdb@leemhuis.info> Content-Language: en-US From: =?UTF-8?B?QXLEsW7DpyDDnE5BTA==?= In-Reply-To: <750a60a6-4585-4bd2-97be-cf944e51fbdb@leemhuis.info> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-GND-Sasl: arinc.unal@arinc9.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240624_231716_710987_C2100F3A X-CRM114-Status: GOOD ( 22.41 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Conor Dooley , linux-kernel@vger.kernel.org, Daniel Golle , linux-mediatek@lists.infradead.org, Matthias Brugger , Krzysztof Kozlowski , Frank Wunderlich , Rob Herring , linux-arm-kernel@lists.infradead.org Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On 25/06/2024 08.56, Linux regression tracking (Thorsten Leemhuis) wrote: > On 17.06.24 13:08, Arınç ÜNAL wrote: >> On 17/06/2024 11:33, Linux regression tracking (Thorsten Leemhuis) wrote: >> [...] >> I've submitted a patch series that fixes the regression. Angelo argued >> against the way the regression is fixed. I've very clearly argued back why >> I find Angelo's approach wrong. There's been no response back. I don't >> understand why reverting the patch is the likely outcome > > Long story short: because that how things like that are handled in the > Linux kernel project, as Linus wants it like that. See some of the > quotes from https://docs.kernel.org/process/handling-regressions.html > for details. > >> whilst the >> standing argument points towards applying the said patch series. If a >> revert happens before this discussion with Angelo finalises, this will set >> a precedent that will tell maintainers that they can have their way by just >> not replying to the ongoing discussions. >> >> That said, the decision of resolving the regression by either reverting the >> patch or applying the patch series shall not depend on whether or not >> Angelo is pleased but rather there're no counter-arguments left on the >> points brought, meaning the decision shall be made depending on the >> argument that stands. >> >> Therefore, I suggest that unless Angelo responds back with a >> counter-argument in the window of a week or two, as you've described, my >> patch series shall be applied. > > It looks more and more like we are stuck here (or was there progress and > I just missed it?) while the 6.10 final is slowly getting closer. Hence: There hasn't been progress at all. I believe I have clearly described the way out of this issue. > > AngeloGioacchino, should we ask the net maintainers to revert > 868ff5f4944aa9 ("net: dsa: mt7530-mdio: read PHY address of switch from > device tree") for now to resolve this regression? Reminder, there is > nothing wrong with that commit per se afaik, it just exposes a problem > that needs to be fixed first before it can be reapplied. Are you suggesting the patch shall be reverted first, then the DT patch applied, then the reverted patch applied back? If only one of the first two steps were done, it would fix the regression so I don't understand why go through this tedious process when we can quite simply apply the DT patch to resolve the regression. Keep in mind that I maintain the MT7530 DSA subdriver and the company I work with has got boards that uses the functionality the commit 868ff5f4944aa9 ("net: dsa: mt7530-mdio: read PHY address of switch from device tree") brings. Arınç