* FW: Re: [PATCH] abefw: Firmware for OMAP5 ABE
@ 2013-07-02 11:20 Hingolikar, Mrinmayee
2013-07-02 14:26 ` Denys Dmytriyenko
0 siblings, 1 reply; 2+ messages in thread
From: Hingolikar, Mrinmayee @ 2013-07-02 11:20 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-ti@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 3067 bytes --]
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@ti.com><mailto:mrinmayee@ti.com>
>> Signed-off-by: Siddharth Heroor <heroor@ti.com><mailto:heroor@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.
>
>
>> 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"<http://git.ti.com>
>> +LICENSE = "MIT"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"<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.
>
>> 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@yoctoproject.org<mailto:meta-ti@yoctoproject.org>
>> https://lists.yoctoproject.org/listinfo/meta-ti
--
Regards,
Mrinmayee
[-- Attachment #2: Type: text/html, Size: 8625 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: FW: Re: [PATCH] abefw: Firmware for OMAP5 ABE
2013-07-02 11:20 FW: Re: [PATCH] abefw: Firmware for OMAP5 ABE Hingolikar, Mrinmayee
@ 2013-07-02 14:26 ` Denys Dmytriyenko
0 siblings, 0 replies; 2+ messages in thread
From: Denys Dmytriyenko @ 2013-07-02 14:26 UTC (permalink / raw)
To: Hingolikar, Mrinmayee; +Cc: meta-ti@yoctoproject.org
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@ti.com>
>
> >> Signed-off-by: Siddharth Heroor <heroor@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@yoctoproject.org
>
> >> https://lists.yoctoproject.org/listinfo/meta-ti
>
>
>
>
>
> --
>
> Regards,
>
> Mrinmayee
>
>
>
>
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-07-02 14:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-02 11:20 FW: Re: [PATCH] abefw: Firmware for OMAP5 ABE Hingolikar, Mrinmayee
2013-07-02 14:26 ` Denys Dmytriyenko
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.