Skip to contentSkip to conversation

u-boot-sh/master

GIT PULL Marek Vasut 3 participants 9 messages Jun 13, 2018 – Jun 18, 2018

lore
Share & tools
lore
#0 lore raw

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

----------------------------------------------------------------
Marek Vasut (8):
pinctrl: renesas: Sync Gen2 PFC tables with Linux v4.17
pinctrl: renesas: Sync Gen3 PFC tables with Linux v4.17
ARM: rmobile: Sync Gen2 DTS with Linux v4.17
ARM: rmobile: Sync Gen3 DTS with Linux v4.17
ARM: rmobile: Zap CONFIG_SYS_CLK_FREQ where applicable
ARM: rmobile: Point load address to more sane area on Gen2
ARM: rmobile: Point load address to more sane area on Gen3
ARM: rmobile: Fix environment placement on Draak

 arch/arm/dts/r8a7790-lager.dts         |  310 +++++++++-----
 arch/arm/dts/r8a7790.dtsi              | 2877
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------------------------------------------------------
 arch/arm/dts/r8a7791-koelsch.dts       |  259 ++++++-----
 arch/arm/dts/r8a7791-porter.dts        |  151 ++++---
 arch/arm/dts/r8a7791.dtsi              | 3008
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------------------------------------
 arch/arm/dts/r8a7792.dtsi              |  587 +++++++++++++------------
 arch/arm/dts/r8a7793-gose.dts          |  262 +++++++-----
 arch/arm/dts/r8a7793.dtsi              | 2409
+++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------------------------
 arch/arm/dts/r8a7794-alt.dts           |   56 ++-
 arch/arm/dts/r8a7794-silk.dts          |  189 ++++++---
 arch/arm/dts/r8a7794.dtsi              | 2416
+++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------------------------
 arch/arm/dts/r8a7795.dtsi              |  602 +++++++++++++++++++++-----
 arch/arm/dts/r8a7796.dtsi              |  556 ++++++++++++++++++------
 arch/arm/dts/r8a77965.dtsi             |  127 +++++-
 arch/arm/dts/r8a77970-eagle.dts        |   66 +--
 arch/arm/dts/r8a77970.dtsi             |  328 ++++++++++++--
 arch/arm/dts/r8a77995-draak.dts        |  124 ++++++
 arch/arm/dts/r8a77995.dtsi             |  423 +++++++++++++++++-
 arch/arm/dts/salvator-common.dtsi      |   85 +++-
 arch/arm/dts/ulcb.dtsi                 |   22 +-
 drivers/pinctrl/renesas/pfc-r8a7790.c  |    8 +-
 drivers/pinctrl/renesas/pfc-r8a7791.c  |   84 +++-
 drivers/pinctrl/renesas/pfc-r8a7794.c  |  473 +++++++++++++++++++++
 drivers/pinctrl/renesas/pfc-r8a7795.c  | 1968
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------
 drivers/pinctrl/renesas/pfc-r8a7796.c  | 1048
+++++++++++++++++++++++++++++++--------------
 drivers/pinctrl/renesas/pfc-r8a77970.c |  873
++++++++++++++++---------------------
 drivers/pinctrl/renesas/pfc-r8a77990.c | 3446
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
 drivers/pinctrl/renesas/pfc-r8a77995.c |  695
+++++++++++++++++++++++++++++-
 drivers/pinctrl/renesas/sh_pfc.h       |    5 +-
 include/configs/draak.h                |    6 +-
 include/configs/ebisu.h                |    4 -
 include/configs/rcar-gen2-common.h     |    3 +-
 include/configs/rcar-gen3-common.h     |    3 +-
 include/configs/salvator-x.h           |    4 -
 include/configs/ulcb.h                 |    4 -
 35 files changed, 16030 insertions(+), 7451 deletions(-)
#1 lore raw
Quoted 5 lines · Marek Vasut

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

Quoted 9 lines

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

NAK. First, there are a lot of should be fix style errors in the new files in drivers/pinctrl/renesas/. Second, you have build failures:
https://travis-ci.org/trini/u-boot/jobs/391819805
https://travis-ci.org/trini/u-boot/jobs/391819826

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180614/b62a9db1/attachment.sig>
#2 lore raw
Quoted 17 lines · Tom Rini
Quoted 5 lines · Marek Vasut

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

Quoted 9 lines

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

Quoted 5 lines

NAK. First, there are a lot of should be fix style errors in the new files in drivers/pinctrl/renesas/. Second, you have build failures:
https://travis-ci.org/trini/u-boot/jobs/391819805
https://travis-ci.org/trini/u-boot/jobs/391819826

Great, thanks!

If the travis didn't vomit constant false positives recently, I'd trust it. With what it does now, I cannot trust it anymore.

Signature · 2 lines
Best regards,
Marek Vasut
#3 lore raw
Quoted 24 lines · Marek Vasut
Quoted 17 lines · Tom Rini
Quoted 5 lines · Marek Vasut

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

Quoted 9 lines

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

Quoted 5 lines

NAK. First, there are a lot of should be fix style errors in the new files in drivers/pinctrl/renesas/. Second, you have build failures:
https://travis-ci.org/trini/u-boot/jobs/391819805
https://travis-ci.org/trini/u-boot/jobs/391819826

Quoted 4 lines

Great, thanks!

If the travis didn't vomit constant false positives recently, I'd trust it. With what it does now, I cannot trust it anymore.

Can you elaborate?

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180614/34f51576/attachment.sig>
#4 lore raw
Quoted 25 lines · Tom Rini
Quoted 24 lines · Marek Vasut
Quoted 22 lines · Tom Rini
Quoted 5 lines · Marek Vasut

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

Quoted 9 lines

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

NAK. First, there are a lot of should be fix style errors in the new files in drivers/pinctrl/renesas/. Second, you have build failures:
https://travis-ci.org/trini/u-boot/jobs/391819805
https://travis-ci.org/trini/u-boot/jobs/391819826

Quoted 6 lines
Quoted 4 lines

Great, thanks!

If the travis didn't vomit constant false positives recently, I'd trust it. With what it does now, I cannot trust it anymore.

Can you elaborate?

Don't you see the constant random timeouts during builds for some targets? I cannot get a successful build on the first try almost never these days, I have to keep restarting until I see that different targets failed during each build with a timeout and then extrapolate that the build is actually fine from that. It's annoying.

And then there is the other category of boards which indicate "red" failure marker due to ie. DT problems, which if you pull DTs from Linux is something you cannot really fix on the U-Boot side right away, but rather fix in Linux and then resync.

Signature · 2 lines
Best regards,
Marek Vasut
#5 lore raw
Quoted 31 lines · Marek Vasut
Quoted 30 lines · Tom Rini
Quoted 28 lines · Marek Vasut
Quoted 17 lines · Tom Rini
Quoted 5 lines · Marek Vasut

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

Quoted 9 lines

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

Quoted 5 lines

NAK. First, there are a lot of should be fix style errors in the new files in drivers/pinctrl/renesas/. Second, you have build failures:
https://travis-ci.org/trini/u-boot/jobs/391819805
https://travis-ci.org/trini/u-boot/jobs/391819826

Great, thanks!

If the travis didn't vomit constant false positives recently, I'd trust it. With what it does now, I cannot trust it anymore.

Quoted 7 lines
Quoted 1 line

Can you elaborate?

Don't you see the constant random timeouts during builds for some targets? I cannot get a successful build on the first try almost never these days, I have to keep restarting until I see that different targets failed during each build with a timeout and then extrapolate that the build is actually fine from that. It's annoying.

I see maybe 2-5 of those per day when I'm busy, which is annoying, but I've never had it fail a second time in a row.

Quoted 4 lines

And then there is the other category of boards which indicate "red" failure marker due to ie. DT problems, which if you pull DTs from Linux is something you cannot really fix on the U-Boot side right away, but rather fix in Linux and then resync.

Yes, there are DT warnings that should be fixed. But they aren't travis errors, only actual fail to builds (like in the above, which indeed are confusingly DT related, but not a travis problem at all, I see them locally too) and new C warnings.

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180614/e0f1009d/attachment.sig>
#6 lore raw
Quoted 34 lines · Tom Rini
Quoted 33 lines · Marek Vasut
Quoted 31 lines · Tom Rini
Quoted 24 lines · Marek Vasut
Quoted 17 lines · Tom Rini
Quoted 5 lines · Marek Vasut

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

Quoted 9 lines

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

Quoted 5 lines

NAK. First, there are a lot of should be fix style errors in the new files in drivers/pinctrl/renesas/. Second, you have build failures:
https://travis-ci.org/trini/u-boot/jobs/391819805
https://travis-ci.org/trini/u-boot/jobs/391819826

Quoted 4 lines

Great, thanks!

If the travis didn't vomit constant false positives recently, I'd trust it. With what it does now, I cannot trust it anymore.

Can you elaborate?

Quoted 8 lines
Quoted 5 lines

Don't you see the constant random timeouts during builds for some targets? I cannot get a successful build on the first try almost never these days, I have to keep restarting until I see that different targets failed during each build with a timeout and then extrapolate that the build is actually fine from that. It's annoying.

I see maybe 2-5 of those per day when I'm busy, which is annoying, but I've never had it fail a second time in a row.

Good for you.

Quoted 9 lines
Quoted 4 lines

And then there is the other category of boards which indicate "red" failure marker due to ie. DT problems, which if you pull DTs from Linux is something you cannot really fix on the U-Boot side right away, but rather fix in Linux and then resync.

Yes, there are DT warnings that should be fixed. But they aren't travis errors, only actual fail to builds (like in the above, which indeed are confusingly DT related, but not a travis problem at all, I see them locally too) and new C warnings.

I'd prefer to see only errors which are relevant in the logs. Oh well.

Signature · 2 lines
Best regards,
Marek Vasut
#7 lore raw
Quoted 51 lines · Marek Vasut
Quoted 34 lines · Tom Rini
Quoted 33 lines · Marek Vasut
Quoted 31 lines · Tom Rini
Quoted 24 lines · Marek Vasut
Quoted 17 lines · Tom Rini
Quoted 5 lines · Marek Vasut

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

Quoted 9 lines

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

Quoted 5 lines

NAK. First, there are a lot of should be fix style errors in the new files in drivers/pinctrl/renesas/. Second, you have build failures:
https://travis-ci.org/trini/u-boot/jobs/391819805
https://travis-ci.org/trini/u-boot/jobs/391819826

Quoted 4 lines

Great, thanks!

If the travis didn't vomit constant false positives recently, I'd trust it. With what it does now, I cannot trust it anymore.

Can you elaborate?

Quoted 8 lines
Quoted 5 lines

Don't you see the constant random timeouts during builds for some targets? I cannot get a successful build on the first try almost never these days, I have to keep restarting until I see that different targets failed during each build with a timeout and then extrapolate that the build is actually fine from that. It's annoying.

I see maybe 2-5 of those per day when I'm busy, which is annoying, but I've never had it fail a second time in a row.

Good for you.

Quoted 5 lines
Quoted 4 lines

And then there is the other category of boards which indicate "red" failure marker due to ie. DT problems, which if you pull DTs from Linux is something you cannot really fix on the U-Boot side right away, but rather fix in Linux and then resync.

Quoted 6 lines
Quoted 4 lines

Yes, there are DT warnings that should be fixed. But they aren't travis errors, only actual fail to builds (like in the above, which indeed are confusingly DT related, but not a travis problem at all, I see them locally too) and new C warnings.

I'd prefer to see only errors which are relevant in the logs. Oh well.

Then fix the kernel dts. I swear when I looked into these last they're just as visible there.

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180615/57f78a54/attachment.sig>
#8 lore raw
In reply to Tom Rini
Quoted 34 lines · Tom Rini
Quoted 33 lines · Marek Vasut
Quoted 31 lines · Tom Rini
Quoted 24 lines · Marek Vasut
Quoted 17 lines · Tom Rini
Quoted 5 lines · Marek Vasut

The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b:

Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08
10:08:20 -0400)

Quoted 9 lines

are available in the Git repository at:

git://git.denx.de/u-boot-sh.git master

for you to fetch changes up to 0a52b00627c4fb6b100745c97910b75c99f3f4a9:

ARM: rmobile: Fix environment placement on Draak (2018-06-10 16:34:43
+0200)

Quoted 5 lines

NAK. First, there are a lot of should be fix style errors in the new files in drivers/pinctrl/renesas/. Second, you have build failures:
https://travis-ci.org/trini/u-boot/jobs/391819805
https://travis-ci.org/trini/u-boot/jobs/391819826

Quoted 4 lines

Great, thanks!

If the travis didn't vomit constant false positives recently, I'd trust it. With what it does now, I cannot trust it anymore.

Can you elaborate?

Quoted 9 lines
Quoted 5 lines

Don't you see the constant random timeouts during builds for some targets? I cannot get a successful build on the first try almost never these days, I have to keep restarting until I see that different targets failed during each build with a timeout and then extrapolate that the build is actually fine from that. It's annoying.

I see maybe 2-5 of those per day when I'm busy, which is annoying, but I've never had it fail a second time in a row.

I guess the instability came from "The command "sudo apt-get install libisl15 -y" failed and exited with 100 during". I saw this almost every build. And even some build wass restarted for the 2nd time, it still failed.

Quoted 10 lines
Quoted 4 lines

And then there is the other category of boards which indicate "red" failure marker due to ie. DT problems, which if you pull DTs from Linux is something you cannot really fix on the U-Boot side right away, but rather fix in Linux and then resync.

Yes, there are DT warnings that should be fixed. But they aren't travis errors, only actual fail to builds (like in the above, which indeed are confusingly DT related, but not a travis problem at all, I see them locally too) and new C warnings.

Regards,
Bin