<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 24, 2014 at 12:09 PM, Lauren Post <span dir="ltr"><<a href="mailto:lauren.post@freescale.com" target="_blank">lauren.post@freescale.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Change-Id: I9ead6170ff5e20be0405d6f162c9f047bb3f8d2a<br>
Signed-off-by: Lauren Post <<a href="mailto:lauren.post@freescale.com">lauren.post@freescale.com</a>><br></blockquote><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

---<br>
 recipes-multimedia/gstreamer/gst-fsl-plugin.inc | Â  Â 8 +++++++-<br>
 1 file changed, 7 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/recipes-multimedia/gstreamer/gst-fsl-plugin.inc b/recipes-multimedia/gstreamer/gst-fsl-plugin.inc<br>
index 46bc6f0..4835b75 100644<br>
--- a/recipes-multimedia/gstreamer/gst-fsl-plugin.inc<br>
+++ b/recipes-multimedia/gstreamer/gst-fsl-plugin.inc<br>
@@ -26,7 +26,7 @@ PLATFORM_mx6sl = "MX6SL"<br>
 PLATFORM_mx28 = "MX28"<br>
<br>
 # Todo add a mechanism to map posible build targets<br>
-EXTRA_OECONF = "PLATFORM=${PLATFORM} \<br>
+EXTRA_OECONF = "PLATFORM=${PLATFORM} Â \<br></blockquote><div><br></div><div>This is a spurious change, I think it is a whitespace change, please revert this hunk.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

  Â  Â  Â  Â  Â  Â  Â  Â CPPFLAGS="-I${STAGING_KERNEL_DIR}/include/uapi -I${STAGING_KERNEL_DIR}/include""<br>
<br>
 PACKAGES =+ "${PN}-libmfwsconf ${PN}-libgplaycore ${PN}-libgstfsl \<br>
@@ -36,6 +36,12 @@ PACKAGES =+ "${PN}-libmfwsconf ${PN}-libgplaycore ${PN}-libgstfsl \<br>
 # FIXME: Add all features<br>
 # feature from excluded mm packages<br>
 PACKAGECONFIG[ac3] += "--enable-ac3dec,--disable-ac3dec,libfslac3codec,libfslac3codec"<br>
+# Â  The ddplus codec does not support mx28<br>
+DDPDEPENDS = "libfslddpcodec"<br>
+DDPDEPENDS_mx28 = ""<br>
+DDP_CONFIG_ARGS = "--enable-ddpdec"<br>
+DDP_CONFIG_ARGS_mx28 = "--disable-ddpdec"<br>
+PACKAGECONFIG[ddp] += "${DDP_CONFIG_ARGS},--disable-ddpdec,${DDPDEPENDS},${DDPDEPENDS}"<br>
 # feature from special mm packages<br>
 PACKAGECONFIG[aacp] += "--enable-aacpdec,--disable-aacpdec,libfslaacpcodec,libfslaacpcodec"<br>
 MSDEPENDS = "libfslmsparser libfslmscodec"<br></blockquote><div><br></div><div>This is wrong.</div><div><br></div><div>PACKAGECONFIG first parameter is to be given /when/ the feature is enabled. So giving a --disable-ddpdec is conceptually wrong.</div>
<div><br></div><div>Please use as:</div><div><br></div><div># ddp should not be enabled in MX23/MX28</div><div>PACKAGECONFIG[ddp] += "--enable-ddpdec,--disable-ddpdec,libfslddpcodec,libfslddpcodec"</div></div><div>
<br></div><div>as this is much easier to read and does not use the PACKAGECONFIG mechanism in a not intended way.</div><div><br></div>-- <br>Otavio Salvador Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  O.S. Systems<br><a href="http://www.ossystems.com.br" target="_blank">http://www.ossystems.com.br</a> Â  Â  Â  Â <a href="http://code.ossystems.com.br" target="_blank">http://code.ossystems.com.br</a><br>
<span>Mobile: +55 (53) 9981-7854 Â  Â  Â  Â  Â  Â Mobile: <span id="gc-number-384" class="" title="Call with Google Voice">+1 (347) 903-9750</span></span><br>
</div></div>