[CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Otavio Salvador]

Hello folks,

The board is fetching the wrong U-Boot script for boot and hance
making it not usable.

Something like:

diff --git a/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
b/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
index 1ffca84..2a992c0 100644
--- a/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
+++ b/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
@@ -13,6 +13,7 @@ S = "${WORKDIR}/git"
 inherit deploy

 BOOTSCRIPT ?= "${S}/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt"
+BOOTSCRIPT_nitrogen6x-lite ?=
"${S}/board/boundary/nit6xlite/6x_bootscript-yocto-3.14.txt"

 UPGRADESCRIPT = "${S}/board/boundary/nitrogen6x/6x_upgrade.txt"

Is needed but the repository does not offer the script for use.

-- 
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

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Ian Coolidge]

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

Hey Otavio,

The current setup is actually correct. It's a bit cryptic in the path used,
implying it only works on the nitrogen6x board. However, our bootscript is
universal across all our boards now and will work, assuming you have the
latest u-boot installed.

I also just tripled checked core-image-sato against the nitrogen6x-lite
board, the bootscript is handling as it should.

Maybe we could add a comment there mentioning the universal nature of the
bootscript?
Thanks,
-Ian Coolidge

On Fri, Nov 6, 2015 at 4:53 AM, Otavio Salvador <
otavio.salvador@ossystems.com.br> wrote:

> Hello folks,
>
> The board is fetching the wrong U-Boot script for boot and hance
> making it not usable.
>
> Something like:
>
> diff --git a/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
> b/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
> index 1ffca84..2a992c0 100644
> --- a/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
> +++ b/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
> @@ -13,6 +13,7 @@ S = "${WORKDIR}/git"
>  inherit deploy
>
>  BOOTSCRIPT ?=
> "${S}/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt"
> +BOOTSCRIPT_nitrogen6x-lite ?=
> "${S}/board/boundary/nit6xlite/6x_bootscript-yocto-3.14.txt"
>
>  UPGRADESCRIPT = "${S}/board/boundary/nitrogen6x/6x_upgrade.txt"
>
> Is needed but the repository does not offer the script for use.
>
> --
> 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: 2778 bytes --]

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Otavio Salvador]

On Fri, Nov 6, 2015 at 4:11 PM, Ian Coolidge <ian@boundarydevices.com> wrote:
> The current setup is actually correct. It's a bit cryptic in the path used,
> implying it only works on the nitrogen6x board. However, our bootscript is
> universal across all our boards now and will work, assuming you have the
> latest u-boot installed.
>
> I also just tripled checked core-image-sato against the nitrogen6x-lite
> board, the bootscript is handling as it should.
>
> Maybe we could add a comment there mentioning the universal nature of the
> bootscript?

On the Nitrogen Lite we have, it did not work at all. It seems it does
not find out the board to use when loading the device tree file.
Remember we are using the 3.14 kernel.

-- 
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

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Ian Coolidge]

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

Have you updated the u-boot on the board? We wrote a blog about it:
https://boundarydevices.com/compiling-latest-u-boot-for-i-mx6-2015-edition/

3.14 kernel requires the new bootscript for our boards. The new bootscript
also requires the new u-boot to be installed on the board.

I would have sworn I made a note of this in the patches I sent in, but it
appears I forgot to mention that.

On Fri, Nov 6, 2015 at 11:27 AM, Otavio Salvador <
otavio.salvador@ossystems.com.br> wrote:

> On Fri, Nov 6, 2015 at 4:11 PM, Ian Coolidge <ian@boundarydevices.com>
> wrote:
> > The current setup is actually correct. It's a bit cryptic in the path
> used,
> > implying it only works on the nitrogen6x board. However, our bootscript
> is
> > universal across all our boards now and will work, assuming you have the
> > latest u-boot installed.
> >
> > I also just tripled checked core-image-sato against the nitrogen6x-lite
> > board, the bootscript is handling as it should.
> >
> > Maybe we could add a comment there mentioning the universal nature of the
> > bootscript?
>
> On the Nitrogen Lite we have, it did not work at all. It seems it does
> not find out the board to use when loading the device tree file.
> Remember we are using the 3.14 kernel.
>
> --
> 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: 2401 bytes --]

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Otavio Salvador]

On Fri, Nov 6, 2015 at 6:01 PM, Ian Coolidge <ian@boundarydevices.com> wrote:
> Have you updated the u-boot on the board? We wrote a blog about it:
> https://boundarydevices.com/compiling-latest-u-boot-for-i-mx6-2015-edition/
>
> 3.14 kernel requires the new bootscript for our boards. The new bootscript
> also requires the new u-boot to be installed on the board.
>
> I would have sworn I made a note of this in the patches I sent in, but it
> appears I forgot to mention that.

Ok; will try that on Monday however be prepared for people to report
errors ;-) To be honest, I would make a backward compatible script
which checks the version and ask for the upgrade in case it is old so
user does not get an ugly error on their screen.

-- 
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

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Ian Coolidge]

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

>
> Ok; will try that on Monday however be prepared for people to report
> errors ;-) To be honest, I would make a backward compatible script
> which checks the version and ask for the upgrade in case it is old so
> user does not get an ugly error on their screen.


Did you see this message as u-boot started?

https://github.com/boundarydevices/u-boot-imx6/blob/boundary-v2015.07/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt#L89-L93

On Fri, Nov 6, 2015 at 12:26 PM, Otavio Salvador <
otavio.salvador@ossystems.com.br> wrote:

> On Fri, Nov 6, 2015 at 6:01 PM, Ian Coolidge <ian@boundarydevices.com>
> wrote:
> > Have you updated the u-boot on the board? We wrote a blog about it:
> >
> https://boundarydevices.com/compiling-latest-u-boot-for-i-mx6-2015-edition/
> >
> > 3.14 kernel requires the new bootscript for our boards. The new
> bootscript
> > also requires the new u-boot to be installed on the board.
> >
> > I would have sworn I made a note of this in the patches I sent in, but it
> > appears I forgot to mention that.
>
> Ok; will try that on Monday however be prepared for people to report
> errors ;-) To be honest, I would make a backward compatible script
> which checks the version and ask for the upgrade in case it is old so
> user does not get an ugly error on their screen.
>
> --
> 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: 2822 bytes --]

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Otavio Salvador]

On Fri, Nov 6, 2015 at 6:37 PM, Ian Coolidge <ian@boundarydevices.com> wrote:
>> Ok; will try that on Monday however be prepared for people to report
>> errors ;-) To be honest, I would make a backward compatible script
>> which checks the version and ask for the upgrade in case it is old so
>> user does not get an ugly error on their screen.
>
>
> Did you see this message as u-boot started?
>
> https://github.com/boundarydevices/u-boot-imx6/blob/boundary-v2015.07/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt#L89-L93

No; it failed here:

https://github.com/boundarydevices/u-boot-imx6/blob/boundary-v2015.07/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt#L26

as board is undefined.

-- 
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

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Ian Coolidge]

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

Ah, okay. Yes, that's a symptom of an older u-boot version.

On Fri, Nov 6, 2015 at 12:56 PM, Otavio Salvador <
otavio.salvador@ossystems.com.br> wrote:

> On Fri, Nov 6, 2015 at 6:37 PM, Ian Coolidge <ian@boundarydevices.com>
> wrote:
> >> Ok; will try that on Monday however be prepared for people to report
> >> errors ;-) To be honest, I would make a backward compatible script
> >> which checks the version and ask for the upgrade in case it is old so
> >> user does not get an ugly error on their screen.
> >
> >
> > Did you see this message as u-boot started?
> >
> >
> https://github.com/boundarydevices/u-boot-imx6/blob/boundary-v2015.07/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt#L89-L93
>
> No; it failed here:
>
>
> https://github.com/boundarydevices/u-boot-imx6/blob/boundary-v2015.07/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt#L26
>
> as board is undefined.
>
> --
> 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: 2297 bytes --]

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Ian Coolidge]

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

Sent in a patch to meta-fsl-arm-extra to bump the u-boot-script-boundary
recipe to add a sanity check for 6x_bootscript looking for the board
variable. If it's not there then u-boot is far outdated and its a
non-starter. That should help address this issue.

A bit off topic: That patch should apply to master, jethro, and fido.
What's the etiquette for sending in patches to apply to multiple branches
like that? Three separate patches with the tag of each branch?

On Fri, Nov 6, 2015 at 12:56 PM, Otavio Salvador <
otavio.salvador@ossystems.com.br> wrote:

> On Fri, Nov 6, 2015 at 6:37 PM, Ian Coolidge <ian@boundarydevices.com>
> wrote:
> >> Ok; will try that on Monday however be prepared for people to report
> >> errors ;-) To be honest, I would make a backward compatible script
> >> which checks the version and ask for the upgrade in case it is old so
> >> user does not get an ugly error on their screen.
> >
> >
> > Did you see this message as u-boot started?
> >
> >
> https://github.com/boundarydevices/u-boot-imx6/blob/boundary-v2015.07/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt#L89-L93
>
> No; it failed here:
>
>
> https://github.com/boundarydevices/u-boot-imx6/blob/boundary-v2015.07/board/boundary/nitrogen6x/6x_bootscript-yocto-3.14.txt#L26
>
> as board is undefined.
>
> --
> 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: 2724 bytes --]

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Otavio Salvador]

On Sat, Nov 7, 2015 at 12:12 AM, Ian Coolidge <ian@boundarydevices.com> wrote:
> Sent in a patch to meta-fsl-arm-extra to bump the u-boot-script-boundary
> recipe to add a sanity check for 6x_bootscript looking for the board
> variable. If it's not there then u-boot is far outdated and its a
> non-starter. That should help address this issue.

Great.

> A bit off topic: That patch should apply to master, jethro, and fido. What's
> the etiquette for sending in patches to apply to multiple branches like
> that? Three separate patches with the tag of each branch?

I applied this for master and jethro. For fido, if we apply this,
other patches need to go in as well as the 3.14.28 upgrade or am I
wrong?

-- 
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

Re: [CRITICAL] Nitrogen6x Lite is broken at 3.14 kernel

[Ian Coolidge]

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

>
> I applied this for master and jethro. For fido, if we apply this,
> other patches need to go in as well as the 3.14.28 upgrade or am I
> wrong?


Fido is at 3.14.28 too. Granted, a few revisions behind but it still would
benefit from having this u-boot-script-boundary update.

https://github.com/Freescale/meta-fsl-arm-extra/blob/fido/recipes-kernel/linux/linux-boundary_3.14.28.bb

On Mon, Nov 9, 2015 at 2:28 AM, Otavio Salvador <
otavio.salvador@ossystems.com.br> wrote:

> On Sat, Nov 7, 2015 at 12:12 AM, Ian Coolidge <ian@boundarydevices.com>
> wrote:
> > Sent in a patch to meta-fsl-arm-extra to bump the u-boot-script-boundary
> > recipe to add a sanity check for 6x_bootscript looking for the board
> > variable. If it's not there then u-boot is far outdated and its a
> > non-starter. That should help address this issue.
>
> Great.
>
> > A bit off topic: That patch should apply to master, jethro, and fido.
> What's
> > the etiquette for sending in patches to apply to multiple branches like
> > that? Three separate patches with the tag of each branch?
>
> I applied this for master and jethro. For fido, if we apply this,
> other patches need to go in as well as the 3.14.28 upgrade or am I
> wrong?
>
> --
> 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: 2556 bytes --]