Re: Test [PATCH] media: ipu-bridge: the Surface Pro 11 rear sensor is mounted upside down

[Sergey Lebedev <lsa.uz@pm.me>]

Thank you. I will carry your tag to the next version.

I want to be sure I understand your result. Please answer yes or no:

  1. Is the image now correct on screen, not upside down?
  2. Did you check this in Gnome Snapshot?
  3. Did you check this in qcam?
  4. Did you check this in Firefox?
  5. Was it correct in the live preview, the moving picture?
  6. Was it correct in a photo file you saved?

Questions 5 and 6 matter most. A program can turn the picture on the screen
and still save the file the old way. I only tested saved files here.

If the answer to 1 is no, that is fine. It does not make the patch wrong.
The patch makes the kernel report the correct rotation. That is all it
claims to do.

I ask because I tested it here. I captured the same scene twice: once with
the patch, once without it. Both images are the same way up. The patch did
not turn the picture. It only changed the number libcamera reports, from 0
to 180.

So if your image is now correct, your program is turning it, using that
number. I could not see that here, and I would like to repeat what you did.
I have the same tablet. Please send:

  uname -r
  pacman -Q libcamera          (or your distribution's command)

and the version of whichever program showed the correct image.

If I can match your setup, I can find which program does the rotation. Then
it is a question for that program, not for the kernel.

Sergey