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

[Karthik Ramanan <a0393906@ti.com>]

Yaniv,

Could you please comment on the below question?

Regards
Karthik


On 01-Oct-14 12:34 PM, Karthik Ramanan wrote:
> 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
>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago