All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
@ 2015-02-11  9:19 Werner Grift
  2015-02-13  0:37 ` Otavio Salvador
  0 siblings, 1 reply; 9+ messages in thread
From: Werner Grift @ 2015-02-11  9:19 UTC (permalink / raw)
  To: meta-freescale; +Cc: Werner Grift

Signed-off-by: Werner Grift <sky.captin@gmail.com>
---
 classes/image_types_fsl.bbclass | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/classes/image_types_fsl.bbclass b/classes/image_types_fsl.bbclass
index 53f8767..91c418a 100644
--- a/classes/image_types_fsl.bbclass
+++ b/classes/image_types_fsl.bbclass
@@ -168,9 +168,9 @@ generate_imx_sdcard () {
 	if test -n "${KERNEL_DEVICETREE}"; then
 		for DTS_FILE in ${KERNEL_DEVICETREE}; do
 			DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print $1}'`
-			if [ -e "${KERNEL_IMAGETYPE}-${DTS_BASE_NAME}.dtb" ]; then
-				kernel_bin="`readlink ${KERNEL_IMAGETYPE}-${MACHINE}.bin`"
-				kernel_bin_for_dtb="`readlink ${KERNEL_IMAGETYPE}-${DTS_BASE_NAME}.dtb | sed "s,$DTS_BASE_NAME,${MACHINE},g;s,\.dtb$,.bin,g"`"
+			if [ -e "${DEPLOY_DIR_IMAGE}/${KERNEL_IMAGETYPE}-${DTS_BASE_NAME}.dtb" ]; then
+				kernel_bin="`readlink ${DEPLOY_DIR_IMAGE}/${KERNEL_IMAGETYPE}-${MACHINE}.bin`"
+				kernel_bin_for_dtb="`readlink ${DEPLOY_DIR_IMAGE}/${KERNEL_IMAGETYPE}-${DTS_BASE_NAME}.dtb | sed "s,$DTS_BASE_NAME,${MACHINE},g;s,\.dtb$,.bin,g"`"
 				if [ $kernel_bin = $kernel_bin_for_dtb ]; then
 					mcopy -i ${WORKDIR}/boot.img -s ${DEPLOY_DIR_IMAGE}/${KERNEL_IMAGETYPE}-${DTS_BASE_NAME}.dtb ::/${DTS_BASE_NAME}.dtb
 				fi
-- 
2.0.0



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

* Re: [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
  2015-02-11  9:19 [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree Werner Grift
@ 2015-02-13  0:37 ` Otavio Salvador
  2015-02-17  7:45   ` Werner Grift
  0 siblings, 1 reply; 9+ messages in thread
From: Otavio Salvador @ 2015-02-13  0:37 UTC (permalink / raw)
  To: Werner Grift; +Cc: meta-freescale

Hello Werner,

On Wed, Feb 11, 2015 at 7:19 AM, Werner Grift <sky.captin@gmail.com> wrote:
> Signed-off-by: Werner Grift <sky.captin@gmail.com>

I am wondering how this has been working previously. Any idea what
changed which causes the behaviour change?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


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

* Re: [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
  2015-02-13  0:37 ` Otavio Salvador
@ 2015-02-17  7:45   ` Werner Grift
  2015-02-17  7:47     ` Werner Grift
  2015-02-17 13:08     ` Otavio Salvador
  0 siblings, 2 replies; 9+ messages in thread
From: Werner Grift @ 2015-02-17  7:45 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: meta-freescale

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

Hi Otavio.

I received the recipes (and a working version of it on our build server)
from a colleague that only introduced the following patch
<http://c7b9611ad0ead17a624fc73a60c321ff249c2214> to oe at
rev c7b9611ad0ead17a624fc73a60c321ff249c2214. The rest of the system was
untouched.

I made a fresh build system on my development machine. My build ran a
couple of times without above mentioned patch, but broke because it was not
applied. After applying the patch my build worked, but the copy device tree
issue remained. (But not on the build server) I then ran ./oebb.sh update
to make sure everything is cozy (which removed the patch mentioned above
without me realizing) and miraculously the build continued to succeed, but
the device tree issue remained. I suspect I broke sstate by doing this. My
sstate is not shared between dev and build.

Therefor, I suspect that the issue was generated in house. Regardless, the
rest of create_image.sdcard does not make assumptions as to pwd, so I
figured the patch is needed for completeness.

Regards,

Werner.


On Fri, Feb 13, 2015 at 2:37 AM, Otavio Salvador <otavio@ossystems.com.br>
wrote:

> Hello Werner,
>
> On Wed, Feb 11, 2015 at 7:19 AM, Werner Grift <sky.captin@gmail.com>
> wrote:
> > Signed-off-by: Werner Grift <sky.captin@gmail.com>
>
> I am wondering how this has been working previously. Any idea what
> changed which causes the behaviour change?
>
> --
> Otavio Salvador                             O.S. Systems
> http://www.ossystems.com.br        http://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750
>

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

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

* Re: [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
  2015-02-17  7:45   ` Werner Grift
@ 2015-02-17  7:47     ` Werner Grift
  2015-02-17 13:08     ` Otavio Salvador
  1 sibling, 0 replies; 9+ messages in thread
From: Werner Grift @ 2015-02-17  7:47 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: meta-freescale

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

I screwed up the patch link in my previous email:
http://hastebin.com/yaxisokuqo.py

On Tue, Feb 17, 2015 at 9:45 AM, Werner Grift <sky.captin@gmail.com> wrote:

> Hi Otavio.
>
> I received the recipes (and a working version of it on our build server)
> from a colleague that only introduced the following patch
> <http://c7b9611ad0ead17a624fc73a60c321ff249c2214> to oe at
> rev c7b9611ad0ead17a624fc73a60c321ff249c2214. The rest of the system was
> untouched.
>
> I made a fresh build system on my development machine. My build ran a
> couple of times without above mentioned patch, but broke because it was not
> applied. After applying the patch my build worked, but the copy device tree
> issue remained. (But not on the build server) I then ran ./oebb.sh update
> to make sure everything is cozy (which removed the patch mentioned above
> without me realizing) and miraculously the build continued to succeed, but
> the device tree issue remained. I suspect I broke sstate by doing this. My
> sstate is not shared between dev and build.
>
> Therefor, I suspect that the issue was generated in house. Regardless,
> the rest of create_image.sdcard does not make assumptions as to pwd, so I
> figured the patch is needed for completeness.
>
> Regards,
>
> Werner.
>
>
> On Fri, Feb 13, 2015 at 2:37 AM, Otavio Salvador <otavio@ossystems.com.br>
> wrote:
>
>> Hello Werner,
>>
>> On Wed, Feb 11, 2015 at 7:19 AM, Werner Grift <sky.captin@gmail.com>
>> wrote:
>> > Signed-off-by: Werner Grift <sky.captin@gmail.com>
>>
>> I am wondering how this has been working previously. Any idea what
>> changed which causes the behaviour change?
>>
>> --
>> Otavio Salvador                             O.S. Systems
>> http://www.ossystems.com.br        http://code.ossystems.com.br
>> Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750
>>
>
>

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

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

* Re: [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
  2015-02-17  7:45   ` Werner Grift
  2015-02-17  7:47     ` Werner Grift
@ 2015-02-17 13:08     ` Otavio Salvador
  2015-02-17 15:09       ` Werner Grift
  1 sibling, 1 reply; 9+ messages in thread
From: Otavio Salvador @ 2015-02-17 13:08 UTC (permalink / raw)
  To: Werner Grift; +Cc: meta-freescale

On Tue, Feb 17, 2015 at 5:45 AM, Werner Grift <sky.captin@gmail.com> wrote:
...
> without me realizing) and miraculously the build continued to succeed, but
> the device tree issue remained. I suspect I broke sstate by doing this. My
> sstate is not shared between dev and build.
...

Which device tree issue?

We need to understand where the issue come from as this is a code used
by all people using meta-fsl-arm and I'd like to avoid to apply
something we don't fully understand the need.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


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

* Re: [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
  2015-02-17 13:08     ` Otavio Salvador
@ 2015-02-17 15:09       ` Werner Grift
  2015-02-17 16:18         ` Otavio Salvador
  0 siblings, 1 reply; 9+ messages in thread
From: Werner Grift @ 2015-02-17 15:09 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: meta-freescale

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

The device tree issue I refer too is the device tree file that is not being
copied to the boot partition because the generated script
create_image.sdcard contains if statements that assume that the current
working directory is ${DEPLOY_DIR_IMAGE} which it is not. In my case it was
${OE_BASE} which makes sense since it is where bitbake is called from.

On Tue, Feb 17, 2015 at 3:08 PM, Otavio Salvador <otavio@ossystems.com.br>
wrote:

> On Tue, Feb 17, 2015 at 5:45 AM, Werner Grift <sky.captin@gmail.com>
> wrote:
> ...
> > without me realizing) and miraculously the build continued to succeed,
> but
> > the device tree issue remained. I suspect I broke sstate by doing this.
> My
> > sstate is not shared between dev and build.
> ...
>
> Which device tree issue?
>
> We need to understand where the issue come from as this is a code used
> by all people using meta-fsl-arm and I'd like to avoid to apply
> something we don't fully understand the need.
>
> --
> Otavio Salvador                             O.S. Systems
> http://www.ossystems.com.br        http://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750
>

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

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

* Re: [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
  2015-02-17 15:09       ` Werner Grift
@ 2015-02-17 16:18         ` Otavio Salvador
  2015-02-17 17:30           ` Werner Grift
  0 siblings, 1 reply; 9+ messages in thread
From: Otavio Salvador @ 2015-02-17 16:18 UTC (permalink / raw)
  To: Werner Grift; +Cc: meta-freescale

On Tue, Feb 17, 2015 at 1:09 PM, Werner Grift <sky.captin@gmail.com> wrote:
> The device tree issue I refer too is the device tree file that is not being
> copied to the boot partition because the generated script
> create_image.sdcard contains if statements that assume that the current
> working directory is ${DEPLOY_DIR_IMAGE} which it is not. In my case it was
> ${OE_BASE} which makes sense since it is where bitbake is called from.

How is your platform setup?

FSL Community BSP usually looks like:

.
├── build
│   └── conf
├── README
├── setup-environment
└── sources
    ├── base
    ├── Documentation
    ├── meta-fsl-arm
    ├── meta-fsl-arm-extra
    ├── meta-fsl-demos
    ├── meta-openembedded
    └── poky

Is yours same layout?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


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

* Re: [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
  2015-02-17 16:18         ` Otavio Salvador
@ 2015-02-17 17:30           ` Werner Grift
  2015-02-17 17:34             ` Otavio Salvador
  0 siblings, 1 reply; 9+ messages in thread
From: Werner Grift @ 2015-02-17 17:30 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: meta-freescale

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

We use the Angstrom distro 1.7

Our structure does not look like that. Whats this Yocto business?

Looks like script is being called from the wrong directory then.



On Tue, Feb 17, 2015 at 6:18 PM, Otavio Salvador <otavio@ossystems.com.br>
wrote:

> On Tue, Feb 17, 2015 at 1:09 PM, Werner Grift <sky.captin@gmail.com>
> wrote:
> > The device tree issue I refer too is the device tree file that is not
> being
> > copied to the boot partition because the generated script
> > create_image.sdcard contains if statements that assume that the current
> > working directory is ${DEPLOY_DIR_IMAGE} which it is not. In my case it
> was
> > ${OE_BASE} which makes sense since it is where bitbake is called from.
>
> How is your platform setup?
>
> FSL Community BSP usually looks like:
>
> .
> ├── build
> │   └── conf
> ├── README
> ├── setup-environment
> └── sources
>     ├── base
>     ├── Documentation
>     ├── meta-fsl-arm
>     ├── meta-fsl-arm-extra
>     ├── meta-fsl-demos
>     ├── meta-openembedded
>     └── poky
>
> Is yours same layout?
>
> --
> Otavio Salvador                             O.S. Systems
> http://www.ossystems.com.br        http://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750
>

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

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

* Re: [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree
  2015-02-17 17:30           ` Werner Grift
@ 2015-02-17 17:34             ` Otavio Salvador
  0 siblings, 0 replies; 9+ messages in thread
From: Otavio Salvador @ 2015-02-17 17:34 UTC (permalink / raw)
  To: Werner Grift; +Cc: meta-freescale

On Tue, Feb 17, 2015 at 3:30 PM, Werner Grift <sky.captin@gmail.com> wrote:
> We use the Angstrom distro 1.7

Ok.

> Our structure does not look like that. Whats this Yocto business?

This is the FSL Community BSP. You can find more information about it
in http://freescale.github.io

> Looks like script is being called from the wrong directory then.

Nothing wrong. The system must to behave right and to support all
different type of platform setups.

Can you give me (might be private - if preferable) more information
about your platform layout?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


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

end of thread, other threads:[~2015-02-17 17:34 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-11  9:19 [meta-fsl-arm][PATCH 73/73] Added absolute paths to checks when copying the device tree Werner Grift
2015-02-13  0:37 ` Otavio Salvador
2015-02-17  7:45   ` Werner Grift
2015-02-17  7:47     ` Werner Grift
2015-02-17 13:08     ` Otavio Salvador
2015-02-17 15:09       ` Werner Grift
2015-02-17 16:18         ` Otavio Salvador
2015-02-17 17:30           ` Werner Grift
2015-02-17 17:34             ` Otavio Salvador

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.