MIME-Version: 1.0
References: <20230206002136.29401-1-prabhakar.mahadev-lad.rj@bp.renesas.com> <CAMuHMdUXXrYPcNiD3M_ONjX2Jn16KKXdY=4JMqJbm9F=9GMBvA@mail.gmail.com>
In-Reply-To: <CAMuHMdUXXrYPcNiD3M_ONjX2Jn16KKXdY=4JMqJbm9F=9GMBvA@mail.gmail.com>
From:   "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
Date:   Fri, 10 Feb 2023 13:45:32 +0000
Message-ID: <CA+V-a8uWyCrziE2+LNy7jK0hU-5bKUC2i++oQqEt8qoqwjm07Q@mail.gmail.com>
Subject: Re: [PATCH] arm64: dts: renesas: Drop specifying the
 GIC_CPU_MASK_SIMPLE() for GICv3 systems
To:     Geert Uytterhoeven <geert@linux-m68k.org>
Cc:     Magnus Damm <magnus.damm@gmail.com>,
        Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Biju Das <biju.das.jz@bp.renesas.com>,
        Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Hi Geert,

Thank you for the review.

On Fri, Feb 10, 2023 at 11:10 AM Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
>
> Hi Prabhakar,
>
> On Mon, Feb 6, 2023 at 1:28 AM Prabhakar <prabhakar.csengg@gmail.com> wrote:
> > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> >
> > The GICv3 interrupts binding does not have a cpumask. The CPU mask only
> > applies to pre-GICv3. So just drop using them from GICv3 systems.
> >
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>
> Thanks for your patch!
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> i.e. will queue in renesas-devel for v6.4...
>
> > ---
> > Note, this patch applies on top of [0]
> >
> > [0] https://patchwork.kernel.org/project/linux-renesas-soc/cover/20230131223529.11905-1-prabhakar.mahadev-lad.rj@bp.renesas.com/
>
> As this patch is a fix, I think it makes sense to drop that dependency.
>
Agreed.

> So I'll do
>
>     git show -- arch/arm64/boot/dts/renesas/r9a07g044.dtsi | sed -e
> 's/044/054/g' | patch -p1 && git commit -a --amend
>
Hmm that's something new for me to learn!

> while applying.
>
Thanks.

Cheers,
Prabhakar