Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 30 May 2023 08:15:47 +0100
Message-ID: <87r0qyjnmk.wl-maz@kernel.org>
From:   Marc Zyngier <maz@kernel.org>
To:     Yan =?UTF-8?B?WmhlbmfvvIjkuKXmlL/vvIk=?= <zhengyan@asrmicro.com>
Cc:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Gao =?UTF-8?B?TWVpdGFv77yI6auY546r5rab77yJ?= 
        <meitaogao@asrmicro.com>,
        =?UTF-8?B?Ilpob3UgUWlhbyjlkajkvqgpIg==?= <qiaozhou@asrmicro.com>,
        "tglx@linutronix.de" <tglx@linutronix.de>,
        =?UTF-8?B?IlpoYW5nIFpoaXpob3Uo?= =?UTF-8?B?5byg5rK75rSyKSI=?= 
        <zhizhouzhang@asrmicro.com>
Subject: Re: [PATCH v2] irqchip/gic-v3: workaround for ASR8601 when reading mpidr
In-Reply-To: <6452e00de4e446ddb25588e29fa9e768@exch03.asrmicro.com>
References: <20230517075500.43516-1-zhengyan@asrmicro.com>
        <20230522110643.3063073-1-zhengyan@asrmicro.com>
        <87sfbfji4q.wl-maz@kernel.org>
        <6452e00de4e446ddb25588e29fa9e768@exch03.asrmicro.com>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue)
 FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/28.2
 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO)
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Tue, 30 May 2023 07:07:21 +0100,
Yan Zheng=EF=BC=88=E4=B8=A5=E6=94=BF=EF=BC=89 <zhengyan@asrmicro.com> wrote:
>=20
> > I came up with the following patch, which keeps the workaround *in
> > a single spot*.
> >=20
> > Let me know if that works for you.
> >=20
> > 	M.
> >=20
>=20
> Really thanks for your help, this patch works well for us.
> My previous patch was too ugly, this one looks better. I didn't find this=
 good solution.
> Should I send a patch for v3, considering that you're the author?

See [1]. I've split the refactoring in its own patch, and kept your
authorship for what is left of the workaround code.

No need to resend, I'll queue that for 6.5.

	M.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms.git/l=
og/?h=3Dirq/gic-v3-asr8601

--=20
Without deviation from the norm, progress is not possible.