Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Date:   Wed, 1 Nov 2023 17:56:19 +0100
From:   Paul Kocialkowski <paul.kocialkowski@bootlin.com>
To:     Conor Dooley <conor@kernel.org>
Cc:     Rob Herring <robh@kernel.org>,
        Mehdi Djait <mehdi.djait@bootlin.com>, mchehab@kernel.org,
        heiko@sntech.de, hverkuil-cisco@xs4all.nl,
        krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
        linux-media@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, thomas.petazzoni@bootlin.com,
        alexandre.belloni@bootlin.com, maxime.chevallier@bootlin.com,
        michael.riesch@wolfvision.net
Subject: Re: [PATCH v9 1/3] media: dt-bindings: media: add bindings for
 Rockchip CIF
Message-ID: <ZUKDM-_Re4qFHH9T@aptenodytes>
References: <cover.1698666612.git.mehdi.djait@bootlin.com>
 <5f83d60031320c4c7b0f0727604903f50ee49058.1698666612.git.mehdi.djait@bootlin.com>
 <20231030193236.GA1995060-robh@kernel.org>
 <ZUDOCywnEqJSuQMM@aptenodytes>
 <20231101-ergonomic-grafted-2edc0290c569@spud>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="tGS4dK5LvN/190zE"
Content-Disposition: inline
In-Reply-To: <20231101-ergonomic-grafted-2edc0290c569@spud>
Precedence: bulk


--tGS4dK5LvN/190zE
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Conor,

On Wed 01 Nov 23, 15:44, Conor Dooley wrote:
> On Tue, Oct 31, 2023 at 10:51:07AM +0100, Paul Kocialkowski wrote:
> > Hi Rob,
> >=20
> > On Mon 30 Oct 23, 14:32, Rob Herring wrote:
> > > On Mon, Oct 30, 2023 at 01:25:12PM +0100, Mehdi Djait wrote:
> > > > Add a documentation for the Rockchip Camera Interface
> > > > binding.
> > > >=20
> > > > Signed-off-by: Mehdi Djait <mehdi.djait@bootlin.com>
> > > > ---
> > > > v8=3D>v9:
> > > > dropped the "Reviewed-by: Rob Herring <robh@kernel.org>"
> > > > because of the following changes:
> > > > - changed the compatible to rk3066-cif: rk3066 is the earliest Rock=
chip SoC
> > > >   that uses cif and it is the first model starting the RK30 lineup.
> > >=20
> > > Is px30 compatible with rk3066? It's not clear because you didn't add=
=20
> > > rk3066 support. If not compatible, then add rk3066 when you have a us=
er.=20
> > > If it is compatible, then you should have a fallback for px30.
> >=20
> > Just to clarify here: we haven't checked that rk3066 uses the exact same
> > programming model as px30 (so there should be no fallback compatible), =
but it
> > is definitely the same unit (in a different version).
>=20
> If you aren't sure about the programming model, are you sure about the
> number of clocks, resets etc that it has? Same IP in a different SoC
> could be integrated differently, right?

Yes absolutely there might be differences in these areas. But we can always=
 add
conditionals in the binding to take care of different cases later.

My main concern is to avoid giving px30 too much importance just because it
happens to be the first platform supported by the Linux driver.

Cheers,

Paul

> > Since the yaml binding document will apply to all generations of the un=
it,
> > the name of the file should be the first generation (rk3066) instead of=
 the
> > px30 which is just one of the many iterations of the unit.
> >=20
> > It would be both confusing and irrelevant to pick px30 just because it =
happens
> > to be the first generation supported in the Linux driver (and described=
 in
> > the binding).


--=20
Paul Kocialkowski, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com

--tGS4dK5LvN/190zE
Content-Type: application/pgp-signature; name="signature.asc"

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

iQEzBAEBCAAdFiEEJZpWjZeIetVBefti3cLmz3+fv9EFAmVCgzMACgkQ3cLmz3+f
v9H3FAf/WjMfxacq7f9Su3oJCd059UA3G2OfjKLNkOogkv52/MsbxkKNVA3zW4qF
na2nBCQiCA46fB2WBNJ0rmCz2acRUBYDxWho//H6GfLrEWGUrJ/fnrzCpWewrSIb
fa+b4v9ayogxqAfhBtK50vCRkdui5gfslh5m9z8snBCBZvvXxRXSnP1DmWz+DAC/
YBG/m/py2WEbfz9yusO3KAkEsR0UuBwD2299yVe8SF750m8q5bzTyE0st5CJUuAw
0yNYo2UGIHayTHZHPOp0Ww35IFIalvqbhDHte6jKZD3VFVQvg3GtF+sbM5rWmO/S
a3RT0uPustfvM8mDIpNQemjMz7DSaA==
=qVos
-----END PGP SIGNATURE-----

--tGS4dK5LvN/190zE--