Re: [for master] [PATCH 1/3] tisdk-connectivity: Add support for omap-a15 based targets

[Karthik Ramanan <a0393906@ti.com>]

On 01-Oct-14 2:53 AM, Denys Dmytriyenko wrote:
> On Thu, Sep 11, 2014 at 06:29:51PM -0400, Denys Dmytriyenko wrote:
>> On Thu, Sep 04, 2014 at 09:04:12PM +0530, Karthik Ramanan wrote:
>>> Signed-off-by: Karthik Ramanan <a0393906@ti.com>
>>> ---
>>>   .../packagegroups/packagegroup-arago-tisdk-connectivity.bb       | 9 ++++++++-
>>>   1 file changed, 8 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb
>>> index f6c8660..7c5c565 100644
>>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb
>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb
>>> @@ -1,6 +1,6 @@
>>>   DESCRIPTION = "Task to install wireless packages into the target FS"
>>>   LICENSE = "MIT"
>>> -PR = "r29"
>>> +PR = "r30"
>>>
>>>   inherit packagegroup
>>>
>>> @@ -63,6 +63,13 @@ CONNECTIVITY_RDEPENDS_ti43x = "\
>>>       ${FIRMWARE_AND_DRIVERS} \
>>>   "
>>>
>>> +CONNECTIVITY_RDEPENDS_omap-a15 = "\
>>> +    ${WLAN_COMMON} \
>>> +    ${BLUETOOTH_STACK} \
>>> +    ${@base_contains('ARAGO_BRAND','glsdk','${FIRMWARE_AND_DRIVERS}','',d)} \
>>> +    bt-firmware \
>>
>> So, I see that wlan and by components are added to omap-a15 SOC, while
>> firmware and drivers are only enabled when built for GLSDK. I'm wondering if
>> that will affect negatively other platforms based on the same SOC, including
>> am57x... And wasn't BT support completely dropped by Sitara? Is it still the
>> case?
>
> Ping on this one - this will re-enable BT for the entire SOC family, that will
> affect DRA7xx as well AM57x. I'm not sure that is intended for other SDK
> products. Should those be conditional on ARAGO_BRAND set to glsdk as well?
>

I will take Yaniv's help here because I am not aware of what is required 
for the other SoC families.

Yaniv,

If you recall, during the previous GLSDK releases we made this change as 
conditional to GLSDK. There was also a check for amsdk.

Here is the link to the older version of this file:
http://arago-project.org/git/projects/meta-arago-glsdk.git?p=projects/meta-arago-glsdk.git;a=blob;f=meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb;h=00460683ce31ba5c32c60b94fc4aa94ed11b682b;hb=a0c23900703dd66bf28760cbdaf6bdec97c9bd72

Would you be able to comment/suggest on what is needed here?

Regards
Karthik


>
>>> +"
>>> +
>>>   RDEPENDS_${PN} = "\
>>>       ${CONNECTIVITY_RDEPENDS} \
>>>   "
>>> --
>>> 2.0.1
>>>
>>> _______________________________________________
>>> 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