Hi Hans,
On 12/7/18 5:35 AM, Hans Verkuil wrote:
Hi Steve,
How to proceed with this w.r.t. the two gpu ipu patches? Are those going
in first through the gpu tree? Or do they have to go in through our tree?
There is only one remaining gpu ipu patch in this series that is
awaiting an ack from Philipp:
"gpu: ipu-csi: Swap fields according to input/output field types"
I pinged him again.
Philipp agreed to allow the two ipu patches in the series to be merged
to media tree.
Steve
In that case I need Acks from whoever maintains that code.
Regards,
Hans
On 10/17/2018 02:00 AM, Steve Longerbeam wrote:
A set of patches that fixes some bugs with capturing from an
interlaced source, and incompatibilites between IDMAC interlace
interweaving and 4:2:0 data write reduction.
History:
v5:
- Added a regression fix to allow empty endpoints to CSI (fix for imx6q
SabreAuto).
- Cleaned up some convoluted code in ipu_csi_init_interface(), suggested
by Philipp Zabel.
- Fixed a regression in csi_setup(), caught by Philipp.
- Removed interweave_offset and replace with boolean interweave_swap,
suggested by Philipp.
- Make clear that it is IDMAC channel that does pixel reordering and
interweave, not the CSI, in the imx.rst doc, caught by Philipp.
v4:
- rebased to latest media-tree master branch.
- Make patch author and SoB email addresses the same.
v3:
- add support for/fix interweaved scan with YUV planar output.
- fix bug in 4:2:0 U/V offset macros.
- add patch that generalizes behavior of field swap in
ipu_csi_init_interface().
- add support for interweaved scan with field order swap.
Suggested by Philipp Zabel.
- in v2, inteweave scan was determined using field types of
CSI (and PRPENCVF) at the sink and source pads. In v3, this
has been moved one hop downstream: interweave is now determined
using field type at source pad, and field type selected at
capture interface. Suggested by Philipp.
- make sure to double CSI crop target height when input field
type in alternate.
- more updates to media driver doc to reflect above.
v2:
- update media driver doc.
- enable idmac interweave only if input field is sequential/alternate,
and output field is 'interlaced*'.
- move field try logic out of *try_fmt and into separate function.
- fix bug with resetting crop/compose rectangles.
- add a patch that fixes a field order bug in VDIC indirect mode.
- remove alternate field type from V4L2_FIELD_IS_SEQUENTIAL() macro
Suggested-by: Nicolas Dufresne <nicolas@xxxxxxxxxxxx>.
- add macro V4L2_FIELD_IS_INTERLACED().
Steve Longerbeam (12):
media: videodev2.h: Add more field helper macros
gpu: ipu-csi: Swap fields according to input/output field types
gpu: ipu-v3: Add planar support to interlaced scan
media: imx: Fix field negotiation
media: imx-csi: Input connections to CSI should be optional
media: imx-csi: Double crop height for alternate fields at sink
media: imx: interweave and odd-chroma-row skip are incompatible
media: imx-csi: Allow skipping odd chroma rows for YVU420
media: imx: vdic: rely on VDIC for correct field order
media: imx-csi: Move crop/compose reset after filling default mbus
fields
media: imx: Allow interweave with top/bottom lines swapped
media: imx.rst: Update doc to reflect fixes to interlaced capture
Documentation/media/v4l-drivers/imx.rst | 103 +++++++----
drivers/gpu/ipu-v3/ipu-cpmem.c | 26 ++-
drivers/gpu/ipu-v3/ipu-csi.c | 119 +++++++++----
drivers/staging/media/imx/imx-ic-prpencvf.c | 46 +++--
drivers/staging/media/imx/imx-media-capture.c | 14 ++
drivers/staging/media/imx/imx-media-csi.c | 168 +++++++++++++-----
drivers/staging/media/imx/imx-media-vdic.c | 12 +-
include/uapi/linux/videodev2.h | 7 +
include/video/imx-ipu-v3.h | 6 +-
9 files changed, 354 insertions(+), 147 deletions(-)
