On 06/02/2026 10:49, Roger Shimizu wrote: > 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.
Where was the review of v2 given? The patch is entirely different. I already said it. I also said what is expected in such case. It is also documented: https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577 Best regards, Krzysztof
