On Fri, Oct 14, 2016 at 07:34:20PM +0200, Philipp Zabel wrote:
> the second round removes the prepare_stream callback and instead lets the
> intermediate subdevices propagate s_stream calls to their sources rather
> than individually calling s_stream on each subdevice from the bridge driver.
> This is similar to how drm bridges recursively call into their next neighbor.
> It makes it easier to do bringup ordering on a per-link level, as long as the
> source preparation can be done at s_power, and the sink can just prepare, call
> s_stream on its source, and then enable itself inside s_stream. Obviously this
> would only work in a generic fashion if all asynchronous subdevices with both
> inputs and outputs would propagate s_stream to their source subdevices.
> Changes since v1:
> - Propagate field and colorspace in ipucsi_subdev_set_format.
> - Remove v4l2_media_subdev_prepare_stream and v4l2_media_subdev_s_stream,
> let subdevices propagate s_stream calls to their upstream subdevices
> - Various fixes (see individual patches for details)
For the whole series:
Tested-by: Gary Bisson <gary.bis...@boundarydevices.com>
Tested on Nitrogen6x + BD_HDMI_MIPI daughter board on linux-next
This required using your v4l2-ctl patch to set the EDID if the source
output can't be forced:
BTW, do you have any update on this? Because it looks like the
VIDIOC_SUBDEV_QUERYCAP hasn't been implemented since your patch (March
Then I followed the procedure you gave here:
For those interested in trying it out, note that kmssink requires to use
I have a few remarks:
- I believe it would help having a patch that sets imx_v6_v7_defconfig
with the proper options in this series
- Not related to this series, I couldn't boot the board unless I disable
the PCIe driver, have you experienced the same issue?
- Is there a way not to set all the links manually using media-ctl? I
expected all the formats to be negotiated automatically once a stream
is properly detected.
- As discussed last week, the Nitrogen6x dtsi file shouldn't be
included, instead an overlay would be more appropriate. Maybe the log
should contain a comment about this.
Let me know if I need to add that Tested-by to every single patch so it
appears on Patchwork.
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html