All of lore.kernel.org
 help / color / mirror / Atom feed
* [dunfell/master PATCH] packagegroup: Add ti-sci-fw-src to tisdk-addons-sdk-host
@ 2020-05-15 18:44 prao
  2020-05-15 18:46 ` Denys Dmytriyenko
  0 siblings, 1 reply; 6+ messages in thread
From: prao @ 2020-05-15 18:44 UTC (permalink / raw)
  To: denys; +Cc: meta-arago, Praveen Rao

From: Praveen Rao <prao@ti.com>

Add source package for k3 image gen in the tisdk-addons-sdk-host

Signed-off-by: Praveen Rao <prao@ti.com>
---
 .../packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb    | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
index 6dcb253c..32d6c98b 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
@@ -38,6 +38,7 @@ UTILS_append_k2g = " \
 
 UTILS_append_k3 = " \
     jailhouse-src \
+    ti-sci-fw-src \
     arm-trusted-firmware-src \
 "
 
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [dunfell/master PATCH] packagegroup: Add ti-sci-fw-src to tisdk-addons-sdk-host
  2020-05-15 18:44 [dunfell/master PATCH] packagegroup: Add ti-sci-fw-src to tisdk-addons-sdk-host prao
@ 2020-05-15 18:46 ` Denys Dmytriyenko
  2020-05-15 19:09   ` Nikhil Devshatwar
  0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2020-05-15 18:46 UTC (permalink / raw)
  To: prao; +Cc: meta-arago

Have you even tried it yourself?


On Sat, May 16, 2020 at 12:14:39AM +0530, prao@ti.com wrote:
> From: Praveen Rao <prao@ti.com>
> 
> Add source package for k3 image gen in the tisdk-addons-sdk-host
> 
> Signed-off-by: Praveen Rao <prao@ti.com>
> ---
>  .../packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb    | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> index 6dcb253c..32d6c98b 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> @@ -38,6 +38,7 @@ UTILS_append_k2g = " \
>  
>  UTILS_append_k3 = " \
>      jailhouse-src \
> +    ti-sci-fw-src \
>      arm-trusted-firmware-src \
>  "
>  
> -- 
> 2.17.1
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dunfell/master PATCH] packagegroup: Add ti-sci-fw-src to tisdk-addons-sdk-host
  2020-05-15 18:46 ` Denys Dmytriyenko
@ 2020-05-15 19:09   ` Nikhil Devshatwar
  2020-05-15 19:31     ` Denys Dmytriyenko
  2020-05-15 20:01     ` Praveen Rao
  0 siblings, 2 replies; 6+ messages in thread
From: Nikhil Devshatwar @ 2020-05-15 19:09 UTC (permalink / raw)
  To: Denys Dmytriyenko, prao; +Cc: meta-arago

[-- Attachment #1: Type: text/plain, Size: 1638 bytes --]

Hi Denys,

On 16/05/20 12:16 am, Denys Dmytriyenko wrote:
> Have you even tried it yourself?
>
Praveen has been debugging some issue where the source package for 
ti-sci-fw is not included in the installer.
I asked him to check in the packagegroup-arago-tisdk-addons-sdk-host.bb 
and he found it was missing

Is this dropped for some reason?
Also, now shall we rename SRCIPK_INSTALL_DIR_pn-ti-sci-fw_k3 to 
board-support/k3-image-gen-${PV} ?

Regards,
Nikhil D
> On Sat, May 16, 2020 at 12:14:39AM +0530, prao@ti.com wrote:
>> From: Praveen Rao <prao@ti.com>
>>
>> Add source package for k3 image gen in the tisdk-addons-sdk-host
>>
>> Signed-off-by: Praveen Rao <prao@ti.com>
>> ---
>>   .../packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb    | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
>> index 6dcb253c..32d6c98b 100644
>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
>> @@ -38,6 +38,7 @@ UTILS_append_k2g = " \
>>   
>>   UTILS_append_k3 = " \
>>       jailhouse-src \
>> +    ti-sci-fw-src \
>>       arm-trusted-firmware-src \
>>   "
>>   
>> -- 
>> 2.17.1
>>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


[-- Attachment #2: Type: text/html, Size: 2662 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dunfell/master PATCH] packagegroup: Add ti-sci-fw-src to tisdk-addons-sdk-host
  2020-05-15 19:09   ` Nikhil Devshatwar
@ 2020-05-15 19:31     ` Denys Dmytriyenko
  2020-05-15 20:01     ` Praveen Rao
  1 sibling, 0 replies; 6+ messages in thread
From: Denys Dmytriyenko @ 2020-05-15 19:31 UTC (permalink / raw)
  To: Nikhil Devshatwar; +Cc: meta-arago, prao

Nikhil,

Even w/o going into all the details and reasoning, my question is very simple: 

has this patch been tested locally by the submitter?

-- 
Denys


On Sat, May 16, 2020 at 12:39:23AM +0530, Nikhil Devshatwar wrote:
> Hi Denys,
> 
> On 16/05/20 12:16 am, Denys Dmytriyenko wrote:
> >Have you even tried it yourself?
> >
> Praveen has been debugging some issue where the source package for ti-sci-fw
> is not included in the installer.
> I asked him to check in the packagegroup-arago-tisdk-addons-sdk-host.bb and
> he found it was missing
> 
> Is this dropped for some reason?
> Also, now shall we rename SRCIPK_INSTALL_DIR_pn-ti-sci-fw_k3 to
> board-support/k3-image-gen-${PV} ?
> 
> Regards,
> Nikhil D
> >On Sat, May 16, 2020 at 12:14:39AM +0530, prao@ti.com wrote:
> >>From: Praveen Rao <prao@ti.com>
> >>
> >>Add source package for k3 image gen in the tisdk-addons-sdk-host
> >>
> >>Signed-off-by: Praveen Rao <prao@ti.com>
> >>---
> >>  .../packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb    | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >>diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> >>index 6dcb253c..32d6c98b 100644
> >>--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> >>+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> >>@@ -38,6 +38,7 @@ UTILS_append_k2g = " \
> >>  UTILS_append_k3 = " \
> >>      jailhouse-src \
> >>+    ti-sci-fw-src \
> >>      arm-trusted-firmware-src \
> >>  "
> >>-- 
> >>2.17.1
> >>
> >_______________________________________________
> >meta-arago mailing list
> >meta-arago@arago-project.org
> >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dunfell/master PATCH] packagegroup: Add ti-sci-fw-src to tisdk-addons-sdk-host
  2020-05-15 19:09   ` Nikhil Devshatwar
  2020-05-15 19:31     ` Denys Dmytriyenko
@ 2020-05-15 20:01     ` Praveen Rao
  2020-05-15 20:02       ` Denys Dmytriyenko
  1 sibling, 1 reply; 6+ messages in thread
From: Praveen Rao @ 2020-05-15 20:01 UTC (permalink / raw)
  To: Nikhil Devshatwar, Denys Dmytriyenko; +Cc: meta-arago

Pls ignore this patch. Encountered build error with it.


On 5/15/20 2:09 PM, Nikhil Devshatwar wrote:
> Hi Denys,
> 
> On 16/05/20 12:16 am, Denys Dmytriyenko wrote:
>> Have you even tried it yourself?
>>
> Praveen has been debugging some issue where the source package for 
> ti-sci-fw is not included in the installer.
> I asked him to check in the packagegroup-arago-tisdk-addons-sdk-host.bb 
> and he found it was missing
> 
> Is this dropped for some reason?
> Also, now shall we rename SRCIPK_INSTALL_DIR_pn-ti-sci-fw_k3 to 
> board-support/k3-image-gen-${PV} ?
> 
> Regards,
> Nikhil D
>> On Sat, May 16, 2020 at 12:14:39AM +0530,prao@ti.com  wrote:
>>> From: Praveen Rao<prao@ti.com>
>>>
>>> Add source package for k3 image gen in the tisdk-addons-sdk-host
>>>
>>> Signed-off-by: Praveen Rao<prao@ti.com>
>>> ---
>>>   .../packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb    | 1 +
>>>   1 file changed, 1 insertion(+)
>>>
>>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
>>> index 6dcb253c..32d6c98b 100644
>>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
>>> @@ -38,6 +38,7 @@ UTILS_append_k2g = " \
>>>   
>>>   UTILS_append_k3 = " \
>>>       jailhouse-src \
>>> +    ti-sci-fw-src \
>>>       arm-trusted-firmware-src \
>>>   "
>>>   
>>> -- 
>>> 2.17.1
>>>
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dunfell/master PATCH] packagegroup: Add ti-sci-fw-src to tisdk-addons-sdk-host
  2020-05-15 20:01     ` Praveen Rao
@ 2020-05-15 20:02       ` Denys Dmytriyenko
  0 siblings, 0 replies; 6+ messages in thread
From: Denys Dmytriyenko @ 2020-05-15 20:02 UTC (permalink / raw)
  To: Praveen Rao; +Cc: meta-arago

Thank you for confirming the issue. Please see commit 
8caaed4e6dd910a8d69e10bf9f3d879159d887bf for details.


On Fri, May 15, 2020 at 03:01:42PM -0500, Praveen Rao wrote:
> Pls ignore this patch. Encountered build error with it.
> 
> 
> On 5/15/20 2:09 PM, Nikhil Devshatwar wrote:
> >Hi Denys,
> >
> >On 16/05/20 12:16 am, Denys Dmytriyenko wrote:
> >>Have you even tried it yourself?
> >>
> >Praveen has been debugging some issue where the source package for
> >ti-sci-fw is not included in the installer.
> >I asked him to check in the packagegroup-arago-tisdk-addons-sdk-host.bb
> >and he found it was missing
> >
> >Is this dropped for some reason?
> >Also, now shall we rename SRCIPK_INSTALL_DIR_pn-ti-sci-fw_k3 to
> >board-support/k3-image-gen-${PV} ?
> >
> >Regards,
> >Nikhil D
> >>On Sat, May 16, 2020 at 12:14:39AM +0530,prao@ti.com  wrote:
> >>>From: Praveen Rao<prao@ti.com>
> >>>
> >>>Add source package for k3 image gen in the tisdk-addons-sdk-host
> >>>
> >>>Signed-off-by: Praveen Rao<prao@ti.com>
> >>>---
> >>>  .../packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb    | 1 +
> >>>  1 file changed, 1 insertion(+)
> >>>
> >>>diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> >>>index 6dcb253c..32d6c98b 100644
> >>>--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> >>>+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
> >>>@@ -38,6 +38,7 @@ UTILS_append_k2g = " \
> >>>  UTILS_append_k3 = " \
> >>>      jailhouse-src \
> >>>+    ti-sci-fw-src \
> >>>      arm-trusted-firmware-src \
> >>>  "
> >>>-- 
> >>>2.17.1
> >>>
> >>_______________________________________________
> >>meta-arago mailing list
> >>meta-arago@arago-project.org
> >>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-05-15 20:05 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-05-15 18:44 [dunfell/master PATCH] packagegroup: Add ti-sci-fw-src to tisdk-addons-sdk-host prao
2020-05-15 18:46 ` Denys Dmytriyenko
2020-05-15 19:09   ` Nikhil Devshatwar
2020-05-15 19:31     ` Denys Dmytriyenko
2020-05-15 20:01     ` Praveen Rao
2020-05-15 20:02       ` 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.