Received-SPF: pass (google.com: domain of linux-kernel+bounces-48364-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Precedence: bulk
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
Date: Thu, 01 Feb 2024 16:07:08 +0100
Message-Id: <CYTU44ED147E.2V11E1VTOPYSE@gmail.com>
Cc: "Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
 "linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>, "Thierry
 Reding" <treding@nvidia.com>
Subject: Re: linux-next: Tree for Jan 29 (soc/tegra/fuse/fuse-tegra30.c)
From: "Thierry Reding" <thierry.reding@gmail.com>
To: "Arnd Bergmann" <arnd@arndb.de>, "Jon Hunter" <jonathanh@nvidia.com>,
 "Randy Dunlap" <rdunlap@infradead.org>, "Stephen Rothwell"
 <sfr@canb.auug.org.au>, "linux-next" <linux-next@vger.kernel.org>, "Kartik"
 <kkartik@nvidia.com>
References: <20240129143030.16647483@canb.auug.org.au>
 <d0e1a7c1-06a2-4fa4-b4f8-df149af8eccb@infradead.org>
 <61e8b6ae-fd30-4443-8473-6429f6de8768@nvidia.com>
 <dafa2fb5-276d-4c87-b003-4b7f30d44f0a@app.fastmail.com>
In-Reply-To: <dafa2fb5-276d-4c87-b003-4b7f30d44f0a@app.fastmail.com>

On Tue Jan 30, 2024 at 11:59 AM CET, Arnd Bergmann wrote:
> On Tue, Jan 30, 2024, at 11:46, Jon Hunter wrote:
> > On 30/01/2024 04:06, Randy Dunlap wrote:
>
> > Looks like we are missing the following ...
> >
> > diff --git a/drivers/soc/tegra/fuse/fuse-tegra30.c=20
> > b/drivers/soc/tegra/fuse/fuse-tegra30.c
> > index 2070d36c510d..eb14e5ff5a0a 100644
> > --- a/drivers/soc/tegra/fuse/fuse-tegra30.c
> > +++ b/drivers/soc/tegra/fuse/fuse-tegra30.c
> > @@ -38,7 +38,8 @@
> >       defined(CONFIG_ARCH_TEGRA_210_SOC) || \
> >       defined(CONFIG_ARCH_TEGRA_186_SOC) || \
> >       defined(CONFIG_ARCH_TEGRA_194_SOC) || \
> > -    defined(CONFIG_ARCH_TEGRA_234_SOC)
> > +    defined(CONFIG_ARCH_TEGRA_234_SOC) || \
> > +    defined(CONFIG_ARCH_TEGRA_241_SOC)
> >   static u32 tegra30_fuse_read_early(struct tegra_fuse *fuse, unsigned=
=20
> > int offset)
> >
> >
> > Kartik, can you send a fix for this?
>
> If I get an Ack for my original patch, I can pick that
> up into the soc tree directly:
>
> https://lore.kernel.org/all/20240103102654.3779458-1-arnd@kernel.org/

Sorry for the delay on this. I guess you can't really apply that to ARM
SoC directly because the patch that introduces Tegra241 support is only
in the Tegra tree.

In any case, I've applied your patch now, so the fixed version should
show up in tomorrow's linux-next.

I'm considering squashing it into the original since that'd preserve
bisectability, though that'd probably be of limited usefulness because
most people doing regression builds involving this will likely have one
of the other bits enabled.

Thierry