On Fri, Feb 6, 2026 at 2:56 AM Krzysztof Kozlowski <[email protected]> wrote: > > 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
Thanks for the guide! Yes, v2 was reviewed in private. I'll inform Hongyang to remove the tag when he sends the next series. -Roger
