On Thu, Feb 5, 2026 at 11:08 PM Krzysztof Kozlowski <[email protected]> wrote: > > On 05/02/2026 21:31, Roger Shimizu wrote: > > On Thu, Feb 5, 2026 at 5:07 AM Krzysztof Kozlowski <[email protected]> wrote: > >> > >> On Wed, Jan 28, 2026 at 07:15:45PM +0800, Hongyang Zhao wrote: > >>> The LT9611 has two DSI input ports (Port A and Port B). Update the > >>> binding to clearly document the port mapping and allow using Port B > >>> alone when DSI is physically connected to Port B only. > >>> > >>> Changes: > >>> - Clarify port@0 corresponds to DSI Port A input > >>> - Clarify port@1 corresponds to DSI Port B input > >>> - Change port requirement from mandatory port@0 to anyOf port@0/port@1, > >>> allowing either port to be used independently > >>> > >>> Signed-off-by: Hongyang Zhao <[email protected]> > >>> Reviewed-by: Roger Shimizu <[email protected]> > >> > >> Where did this review happen? V1 had this tag, but the patch was > >> completely different, which means you were supposed to drop the tag. > >> Please perform review in public. > > > > FYI. v2 was updated per review feedback, which is public: > > https://lore.kernel.org/all/[email protected]/ > > Link above is not from Roger, so again - where did the review leading to > above tag happen?
Per feedback of v1, v2 was quite different than v1. For v2, it's close to initial review, because it looks like a new patch. Of course, if you don't like this way, we can drop this next time. -Roger
