Hi Krzysztof > >-----Original Message----- >From: Krzysztof Kozlowski <[email protected]> >Sent: Friday, October 30, 2026 5:56 PM >To: Hermes Wu (吳佳宏) <[email protected]> >Cc: Andrzej Hajda <[email protected]>; Neil Armstrong ><[email protected]>; Robert Foss <[email protected]>; Laurent Pinchart ><[email protected]>; Jonas Karlman <[email protected]>; Jernej >Skrabec <[email protected]>; David Airlie <[email protected]>; Simona >Vetter <[email protected]>; Maarten Lankhorst ><[email protected]>; Maxime Ripard <[email protected]>; >Thomas Zimmermann <[email protected]>; Rob Herring <[email protected]>; >Krzysztof Kozlowski <[email protected]>; Conor Dooley <[email protected]>; >Luca Ceresoli <[email protected]>; Pet Weng (翁玉芬) ><[email protected]>; Kenneth Hung (洪家倫) <[email protected]>; >[email protected]; [email protected]; >[email protected]; Dmitry Baryshkov ><[email protected]> >Subject: Re: [PATCH v5 1/2] dt-bindings: display: bridge: Add ITE IT6162 MIPI >DSI to HDMI bridge > >On Thu, Oct 29, 2026 at 06:12:30PM +0800, Hermes Wu wrote: >> Add device tree binding documentation for the ITE IT6162 MIPI DSI to >> HDMI 2.0 bridge chip. The IT6162 is an I2C-controlled bridge that >> supports the following configurations: >> >> - Single MIPI DSI input: up to 4K @ 30Hz >> - Dual MIPI DSI input (combined): up to 4K @ 60Hz >> >> The chip also supports up to 8-channel audio output via 4 I2S data >> channels. >> >> Assisted-by: Claude Opus 5.5 > >Drop the tag now, rules changed. > Thanks, I will drop it in v6.
>Please run scripts/checkpatch.pl on the patches and fix reported warnings. >After that, run also 'scripts/checkpatch.pl --strict' on the patches and >(probably) fix more warnings. Some warnings can be ignored, especially from >--strict run, but the code here looks like it needs a fix. Feel free to get in >touch if the warning is not clear. > I ran checkpatch.pl, with and without --strict, on both patches. The only remaining warning is "does MAINTAINERS need updating?" on this patch. The MAINTAINERS entry that covers ite,it6162.yaml is added in patch 2/2, so I will keep it there unless you prefer it in this patch. >Reviewed-by: Krzysztof Kozlowski <[email protected]> Thank you for the review. >Best regards, >Krzysztof > Regards, Hermes
