Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33;
Message-ID: <f76f9d460b6d7e37aadc4100dbbd57321e5330c2.camel@collabora.com>
Subject: Re: [PATCH v2 2/2] media: chips-media: wave5: Remove K3 References
From:   Nicolas Dufresne <nicolas.dufresne@collabora.com>
To:     Brandon Brnich <b-brnich@ti.com>, linux-media@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Nas Chung <nas.chung@chipsnmedia.com>,
        Jackson Lee <jackson.lee@chipsnmedia.com>,
        Hans Verkuil <hverkuil-cisco@xs4all.nl>,
        Sebastian Fricke <sebastian.fricke@collabora.com>,
        Robert Beckett <bob.beckett@collabora.com>,
        Geert Uytterhoeven <geert@linux-m68k.org>
Cc:     Nishanth Menon <nm@ti.com>, Darren Etheridge <detheridge@ti.com>
Date:   Wed, 06 Dec 2023 14:55:07 -0500
In-Reply-To: <20231206185254.1748473-3-b-brnich@ti.com>
References: <20231206185254.1748473-1-b-brnich@ti.com>
         <20231206185254.1748473-3-b-brnich@ti.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.48.4 (3.48.4-1.fc38) 
MIME-Version: 1.0
Precedence: bulk

As this is a DT call made by TI, there is not much to review.

Le mercredi 06 d=C3=A9cembre 2023 =C3=A0 12:52 -0600, Brandon Brnich a =C3=
=A9crit=C2=A0:
> Change compatible string to match dt bindings for TI devices. K3 family
> prefix should not be included as it deviates from naming convention.
>=20
> Fixes: 9707a6254a8a ("media: chips-media: wave5: Add the v4l2 layer")
> Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
> Closes: https://lore.kernel.org/all/CAMuHMdUYOq=3Dq1j=3Dd+Eac28hthOUAaNUk=
uvxmRu-mUN1pLKq69g@mail.gmail.com/
> Signed-off-by: Brandon Brnich <b-brnich@ti.com>

This should just be picked.

Acked-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>


> ---
>  V1 -> V2: Fix style issues in commit message
>=20
>  drivers/media/platform/chips-media/wave5/wave5-vpu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>=20
> diff --git a/drivers/media/platform/chips-media/wave5/wave5-vpu.c b/drive=
rs/media/platform/chips-media/wave5/wave5-vpu.c
> index bfe4caa79cc9..0d90b5820bef 100644
> --- a/drivers/media/platform/chips-media/wave5/wave5-vpu.c
> +++ b/drivers/media/platform/chips-media/wave5/wave5-vpu.c
> @@ -272,7 +272,7 @@ static const struct wave5_match_data ti_wave521c_data=
 =3D {
>  };
> =20
>  static const struct of_device_id wave5_dt_ids[] =3D {
> -	{ .compatible =3D "ti,k3-j721s2-wave521c", .data =3D &ti_wave521c_data =
},
> +	{ .compatible =3D "ti,j721s2-wave521c", .data =3D &ti_wave521c_data },
>  	{ /* sentinel */ }
>  };
>  MODULE_DEVICE_TABLE(of, wave5_dt_ids);