Hi Conor, Thanks for your feedback. On 2024-05-27 17:37:21 +0100, Conor Dooley wrote: > On Mon, May 27, 2024 at 03:18:49PM +0200, Niklas Söderlund wrote: > > Document support for the VIN module in the Renesas V4M (r8a779h0) SoC. > > Which is different from the other devices how? Compared to the other Gen4 SoC supported it only supports D-PHY. I will add this to next version, thanks for spotting it. > Should be with the driver: > https://lore.kernel.org/all/20240527132429.1683547-1-niklas.soderlund+renesas@xxxxxxxxxxxx/ As I mentioned in the other thread about the ISPCS bindings, I intentionally posted the bindings separately to allow parallel upstreaming of driver and DT users. Is it really a bad idea to do it this way? For other work I have done that involves more complex DT changes then adding a compatible, such as adding a new device or adding more properties to cover more features only available in a later version of a device. I always post the DT parts first as this can spur discussions about the design and only after they are agreed upon do I post the driver parts that make use of them. Seems like this would consume less review resources as the bindings can be agreed upon first, before anyone have to spend time reviewing a driver that might need to be redesigned as the bindings could be improved. > > Thanks, > Conor. > > > > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/media/renesas,vin.yaml | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/Documentation/devicetree/bindings/media/renesas,vin.yaml b/Documentation/devicetree/bindings/media/renesas,vin.yaml > > index 5539d0f8e74d..168cb02f8abe 100644 > > --- a/Documentation/devicetree/bindings/media/renesas,vin.yaml > > +++ b/Documentation/devicetree/bindings/media/renesas,vin.yaml > > @@ -54,6 +54,7 @@ properties: > > - renesas,vin-r8a77995 # R-Car D3 > > - renesas,vin-r8a779a0 # R-Car V3U > > - renesas,vin-r8a779g0 # R-Car V4H > > + - renesas,vin-r8a779h0 # R-Car V4M > > > > reg: > > maxItems: 1 > > -- > > 2.45.1 > > -- Kind Regards, Niklas Söderlund
