On Sun, 2 Feb 2020 at 12:50, Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxxx> wrote: > > The objective of this series is to add support for GBR and BGR media bus formats > for the source pad of debayer subdevices of the vimc driver. > Can you elaborate why is this needed, e.g. what use-case is this enabling, or what is this fixing? Thanks, Ezequiel > Since the GBR media bus code doesn't have a corresponding pixelformat, it needed > to use the pixelformat of another bus code. > > The first patch makes it possible to have multiple media bus codes mapping to > the same pixelformat. > > The second patch adds the GBR media bus code, using the RGB pixelformat. > > The third patch adds support for GBR and BGR media bus formats on the source > pad of the debayer subdevice. > > This patch series passed all tests of v4l2-compliance: > $ compliance_git -m /dev/media0 > v4l2-compliance SHA: c4a62f26c5c3ecd856ca10cf2f0d35d100283d7f, 64 bits, 64-bit time_t > > Grand Total for vimc device /dev/media0: 461, Succeeded: 461, Failed: 0, Warnings: 0 > > Nícolas F. R. A. Prado (3): > media: vimc: Support multiple buscodes for each pixelformat > media: vimc: Add GBR media bus code > media: vimc: deb: Add support for GBR and BGR bus formats on source > pad > > drivers/media/platform/vimc/vimc-common.c | 68 +++++++++++++--------- > drivers/media/platform/vimc/vimc-common.h | 9 ++- > drivers/media/platform/vimc/vimc-debayer.c | 53 +++++++++++++---- > drivers/media/platform/vimc/vimc-scaler.c | 10 +++- > drivers/media/platform/vimc/vimc-sensor.c | 6 +- > 5 files changed, 102 insertions(+), 44 deletions(-) > > -- > 2.25.0 > >
