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:   Fri, 25 Feb 2022 13:51:02 +0100
From:   Thierry Reding <thierry.reding@gmail.com>
To:     Rob Herring <robh@kernel.org>
Cc:     Mohan Kumar <mkumard@nvidia.com>, broonie@kernel.org,
        lgirdwood@gmail.com, tiwai@suse.com, jonathanh@nvidia.com,
        spujar@nvidia.com, alsa-devel@alsa-project.org,
        devicetree@vger.kernel.org, linux-tegra@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 4/6] dt-bindings: Add HDA support for Tegra234
Message-ID: <YhjQtrueEYNGQrSA@orome>
References: <20220216092240.26464-1-mkumard@nvidia.com>
 <20220216092240.26464-5-mkumard@nvidia.com>
 <YhfdHd2LPc+ifFTS@robh.at.kernel.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="VMSq3re6wkdG1hOh"
Content-Disposition: inline
In-Reply-To: <YhfdHd2LPc+ifFTS@robh.at.kernel.org>
User-Agent: Mutt/2.2.1 (c8109e14) (2022-02-19)
Precedence: bulk


--VMSq3re6wkdG1hOh
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Feb 24, 2022 at 01:31:41PM -0600, Rob Herring wrote:
> On Wed, Feb 16, 2022 at 02:52:38PM +0530, Mohan Kumar wrote:
> > Add hda clocks, memory ,power and reset binding entries
> > for Tegra234.
> >=20
> > Signed-off-by: Mohan Kumar <mkumard@nvidia.com>
> > ---
> >  include/dt-bindings/clock/tegra234-clock.h     | 4 ++++
> >  include/dt-bindings/memory/tegra234-mc.h       | 6 ++++++
> >  include/dt-bindings/power/tegra234-powergate.h | 9 +++++++++
> >  include/dt-bindings/reset/tegra234-reset.h     | 2 ++
> >  4 files changed, 21 insertions(+)
> >  create mode 100644 include/dt-bindings/power/tegra234-powergate.h
> >=20
> > diff --git a/include/dt-bindings/clock/tegra234-clock.h b/include/dt-bi=
ndings/clock/tegra234-clock.h
> > index 2529e7ec0bf4..2ab651916127 100644
> > --- a/include/dt-bindings/clock/tegra234-clock.h
> > +++ b/include/dt-bindings/clock/tegra234-clock.h
> > @@ -64,4 +64,8 @@
> >  #define TEGRA234_CLK_PLLC4			237U
> >  /** @brief 32K input clock provided by PMIC */
> >  #define TEGRA234_CLK_CLK_32K			289U
> > +/** @brief CLK_RST_CONTROLLER_AZA2XBITCLK_OUT_SWITCH_DIVIDER switch di=
vider output (aza_2xbitclk) */
> > +#define TEGRA234_CLK_AZA_2XBIT			457U
> > +/** @brief aza_2xbitclk / 2 (aza_bitclk) */
> > +#define TEGRA234_CLK_AZA_BIT			458U
> >  #endif
> > diff --git a/include/dt-bindings/memory/tegra234-mc.h b/include/dt-bind=
ings/memory/tegra234-mc.h
> > index 2662f70c15c6..c42fe61fba5e 100644
> > --- a/include/dt-bindings/memory/tegra234-mc.h
> > +++ b/include/dt-bindings/memory/tegra234-mc.h
> > @@ -7,6 +7,8 @@
> >  #define TEGRA234_SID_INVALID		0x00
> >  #define TEGRA234_SID_PASSTHROUGH	0x7f
> > =20
> > +/* NISO0 stream IDs */
> > +#define TEGRA234_SID_HDA		0x03
> > =20
> >  /* NISO1 stream IDs */
> >  #define TEGRA234_SID_SDMMC4	0x02
> > @@ -16,6 +18,10 @@
> >   * memory client IDs
> >   */
> > =20
> > +/* High-definition audio (HDA) read clients */
> > +#define TEGRA234_MEMORY_CLIENT_HDAR 0x15
> > +/* High-definition audio (HDA) write clients */
> > +#define TEGRA234_MEMORY_CLIENT_HDAW 0x35
> >  /* sdmmcd memory read client */
> >  #define TEGRA234_MEMORY_CLIENT_SDMMCRAB 0x63
> >  /* sdmmcd memory write client */
> > diff --git a/include/dt-bindings/power/tegra234-powergate.h b/include/d=
t-bindings/power/tegra234-powergate.h
> > new file mode 100644
> > index 000000000000..3c5575a51296
> > --- /dev/null
> > +++ b/include/dt-bindings/power/tegra234-powergate.h
> > @@ -0,0 +1,9 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
>=20
> Dual license please.

We've got a bunch of similar files in this directory that are not
dual licensed. These files originally are part of the BPMP firmware
tree, so I'll check with the owners if relicensing would be possible.
If so, I think we should be able to relicense all of those header files.

Thierry

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

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

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmIY0LMACgkQ3SOs138+
s6Ez8xAAn+49vmQ+ErSD9Sg2/t6O8Sil+0KTyZF+OSNKNEOKafvj38Sv+zjGH7ZN
OH0666hghTwlDgDBX9L/ZI4fItEVAXr7l/lMjoIiXtZ0CYb0xIXBXqIRvP8R7xOp
TjPWFBe1oTBqALv+GSRE0f4ZPT1OsWtCddlONIwMtzO4SmHjtTKTx6PvCvOL62AC
s+xHeUtfCeh8orT1Qcmr9XK0XGDikxqVi1SFfl7rgUBX2TS+7zmjHVDF5vmSkpw7
lTQ3rZsRTWlyScBolyIhNAP9y6N4/Ix3ASVrEQSPjkRx6S8UgnlCsjN9qjXS0I4W
C7rQgFPPHQVpIEFxdfZBWaP1ReyDUN/5kseJzPVqB+XQ+/cUXsFYsuP8Nnx8+F4m
/GlQwsmtLzanpKOgwyMEN9sgJcWLbXa+9ZtT2lOEr9dDMFbnMq27bZZ319DvTQ6o
Fh9aahMgLu45+gRhGMJsrGRaFqoVYzFPUh6+7bJmSshVIvvp1esNNy9ofZXdBKIW
oPzfNjAd0l4KQ2eDmZdjfZyYjgcnU6gTeSoXIPvjwf3i6Vp3kjcB5eAR6qwcKxDc
8r+dsJaPfBoS0qMWpe2zMPgiHJ5N034/Au6JqDjuv94vY1sVDCRvS1UejJlItHe9
9iTv9lVa4Fa+6r6vj0MnPhLSCk8PpmXqiuGWBMyAc8MyWMbtHyA=
=xQrv
-----END PGP SIGNATURE-----

--VMSq3re6wkdG1hOh--