On 07/07/2016 12:59 AM, Steve Longerbeam wrote: > Autodetect was likely broken only because access to the > interrupt registers were broken, so there were no standard > change interrupts. After fixing that, and reverting this, > autodetect seems to work just fine on an i.mx6q SabreAuto. > > This reverts commit 937feeed3f0ae8a0389d5732f6db63dd912acd99. The brokenness the commit refers to is conceptual not functional. The driver simply implemented the API incorrect. A subdev driver is not allowed to automatically switch the output format/resolution randomly. In the best case this will confuse the receiver which is not prepared to receive the changed resolution, in the worst case it will cause buffer overruns with hardware that has no boundary checks. This is why this was removed from the driver. The correct sequence is for the driver to generate a change notification and then have userspace react to that notification by stopping the current stream, query the new format/resolution, reconfigure the video pipeline for the new format/resolution and re-start the stream. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
