| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <a3acd5b9-6270-1b2e-6fdc-732c42855fd8@gmail.com>
Date: Wed, 7 Jun 2017 12:05:36 -0700
From: Steve Longerbeam <slongerbeam@...il.com>
To: Hans Verkuil <hverkuil@...all.nl>, robh+dt@...nel.org,
mark.rutland@....com, shawnguo@...nel.org, kernel@...gutronix.de,
fabio.estevam@....com, linux@...linux.org.uk, mchehab@...nel.org,
nick@...anahar.org, markus.heiser@...marIT.de,
p.zabel@...gutronix.de, laurent.pinchart+renesas@...asonboard.com,
bparrot@...com, geert@...ux-m68k.org, arnd@...db.de,
sudipm.mukherjee@...il.com, minghsiu.tsai@...iatek.com,
tiffany.lin@...iatek.com, jean-christophe.trotin@...com,
horms+renesas@...ge.net.au, niklas.soderlund+renesas@...natech.se,
robert.jarzmik@...e.fr, songjun.wu@...rochip.com,
andrew-ct.chen@...iatek.com, gregkh@...uxfoundation.org,
shuah@...nel.org, sakari.ailus@...ux.intel.com, pavel@....cz
Cc: devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-media@...r.kernel.org,
devel@...verdev.osuosl.org,
Steve Longerbeam <steve_longerbeam@...tor.com>
Subject: Re: [PATCH v8 00/34] i.MX Media Driver
On 06/07/2017 12:02 PM, Hans Verkuil wrote:
> We're still waiting for an Ack for patch 02/34, right?
>
Hi Hans, Yes still waiting for an ack for the imx-media bindings.
> Other than that everything is ready AFAICT.
Agreed.
Steve
>
> Regards,
>
> Hans
>
> On 07/06/17 20:33, Steve Longerbeam wrote:
>> In version 8:
>>
>> - Switched to v4l2_fwnode APIs.
>>
>> - Always pass a valid CSI id to ipu_set_ic_src_mux() in imx-ic-prp, even
>> if the IC is receiving from the VDIC. The reason is due to a bug in the
>> i.MX6 reference manual: from experiment it is determined that the CSI id
>> select bit in IPU_CONF register selects which CSI is routed to either
>> the VDIC or the IC, and is independent of whether the IC is set to
>> receive from a CSI or the VDIC. Sugested by Marek Vasut <marex@...x.de>.
>>
>> - ov5640: propagate error codes from all i2c register accesses.
>> Sugested by Sakari Ailus <sakari.ailus@....fi>.
>>
>> - ov5640: drop the entity stream count check in ov5640_s_stream().
>> Sugested by Sakari Ailus <sakari.ailus@....fi>.
>>
>> - ov5640: Fix manual exposure control. The manual exposure setting
>> (in line periods) cannot exceed the max exposure value in registers
>> {0x380E, 0x380F} + {0x350C,0x350D}, however the max eposure value was
>> not being calcualted correctly.
>>
>> - ov5640: the video mode register tables require auto gain/exposure be
>> disabled before programming the register set. However auto gain/exp was
>> being disabled by direct register write. This caused the auto gain/exp
>> control values to be inconsistent with the actual hardware setting. Fixed
>> by going through v4l2-ctrl when disabling auto gain/exp.
>>
>> - ov5640: converted virtual channel macro to a module parameter, default
>> to channel 0.
>>
>> - ov5640: override the v4l2-ctl lock to use the ov5640 subdev driver's
>> lock. Sugested by Sakari Ailus <sakari.ailus@....fi>.
>>
>> - ov5640: switch to unit-less V4L2_CID_EXPOSURE. Using
>> V4L2_CID_EXPOSURE_ABSOLUTE will require converting from 100-usec units
>> to line periods and vice-versa. Sugested by Sakari Ailus and
>> Pavel Machek <pavel@....cz>.
>>
>> - ov5640: drop dangling regulator_bulk_disable() from probe/remove.
>> Sugested by Sakari Ailus <sakari.ailus@....fi>.
>>
>> - FIM: move input capture channel selection out of the device-tree and
>> make this a V4L2 control. In order to support attaching a FIM to prpencvf,
>> the FIM cannot have any device-tree configuration, because prpencvf has
>> no device node. The FIM now is completely configurable via its V4L2
>> controls.
>>
>> - FIM: drop imx_media_fim_set_power(), and move the input capture channel
>> request to imx_media_fim_set_stream(). This allows to drop csi_s_power()
>> as well, since the latter only called imx_media_fim_set_power().
>>
>> - FIM: add a spinlock to protect the frame_interval_monitor() from the
>> setting of new control values. The frame_interval_monitor() is called
>> from interrupt context so a spinlock must be used.
>>
>> - Updated to version 8 video-mux patchset from Philipp Zabel
>> <p.zabel@...gutronix.de>.
>>
>>
>> Marek Vasut (1):
>> media: imx: Drop warning upon multiple S_STREAM disable calls
>>
>> Philipp Zabel (8):
>> dt-bindings: Add bindings for video-multiplexer device
>> ARM: dts: imx6qdl: add multiplexer controls
>> ARM: dts: imx6qdl: Add video multiplexers, mipi_csi, and their
>> connections
>> add mux and video interface bridge entity functions
>> platform: add video-multiplexer subdevice driver
>> media: imx: csi: increase burst size for YUV formats
>> media: imx: csi: add frame skipping support
>> media: imx: csi: add sink selection rectangles
>>
>> Russell King (3):
>> media: imx: csi: add support for bayer formats
>> media: imx: csi: add frame size/interval enumeration
>> media: imx: capture: add frame sizes/interval enumeration
>>
>> Steve Longerbeam (22):
>> [media] dt-bindings: Add bindings for i.MX media driver
>> [media] dt/bindings: Add bindings for OV5640
>> ARM: dts: imx6qdl: Add compatible, clocks, irqs to MIPI CSI-2 node
>> ARM: dts: imx6qdl: add capture-subsystem device
>> ARM: dts: imx6qdl-sabrelite: remove erratum ERR006687 workaround
>> ARM: dts: imx6-sabrelite: add OV5642 and OV5640 camera sensors
>> ARM: dts: imx6-sabresd: add OV5642 and OV5640 camera sensors
>> ARM: dts: imx6-sabreauto: create i2cmux for i2c3
>> ARM: dts: imx6-sabreauto: add reset-gpios property for max7310_b
>> ARM: dts: imx6-sabreauto: add pinctrl for gpt input capture
>> ARM: dts: imx6-sabreauto: add the ADV7180 video decoder
>> [media] add Omnivision OV5640 sensor driver
>> media: Add userspace header file for i.MX
>> media: Add i.MX media core driver
>> media: imx: Add a TODO file
>> media: imx: Add Capture Device Interface
>> media: imx: Add CSI subdev driver
>> media: imx: Add VDIC subdev driver
>> media: imx: Add IC subdev drivers
>> media: imx: Add MIPI CSI-2 Receiver subdev driver
>> media: imx: set and propagate default field, colorimetry
>> ARM: imx_v6_v7_defconfig: Enable staging video4linux drivers
>>
>> .../devicetree/bindings/media/i2c/ov5640.txt | 45 +
>> Documentation/devicetree/bindings/media/imx.txt | 47 +
>> .../devicetree/bindings/media/video-mux.txt | 60 +
>> Documentation/media/uapi/mediactl/media-types.rst | 21 +
>> Documentation/media/v4l-drivers/imx.rst | 614 +++++
>> arch/arm/boot/dts/imx6dl-sabrelite.dts | 5 +
>> arch/arm/boot/dts/imx6dl-sabresd.dts | 5 +
>> arch/arm/boot/dts/imx6dl.dtsi | 189 ++
>> arch/arm/boot/dts/imx6q-sabrelite.dts | 5 +
>> arch/arm/boot/dts/imx6q-sabresd.dts | 5 +
>> arch/arm/boot/dts/imx6q.dtsi | 125 ++
>> arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 136 +-
>> arch/arm/boot/dts/imx6qdl-sabrelite.dtsi | 152 +-
>> arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 114 +-
>> arch/arm/boot/dts/imx6qdl.dtsi | 20 +-
>> arch/arm/configs/imx_v6_v7_defconfig | 11 +
>> drivers/media/i2c/Kconfig | 10 +
>> drivers/media/i2c/Makefile | 1 +
>> drivers/media/i2c/ov5640.c | 2344 ++++++++++++++++++++
>> drivers/media/platform/Kconfig | 6 +
>> drivers/media/platform/Makefile | 2 +
>> drivers/media/platform/video-mux.c | 334 +++
>> drivers/staging/media/Kconfig | 2 +
>> drivers/staging/media/Makefile | 1 +
>> drivers/staging/media/imx/Kconfig | 21 +
>> drivers/staging/media/imx/Makefile | 12 +
>> drivers/staging/media/imx/TODO | 23 +
>> drivers/staging/media/imx/imx-ic-common.c | 113 +
>> drivers/staging/media/imx/imx-ic-prp.c | 518 +++++
>> drivers/staging/media/imx/imx-ic-prpencvf.c | 1309 +++++++++++
>> drivers/staging/media/imx/imx-ic.h | 38 +
>> drivers/staging/media/imx/imx-media-capture.c | 775 +++++++
>> drivers/staging/media/imx/imx-media-csi.c | 1816 +++++++++++++++
>> drivers/staging/media/imx/imx-media-dev.c | 666 ++++++
>> drivers/staging/media/imx/imx-media-fim.c | 494 +++++
>> drivers/staging/media/imx/imx-media-internal-sd.c | 349 +++
>> drivers/staging/media/imx/imx-media-of.c | 270 +++
>> drivers/staging/media/imx/imx-media-utils.c | 896 ++++++++
>> drivers/staging/media/imx/imx-media-vdic.c | 1009 +++++++++
>> drivers/staging/media/imx/imx-media.h | 325 +++
>> drivers/staging/media/imx/imx6-mipi-csi2.c | 698 ++++++
>> include/linux/imx-media.h | 29 +
>> include/media/imx.h | 15 +
>> include/uapi/linux/media.h | 6 +
>> include/uapi/linux/v4l2-controls.h | 4 +
>> 45 files changed, 13613 insertions(+), 27 deletions(-)
>> create mode 100644 Documentation/devicetree/bindings/media/i2c/ov5640.txt
>> create mode 100644 Documentation/devicetree/bindings/media/imx.txt
>> create mode 100644 Documentation/devicetree/bindings/media/video-mux.txt
>> create mode 100644 Documentation/media/v4l-drivers/imx.rst
>> create mode 100644 drivers/media/i2c/ov5640.c
>> create mode 100644 drivers/media/platform/video-mux.c
>> create mode 100644 drivers/staging/media/imx/Kconfig
>> create mode 100644 drivers/staging/media/imx/Makefile
>> create mode 100644 drivers/staging/media/imx/TODO
>> create mode 100644 drivers/staging/media/imx/imx-ic-common.c
>> create mode 100644 drivers/staging/media/imx/imx-ic-prp.c
>> create mode 100644 drivers/staging/media/imx/imx-ic-prpencvf.c
>> create mode 100644 drivers/staging/media/imx/imx-ic.h
>> create mode 100644 drivers/staging/media/imx/imx-media-capture.c
>> create mode 100644 drivers/staging/media/imx/imx-media-csi.c
>> create mode 100644 drivers/staging/media/imx/imx-media-dev.c
>> create mode 100644 drivers/staging/media/imx/imx-media-fim.c
>> create mode 100644 drivers/staging/media/imx/imx-media-internal-sd.c
>> create mode 100644 drivers/staging/media/imx/imx-media-of.c
>> create mode 100644 drivers/staging/media/imx/imx-media-utils.c
>> create mode 100644 drivers/staging/media/imx/imx-media-vdic.c
>> create mode 100644 drivers/staging/media/imx/imx-media.h
>> create mode 100644 drivers/staging/media/imx/imx6-mipi-csi2.c
>> create mode 100644 include/linux/imx-media.h
>> create mode 100644 include/media/imx.h
>>
>
Powered by blists - more mailing lists