From: Hans Verkuil <hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
To: Boris Brezillon
<boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Mauro Carvalho Chehab
<m.chehab-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Hans Verkuil
<hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>,
Laurent Pinchart
<laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>,
linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Sakari Ailus <sakari.ailus-X3B1VOXEql0@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Guennadi Liakhovetski
<g.liakhovetski-Mmb7MZpHnFY@public.gmane.org>
Subject: Re: [PATCH v3 00/10] [media] Make mediabus format subsystem neutral
Date: Fri, 07 Nov 2014 15:51:05 +0100 [thread overview]
Message-ID: <545CDC59.8020504@xs4all.nl> (raw)
In-Reply-To: <1415369269-5064-1-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
On 11/07/14 15:07, Boris Brezillon wrote:
> Hello,
>
> This patch series prepares the use of media bus formats outside of
> the V4L2 subsytem (my final goal is to use it in the Atmel HLCDC DRM
> driver where I have to configure my DPI/RGB bus according to the
> connected display).
>
> The series first defines MEDIA_BUS_FMT_ macros, and then replace all
> references to the v4l2_mbus_pixelcode enum and its values within the
> kernel.
Looks good!
For patches 1-9:
Acked-by: Hans Verkuil <hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>
Patch 10 needs a bit more work, see my reply to that patch. If I get a
v4 for that patch and nobody else has any further comments, then I'll
make a pull request for this on Monday.
Regards,
Hans
> Best Regards,
>
> Boris
>
> Changes since v2:
> - drop media_bus_format enum and replace its values with pre-processor
> macros
>
> Changes since v1:
> - drop patches deprecating v4l2_mbus_pixelcode for user-space users
> - put V4L2 legacy format definitions into media-bus-format.h
>
> Boris Brezillon (10):
> [media] Move mediabus format definition to a more standard place
> [media] v4l: Update subdev-formats doc with new MEDIA_BUS_FMT values
> [media] Make use of the new media_bus_format definitions
> [media] i2c: Make use of media_bus_format enum
> [media] pci: Make use of MEDIA_BUS_FMT definitions
> [media] platform: Make use of media_bus_format enum
> [media] usb: Make use of media_bus_format enum
> staging: media: Make use of MEDIA_BUS_FMT_ definitions
> gpu: ipu-v3: Make use of media_bus_format enum
> [media] v4l: Forbid usage of V4L2_MBUS_FMT definitions inside the
> kernel
>
> Documentation/DocBook/media/v4l/subdev-formats.xml | 308 ++++++++++-----------
> Documentation/video4linux/soc-camera.txt | 2 +-
> arch/arm/mach-davinci/board-dm355-evm.c | 2 +-
> arch/arm/mach-davinci/board-dm365-evm.c | 4 +-
> arch/arm/mach-davinci/dm355.c | 7 +-
> arch/arm/mach-davinci/dm365.c | 7 +-
> arch/arm/mach-shmobile/board-mackerel.c | 2 +-
> arch/sh/boards/mach-ap325rxa/setup.c | 2 +-
> drivers/gpu/ipu-v3/ipu-csi.c | 66 ++---
> drivers/media/i2c/adv7170.c | 16 +-
> drivers/media/i2c/adv7175.c | 16 +-
> drivers/media/i2c/adv7180.c | 6 +-
> drivers/media/i2c/adv7183.c | 6 +-
> drivers/media/i2c/adv7604.c | 72 ++---
> drivers/media/i2c/adv7842.c | 6 +-
> drivers/media/i2c/ak881x.c | 8 +-
> drivers/media/i2c/cx25840/cx25840-core.c | 2 +-
> drivers/media/i2c/m5mols/m5mols_core.c | 6 +-
> drivers/media/i2c/ml86v7667.c | 6 +-
> drivers/media/i2c/mt9m032.c | 6 +-
> drivers/media/i2c/mt9p031.c | 8 +-
> drivers/media/i2c/mt9t001.c | 8 +-
> drivers/media/i2c/mt9v011.c | 6 +-
> drivers/media/i2c/mt9v032.c | 12 +-
> drivers/media/i2c/noon010pc30.c | 12 +-
> drivers/media/i2c/ov7670.c | 16 +-
> drivers/media/i2c/ov9650.c | 10 +-
> drivers/media/i2c/s5c73m3/s5c73m3.h | 6 +-
> drivers/media/i2c/s5k4ecgx.c | 4 +-
> drivers/media/i2c/s5k5baf.c | 14 +-
> drivers/media/i2c/s5k6a3.c | 2 +-
> drivers/media/i2c/s5k6aa.c | 8 +-
> drivers/media/i2c/saa6752hs.c | 6 +-
> drivers/media/i2c/saa7115.c | 2 +-
> drivers/media/i2c/saa717x.c | 2 +-
> drivers/media/i2c/smiapp/smiapp-core.c | 32 +--
> drivers/media/i2c/soc_camera/imx074.c | 8 +-
> drivers/media/i2c/soc_camera/mt9m001.c | 14 +-
> drivers/media/i2c/soc_camera/mt9m111.c | 70 ++---
> drivers/media/i2c/soc_camera/mt9t031.c | 10 +-
> drivers/media/i2c/soc_camera/mt9t112.c | 22 +-
> drivers/media/i2c/soc_camera/mt9v022.c | 26 +-
> drivers/media/i2c/soc_camera/ov2640.c | 54 ++--
> drivers/media/i2c/soc_camera/ov5642.c | 8 +-
> drivers/media/i2c/soc_camera/ov6650.c | 58 ++--
> drivers/media/i2c/soc_camera/ov772x.c | 20 +-
> drivers/media/i2c/soc_camera/ov9640.c | 40 +--
> drivers/media/i2c/soc_camera/ov9740.c | 12 +-
> drivers/media/i2c/soc_camera/rj54n1cb0c.c | 54 ++--
> drivers/media/i2c/soc_camera/tw9910.c | 10 +-
> drivers/media/i2c/sr030pc30.c | 14 +-
> drivers/media/i2c/tvp514x.c | 12 +-
> drivers/media/i2c/tvp5150.c | 6 +-
> drivers/media/i2c/tvp7002.c | 10 +-
> drivers/media/i2c/vs6624.c | 18 +-
> drivers/media/pci/cx18/cx18-av-core.c | 2 +-
> drivers/media/pci/cx18/cx18-controls.c | 2 +-
> drivers/media/pci/cx18/cx18-ioctl.c | 2 +-
> drivers/media/pci/cx23885/cx23885-video.c | 2 +-
> drivers/media/pci/ivtv/ivtv-controls.c | 2 +-
> drivers/media/pci/ivtv/ivtv-ioctl.c | 2 +-
> drivers/media/pci/saa7134/saa7134-empress.c | 4 +-
> drivers/media/platform/blackfin/bfin_capture.c | 14 +-
> drivers/media/platform/davinci/vpbe.c | 2 +-
> drivers/media/platform/davinci/vpfe_capture.c | 4 +-
> drivers/media/platform/exynos-gsc/gsc-core.c | 8 +-
> drivers/media/platform/exynos-gsc/gsc-core.h | 2 +-
> drivers/media/platform/exynos4-is/fimc-capture.c | 2 +-
> drivers/media/platform/exynos4-is/fimc-core.c | 14 +-
> drivers/media/platform/exynos4-is/fimc-core.h | 4 +-
> drivers/media/platform/exynos4-is/fimc-isp.c | 16 +-
> drivers/media/platform/exynos4-is/fimc-lite-reg.c | 26 +-
> drivers/media/platform/exynos4-is/fimc-lite.c | 14 +-
> drivers/media/platform/exynos4-is/fimc-reg.c | 14 +-
> drivers/media/platform/exynos4-is/mipi-csis.c | 14 +-
> drivers/media/platform/marvell-ccic/mcam-core.c | 21 +-
> drivers/media/platform/marvell-ccic/mcam-core.h | 2 +-
> drivers/media/platform/omap3isp/ispccdc.c | 112 ++++----
> drivers/media/platform/omap3isp/ispccp2.c | 18 +-
> drivers/media/platform/omap3isp/ispcsi2.c | 42 +--
> drivers/media/platform/omap3isp/isppreview.c | 60 ++--
> drivers/media/platform/omap3isp/ispresizer.c | 19 +-
> drivers/media/platform/omap3isp/ispvideo.c | 95 ++++---
> drivers/media/platform/omap3isp/ispvideo.h | 10 +-
> drivers/media/platform/s3c-camif/camif-capture.c | 10 +-
> drivers/media/platform/s3c-camif/camif-regs.c | 8 +-
> drivers/media/platform/s5p-tv/hdmi_drv.c | 2 +-
> drivers/media/platform/s5p-tv/sdo_drv.c | 2 +-
> drivers/media/platform/sh_vou.c | 8 +-
> drivers/media/platform/soc_camera/atmel-isi.c | 22 +-
> drivers/media/platform/soc_camera/mx2_camera.c | 26 +-
> drivers/media/platform/soc_camera/mx3_camera.c | 6 +-
> drivers/media/platform/soc_camera/omap1_camera.c | 36 +--
> drivers/media/platform/soc_camera/pxa_camera.c | 16 +-
> drivers/media/platform/soc_camera/rcar_vin.c | 14 +-
> .../platform/soc_camera/sh_mobile_ceu_camera.c | 20 +-
> drivers/media/platform/soc_camera/sh_mobile_csi2.c | 38 +--
> drivers/media/platform/soc_camera/soc_camera.c | 2 +-
> .../platform/soc_camera/soc_camera_platform.c | 2 +-
> drivers/media/platform/soc_camera/soc_mediabus.c | 78 +++---
> drivers/media/platform/via-camera.c | 8 +-
> drivers/media/platform/vsp1/vsp1_bru.c | 14 +-
> drivers/media/platform/vsp1/vsp1_hsit.c | 12 +-
> drivers/media/platform/vsp1/vsp1_lif.c | 10 +-
> drivers/media/platform/vsp1/vsp1_lut.c | 14 +-
> drivers/media/platform/vsp1/vsp1_rwpf.c | 10 +-
> drivers/media/platform/vsp1/vsp1_sru.c | 12 +-
> drivers/media/platform/vsp1/vsp1_uds.c | 10 +-
> drivers/media/platform/vsp1/vsp1_video.c | 42 +--
> drivers/media/usb/cx231xx/cx231xx-417.c | 2 +-
> drivers/media/usb/cx231xx/cx231xx-video.c | 4 +-
> drivers/media/usb/em28xx/em28xx-camera.c | 2 +-
> drivers/media/usb/go7007/go7007-v4l2.c | 2 +-
> drivers/media/usb/pvrusb2/pvrusb2-hdw.c | 2 +-
> drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 18 +-
> .../staging/media/davinci_vpfe/dm365_ipipe_hw.c | 26 +-
> drivers/staging/media/davinci_vpfe/dm365_ipipeif.c | 100 +++----
> drivers/staging/media/davinci_vpfe/dm365_isif.c | 90 +++---
> drivers/staging/media/davinci_vpfe/dm365_resizer.c | 98 +++----
> .../staging/media/davinci_vpfe/vpfe_mc_capture.c | 18 +-
> drivers/staging/media/omap4iss/iss_csi2.c | 62 ++---
> drivers/staging/media/omap4iss/iss_ipipe.c | 16 +-
> drivers/staging/media/omap4iss/iss_ipipeif.c | 28 +-
> drivers/staging/media/omap4iss/iss_resizer.c | 26 +-
> drivers/staging/media/omap4iss/iss_video.c | 78 +++---
> drivers/staging/media/omap4iss/iss_video.h | 10 +-
> include/media/davinci/vpbe.h | 2 +-
> include/media/davinci/vpbe_venc.h | 5 +-
> include/media/exynos-fimc.h | 2 +-
> include/media/soc_camera.h | 2 +-
> include/media/soc_mediabus.h | 6 +-
> include/media/v4l2-mediabus.h | 2 +-
> include/media/v4l2-subdev.h | 2 +-
> include/uapi/linux/Kbuild | 1 +
> include/uapi/linux/media-bus-format.h | 125 +++++++++
> include/uapi/linux/v4l2-mediabus.h | 189 ++++++-------
> include/uapi/linux/v4l2-subdev.h | 6 +-
> 137 files changed, 1569 insertions(+), 1470 deletions(-)
> create mode 100644 include/uapi/linux/media-bus-format.h
>
next prev parent reply other threads:[~2014-11-07 14:51 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-07 14:07 [PATCH v3 00/10] [media] Make mediabus format subsystem neutral Boris Brezillon
2014-11-07 14:07 ` [PATCH v3 01/10] [media] Move mediabus format definition to a more standard place Boris Brezillon
2014-11-07 15:24 ` Sakari Ailus
[not found] ` <20141107152416.GC3136-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org>
2014-11-07 16:09 ` Boris Brezillon
2014-11-07 16:28 ` [PATCH v4 " Boris Brezillon
2014-11-07 14:07 ` [PATCH v3 02/10] [media] v4l: Update subdev-formats doc with new MEDIA_BUS_FMT values Boris Brezillon
2014-11-07 14:07 ` [PATCH v3 03/10] [media] Make use of the new media_bus_format definitions Boris Brezillon
2014-11-07 14:07 ` [PATCH v3 04/10] [media] i2c: Make use of media_bus_format enum Boris Brezillon
[not found] ` <1415369269-5064-5-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-11-08 9:23 ` Prabhakar Lad
2014-11-07 14:07 ` [PATCH v3 05/10] [media] pci: Make use of MEDIA_BUS_FMT definitions Boris Brezillon
2014-11-07 14:07 ` [PATCH v3 06/10] [media] platform: Make use of media_bus_format enum Boris Brezillon
2014-11-08 9:18 ` Prabhakar Lad
2014-11-11 4:42 ` Sekhar Nori
2014-11-11 7:49 ` Sekhar Nori
2014-11-07 14:07 ` [PATCH v3 08/10] staging: media: Make use of MEDIA_BUS_FMT_ definitions Boris Brezillon
2014-11-08 9:11 ` Prabhakar Lad
2014-11-07 14:07 ` [PATCH v3 09/10] gpu: ipu-v3: Make use of media_bus_format enum Boris Brezillon
2014-11-07 15:05 ` Philipp Zabel
2014-11-07 14:07 ` [PATCH v3 10/10] [media] v4l: Forbid usage of V4L2_MBUS_FMT definitions inside the kernel Boris Brezillon
2014-11-07 14:47 ` Hans Verkuil
[not found] ` <545CDB8D.4080406-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2014-11-07 15:31 ` Boris Brezillon
2014-11-07 16:27 ` [PATCH v4 " Boris Brezillon
2014-11-07 22:47 ` Sakari Ailus
[not found] ` <545D4BFD.6000206-X3B1VOXEql0@public.gmane.org>
2014-11-08 0:38 ` Boris Brezillon
[not found] ` <1415369269-5064-1-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2014-11-07 14:07 ` [PATCH v3 07/10] [media] usb: Make use of media_bus_format enum Boris Brezillon
2014-11-07 14:51 ` Hans Verkuil [this message]
2014-11-08 15:47 ` [PATCH v5 10/10] [media] v4l: Forbid usage of V4L2_MBUS_FMT definitions inside the kernel Boris Brezillon
2014-11-10 11:09 ` Hans Verkuil
[not found] ` <54609CDF.8030303-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2014-11-10 17:02 ` Boris Brezillon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=545CDC59.8020504@xs4all.nl \
--to=hverkuil-qwit8jrvyhvmr6xm/wnwpw@public.gmane.org \
--cc=boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b@public.gmane.org \
--cc=g.liakhovetski-Mmb7MZpHnFY@public.gmane.org \
--cc=hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org \
--cc=laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org \
--cc=linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=m.chehab-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=sakari.ailus-X3B1VOXEql0@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).