Openembedded Devel Discussions
 help / color / mirror / Atom feed
* [PATCH 1/2] x-load_git: defined config for dm37x and am37x machines
@ 2010-07-29 12:26 Chase Maupin
  2010-07-29 12:26 ` [PATCH 2/2] x-load: add signGP to sourceipk Chase Maupin
  2010-07-29 12:47 ` [PATCH 1/2] x-load_git: defined config for dm37x and am37x machines Koen Kooi
  0 siblings, 2 replies; 6+ messages in thread
From: Chase Maupin @ 2010-07-29 12:26 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Chase Maupin

* Defined the XLOAD_MACHINE value for dm37x-evm and am37x-evm
  machine types.

Signed-off-by: Chase Maupin <chase.maupin@ti.com>
---
 recipes/x-load/x-load_git.bb |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/recipes/x-load/x-load_git.bb b/recipes/x-load/x-load_git.bb
index cbcbf17..a14d6cb 100644
--- a/recipes/x-load/x-load_git.bb
+++ b/recipes/x-load/x-load_git.bb
@@ -9,7 +9,7 @@ SRCREV_beagleboard = "1c9276af4d6a5b7014a7630a1abeddf3b3177563"
 
 PV = "1.42+${PR}+gitr${SRCREV}"
 PV_beagleboard = "1.44+${PR}+gitr${SRCREV}"
-PR ="r15"
+PR ="r16"
 PE = "1"
 
 SRC_URI = "git://gitorious.org/x-load-omap3/mainline.git;branch=master;protocol=git"
@@ -34,11 +34,13 @@ PV_omap3evm = "1.46+${PR}+gitr${SRCREV}"
 SRC_URI_dm37x-evm = "git://arago-project.org/git/projects/x-load-omap3.git;protocol=git"
 SRCREV_dm37x-evm = "fc6d5be15c703d21aef0ae0b8c02177721f0445f"
 PV_dm37x-evm = "1.46+${PR}+gitr${SRCREV}"
+XLOAD_MACHINE_dm37x-evm = "omap3evm_config"
 
 # TI PSP v1.46_OMAPPSP_03.00.01.06 (Tag is one commit different)
 SRC_URI_am37x-evm = "git://arago-project.org/git/projects/x-load-omap3.git;protocol=git"
 SRCREV_am37x-evm = "fc6d5be15c703d21aef0ae0b8c02177721f0445f"
 PV_am37x-evm = "1.46+${PR}+gitr${SRCREV}"
+XLOAD_MACHINE_dm37x-evm = "omap3evm_config"
 
 # TI PSP v1.46_OMAPPSP_03.00.01.06 (Tag is one commit different)
 SRC_URI_am3517-evm = "git://arago-project.org/git/projects/x-load-omap3.git;protocol=git"
-- 
1.7.0.4




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

* [PATCH 2/2] x-load: add signGP to sourceipk
  2010-07-29 12:26 [PATCH 1/2] x-load_git: defined config for dm37x and am37x machines Chase Maupin
@ 2010-07-29 12:26 ` Chase Maupin
  2010-07-29 12:48   ` Koen Kooi
  2010-07-29 15:11   ` John Willis
  2010-07-29 12:47 ` [PATCH 1/2] x-load_git: defined config for dm37x and am37x machines Koen Kooi
  1 sibling, 2 replies; 6+ messages in thread
From: Chase Maupin @ 2010-07-29 12:26 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Chase Maupin

* When creating a source ipk for the x-load program copy the
  signGP binary as well as this is needed when rebuilding.
* Bump the PR for x-load-omap3pandora recipe.

Signed-off-by: Chase Maupin <chase.maupin@ti.com>
---
 recipes/x-load/x-load-omap3pandora_git.bb |    2 +-
 recipes/x-load/x-load.inc                 |    5 +++++
 2 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/recipes/x-load/x-load-omap3pandora_git.bb b/recipes/x-load/x-load-omap3pandora_git.bb
index 05dfc20..e9e6e49 100644
--- a/recipes/x-load/x-load-omap3pandora_git.bb
+++ b/recipes/x-load/x-load-omap3pandora_git.bb
@@ -9,7 +9,7 @@ SRCREV = "b3646a83f9e67ec9a2de3182fd254557fd6ad9aa"
 PROVIDES = "x-load"
 
 PV = "1.42+${PR}+gitr${SRCREV}"
-PR ="r2"
+PR ="r3"
 PE = "1"
 
 SRC_URI = "git://git.openpandora.org/pandora-x-loader.git;branch=master;protocol=git"
diff --git a/recipes/x-load/x-load.inc b/recipes/x-load/x-load.inc
index 0699c01..e786ac1 100644
--- a/recipes/x-load/x-load.inc
+++ b/recipes/x-load/x-load.inc
@@ -38,6 +38,11 @@ do_install () {
 
 FILES_${PN} = "/boot"
 
+do_create_srcipk_prepend() {
+    mkdir -p ${WORKDIR}/sourceipk-tmp/psp/x-load/host-tools
+    cp `which signGP` ${WORKDIR}/sourceipk-tmp/psp/x-load/host-tools/
+}
+
 do_deploy () {
 	install -d ${DEPLOY_DIR_IMAGE}
 	install ${S}/x-load.bin.ift ${DEPLOY_DIR_IMAGE}/${XLOAD_IMAGE}
-- 
1.7.0.4




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

* Re: [PATCH 1/2] x-load_git: defined config for dm37x and am37x machines
  2010-07-29 12:26 [PATCH 1/2] x-load_git: defined config for dm37x and am37x machines Chase Maupin
  2010-07-29 12:26 ` [PATCH 2/2] x-load: add signGP to sourceipk Chase Maupin
@ 2010-07-29 12:47 ` Koen Kooi
  1 sibling, 0 replies; 6+ messages in thread
From: Koen Kooi @ 2010-07-29 12:47 UTC (permalink / raw)
  To: openembedded-devel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 29-07-10 14:26, Chase Maupin wrote:
> * Defined the XLOAD_MACHINE value for dm37x-evm and am37x-evm
>   machine types.
> 
> Signed-off-by: Chase Maupin <chase.maupin@ti.com>
> ---
>  recipes/x-load/x-load_git.bb |    4 +++-

NAK, please put those in the machine.confs, that would make it work for
xload releases as well.

regards,

Koen

>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/recipes/x-load/x-load_git.bb b/recipes/x-load/x-load_git.bb
> index cbcbf17..a14d6cb 100644
> --- a/recipes/x-load/x-load_git.bb
> +++ b/recipes/x-load/x-load_git.bb
> @@ -9,7 +9,7 @@ SRCREV_beagleboard = "1c9276af4d6a5b7014a7630a1abeddf3b3177563"
>  
>  PV = "1.42+${PR}+gitr${SRCREV}"
>  PV_beagleboard = "1.44+${PR}+gitr${SRCREV}"
> -PR ="r15"
> +PR ="r16"
>  PE = "1"
>  
>  SRC_URI = "git://gitorious.org/x-load-omap3/mainline.git;branch=master;protocol=git"
> @@ -34,11 +34,13 @@ PV_omap3evm = "1.46+${PR}+gitr${SRCREV}"
>  SRC_URI_dm37x-evm = "git://arago-project.org/git/projects/x-load-omap3.git;protocol=git"
>  SRCREV_dm37x-evm = "fc6d5be15c703d21aef0ae0b8c02177721f0445f"
>  PV_dm37x-evm = "1.46+${PR}+gitr${SRCREV}"
> +XLOAD_MACHINE_dm37x-evm = "omap3evm_config"
>  
>  # TI PSP v1.46_OMAPPSP_03.00.01.06 (Tag is one commit different)
>  SRC_URI_am37x-evm = "git://arago-project.org/git/projects/x-load-omap3.git;protocol=git"
>  SRCREV_am37x-evm = "fc6d5be15c703d21aef0ae0b8c02177721f0445f"
>  PV_am37x-evm = "1.46+${PR}+gitr${SRCREV}"
> +XLOAD_MACHINE_dm37x-evm = "omap3evm_config"
>  
>  # TI PSP v1.46_OMAPPSP_03.00.01.06 (Tag is one commit different)
>  SRC_URI_am3517-evm = "git://arago-project.org/git/projects/x-load-omap3.git;protocol=git"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFMUXh1MkyGM64RGpERAk+kAJ46GwRDNgiFhm6WTTPKu75PB9Db5ACfb/V7
1JZxc1cbt+oBtdygOlw0m54=
=OPfU
-----END PGP SIGNATURE-----




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

* Re: [PATCH 2/2] x-load: add signGP to sourceipk
  2010-07-29 12:26 ` [PATCH 2/2] x-load: add signGP to sourceipk Chase Maupin
@ 2010-07-29 12:48   ` Koen Kooi
  2010-07-29 15:11   ` John Willis
  1 sibling, 0 replies; 6+ messages in thread
From: Koen Kooi @ 2010-07-29 12:48 UTC (permalink / raw)
  To: openembedded-devel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 29-07-10 14:26, Chase Maupin wrote:
> * When creating a source ipk for the x-load program copy the
>   signGP binary as well as this is needed when rebuilding.
> * Bump the PR for x-load-omap3pandora recipe.

NAK. If you want to have signgp, bitbake signgp-native. Including
'random' binaries from other recipes in packages is a no-no.

regards,

Koen


> 
> Signed-off-by: Chase Maupin <chase.maupin@ti.com>
> ---
>  recipes/x-load/x-load-omap3pandora_git.bb |    2 +-
>  recipes/x-load/x-load.inc                 |    5 +++++
>  2 files changed, 6 insertions(+), 1 deletions(-)
> 
> diff --git a/recipes/x-load/x-load-omap3pandora_git.bb b/recipes/x-load/x-load-omap3pandora_git.bb
> index 05dfc20..e9e6e49 100644
> --- a/recipes/x-load/x-load-omap3pandora_git.bb
> +++ b/recipes/x-load/x-load-omap3pandora_git.bb
> @@ -9,7 +9,7 @@ SRCREV = "b3646a83f9e67ec9a2de3182fd254557fd6ad9aa"
>  PROVIDES = "x-load"
>  
>  PV = "1.42+${PR}+gitr${SRCREV}"
> -PR ="r2"
> +PR ="r3"
>  PE = "1"
>  
>  SRC_URI = "git://git.openpandora.org/pandora-x-loader.git;branch=master;protocol=git"
> diff --git a/recipes/x-load/x-load.inc b/recipes/x-load/x-load.inc
> index 0699c01..e786ac1 100644
> --- a/recipes/x-load/x-load.inc
> +++ b/recipes/x-load/x-load.inc
> @@ -38,6 +38,11 @@ do_install () {
>  
>  FILES_${PN} = "/boot"
>  
> +do_create_srcipk_prepend() {
> +    mkdir -p ${WORKDIR}/sourceipk-tmp/psp/x-load/host-tools
> +    cp `which signGP` ${WORKDIR}/sourceipk-tmp/psp/x-load/host-tools/
> +}
> +
>  do_deploy () {
>  	install -d ${DEPLOY_DIR_IMAGE}
>  	install ${S}/x-load.bin.ift ${DEPLOY_DIR_IMAGE}/${XLOAD_IMAGE}

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFMUXitMkyGM64RGpERAhUYAKC6fLMHo4tdV74afqFSzzfW7JZ0XgCgrNnS
EbRbIAK7+yFW6gldUT+ETqs=
=JZx8
-----END PGP SIGNATURE-----




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

* Re: [PATCH 2/2] x-load: add signGP to sourceipk
  2010-07-29 12:26 ` [PATCH 2/2] x-load: add signGP to sourceipk Chase Maupin
  2010-07-29 12:48   ` Koen Kooi
@ 2010-07-29 15:11   ` John Willis
  2010-07-29 17:41     ` Koen Kooi
  1 sibling, 1 reply; 6+ messages in thread
From: John Willis @ 2010-07-29 15:11 UTC (permalink / raw)
  To: openembedded-devel; +Cc: 'Chase Maupin'

> * When creating a source ipk for the x-load program copy the
>   signGP binary as well as this is needed when rebuilding.
> * Bump the PR for x-load-omap3pandora recipe.
> 
> Signed-off-by: Chase Maupin <chase.maupin@ti.com>

Nack, this seems totally pointless, signGP is in the tree (signgp-native) so
is it not better to bring that in?

Out of interest, what is the problem you see with the current setup, the
recipes as they stand will give people signed and unsigned X-Loader binaries
for just about all purposes you could want them on the OMAP3. Or is this
patch aimed at using source IPK's for use in another non-OE SDK?

Shipping binaries in a source IPK seems very wrong regardless of the need,
it's not really source at that point ;-).

Regards,

John





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

* Re: [PATCH 2/2] x-load: add signGP to sourceipk
  2010-07-29 15:11   ` John Willis
@ 2010-07-29 17:41     ` Koen Kooi
  0 siblings, 0 replies; 6+ messages in thread
From: Koen Kooi @ 2010-07-29 17:41 UTC (permalink / raw)
  To: openembedded-devel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 29-07-10 17:11, John Willis wrote:
>> * When creating a source ipk for the x-load program copy the
>>   signGP binary as well as this is needed when rebuilding.
>> * Bump the PR for x-load-omap3pandora recipe.
>>
>> Signed-off-by: Chase Maupin <chase.maupin@ti.com>
> 
> Nack, this seems totally pointless, signGP is in the tree (signgp-native) so
> is it not better to bring that in?
> 
> Out of interest, what is the problem you see with the current setup, the
> recipes as they stand will give people signed and unsigned X-Loader binaries
> for just about all purposes you could want them on the OMAP3. Or is this
> patch aimed at using source IPK's for use in another non-OE SDK?
> 
> Shipping binaries in a source IPK seems very wrong regardless of the need,
> it's not really source at that point ;-).

What Chase actually needed was an -sdk package, so:
http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=962d52ae8358d8fd11ae8a7054fa785d102cf274

regards,

Koen
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFMUb1iMkyGM64RGpERArLhAJoD27ERUKmy2OrmVZNK5kzEzVWi8QCgmIpZ
s2B/iRmjYI0hrFSLpiz9SBY=
=4MXC
-----END PGP SIGNATURE-----




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

end of thread, other threads:[~2010-07-29 17:42 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-29 12:26 [PATCH 1/2] x-load_git: defined config for dm37x and am37x machines Chase Maupin
2010-07-29 12:26 ` [PATCH 2/2] x-load: add signGP to sourceipk Chase Maupin
2010-07-29 12:48   ` Koen Kooi
2010-07-29 15:11   ` John Willis
2010-07-29 17:41     ` Koen Kooi
2010-07-29 12:47 ` [PATCH 1/2] x-load_git: defined config for dm37x and am37x machines Koen Kooi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox