tools: logos: crop Toradex logo slightly

PATCH New Stefan Agner 4 participants 15 messages Dec 6, 2018 – Jan 10, 2019

lore

Share & tools

lore

Stefan Agner

author

2018-12-06 13:53 UTC #0 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

From: Stefan Agner <[email protected]>

The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.

Signed-off-by Stefan Agner <[email protected]>

---

 tools/logos/toradex.bmp | Bin 24982 -> 23298 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)

tools/logos/toradex.bmp binary +0 -0

index 3e2dcf23358dd46fc7b1bb0dae70d3ba985606ee..8b5cb18e8475b3eda72c9f0e821b2aad17bc1bae 100644
GIT binary patch

zcmYLEO-oxr6rCd>(HDKjZ>@>Jpis10rD|SMpO}a~;ztZ>#n#YvXDMP^LN{G at Ros~Y z7ZzGv2rXT;2fM2V{0-tp-MG+&1*NzVD!qwH2j<L~d(J&XLSefe<B*6D4JJ=_0FZ# zqA&|eIdVc37wR8om8$e!Zz<*l6?djGE9YG(S~;)!9=&quRG=vfFLUm4-15 at m|9r0G zmizeCFm60Pec_#Ey3_){ltsV(*{R;a?MtR!GlVHyVU}jnm+JXtq3Lc0*@L5*6#@X zZWM?FBG5Uvi_1-;;7M@9V643tT)ro*yB*RRyf5r!m%wIs7(X6Z&}P4>2dpwK^!TCA z#vApR)u#7*|L~|!Y at L=^>|}#ZB|Th=)POrtDVzOa=q<<Og53sg4CGC*R at M$yfp)Br zTCNWr4?o94(4CgaqoH;%G%TNcHS!SLa%5g3A at fQ&UylZ%jYNRCF+X@VE-pJ+(CWB8 zDGB)@pZ0ionrZ=CrbK@;9R;^%B+8u4K>wYo&*H6Kdi+g)O1ODE*9if$Qv)_#kr8 Tu-(CC?xpL%<TJT_e17>4;b52`

delta 656 zcmXw!ODIHP6vw|?uE!njFhgTzOi3nYyasb;yv8sxo^!iK9tlwv3RzGrG(|QZDZa9j zB?~JZ+1My$i;az at WKD$CWZ~X(xb^k>&j0++`QLBjQaHR3{E^t<G{Y~D##p(2>lYa7 z)+w+|)TbsGvzX}UqTV=PO%wUG_^qEgSQ8tBeJ!7_i5^I%n~giXW{@^6RYN>W!v46l zZnTmTwxA%p5v^)Y2)QGZeq#<|oysNFT8OjO0P4s2<a;b2$~I!OkeIL&FCDc1-KiRL zf{70}efyfocU>k(xMn2%=P%%;Ou?--52dJHO4+QSl^!nSw-~J;C6l%lwr~5F=qKz zmy?(Jy*yd_Wjye{4y=ZDsxrI1-IXU5n+X*mIJivEJ(i7hzuVMUi3JcX#IAJ8&;aF zsBO3WN1EXwgprVHrNB5$D^R(R&&puNHN|Pe+lV at v!3JlZpi4&OG#JQJUH6Lh5}x zJarfUd2L|nEy26~M84?DfS$fmoLlIpxszBE*3T+sI<+ON_XaQ<ZsTQGzYS9D9-_I) WVdOzbgrkvqwA_u-FJz9T1ojITOUl0h

Signature · 1 line

2.19.2

Lukasz Majewski

2018-12-06 14:41 UTC #1 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Quoted 26 lines · On Thu, 6 Dec 2018 14:53:02 +0100 Stefan Agner <[email protected]> wrote:

From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
Signed-off-by Stefan Agner <[email protected]>
---
 tools/logos/toradex.bmp | Bin 24982 -> 23298 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)
tools/logos/toradex.bmp +0 -0
index 3e2dcf23358dd46fc7b1bb0dae70d3ba985606ee..8b5cb18e8475b3eda72c9f0e821b2aad17bc1bae 100644 GIT binary patch delta 505 zcmYLEO-oxr6rCd>(HDKjZ>@>Jpis10rD|SMpO}a~;ztZ>#n#YvXDMP^LN{G at Ros~Y z7ZzGv2rXT;2fM2V{0-tp-MG+&1*NzVD!qwH2j<L~d(J&XLSefe<B*6D4JJ=_0FZ# zqA&|eIdVc37wR8om8$e!Zz<*l6?djGE9YG(S~;)!9=&quRG=vfFLUm4-15@m|9r0G zmizeCFm60Pec_#Ey3_){ltsV(*{R;a?MtR!GlVHyVU}jnm+JXtq3Lc0*@L5*6#@X zZWM?FBG5Uvi_1-;;7M at 9V643tT)ro*yB*RRyf5r!m%wIs7(X6Z&}P4>2dpwK^!TCA z#vApR)u#7*|L~|!Y at L=^>|}#ZB|Th=)POrtDVzOa=q<<Og53sg4CGC*R@M$yfp)Br zTCNWr4?o94(4CgaqoH;%G%TNcHS!SLa%5g3A at fQ&UylZ%jYNRCF+X at VE-pJ+(CWB8 zDGB)@pZ0ionrZ=CrbK@;9R;^%B+8u4K>wYo&*H6Kdi+g)O1ODE*9if$Qv)_#kr8 Tu-(CC?xpL%<TJT_e17>4;b52`

Quoted 11 lines

delta 656 zcmXw!ODIHP6vw|?uE!njFhgTzOi3nYyasb;yv8sxo^!iK9tlwv3RzGrG(|QZDZa9j zB?~JZ+1My$i;az at WKD$CWZ~X(xb^k>&j0++`QLBjQaHR3{E^t<G{Y~D##p(2>lYa7 z)+w+|)TbsGvzX}UqTV=PO%wUG_^qEgSQ8tBeJ!7_i5^I%n~giXW{@^6RYN>W!v46l zZnTmTwxA%p5v^)Y2)QGZeq#<|oysNFT8OjO0P4s2<a;b2$~I!OkeIL&FCDc1-KiRL zf{70}efyfocU>k(xMn2%=P%%;Ou?--52dJHO4+QSl^!nSw-~J;C6l%lwr~5F=qKz zmy?(Jy*yd_Wjye{4y=ZDsxrI1-IXU5n+X*mIJivEJ(i7hzuVMUi3JcX#IAJ8&;aF zsBO3WN1EXwgprVHrNB5$D^R(R&&puNHN|Pe+lV at v!3JlZpi4&OG#JQJUH6Lh5}x zJarfUd2L|nEy26~M84?DfS$fmoLlIpxszBE*3T+sI<+ON_XaQ<ZsTQGzYS9D9-_I) WVdOzbgrkvqwA_u-FJz9T1ojITOUl0h

Reviewed-by Lukasz Majewski <[email protected]>

Best regards,

Lukasz Majewski

DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20181206/30a4c06f/attachment.sig>

Lukasz Majewski

2018-12-08 16:48 UTC #2 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

In reply to Stefan Agner

Hi Stefan,

Quoted 5 lines

From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.

I can confirm that those changes, as well as

[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B [PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer

Reduce the u-boot.vyb binary for colibri_vf, so it is possible to put more device tree description data to vf.dtsi and hence convert some vf610 boards to DTS/DM.

Tested-by Lukasz Majewski <[email protected]>

Thanks Stefan :-)

Quoted 21 lines

Signed-off-by Stefan Agner <[email protected]>
---
 tools/logos/toradex.bmp | Bin 24982 -> 23298 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)
tools/logos/toradex.bmp +0 -0
index 3e2dcf23358dd46fc7b1bb0dae70d3ba985606ee..8b5cb18e8475b3eda72c9f0e821b2aad17bc1bae 100644 GIT binary patch delta 505 zcmYLEO-oxr6rCd>(HDKjZ>@>Jpis10rD|SMpO}a~;ztZ>#n#YvXDMP^LN{G at Ros~Y z7ZzGv2rXT;2fM2V{0-tp-MG+&1*NzVD!qwH2j<L~d(J&XLSefe<B*6D4JJ=_0FZ# zqA&|eIdVc37wR8om8$e!Zz<*l6?djGE9YG(S~;)!9=&quRG=vfFLUm4-15@m|9r0G zmizeCFm60Pec_#Ey3_){ltsV(*{R;a?MtR!GlVHyVU}jnm+JXtq3Lc0*@L5*6#@X zZWM?FBG5Uvi_1-;;7M at 9V643tT)ro*yB*RRyf5r!m%wIs7(X6Z&}P4>2dpwK^!TCA z#vApR)u#7*|L~|!Y at L=^>|}#ZB|Th=)POrtDVzOa=q<<Og53sg4CGC*R@M$yfp)Br zTCNWr4?o94(4CgaqoH;%G%TNcHS!SLa%5g3A at fQ&UylZ%jYNRCF+X at VE-pJ+(CWB8 zDGB)@pZ0ionrZ=CrbK@;9R;^%B+8u4K>wYo&*H6Kdi+g)O1ODE*9if$Qv)_#kr8 Tu-(CC?xpL%<TJT_e17>4;b52`

Quoted 11 lines

delta 656 zcmXw!ODIHP6vw|?uE!njFhgTzOi3nYyasb;yv8sxo^!iK9tlwv3RzGrG(|QZDZa9j zB?~JZ+1My$i;az at WKD$CWZ~X(xb^k>&j0++`QLBjQaHR3{E^t<G{Y~D##p(2>lYa7 z)+w+|)TbsGvzX}UqTV=PO%wUG_^qEgSQ8tBeJ!7_i5^I%n~giXW{@^6RYN>W!v46l zZnTmTwxA%p5v^)Y2)QGZeq#<|oysNFT8OjO0P4s2<a;b2$~I!OkeIL&FCDc1-KiRL zf{70}efyfocU>k(xMn2%=P%%;Ou?--52dJHO4+QSl^!nSw-~J;C6l%lwr~5F=qKz zmy?(Jy*yd_Wjye{4y=ZDsxrI1-IXU5n+X*mIJivEJ(i7hzuVMUi3JcX#IAJ8&;aF zsBO3WN1EXwgprVHrNB5$D^R(R&&puNHN|Pe+lV at v!3JlZpi4&OG#JQJUH6Lh5}x zJarfUd2L|nEy26~M84?DfS$fmoLlIpxszBE*3T+sI<+ON_XaQ<ZsTQGzYS9D9-_I) WVdOzbgrkvqwA_u-FJz9T1ojITOUl0h

Best regards,

Lukasz Majewski

Stefan Agner

author

2018-12-14 13:13 UTC #3 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Hi Tom,

Quoted 8 lines · Lukasz Majewski

Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.

Quoted 5 lines

I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B [PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer

I think all of those go directly into your tree?

--
Stefan

Quoted 54 lines

Reduce the u-boot.vyb binary for colibri_vf, so it is possible to put more device tree description data to vf.dtsi and hence convert some vf610 boards to DTS/DM.
Tested-by Lukasz Majewski <[email protected]>
Thanks Stefan :-)
Quoted 21 lines
Signed-off-by Stefan Agner <[email protected]>
---
 tools/logos/toradex.bmp | Bin 24982 -> 23298 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)
tools/logos/toradex.bmp +0 -0
index 3e2dcf23358dd46fc7b1bb0dae70d3ba985606ee..8b5cb18e8475b3eda72c9f0e821b2aad17bc1bae 100644 GIT binary patch delta 505 zcmYLEO-oxr6rCd>(HDKjZ>@>Jpis10rD|SMpO}a~;ztZ>#n#YvXDMP^LN{G at Ros~Y z7ZzGv2rXT;2fM2V{0-tp-MG+&1*NzVD!qwH2j<L~d(J&XLSefe<B*6D4JJ=_0FZ# zqA&|eIdVc37wR8om8$e!Zz<*l6?djGE9YG(S~;)!9=&quRG=vfFLUm4-15@m|9r0G zmizeCFm60Pec_#Ey3_){ltsV(*{R;a?MtR!GlVHyVU}jnm+JXtq3Lc0*@L5*6#@X zZWM?FBG5Uvi_1-;;7M at 9V643tT)ro*yB*RRyf5r!m%wIs7(X6Z&}P4>2dpwK^!TCA z#vApR)u#7*|L~|!Y at L=^>|}#ZB|Th=)POrtDVzOa=q<<Og53sg4CGC*R@M$yfp)Br zTCNWr4?o94(4CgaqoH;%G%TNcHS!SLa%5g3A at fQ&UylZ%jYNRCF+X at VE-pJ+(CWB8 zDGB)@pZ0ionrZ=CrbK@;9R;^%B+8u4K>wYo&*H6Kdi+g)O1ODE*9if$Qv)_#kr8 Tu-(CC?xpL%<TJT_e17>4;b52`
Quoted 11 lines
delta 656 zcmXw!ODIHP6vw|?uE!njFhgTzOi3nYyasb;yv8sxo^!iK9tlwv3RzGrG(|QZDZa9j zB?~JZ+1My$i;az at WKD$CWZ~X(xb^k>&j0++`QLBjQaHR3{E^t<G{Y~D##p(2>lYa7 z)+w+|)TbsGvzX}UqTV=PO%wUG_^qEgSQ8tBeJ!7_i5^I%n~giXW{@^6RYN>W!v46l zZnTmTwxA%p5v^)Y2)QGZeq#<|oysNFT8OjO0P4s2<a;b2$~I!OkeIL&FCDc1-KiRL zf{70}efyfocU>k(xMn2%=P%%;Ou?--52dJHO4+QSl^!nSw-~J;C6l%lwr~5F=qKz zmy?(Jy*yd_Wjye{4y=ZDsxrI1-IXU5n+X*mIJivEJ(i7hzuVMUi3JcX#IAJ8&;aF zsBO3WN1EXwgprVHrNB5$D^R(R&&puNHN|Pe+lV at v!3JlZpi4&OG#JQJUH6Lh5}x zJarfUd2L|nEy26~M84?DfS$fmoLlIpxszBE*3T+sI<+ON_XaQ<ZsTQGzYS9D9-_I) WVdOzbgrkvqwA_u-FJz9T1ojITOUl0h
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de

Tom Rini

2018-12-14 13:20 UTC #4 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Quoted 11 lines · Stefan Agner

Hi Tom,
Quoted 8 lines · Lukasz Majewski
Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.

Quoted 7 lines

Quoted 5 lines
I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B [PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer
I think all of those go directly into your tree?

I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.

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/20181214/389da121/attachment.sig>

Stefan Agner

author

2018-12-14 13:26 UTC #5 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Quoted 14 lines · Tom Rini

Quoted 13 lines · Stefan Agner
Hi Tom,
Quoted 10 lines · Lukasz Majewski
Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
I can confirm that those changes, as well as

Quoted 8 lines

Quoted 5 lines
Quoted 3 lines
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B [PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer
I think all of those go directly into your tree?
I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.

I guess the mtd change is rather global...

Currently board/toradex is not specifically marked to be maintained by Stefano (we also have Tegra boards). But yeah for Vybrid it probably makes sense.

--
Stefan

Tom Rini

2018-12-14 13:30 UTC #6 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Quoted 19 lines · Stefan Agner

Quoted 18 lines · Tom Rini
Quoted 17 lines · Stefan Agner
Hi Tom,
Quoted 13 lines · Lukasz Majewski
Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B [PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer

Quoted 6 lines

Quoted 4 lines
Quoted 1 line
I think all of those go directly into your tree?
I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.
I guess the mtd change is rather global...

Ah right. Maybe I gave it to myself? I know I assigned it in patchwork.

Quoted 3 lines

Currently board/toradex is not specifically marked to be maintained by Stefano (we also have Tegra boards). But yeah for Vybrid it probably makes sense.

Right, I try and assign boards based on the underlying SoC correctly but do also from time to time get them wrong (there's a handful of "wait that's not TI-based that's iMX-based, oops" I catch myself on).

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/20181214/50a6aac6/attachment.sig>

Stefano Babic

2018-12-14 13:40 UTC #7 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Quoted 22 lines · Tom Rini

Quoted 21 lines · Stefan Agner
Quoted 20 lines · Tom Rini
Quoted 18 lines · Stefan Agner
Hi Tom,
Quoted 8 lines · Lukasz Majewski
Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
Quoted 5 lines
I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B [PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer
I think all of those go directly into your tree?

Quoted 12 lines

Quoted 4 lines
Quoted 2 lines
I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.
I guess the mtd change is rather global...
Ah right. Maybe I gave it to myself? I know I assigned it in patchwork.
Quoted 3 lines
Currently board/toradex is not specifically marked to be maintained by Stefano (we also have Tegra boards). But yeah for Vybrid it probably makes sense.

Yes, I saw your patches, I will picked them up after current PR (sent now) will be merged.

Quoted 4 lines

Right, I try and assign boards based on the underlying SoC correctly but do also from time to time get them wrong (there's a handful of "wait that's not TI-based that's iMX-based, oops" I catch myself on).

Regards,
Stefano

Signature · 5 lines

=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

Stefan Agner

author

2019-01-04 15:05 UTC #8 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Hi Stefano,

Quoted 31 lines · Stefano Babic

Quoted 30 lines · Tom Rini
Quoted 19 lines · Stefan Agner
Quoted 18 lines · Tom Rini
Quoted 17 lines · Stefan Agner
Hi Tom,
Quoted 13 lines · Lukasz Majewski
Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B [PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer
Quoted 6 lines
Quoted 4 lines
Quoted 1 line
I think all of those go directly into your tree?
I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.
I guess the mtd change is rather global...
Ah right. Maybe I gave it to myself? I know I assigned it in patchwork.

Quoted 7 lines

Quoted 4 lines
Quoted 3 lines
Currently board/toradex is not specifically marked to be maintained by Stefano (we also have Tegra boards). But yeah for Vybrid it probably makes sense.
Yes, I saw your patches, I will picked them up after current PR (sent now) will be merged.

This still did not make it. Can it still be considered for the next release?

There is also:
- board: toradex: colibri_vf: unset NFS and LOADS/B
- ARM: vf610: ddrmc: program Dummy DDRBYTE1/2
- ddr: vybrid: various fixes (patchset)

--
Stefan

Quoted 8 lines

Quoted 5 lines
Right, I try and assign boards based on the underlying SoC correctly but do also from time to time get them wrong (there's a handful of "wait that's not TI-based that's iMX-based, oops" I catch myself on).
Regards,
Stefano

Stefano Babic

2019-01-09 15:41 UTC #9 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Hi Stefan, Lukasz,

Quoted 21 lines · Stefan Agner

Hi Stefano,
Quoted 18 lines · Stefano Babic
Quoted 17 lines · Tom Rini
Quoted 16 lines · Stefan Agner
Quoted 15 lines · Tom Rini
Quoted 14 lines · Stefan Agner
Hi Tom,
Quoted 11 lines · Lukasz Majewski
Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B

You're right, this was slipped away, I pick it now.

Quoted 2 lines

Quoted 2 lines
Quoted 2 lines
Quoted 2 lines
Quoted 2 lines
Quoted 2 lines
Quoted 2 lines
[PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer

Both of them are outside u-boot-imx, so they will be merged by Scott / Tom.

Quoted 19 lines

Quoted 11 lines
Quoted 11 lines
Quoted 7 lines
Quoted 5 lines
Quoted 2 lines
I think all of those go directly into your tree?
I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.
I guess the mtd change is rather global...
Ah right. Maybe I gave it to myself? I know I assigned it in patchwork.
Quoted 7 lines
Quoted 4 lines
Quoted 3 lines
Currently board/toradex is not specifically marked to be maintained by Stefano (we also have Tegra boards). But yeah for Vybrid it probably makes sense.
Yes, I saw your patches, I will picked them up after current PR (sent now) will be merged.

Quoted 6 lines

This still did not make it. Can it still be considered for the next release?
There is also:
- board: toradex: colibri_vf: unset NFS and LOADS/B
- ARM: vf610: ddrmc: program Dummy DDRBYTE1/2

Applied to u-boot-imx.

Quoted 1 line

- ddr: vybrid: various fixes (patchset)

There is a (slight) conflict with Lukasz's patches regarding DDR calibration. I won't pick Lukasz's for 2019.01, and I could solve the conflict in -next branch. IMHO I have no concerns (open points) regarding usage of bit shifting instead of BIT() macro. So I pushed this patchset and I delay Lukasz's to next release.

Best regards,
Stefano

Signature · 5 lines

=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

Lukasz Majewski

2019-01-09 21:38 UTC #10 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Quoted 60 lines · On Wed, 9 Jan 2019 16:41:40 +0100 Stefano Babic <[email protected]> wrote:

Hi Stefan, Lukasz,
Quoted 21 lines · Stefan Agner
Hi Stefano,
Quoted 18 lines · Stefano Babic
Quoted 17 lines · Tom Rini
Quoted 16 lines · Stefan Agner
Quoted 15 lines · Tom Rini
Quoted 14 lines · Stefan Agner
Hi Tom,
Quoted 11 lines · Lukasz Majewski
Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B
You're right, this was slipped away, I pick it now.
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
[PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer
Both of them are outside u-boot-imx, so they will be merged by Scott / Tom.
Quoted 19 lines
Quoted 11 lines
Quoted 11 lines
Quoted 7 lines
Quoted 5 lines
Quoted 2 lines
I think all of those go directly into your tree?
I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.
I guess the mtd change is rather global...
Ah right. Maybe I gave it to myself? I know I assigned it in patchwork.
Quoted 7 lines
Quoted 4 lines
Quoted 3 lines
Currently board/toradex is not specifically marked to be maintained by Stefano (we also have Tegra boards). But yeah for Vybrid it probably makes sense.
Yes, I saw your patches, I will picked them up after current PR (sent now) will be merged.
Quoted 6 lines
This still did not make it. Can it still be considered for the next release?
There is also:
- board: toradex: colibri_vf: unset NFS and LOADS/B
- ARM: vf610: ddrmc: program Dummy DDRBYTE1/2

Quoted 6 lines

Applied to u-boot-imx.
Quoted 1 line
- ddr: vybrid: various fixes (patchset)
There is a (slight) conflict with Lukasz's patches regarding DDR calibration. I won't pick Lukasz's for 2019.01,

Ok. No problem.
( The DDR patch series itself:
http://patchwork.ozlabs.org/cover/1008330/)

Quoted 4 lines

and I could solve the
conflict in -next branch. IMHO I have no concerns (open points) regarding usage of bit shifting instead of BIT() macro. So I pushed this patchset and I delay Lukasz's to next release.

Ok. Thanks for the update.

Quoted 4 lines

Best regards,
Stefano

Best regards,

Lukasz Majewski

Stefano Babic

2019-01-10 08:15 UTC #11 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Quoted 65 lines · Lukasz Majewski

Quoted 62 lines · On Wed, 9 Jan 2019 16:41:40 +0100 Stefano Babic <[email protected]> wrote:
Hi Stefan, Lukasz,
Quoted 21 lines · Stefan Agner
Hi Stefano,
Quoted 18 lines · Stefano Babic
Quoted 17 lines · Tom Rini
Quoted 16 lines · Stefan Agner
Quoted 15 lines · Tom Rini
Quoted 14 lines · Stefan Agner
Hi Tom,
Quoted 11 lines · Lukasz Majewski
Hi Stefan,
Quoted 5 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B
You're right, this was slipped away, I pick it now.
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
[PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer
Both of them are outside u-boot-imx, so they will be merged by Scott / Tom.
Quoted 19 lines
Quoted 11 lines
Quoted 11 lines
Quoted 7 lines
Quoted 5 lines
Quoted 2 lines
I think all of those go directly into your tree?
I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.
I guess the mtd change is rather global...
Ah right. Maybe I gave it to myself? I know I assigned it in patchwork.
Quoted 7 lines
Quoted 4 lines
Quoted 3 lines
Currently board/toradex is not specifically marked to be maintained by Stefano (we also have Tegra boards). But yeah for Vybrid it probably makes sense.
Yes, I saw your patches, I will picked them up after current PR (sent now) will be merged.
Quoted 6 lines
This still did not make it. Can it still be considered for the next release?
There is also:
- board: toradex: colibri_vf: unset NFS and LOADS/B
- ARM: vf610: ddrmc: program Dummy DDRBYTE1/2
Applied to u-boot-imx.

Quoted 9 lines

Quoted 4 lines
Quoted 1 line
- ddr: vybrid: various fixes (patchset)
There is a (slight) conflict with Lukasz's patches regarding DDR calibration. I won't pick Lukasz's for 2019.01,
Ok. No problem.
( The DDR patch series itself:
http://patchwork.ozlabs.org/cover/1008330/)

There is a topic I would like to check. Stefan introduces with his patches a DDRMC_CR80_MC_INIT_COMPLETE that is still missing in your patches. I solved the conflict (not yet pushed to -next) as :

while (!(readl(&ddrmr->cr[80]) & DDRMC_CR80_MC_INIT_COMPLETE))

                 udelay(10);

writel(DDRMC_CR80_MC_INIT_COMPLETE, &ddrmr->cr[81]);

 #ifdef CONFIG_DDRMC_VF610_CALIBRATION
         ddrmc_calibration(ddrmr);

#endif

But I have no idea if DDRMC_CR80_MC_INIT_COMPLETE should be done before the calibration of maybe after. Could you help me ?

Quoted 7 lines

Quoted 4 lines
and I could solve the
conflict in -next branch. IMHO I have no concerns (open points) regarding usage of bit shifting instead of BIT() macro. So I pushed this patchset and I delay Lukasz's to next release.
Ok. Thanks for the update.

Best regards,
Stefano

Signature · 5 lines

=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

Lukasz Majewski

2019-01-10 09:29 UTC #12 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Hi Stefano,

Quoted 87 lines · Lukasz Majewski

Quoted 70 lines
Quoted 67 lines · On Wed, 9 Jan 2019 16:41:40 +0100 Stefano Babic <[email protected]> wrote:
Hi Stefan, Lukasz,
Quoted 25 lines · Stefan Agner
Hi Stefano,
Quoted 22 lines · Stefano Babic
Quoted 21 lines · Tom Rini
Quoted 19 lines · Stefan Agner
Quoted 18 lines · Tom Rini
Quoted 16 lines · Stefan Agner
Hi Tom,
Quoted 13 lines · Lukasz Majewski
Hi Stefan,
Quoted 6 lines
From: Stefan Agner <[email protected]>
The current bitmap is a bit larger than necessary, it has a black border around the Toradex logo. Crop the logo slightly which safes some space, useful especially on Colibri VFxx.
I can confirm that those changes, as well as
[PATCH v1] board: toradex: colibri_vf: unset NFS and LOADS/B
You're right, this was slipped away, I pick it now.
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
[PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts [PATCH v1] fs: fat: dynamically allocate memory for temporary buffer
Both of them are outside u-boot-imx, so they will be merged by Scott / Tom.
Quoted 20 lines
Quoted 12 lines
Quoted 12 lines
Quoted 8 lines
Quoted 6 lines
Quoted 2 lines
I think all of those go directly into your tree?
I had filed vybird stuff off to Stefano for some, hopefully correct reason. The FAT change and logo change are me, yes.
I guess the mtd change is rather global...
Ah right. Maybe I gave it to myself? I know I assigned it in patchwork.
Quoted 7 lines
Quoted 4 lines
Quoted 3 lines
Currently board/toradex is not specifically marked to be maintained by Stefano (we also have Tegra boards). But yeah for Vybrid it probably makes sense.
Yes, I saw your patches, I will picked them up after current PR (sent now) will be merged.
Quoted 6 lines
This still did not make it. Can it still be considered for the next release?
There is also:
- board: toradex: colibri_vf: unset NFS and LOADS/B
- ARM: vf610: ddrmc: program Dummy DDRBYTE1/2
Applied to u-boot-imx.
Quoted 9 lines
Quoted 4 lines
Quoted 1 line
- ddr: vybrid: various fixes (patchset)
There is a (slight) conflict with Lukasz's patches regarding DDR calibration. I won't pick Lukasz's for 2019.01,
Ok. No problem.
( The DDR patch series itself:
http://patchwork.ozlabs.org/cover/1008330/)
There is a topic I would like to check. Stefan introduces with his patches a DDRMC_CR80_MC_INIT_COMPLETE that is still missing in your patches. I solved the conflict (not yet pushed to -next) as :
while (!(readl(&ddrmr->cr[80]) & DDRMC_CR80_MC_INIT_COMPLETE))
                 udelay(10);

Quoted 8 lines

writel(DDRMC_CR80_MC_INIT_COMPLETE, &ddrmr->cr[81]);
 #ifdef CONFIG_DDRMC_VF610_CALIBRATION
         ddrmc_calibration(ddrmr);
#endif
But I have no idea if DDRMC_CR80_MC_INIT_COMPLETE should be done before the calibration of maybe after. Could you help me ?

I've already rebased (DDR calibration code) on top of Stefan's patches:

http://git.denx.de/?p=u-boot/u-boot-dfu.git;a=shortlog;h=refs/heads/testing

The code in question looks like in below link:
http://git.denx.de/?p=u-boot/u-boot-dfu.git;a=blob;f=arch/arm/mach-imx/ddrmc-vf610.c;h=9a8e2f6aad10adc45fa791d24ffad52c42c5c492;hb=refs/heads/testing#l231

Quoted 6 lines

Quoted 6 lines
Quoted 5 lines
and I could solve the
conflict in -next branch. IMHO I have no concerns (open points) regarding usage of bit shifting instead of BIT() macro. So I pushed this patchset and I delay Lukasz's to next release.

Quoted 7 lines

Quoted 2 lines
Ok. Thanks for the update.
Best regards,
Stefano

Best regards,

Lukasz Majewski

Stefano Babic

2019-01-10 09:34 UTC #13 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Hi Lukasz,

Quoted 2 lines · Lukasz Majewski

Hi Stefano,

[snip]

Quoted 15 lines

Quoted 6 lines
There is a topic I would like to check. Stefan introduces with his patches a DDRMC_CR80_MC_INIT_COMPLETE that is still missing in your patches. I solved the conflict (not yet pushed to -next) as :
while (!(readl(&ddrmr->cr[80]) & DDRMC_CR80_MC_INIT_COMPLETE))
                 udelay(10);
Quoted 8 lines
writel(DDRMC_CR80_MC_INIT_COMPLETE, &ddrmr->cr[81]);
 #ifdef CONFIG_DDRMC_VF610_CALIBRATION
         ddrmc_calibration(ddrmr);
#endif
But I have no idea if DDRMC_CR80_MC_INIT_COMPLETE should be done before the calibration of maybe after. Could you help me ?

Quoted 6 lines

I've already rebased (DDR calibration code) on top of Stefan's patches:
http://git.denx.de/?p=u-boot/u-boot-dfu.git;a=shortlog;h=refs/heads/testing
The code in question looks like in below link:
http://git.denx.de/?p=u-boot/u-boot-dfu.git;a=blob;f=arch/arm/mach-imx/ddrmc-vf610.c;h=9a8e2f6aad10adc45fa791d24ffad52c42c5c492;hb=refs/heads/testing#l231

ok, thanks - so the DDRMC_CR80_MC_INIT_COMPLETE is called before calibration. I have already did this by rebasing myself. There is no need you repost, I have merged yours patches into my -next branch.

Regards,
Stefano

Signature · 5 lines

=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

Lukasz Majewski

2019-01-10 09:47 UTC #14 lore raw

PATCH 1/1 tools: logos: crop Toradex logo slightly

Quoted 26 lines · On Thu, 10 Jan 2019 10:34:21 +0100 Stefano Babic <[email protected]> wrote:

Hi Lukasz,
Quoted 2 lines · Lukasz Majewski
Hi Stefano,
[snip]
Quoted 18 lines
Quoted 6 lines
There is a topic I would like to check. Stefan introduces with his patches a DDRMC_CR80_MC_INIT_COMPLETE that is still missing in your patches. I solved the conflict (not yet pushed to -next) as :
while (!(readl(&ddrmr->cr[80]) & DDRMC_CR80_MC_INIT_COMPLETE))
                 udelay(10);
Quoted 8 lines
writel(DDRMC_CR80_MC_INIT_COMPLETE, &ddrmr->cr[81]);
 #ifdef CONFIG_DDRMC_VF610_CALIBRATION
         ddrmc_calibration(ddrmr);
#endif
But I have no idea if DDRMC_CR80_MC_INIT_COMPLETE should be done before the calibration of maybe after. Could you help me ?
I've already rebased (DDR calibration code) on top of Stefan's patches:

Quoted 9 lines

Quoted 4 lines
http://git.denx.de/?p=u-boot/u-boot-dfu.git;a=shortlog;h=refs/heads/testing
The code in question looks like in below link:
http://git.denx.de/?p=u-boot/u-boot-dfu.git;a=blob;f=arch/arm/mach-imx/ddrmc-vf610.c;h=9a8e2f6aad10adc45fa791d24ffad52c42c5c492;hb=refs/heads/testing#l231
ok, thanks - so the DDRMC_CR80_MC_INIT_COMPLETE is called before calibration. I have already did this by rebasing myself. There is no need you repost, I have merged yours patches into my -next branch.

Thanks :-) (And I do have more vf610 patches in the queue to move it to DM - as it uses only single u-boot binary).

I will post them when we release v2019.01

Quoted 3 lines

Regards,
Stefano

Best regards,

Lukasz Majewski