[meta-ti] FW: Re: [PATCH] abefw: Firmware for OMAP5 ABE

[Denys Dmytriyenko]

On Tue, Jul 02, 2013 at 07:20:04AM -0400, Hingolikar, Mrinmayee wrote:
> 
>  On 7/1/2013 9:23 PM, Denys Dmytriyenko wrote:
> 
>  > On Mon, Jul 01, 2013 at 09:11:48PM +0530, Mrinmayee Hingolikar wrote:
> 
>  >> * The OMAP5 Audio Backend requires a 'firmware' to enable
> 
>  >>   sound. This recipe installs a prebuilt firmware. This
> 
>  >>   part is a hack, as in the current form, the firmware
> 
>  >>   cannot be cross-compiled. There is a plan to fix this
> 
>  >>   and hopefully, this recipe is short-lived. Once the code
> 
>  >>   is fixed, we should have new recipes that build the firmware
> 
>  >>   and clean up this recipe.
> 
>  >> 
> 
>  >> Signed-off-by: Mrinmayee Hingolikar <mrinmayee at ti.com>
> 
>  >> Signed-off-by: Siddharth Heroor <heroor at ti.com>
> 
>  >> ---
> 
>  >>  recipes-bsp/abefw/abefw.inc       |    8 ++++++++
> 
>  >>  recipes-bsp/abefw/abefw_1.0.14.bb |   14 ++++++++++++++
> 
>  > What is the reason to have it split in 2 files - inc and bb?
> 
>  Reason to split because the there might be a version upgrade in the
> 
>  future. And also currently it just copies the prebuilt firmware,
> 
>  there should be a better recipe for this in future.

That is not a good reason for doing it now. Moreover, the goal is to not have 
more than one version of the same recipe, unless really-really needed, e.g. 
they are incompatible etc.


>  >>  2 files changed, 22 insertions(+), 0 deletions(-)
> 
>  >>  create mode 100644 recipes-bsp/abefw/abefw.inc
> 
>  >>  create mode 100644 recipes-bsp/abefw/abefw_1.0.14.bb
> 
>  >> 
> 
>  >> diff --git a/recipes-bsp/abefw/abefw.inc b/recipes-bsp/abefw/abefw.inc
> 
>  >> new file mode 100644
> 
>  >> index 0000000..4b6e9dc
> 
>  >> --- /dev/null
> 
>  >> +++ b/recipes-bsp/abefw/abefw.inc
> 
>  >> @@ -0,0 +1,8 @@
> 
>  >> +SUMMARY = "Firmware for OMAP5 ABE"
> 
>  >> +HOMEPAGE = "http://git.ti.com"
> 
>  >> +LICENSE = "MIT"
> 
>  >> +LIC_FILES_CHKSUM = "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
> 
>  >> +
> 
>  >> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> 
>  >> +
> 
>  >> +COMPATIBLE_MACHINE = "omap5-evm"
> 
>  > Is it limited to omap5-evm only? What about J6/DRA7xx (or pandaboard)?
> 
>  > Especially considering the repo is called omap4plus...
> 
>  J6 uses McASP and it does not require this firmware. Since this release
> 
>  is targetted for omap5 and J6, we limited it to omap5-evm. 

Ok, this makes sense then.


>  >> diff --git a/recipes-bsp/abefw/abefw_1.0.14.bb b/recipes-bsp/abefw/abefw_1.0.14.bb
> 
>  >> new file mode 100644
> 
>  >> index 0000000..df68080
> 
>  >> --- /dev/null
> 
>  >> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
> 
>  >> @@ -0,0 +1,14 @@
> 
>  >> +require abefw.inc
> 
>  >> +
> 
>  >> +SRC_URI = "git://git.ti.com/glsdk/abefw-omap4plus.git;protocol=git"
> 
>  >> +
> 
>  >> +S = "${WORKDIR}/git"
> 
>  >> +
> 
>  >> +SRCREV = "ceccc0332264e39bdc51e54f80ea7256a3886c58"
> 
>  >> +
> 
>  >> +do_install() {
> 
>  >> +        mkdir -p ${D}/lib/firmware
> 
>  >> +        cp ${S}/firmware/omap4_abe_new ${D}/lib/firmware/
> 
>  >> +}
> 
>  >> +
> 
>  >> +FILES_${PN} += "/lib/firmware/omap4_abe_new"
> 
>  >> --
> 
>  >> 1.7.0.4
> 
>  >> 
> 
>  >> _______________________________________________
> 
>  >> meta-ti mailing list
> 
>  >> meta-ti at yoctoproject.org
> 
>  >> https://lists.yoctoproject.org/listinfo/meta-ti
> 
>   
> 
>   
> 
>  --
> 
>  Regards,
> 
>  Mrinmayee
> 
>   
> 
>     
> 
>