Commit ef3711cc authored Feb 08, 2014 by Dheeraj CVR

libstagefright: unbreak hardware OMX on OMAP3

http://review.cyanogenmod.org/58070 has mistakenly reverted
http://review.cyanogenmod.org/44365 which broke hardware OMX
on OMAP3.

The fix for OMAP3 was just a logic change to prevent an
indefinate loop since the nIndex value was being overriden by
the OMX component. The code was modified to use a seprate control
value other than nIndex itself and doesnt pose any issues with
other platforms.

Change-Id: I8ca4f9d48c3f9241f21c9fe6ab98d0be5783ceab

parent 64b52c43

media/libstagefright/OMXCodec.cpp

+2 −1

Original line number	Diff line number	Diff line
		@@ -5350,7 +5350,8 @@ status_t QueryCodec(
		#else
		portFormat.nPortIndex = !isEncoder ? 1 : 0;
		#endif
		for (portFormat.nIndex = 0;; ++portFormat.nIndex) {
		for (OMX_U32 index = 0;;index++) {
		portFormat.nIndex = index;
		err = omx->getParameter(
		node, OMX_IndexParamVideoPortFormat,
		&portFormat, sizeof(portFormat));