* [PATCHv2] abefw: Firmware for OMAP5 ABE
@ 2013-07-12 6:43 Hingolikar, Mrinmayee
2013-07-12 13:11 ` Cooper Jr., Franklin
2013-07-19 6:01 ` Hingolikar, Mrinmayee
0 siblings, 2 replies; 14+ messages in thread
From: Hingolikar, Mrinmayee @ 2013-07-12 6:43 UTC (permalink / raw)
To: meta-arago@arago-project.org
* 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.
v2:
* Merged contents from abefw.inc and removed abefw.inc based
on review.
Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
Signed-off-by: Siddharth Heroor <heroor@ti.com>
---
recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644 recipes-bsp/abefw/abefw_1.0.14.bb
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..8db71e4
--- /dev/null
+++ b/recipes-bsp/abefw/abefw_1.0.14.bb
@@ -0,0 +1,20 @@
+SUMMARY = "Firmware for OMAP5 ABE"
+HOMEPAGE = "http://git.ti.com"
+LICENSE = "BSD/GPLv2"
+LIC_FILES_CHKSUM = "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+COMPATIBLE_MACHINE = "omap5-evm"
+
+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
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-07-12 6:43 [PATCHv2] abefw: Firmware for OMAP5 ABE Hingolikar, Mrinmayee
@ 2013-07-12 13:11 ` Cooper Jr., Franklin
2013-07-22 4:13 ` Siddharth Heroor
2013-07-19 6:01 ` Hingolikar, Mrinmayee
1 sibling, 1 reply; 14+ messages in thread
From: Cooper Jr., Franklin @ 2013-07-12 13:11 UTC (permalink / raw)
To: Hingolikar, Mrinmayee; +Cc: meta-arago@arago-project.org
On Jul 12, 2013, at 1:43 AM, "Hingolikar, Mrinmayee" <mrinmayee@ti.com> 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.
>
> v2:
> * Merged contents from abefw.inc and removed abefw.inc based
> on review.
>
> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
> Signed-off-by: Siddharth Heroor <heroor@ti.com>
> ---
> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
> 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644 recipes-bsp/abefw/abefw_1.0.14.bb
>
> 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..8db71e4
> --- /dev/null
> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
> @@ -0,0 +1,20 @@
> +SUMMARY = "Firmware for OMAP5 ABE"
> +HOMEPAGE = "http://git.ti.com"
> +LICENSE = "BSD/GPLv2"
I'm assuming this means BSD or GPLv2.
The proper syntax is
LICENSE = "BSD | GPLv2"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
I don't see any machine based overrides or appends in this recipe. This isn't needed.
> +COMPATIBLE_MACHINE = "omap5-evm"
I'm just curious if the dra7xx or any other possible future boards will be using this firmware/recipe. If so any harm in making this generic and specifying omap-a15?
> +
> +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-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-07-12 13:11 ` Cooper Jr., Franklin
@ 2013-07-22 4:13 ` Siddharth Heroor
2013-07-22 12:12 ` Cooper Jr., Franklin
0 siblings, 1 reply; 14+ messages in thread
From: Siddharth Heroor @ 2013-07-22 4:13 UTC (permalink / raw)
To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org
On 7/12/2013 6:41 PM, Cooper Jr., Franklin wrote:
>
>
> On Jul 12, 2013, at 1:43 AM, "Hingolikar, Mrinmayee" <mrinmayee@ti.com> 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.
>>
>> v2:
>> * Merged contents from abefw.inc and removed abefw.inc based
>> on review.
>>
>> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>> ---
>> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
>> 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644 recipes-bsp/abefw/abefw_1.0.14.bb
>>
>> 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..8db71e4
>> --- /dev/null
>> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
>> @@ -0,0 +1,20 @@
>> +SUMMARY = "Firmware for OMAP5 ABE"
>> +HOMEPAGE = "http://git.ti.com"
>> +LICENSE = "BSD/GPLv2"
> I'm assuming this means BSD or GPLv2.
>
> The proper syntax is
> LICENSE = "BSD | GPLv2"
>
>
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
>> +
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> I don't see any machine based overrides or appends in this recipe. This isn't needed.
>
>> +COMPATIBLE_MACHINE = "omap5-evm"
> I'm just curious if the dra7xx or any other possible future boards will be using this firmware/recipe. If so any harm in making this generic and specifying omap-a15?
This is an IP *specific* to OMAP4/5. On OMAP5, the ABE (Audio Backend)
is present to allow for low power audio playback. Refer TRM
(http://www.ti.com/lit/ug/swpu249w/swpu249w.pdf) section 13. - Audio
Subsystem.
On DRA7xx ABE isn't present. Instead we have multiple McASPs (upto 8)
for which no firmware is required. McASP has its own sound driver (which
should be common to other devices).
By setting COMPATIBILITY and PACKAGE_ARCH to be limited to omap5, we
make it obvious that the IP is OMAP5 specific and not generic for all
omap-a15 devices.
>> +
>> +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-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-07-22 4:13 ` Siddharth Heroor
@ 2013-07-22 12:12 ` Cooper Jr., Franklin
2013-07-22 12:15 ` Maupin, Chase
2013-07-22 16:24 ` Denys Dmytriyenko
0 siblings, 2 replies; 14+ messages in thread
From: Cooper Jr., Franklin @ 2013-07-22 12:12 UTC (permalink / raw)
To: Heroor, Siddharth; +Cc: meta-arago@arago-project.org
On Jul 21, 2013, at 11:13 PM, "Heroor, Siddharth" <heroor@ti.com> wrote:
> On 7/12/2013 6:41 PM, Cooper Jr., Franklin wrote:
>>
>>
>> On Jul 12, 2013, at 1:43 AM, "Hingolikar, Mrinmayee" <mrinmayee@ti.com> 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.
>>>
>>> v2:
>>> * Merged contents from abefw.inc and removed abefw.inc based
>>> on review.
>>>
>>> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
>>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>>> ---
>>> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
>>> 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644 recipes-bsp/abefw/abefw_1.0.14.bb
>>>
>>> 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..8db71e4
>>> --- /dev/null
>>> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
>>> @@ -0,0 +1,20 @@
>>> +SUMMARY = "Firmware for OMAP5 ABE"
>>> +HOMEPAGE = "http://git.ti.com"
>>> +LICENSE = "BSD/GPLv2"
>> I'm assuming this means BSD or GPLv2.
>>
>> The proper syntax is
>> LICENSE = "BSD | GPLv2"
>>
>>
>>> +LIC_FILES_CHKSUM = "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
>>> +
>>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> I don't see any machine based overrides or appends in this recipe. This isn't needed.
>>
>>> +COMPATIBLE_MACHINE = "omap5-evm"
>> I'm just curious if the dra7xx or any other possible future boards will be using this firmware/recipe. If so any harm in making this generic and specifying omap-a15?
>
> This is an IP *specific* to OMAP4/5. On OMAP5, the ABE (Audio Backend)
> is present to allow for low power audio playback. Refer TRM
> (http://www.ti.com/lit/ug/swpu249w/swpu249w.pdf) section 13. - Audio
> Subsystem.
>
> On DRA7xx ABE isn't present. Instead we have multiple McASPs (upto 8)
> for which no firmware is required. McASP has its own sound driver (which
> should be common to other devices).
>
> By setting COMPATIBILITY and PACKAGE_ARCH to be limited to omap5, we
> make it obvious that the IP is OMAP5 specific and not generic for all
> omap-a15 devices.
For COMPATIBILITY that's fine since it will prevent any other board from building the recipe but PACKAGE_ARCH still isn't needed so it should be removed.
>
>>> +
>>> +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-arago mailing list
>>> meta-arago@arago-project.org
>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-07-22 12:12 ` Cooper Jr., Franklin
@ 2013-07-22 12:15 ` Maupin, Chase
2013-07-22 16:24 ` Denys Dmytriyenko
1 sibling, 0 replies; 14+ messages in thread
From: Maupin, Chase @ 2013-07-22 12:15 UTC (permalink / raw)
To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org
Don't forget the licensing comment as well.
Sincerely,
Chase Maupin
On Jul 22, 2013, at 1:12 PM, "Cooper Jr., Franklin" <fcooper@ti.com> wrote:
>
>
> On Jul 21, 2013, at 11:13 PM, "Heroor, Siddharth" <heroor@ti.com> wrote:
>
>> On 7/12/2013 6:41 PM, Cooper Jr., Franklin wrote:
>>>
>>>
>>> On Jul 12, 2013, at 1:43 AM, "Hingolikar, Mrinmayee" <mrinmayee@ti.com> 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.
>>>>
>>>> v2:
>>>> * Merged contents from abefw.inc and removed abefw.inc based
>>>> on review.
>>>>
>>>> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
>>>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>>>> ---
>>>> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
>>>> 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644 recipes-bsp/abefw/abefw_1.0.14.bb
>>>>
>>>> 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..8db71e4
>>>> --- /dev/null
>>>> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
>>>> @@ -0,0 +1,20 @@
>>>> +SUMMARY = "Firmware for OMAP5 ABE"
>>>> +HOMEPAGE = "http://git.ti.com"
>>>> +LICENSE = "BSD/GPLv2"
>>> I'm assuming this means BSD or GPLv2.
>>>
>>> The proper syntax is
>>> LICENSE = "BSD | GPLv2"
>>>
>>>
>>>> +LIC_FILES_CHKSUM = "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
>>>> +
>>>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>>> I don't see any machine based overrides or appends in this recipe. This isn't needed.
>>>
>>>> +COMPATIBLE_MACHINE = "omap5-evm"
>>> I'm just curious if the dra7xx or any other possible future boards will be using this firmware/recipe. If so any harm in making this generic and specifying omap-a15?
>>
>> This is an IP *specific* to OMAP4/5. On OMAP5, the ABE (Audio Backend)
>> is present to allow for low power audio playback. Refer TRM
>> (http://www.ti.com/lit/ug/swpu249w/swpu249w.pdf) section 13. - Audio
>> Subsystem.
>>
>> On DRA7xx ABE isn't present. Instead we have multiple McASPs (upto 8)
>> for which no firmware is required. McASP has its own sound driver (which
>> should be common to other devices).
>>
>> By setting COMPATIBILITY and PACKAGE_ARCH to be limited to omap5, we
>> make it obvious that the IP is OMAP5 specific and not generic for all
>> omap-a15 devices.
> For COMPATIBILITY that's fine since it will prevent any other board from building the recipe but PACKAGE_ARCH still isn't needed so it should be removed.
>
>>
>>>> +
>>>> +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-arago mailing list
>>>> meta-arago@arago-project.org
>>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>>> _______________________________________________
>>> meta-arago mailing list
>>> meta-arago@arago-project.org
>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-07-22 12:12 ` Cooper Jr., Franklin
2013-07-22 12:15 ` Maupin, Chase
@ 2013-07-22 16:24 ` Denys Dmytriyenko
1 sibling, 0 replies; 14+ messages in thread
From: Denys Dmytriyenko @ 2013-07-22 16:24 UTC (permalink / raw)
To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org
On Mon, Jul 22, 2013 at 12:12:23PM +0000, Cooper Jr., Franklin wrote:
>
>
> On Jul 21, 2013, at 11:13 PM, "Heroor, Siddharth" <heroor@ti.com> wrote:
>
> > On 7/12/2013 6:41 PM, Cooper Jr., Franklin wrote:
> >>
> >>
> >> On Jul 12, 2013, at 1:43 AM, "Hingolikar, Mrinmayee" <mrinmayee@ti.com> 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.
> >>>
> >>> v2:
> >>> * Merged contents from abefw.inc and removed abefw.inc based
> >>> on review.
> >>>
> >>> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
> >>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
> >>> ---
> >>> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
> >>> 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644 recipes-bsp/abefw/abefw_1.0.14.bb
> >>>
> >>> 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..8db71e4
> >>> --- /dev/null
> >>> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
> >>> @@ -0,0 +1,20 @@
> >>> +SUMMARY = "Firmware for OMAP5 ABE"
> >>> +HOMEPAGE = "http://git.ti.com"
> >>> +LICENSE = "BSD/GPLv2"
> >> I'm assuming this means BSD or GPLv2.
> >>
> >> The proper syntax is
> >> LICENSE = "BSD | GPLv2"
Ack on this comment - please use the correct syntax for the LICENSE field, as
it has changed since Classic OE and is no longer a free form.
> >>> +LIC_FILES_CHKSUM = "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
> >>> +
> >>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> >> I don't see any machine based overrides or appends in this recipe. This isn't needed.
> >>
> >>> +COMPATIBLE_MACHINE = "omap5-evm"
> >> I'm just curious if the dra7xx or any other possible future boards will
> >> be using this firmware/recipe. If so any harm in making this generic and
> >> specifying omap-a15?
> >
> > This is an IP *specific* to OMAP4/5. On OMAP5, the ABE (Audio Backend)
> > is present to allow for low power audio playback. Refer TRM
> > (http://www.ti.com/lit/ug/swpu249w/swpu249w.pdf) section 13. - Audio
> > Subsystem.
> >
> > On DRA7xx ABE isn't present. Instead we have multiple McASPs (upto 8)
> > for which no firmware is required. McASP has its own sound driver (which
> > should be common to other devices).
> >
> > By setting COMPATIBILITY and PACKAGE_ARCH to be limited to omap5, we
> > make it obvious that the IP is OMAP5 specific and not generic for all
> > omap-a15 devices.
> For COMPATIBILITY that's fine since it will prevent any other board from
> building the recipe but PACKAGE_ARCH still isn't needed so it should be
> removed.
Well, unless you have common feeds and want to prevent the package to be
accidentally installed on dra7xx... But I'd like to hear if Sid's team is
planning to offer feeds. Either way PACKAGE_ARCH is good to have in this case.
--
Denys
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-07-12 6:43 [PATCHv2] abefw: Firmware for OMAP5 ABE Hingolikar, Mrinmayee
2013-07-12 13:11 ` Cooper Jr., Franklin
@ 2013-07-19 6:01 ` Hingolikar, Mrinmayee
2013-07-19 11:54 ` Maupin, Chase
1 sibling, 1 reply; 14+ messages in thread
From: Hingolikar, Mrinmayee @ 2013-07-19 6:01 UTC (permalink / raw)
To: Hingolikar, Mrinmayee, meta-arago@arago-project.org
Any comments on the v2?
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Hingolikar, Mrinmayee
> Sent: Friday, July 12, 2013 12:13 PM
> To: meta-arago@arago-project.org
> Subject: [meta-arago] [PATCHv2] abefw: Firmware for OMAP5 ABE
>
>
> * 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.
>
> v2:
> * Merged contents from abefw.inc and removed abefw.inc based
> on review.
>
> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
> Signed-off-by: Siddharth Heroor <heroor@ti.com>
> ---
> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
> 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644
> recipes-bsp/abefw/abefw_1.0.14.bb
>
> 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..8db71e4
> --- /dev/null
> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
> @@ -0,0 +1,20 @@
> +SUMMARY = "Firmware for OMAP5 ABE"
> +HOMEPAGE = "http://git.ti.com"
> +LICENSE = "BSD/GPLv2"
> +LIC_FILES_CHKSUM =
> "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +COMPATIBLE_MACHINE = "omap5-evm"
> +
> +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-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-07-19 6:01 ` Hingolikar, Mrinmayee
@ 2013-07-19 11:54 ` Maupin, Chase
0 siblings, 0 replies; 14+ messages in thread
From: Maupin, Chase @ 2013-07-19 11:54 UTC (permalink / raw)
To: Hingolikar, Mrinmayee, meta-arago@arago-project.org
Mryinmayee,
Franklin Cooper already gave you comments on the v2 earlier this week about the LICENSE value amongst othre things. You should address that feedback.
>-----Original Message-----
>From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>bounces@arago-project.org] On Behalf Of Hingolikar, Mrinmayee
>Sent: Friday, July 19, 2013 1:01 AM
>To: Hingolikar, Mrinmayee; meta-arago@arago-project.org
>Subject: Re: [meta-arago] [PATCHv2] abefw: Firmware for OMAP5 ABE
>
>Any comments on the v2?
>
>> -----Original Message-----
>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>> bounces@arago-project.org] On Behalf Of Hingolikar, Mrinmayee
>> Sent: Friday, July 12, 2013 12:13 PM
>> To: meta-arago@arago-project.org
>> Subject: [meta-arago] [PATCHv2] abefw: Firmware for OMAP5 ABE
>>
>>
>> * 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.
>>
>> v2:
>> * Merged contents from abefw.inc and removed abefw.inc based
>> on review.
>>
>> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>> ---
>> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
>> 1 files changed, 20 insertions(+), 0 deletions(-) create mode
>100644
>> recipes-bsp/abefw/abefw_1.0.14.bb
>>
>> 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..8db71e4
>> --- /dev/null
>> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
>> @@ -0,0 +1,20 @@
>> +SUMMARY = "Firmware for OMAP5 ABE"
>> +HOMEPAGE = "http://git.ti.com"
>> +LICENSE = "BSD/GPLv2"
>> +LIC_FILES_CHKSUM =
>> "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
>> +
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> +COMPATIBLE_MACHINE = "omap5-evm"
>> +
>> +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-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>_______________________________________________
>meta-arago mailing list
>meta-arago@arago-project.org
>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 14+ messages in thread
[parent not found: <1372849303-21311-1-git-send-email-mrinmayee@ti.com>]
* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
[not found] <1372849303-21311-1-git-send-email-mrinmayee@ti.com>
@ 2013-08-21 11:48 ` Hingolikar, Mrinmayee
2013-08-21 12:52 ` Cooper Jr., Franklin
2013-08-21 15:31 ` Maupin, Chase
0 siblings, 2 replies; 14+ messages in thread
From: Hingolikar, Mrinmayee @ 2013-08-21 11:48 UTC (permalink / raw)
To: Hingolikar, Mrinmayee, meta-ti@yoctoproject.org
Ping for comments!
> -----Original Message-----
> From: Hingolikar, Mrinmayee
> Sent: Wednesday, July 03, 2013 4:32 PM
> To: meta-ti@yoctoproject.org
> Cc: Hingolikar, Mrinmayee; Heroor, Siddharth
> Subject: [PATCHv2] abefw: Firmware for OMAP5 ABE
>
> * 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.
>
> v2:
> * Merged contents from abefw.inc and removed abefw.inc based
> on review.
>
> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
> Signed-off-by: Siddharth Heroor <heroor@ti.com>
> ---
> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
> 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644
> recipes-bsp/abefw/abefw_1.0.14.bb
>
> 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..8db71e4
> --- /dev/null
> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
> @@ -0,0 +1,20 @@
> +SUMMARY = "Firmware for OMAP5 ABE"
> +HOMEPAGE = "http://git.ti.com"
> +LICENSE = "BSD/GPLv2"
> +LIC_FILES_CHKSUM =
> "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +COMPATIBLE_MACHINE = "omap5-evm"
> +
> +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
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-08-21 11:48 ` Hingolikar, Mrinmayee
@ 2013-08-21 12:52 ` Cooper Jr., Franklin
2013-08-21 15:31 ` Maupin, Chase
1 sibling, 0 replies; 14+ messages in thread
From: Cooper Jr., Franklin @ 2013-08-21 12:52 UTC (permalink / raw)
To: Hingolikar, Mrinmayee; +Cc: meta-ti@yoctoproject.org
Only one real comment below but overall looks good
On Aug 21, 2013, at 6:48 AM, "Hingolikar, Mrinmayee" <mrinmayee@ti.com> wrote:
> Ping for comments!
>
>> -----Original Message-----
>> From: Hingolikar, Mrinmayee
>> Sent: Wednesday, July 03, 2013 4:32 PM
>> To: meta-ti@yoctoproject.org
>> Cc: Hingolikar, Mrinmayee; Heroor, Siddharth
>> Subject: [PATCHv2] abefw: Firmware for OMAP5 ABE
>>
>> * 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.
>>
>> v2:
>> * Merged contents from abefw.inc and removed abefw.inc based
>> on review.
>>
>> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>> ---
>> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
>> 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644
>> recipes-bsp/abefw/abefw_1.0.14.bb
>>
>> 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..8db71e4
>> --- /dev/null
>> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
>> @@ -0,0 +1,20 @@
>> +SUMMARY = "Firmware for OMAP5 ABE"
>> +HOMEPAGE = "http://git.ti.com"
Not a big deal but probably should use the git.ti link to the actual repository so people can find it easier"
>> +LICENSE = "BSD/GPLv2"
This should be "BSD | GPLv2"
>> +LIC_FILES_CHKSUM =
>> "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
>> +
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> +COMPATIBLE_MACHINE = "omap5-evm"
>> +
>> +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
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-08-21 11:48 ` Hingolikar, Mrinmayee
2013-08-21 12:52 ` Cooper Jr., Franklin
@ 2013-08-21 15:31 ` Maupin, Chase
2013-08-21 15:49 ` Denys Dmytriyenko
1 sibling, 1 reply; 14+ messages in thread
From: Maupin, Chase @ 2013-08-21 15:31 UTC (permalink / raw)
To: Hingolikar, Mrinmayee, meta-ti@yoctoproject.org
>-----Original Message-----
>From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>bounces@yoctoproject.org] On Behalf Of Hingolikar, Mrinmayee
>Sent: Wednesday, August 21, 2013 6:48 AM
>To: Hingolikar, Mrinmayee; meta-ti@yoctoproject.org
>Subject: Re: [meta-ti] [PATCHv2] abefw: Firmware for OMAP5 ABE
>
>Ping for comments!
For some reason I don't see this in my e-mails for the original post. Sorry for the delay.
>
>> -----Original Message-----
>> From: Hingolikar, Mrinmayee
>> Sent: Wednesday, July 03, 2013 4:32 PM
>> To: meta-ti@yoctoproject.org
>> Cc: Hingolikar, Mrinmayee; Heroor, Siddharth
>> Subject: [PATCHv2] abefw: Firmware for OMAP5 ABE
>>
>> * 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.
>>
>> v2:
>> * Merged contents from abefw.inc and removed abefw.inc based
>> on review.
>>
>> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>> ---
>> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
>> 1 files changed, 20 insertions(+), 0 deletions(-) create mode
>100644
>> recipes-bsp/abefw/abefw_1.0.14.bb
>>
>> 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..8db71e4
>> --- /dev/null
>> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
>> @@ -0,0 +1,20 @@
>> +SUMMARY = "Firmware for OMAP5 ABE"
>> +HOMEPAGE = "http://git.ti.com"
>> +LICENSE = "BSD/GPLv2"
This seems wrong. I thought the convention is "BSD | GPLv2" for either license or "BSD & GPLv2" if mixed license.
If it is mixed then you probably need 2 entries in LIC_FILES_CHKSUM, one for each license.
Also, does the GPLv2 use the "or later" clause such that it should be GPLv2+ ?
>> +LIC_FILES_CHKSUM =
>> "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
>> +
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> +COMPATIBLE_MACHINE = "omap5-evm"
>> +
>> +SRC_URI = "git://git.ti.com/glsdk/abefw-
>omap4plus.git;protocol=git"
Not strictly required but PR = r0 here would help remind you to bump the PR in the future.
>> +
>> +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"
I'm sure this has been discussed before, but do you see any confusion about this being omap5-evm only and yet the FW is omap4_abe_new?
>> --
>> 1.7.0.4
>
>_______________________________________________
>meta-ti mailing list
>meta-ti@yoctoproject.org
>https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-08-21 15:31 ` Maupin, Chase
@ 2013-08-21 15:49 ` Denys Dmytriyenko
2013-09-03 17:42 ` Denys Dmytriyenko
0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2013-08-21 15:49 UTC (permalink / raw)
To: Maupin, Chase; +Cc: meta-ti@yoctoproject.org
On Wed, Aug 21, 2013 at 03:31:42PM +0000, Maupin, Chase wrote:
> >-----Original Message-----
> >From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> >bounces@yoctoproject.org] On Behalf Of Hingolikar, Mrinmayee
> >Sent: Wednesday, August 21, 2013 6:48 AM
> >To: Hingolikar, Mrinmayee; meta-ti@yoctoproject.org
> >Subject: Re: [meta-ti] [PATCHv2] abefw: Firmware for OMAP5 ABE
> >
> >Ping for comments!
>
> For some reason I don't see this in my e-mails for the original post. Sorry
> for the delay.
You are correct - there were no v2 submissions to the list on July 3rd. We
only reviewed the original patch before.
> >> -----Original Message-----
> >> From: Hingolikar, Mrinmayee
> >> Sent: Wednesday, July 03, 2013 4:32 PM
> >> To: meta-ti@yoctoproject.org
> >> Cc: Hingolikar, Mrinmayee; Heroor, Siddharth
> >> Subject: [PATCHv2] abefw: Firmware for OMAP5 ABE
> >>
> >> * 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.
> >>
> >> v2:
> >> * Merged contents from abefw.inc and removed abefw.inc based
> >> on review.
> >>
> >> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
> >> Signed-off-by: Siddharth Heroor <heroor@ti.com>
> >> ---
> >> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
> >> 1 files changed, 20 insertions(+), 0 deletions(-) create mode
> >100644
> >> recipes-bsp/abefw/abefw_1.0.14.bb
> >>
> >> 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..8db71e4
> >> --- /dev/null
> >> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
> >> @@ -0,0 +1,20 @@
> >> +SUMMARY = "Firmware for OMAP5 ABE"
> >> +HOMEPAGE = "http://git.ti.com"
> >> +LICENSE = "BSD/GPLv2"
>
> This seems wrong. I thought the convention is "BSD | GPLv2" for either
> license or "BSD & GPLv2" if mixed license.
>
> If it is mixed then you probably need 2 entries in LIC_FILES_CHKSUM, one for
> each license.
>
> Also, does the GPLv2 use the "or later" clause such that it should be GPLv2+
> ?
This was already mentioned in one of the other patches.
> >> +LIC_FILES_CHKSUM =
> >> "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
> >> +
> >> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> >> +COMPATIBLE_MACHINE = "omap5-evm"
> >> +
> >> +SRC_URI = "git://git.ti.com/glsdk/abefw-
> >omap4plus.git;protocol=git"
The git repo itself says omap4plus - why is it limited to OMAP5 only? Can it
be enabled for OMAP4/Pandaboard too?
> Not strictly required but PR = r0 here would help remind you to bump the PR
> in the future.
>
> >> +
> >> +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"
>
> I'm sure this has been discussed before, but do you see any confusion about
> this being omap5-evm only and yet the FW is omap4_abe_new?
>
> >> --
> >> 1.7.0.4
> >
> >_______________________________________________
> >meta-ti mailing list
> >meta-ti@yoctoproject.org
> >https://lists.yoctoproject.org/listinfo/meta-ti
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-08-21 15:49 ` Denys Dmytriyenko
@ 2013-09-03 17:42 ` Denys Dmytriyenko
2013-10-18 10:29 ` Mrinmayee Hingolikar
0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2013-09-03 17:42 UTC (permalink / raw)
To: meta-ti@yoctoproject.org
Mrinmayee,
Ping. Do you have an ETA for addressing the comments below? Thanks.
On Wed, Aug 21, 2013 at 11:49:54AM -0400, Denys Dmytriyenko wrote:
> On Wed, Aug 21, 2013 at 03:31:42PM +0000, Maupin, Chase wrote:
> > >-----Original Message-----
> > >From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > >bounces@yoctoproject.org] On Behalf Of Hingolikar, Mrinmayee
> > >Sent: Wednesday, August 21, 2013 6:48 AM
> > >To: Hingolikar, Mrinmayee; meta-ti@yoctoproject.org
> > >Subject: Re: [meta-ti] [PATCHv2] abefw: Firmware for OMAP5 ABE
> > >
> > >Ping for comments!
> >
> > For some reason I don't see this in my e-mails for the original post. Sorry
> > for the delay.
>
> You are correct - there were no v2 submissions to the list on July 3rd. We
> only reviewed the original patch before.
>
>
> > >> -----Original Message-----
> > >> From: Hingolikar, Mrinmayee
> > >> Sent: Wednesday, July 03, 2013 4:32 PM
> > >> To: meta-ti@yoctoproject.org
> > >> Cc: Hingolikar, Mrinmayee; Heroor, Siddharth
> > >> Subject: [PATCHv2] abefw: Firmware for OMAP5 ABE
> > >>
> > >> * 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.
> > >>
> > >> v2:
> > >> * Merged contents from abefw.inc and removed abefw.inc based
> > >> on review.
> > >>
> > >> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
> > >> Signed-off-by: Siddharth Heroor <heroor@ti.com>
> > >> ---
> > >> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
> > >> 1 files changed, 20 insertions(+), 0 deletions(-) create mode
> > >100644
> > >> recipes-bsp/abefw/abefw_1.0.14.bb
> > >>
> > >> 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..8db71e4
> > >> --- /dev/null
> > >> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
> > >> @@ -0,0 +1,20 @@
> > >> +SUMMARY = "Firmware for OMAP5 ABE"
> > >> +HOMEPAGE = "http://git.ti.com"
> > >> +LICENSE = "BSD/GPLv2"
> >
> > This seems wrong. I thought the convention is "BSD | GPLv2" for either
> > license or "BSD & GPLv2" if mixed license.
> >
> > If it is mixed then you probably need 2 entries in LIC_FILES_CHKSUM, one for
> > each license.
> >
> > Also, does the GPLv2 use the "or later" clause such that it should be GPLv2+
> > ?
>
> This was already mentioned in one of the other patches.
>
>
> > >> +LIC_FILES_CHKSUM =
> > >> "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
> > >> +
> > >> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> > >> +COMPATIBLE_MACHINE = "omap5-evm"
> > >> +
> > >> +SRC_URI = "git://git.ti.com/glsdk/abefw-
> > >omap4plus.git;protocol=git"
>
> The git repo itself says omap4plus - why is it limited to OMAP5 only? Can it
> be enabled for OMAP4/Pandaboard too?
>
>
> > Not strictly required but PR = r0 here would help remind you to bump the PR
> > in the future.
> >
> > >> +
> > >> +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"
> >
> > I'm sure this has been discussed before, but do you see any confusion about
> > this being omap5-evm only and yet the FW is omap4_abe_new?
> >
> > >> --
> > >> 1.7.0.4
> > >
> > >_______________________________________________
> > >meta-ti mailing list
> > >meta-ti@yoctoproject.org
> > >https://lists.yoctoproject.org/listinfo/meta-ti
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCHv2] abefw: Firmware for OMAP5 ABE
2013-09-03 17:42 ` Denys Dmytriyenko
@ 2013-10-18 10:29 ` Mrinmayee Hingolikar
0 siblings, 0 replies; 14+ messages in thread
From: Mrinmayee Hingolikar @ 2013-10-18 10:29 UTC (permalink / raw)
To: meta-ti
On 9/3/2013 11:12 PM, Denys Dmytriyenko wrote:
> Mrinmayee,
>
> Ping. Do you have an ETA for addressing the comments below? Thanks.
Sorry for the delayed response. I am sending the v3 for this patch
addressing all the comments.
>
>
> On Wed, Aug 21, 2013 at 11:49:54AM -0400, Denys Dmytriyenko wrote:
>> On Wed, Aug 21, 2013 at 03:31:42PM +0000, Maupin, Chase wrote:
>>>> -----Original Message-----
>>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>>>> bounces@yoctoproject.org] On Behalf Of Hingolikar, Mrinmayee
>>>> Sent: Wednesday, August 21, 2013 6:48 AM
>>>> To: Hingolikar, Mrinmayee; meta-ti@yoctoproject.org
>>>> Subject: Re: [meta-ti] [PATCHv2] abefw: Firmware for OMAP5 ABE
>>>>
>>>> Ping for comments!
>>> For some reason I don't see this in my e-mails for the original post. Sorry
>>> for the delay.
>> You are correct - there were no v2 submissions to the list on July 3rd. We
>> only reviewed the original patch before.
>>
>>
>>>>> -----Original Message-----
>>>>> From: Hingolikar, Mrinmayee
>>>>> Sent: Wednesday, July 03, 2013 4:32 PM
>>>>> To: meta-ti@yoctoproject.org
>>>>> Cc: Hingolikar, Mrinmayee; Heroor, Siddharth
>>>>> Subject: [PATCHv2] abefw: Firmware for OMAP5 ABE
>>>>>
>>>>> * 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.
>>>>>
>>>>> v2:
>>>>> * Merged contents from abefw.inc and removed abefw.inc based
>>>>> on review.
>>>>>
>>>>> Signed-off-by: Mrinmayee Hingolikar <mrinmayee@ti.com>
>>>>> Signed-off-by: Siddharth Heroor <heroor@ti.com>
>>>>> ---
>>>>> recipes-bsp/abefw/abefw_1.0.14.bb | 20 ++++++++++++++++++++
>>>>> 1 files changed, 20 insertions(+), 0 deletions(-) create mode
>>>> 100644
>>>>> recipes-bsp/abefw/abefw_1.0.14.bb
>>>>>
>>>>> 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..8db71e4
>>>>> --- /dev/null
>>>>> +++ b/recipes-bsp/abefw/abefw_1.0.14.bb
>>>>> @@ -0,0 +1,20 @@
>>>>> +SUMMARY = "Firmware for OMAP5 ABE"
>>>>> +HOMEPAGE = "http://git.ti.com"
>>>>> +LICENSE = "BSD/GPLv2"
>>> This seems wrong. I thought the convention is "BSD | GPLv2" for either
>>> license or "BSD & GPLv2" if mixed license.
>>>
>>> If it is mixed then you probably need 2 entries in LIC_FILES_CHKSUM, one for
>>> each license.
>>>
>>> Also, does the GPLv2 use the "or later" clause such that it should be GPLv2+
>>> ?
>> This was already mentioned in one of the other patches.
>>
>>
>>>>> +LIC_FILES_CHKSUM =
>>>>> "file://COPYING;md5=923db086ed9463ab3215b24d87e05ec5"
>>>>> +
>>>>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>>>>> +COMPATIBLE_MACHINE = "omap5-evm"
>>>>> +
>>>>> +SRC_URI = "git://git.ti.com/glsdk/abefw-
>>>> omap4plus.git;protocol=git"
>> The git repo itself says omap4plus - why is it limited to OMAP5 only? Can it
>> be enabled for OMAP4/Pandaboard too?
>>
>>
>>> Not strictly required but PR = r0 here would help remind you to bump the PR
>>> in the future.
>>>
>>>>> +
>>>>> +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"
>>> I'm sure this has been discussed before, but do you see any confusion about
>>> this being omap5-evm only and yet the FW is omap4_abe_new?
>>>
>>>>> --
>>>>> 1.7.0.4
>>>> _______________________________________________
>>>> meta-ti mailing list
>>>> meta-ti@yoctoproject.org
>>>> https://lists.yoctoproject.org/listinfo/meta-ti
>>> _______________________________________________
>>> meta-ti mailing list
>>> meta-ti@yoctoproject.org
>>> https://lists.yoctoproject.org/listinfo/meta-ti
>> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
--
Regards,
Mrinmayee
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2013-10-18 10:29 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-12 6:43 [PATCHv2] abefw: Firmware for OMAP5 ABE Hingolikar, Mrinmayee
2013-07-12 13:11 ` Cooper Jr., Franklin
2013-07-22 4:13 ` Siddharth Heroor
2013-07-22 12:12 ` Cooper Jr., Franklin
2013-07-22 12:15 ` Maupin, Chase
2013-07-22 16:24 ` Denys Dmytriyenko
2013-07-19 6:01 ` Hingolikar, Mrinmayee
2013-07-19 11:54 ` Maupin, Chase
[not found] <1372849303-21311-1-git-send-email-mrinmayee@ti.com>
2013-08-21 11:48 ` Hingolikar, Mrinmayee
2013-08-21 12:52 ` Cooper Jr., Franklin
2013-08-21 15:31 ` Maupin, Chase
2013-08-21 15:49 ` Denys Dmytriyenko
2013-09-03 17:42 ` Denys Dmytriyenko
2013-10-18 10:29 ` Mrinmayee Hingolikar
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.