Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates
 216.228.112.34 as permitted sender) receiver=protection.outlook.com;
 client-ip=216.228.112.34; helo=mail.nvidia.com;
Date:   Thu, 3 Jun 2021 14:18:49 +0200
From:   Thierry Reding <treding@nvidia.com>
To:     Dmitry Osipenko <digetx@gmail.com>
CC:     Stephen Rothwell <sfr@canb.auug.org.au>,
        Colin Cross <ccross@android.com>,
        Olof Johansson <olof@lixom.net>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>
Subject: Re: linux-next: build warning after merge of the tegra tree
Message-ID: <YLjIqfeiaJqeJQtw@orome.fritz.box>
References: <20210603103507.304b7626@canb.auug.org.au>
 <8d8f947e-2ba4-f7b8-cb85-dcee940c96b4@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="+UJqigYrIox1WPeM"
Content-Disposition: inline
In-Reply-To: <8d8f947e-2ba4-f7b8-cb85-dcee940c96b4@gmail.com>
User-Agent: Mutt/2.0.6 (98f8cb83) (2021-03-06)
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Jun 2021 12:17:15.0335
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: c7f0f32f-d5a4-4bc1-26a3-08d92689861c
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a;Ip=[216.228.112.34];Helo=[mail.nvidia.com]
X-MS-Exchange-CrossTenant-AuthSource: BN8NAM11FT049.eop-nam11.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH2PR12MB3991
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

--+UJqigYrIox1WPeM
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Jun 03, 2021 at 05:01:48AM +0300, Dmitry Osipenko wrote:
> 03.06.2021 03:35, Stephen Rothwell =D0=BF=D0=B8=D1=88=D0=B5=D1=82:
> > Hi all,
> >=20
> > After merging the tegra tree, today's linux-next build (x86_64
> > allmodconfig) produced this warning:
> >=20
> > WARNING: unmet direct dependencies detected for TEGRA210_EMC_TABLE
> >   Depends on [n]: MEMORY [=3Dy] && TEGRA_MC [=3Dy] && ARCH_TEGRA_210_SO=
C [=3Dn]
> >   Selected by [m]:
> >   - TEGRA210_EMC [=3Dm] && MEMORY [=3Dy] && TEGRA_MC [=3Dy] && (ARCH_TE=
GRA_210_SOC [=3Dn] || COMPILE_TEST [=3Dy])
> >=20
> > Probably introduced by commit
> >=20
> >   08decdd5b448 ("memory: tegra: Enable compile testing for all drivers")
> >=20
>=20
> Thank you. This is a new warning to me, apparently this case wasn't previ=
ously tested by kernel build bot.
>=20
> Perhaps this should fix it:
>=20
> diff --git a/drivers/memory/tegra/Kconfig b/drivers/memory/tegra/Kconfig
> index 71bba2345bce..3f2fa7750118 100644
> --- a/drivers/memory/tegra/Kconfig
> +++ b/drivers/memory/tegra/Kconfig
> @@ -47,7 +47,6 @@ config TEGRA124_EMC
> =20
>  config TEGRA210_EMC_TABLE
>  	bool
> -	depends on ARCH_TEGRA_210_SOC

Why not just add a || COMPILE_TEST like we do for TEGRA210_EMC? Because
TEGRA210_EMC being pulled in under COMPILE_TEST (and then pulling in
TEGRA210_EMC_TABLE which is missing the alternative path) seems to be
the root cause for this.

Thierry

> =20
>  config TEGRA210_EMC
>  	tristate "NVIDIA Tegra210 External Memory Controller driver"
>=20
>=20
> Thierry, please squash this change into the original patch if it looks go=
od to you.

--+UJqigYrIox1WPeM
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmC4yKcACgkQ3SOs138+
s6FH9w/9FXuzZM1yIlevTJnP7FbiciRuxPp1HRk1QLoR1fPYyTRF0iuH/9IrOpPC
gejiE4vbHXrcezjLJ6sdYvBuqhZ0enRxnEZOYVBSEgJeNuDh/MEeJ2IE+KiHluqi
anl5YgM8nlNO+0VzjRBw4seWi0e64YyjblYAJCcX3tHQZTJ6X4iC0FkU5ApSlGvx
4cxT9Fh55r4HmP7L2QY728zECXIsIIiA1yaBggS22CUG20Q9qpH6pKsSe1Nj/Dkz
f65uSse3eDyDn/NFQy3a8kJPXSDTJlmHmj9hEmHMZF2N1K2dYxWuIjxk6C9a/Vli
jO3jeXCHm2mbI8hJhINNgR34oQplhYUkVFiYeh5ShRb10XxWrChbI8xVcxrm8kTE
H4PJuzKf7y5Ko25aAeW6rVNn7/fHMHgoGVKfJRvn046mibSU5ptgumz/l1ooeXsH
6lEu1yUFn1AajpJTE+aqpUDSCYA2Y5OeS5mc9GXBGsijnET9r6Z6CyDoTvUx1F5I
aoHCQrP1G+ohFWhNx4ukOKGuy69nySXRHQAkxXDQCZ5UXFxQG6v2mJICkx9cQw+q
MRiqmZOn/C4GUrJXV5cLQvuYCkrzDp/FTks88Pqz7ysEn9C++7vOOtJhSIlxGBQ9
V0agHGt+v0citHlYvg/g2JTfsFSlW7vX5WiqqzFFDWrNeoTL/fI=
=9Y/K
-----END PGP SIGNATURE-----

--+UJqigYrIox1WPeM--