[U-Boot] [PATCH v1] tools: logos: crop Toradex logo slightly

From: Stefan Agner stefan.agner@toradex.com
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 stefan.agner@toradex.com ---
tools/logos/toradex.bmp | Bin 24982 -> 23298 bytes 1 file changed, 0 insertions(+), 0 deletions(-)
diff --git a/tools/logos/toradex.bmp b/tools/logos/toradex.bmp index 3e2dcf23358dd46fc7b1bb0dae70d3ba985606ee..8b5cb18e8475b3eda72c9f0e821b2aad17bc1bae 100644 GIT binary patch delta 505 zcmYLEO-oxr6rCd>(HDKjZ>@>Jpis10rD|SMpO}a~;ztZ>#n#YvXDMP^LN{G@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@9V643tT)ro*yB*RRyf5r!m%wIs7(X6Z&}P4>2dpwK^!TCA z#vApR)u#7*|L~|!Y@L=^>|}#ZB|Th=)POrtDVzOa=q<<Og53sg4CGC*R@M$yfp)Br zTCNWr4?o94(4CgaqoH;%G%TNcHS!SLa%5g3A@fQ&UylZ%jYNRCF+X@VE-pJ+(CWB8 zDGB)@pZ0ionrZ=CrbK@;9R;^%B+8u4K>wYo&*H6Kdi+g)O1ODE*9`if$Qv)_#k`r8 Tu-(CC?xpL%<TJT_e17>4;b52`
delta 656 zcmXw!ODIHP6vw|?uE!njFhgTzOi3nYyasb;yv8sxo^!iK9tlwv3RzGrG(|QZDZa9j zB?~JZ+1My$i;az@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+QS`l^!nSw-~J;C6l%lwr~5F=qKz zmy?(Jy*yd_Wjye{4y`=ZDsxrI1-IXU5n+X*mIJivEJ(i7hzuVMUi3JcX#IAJ8&;aF zsBO3WN1EXwgprVHrN`B`5$D^R(R&&puNHN|Pe+lV@v!3JlZpi4&OG#JQJUH6Lh5}x zJarfUd2L|nEy26~M84?DfS$fmoLlIpxszBE*3T+sI<+ON_XaQ<ZsTQGzYS9D9-_I) WVdOzbgrkvqwA_u-FJz9T1ojITOUl0h

On Thu, 6 Dec 2018 14:53:02 +0100 Stefan Agner stefan@agner.ch wrote:
From: Stefan Agner stefan.agner@toradex.com
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 stefan.agner@toradex.com
tools/logos/toradex.bmp | Bin 24982 -> 23298 bytes 1 file changed, 0 insertions(+), 0 deletions(-)
diff --git a/tools/logos/toradex.bmp b/tools/logos/toradex.bmp index 3e2dcf23358dd46fc7b1bb0dae70d3ba985606ee..8b5cb18e8475b3eda72c9f0e821b2aad17bc1bae 100644 GIT binary patch delta 505 zcmYLEO-oxr6rCd>(HDKjZ>@>Jpis10rD|SMpO}a~;ztZ>#n#YvXDMP^LN{G@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@9V643tT)ro*yB*RRyf5r!m%wIs7(X6Z&}P4>2dpwK^!TCA z#vApR)u#7*|L~|!Y@L=^>|}#ZB|Th=)POrtDVzOa=q<<Og53sg4CGC*R@M$yfp)Br zTCNWr4?o94(4CgaqoH;%G%TNcHS!SLa%5g3A@fQ&UylZ%jYNRCF+X@VE-pJ+(CWB8 zDGB)@pZ0ionrZ=CrbK@;9R;^%B+8u4K>wYo&*H6Kdi+g)O1ODE*9`if$Qv)_#k`r8 Tu-(CC?xpL%<TJT_e17>4;b52`
delta 656 zcmXw!ODIHP6vw|?uE!njFhgTzOi3nYyasb;yv8sxo^!iK9tlwv3RzGrG(|QZDZa9j zB?~JZ+1My$i;az@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+QS`l^!nSw-~J;C6l%lwr~5F=qKz zmy?(Jy*yd_Wjye{4y`=ZDsxrI1-IXU5n+X*mIJivEJ(i7hzuVMUi3JcX#IAJ8&;aF zsBO3WN1EXwgprVHrN`B`5$D^R(R&&puNHN|Pe+lV@v!3JlZpi4&OG#JQJUH6Lh5}x zJarfUd2L|nEy26~M84?DfS$fmoLlIpxszBE*3T+sI<+ON_XaQ<ZsTQGzYS9D9-_I) WVdOzbgrkvqwA_u-FJz9T1ojITOUl0h
Reviewed-by: Lukasz Majewski lukma@denx.de
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@denx.de

Hi Stefan,
From: Stefan Agner stefan.agner@toradex.com
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 lukma@denx.de
Thanks Stefan :-)
Signed-off-by: Stefan Agner stefan.agner@toradex.com
tools/logos/toradex.bmp | Bin 24982 -> 23298 bytes 1 file changed, 0 insertions(+), 0 deletions(-)
diff --git a/tools/logos/toradex.bmp b/tools/logos/toradex.bmp index 3e2dcf23358dd46fc7b1bb0dae70d3ba985606ee..8b5cb18e8475b3eda72c9f0e821b2aad17bc1bae 100644 GIT binary patch delta 505 zcmYLEO-oxr6rCd>(HDKjZ>@>Jpis10rD|SMpO}a~;ztZ>#n#YvXDMP^LN{G@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@9V643tT)ro*yB*RRyf5r!m%wIs7(X6Z&}P4>2dpwK^!TCA z#vApR)u#7*|L~|!Y@L=^>|}#ZB|Th=)POrtDVzOa=q<<Og53sg4CGC*R@M$yfp)Br zTCNWr4?o94(4CgaqoH;%G%TNcHS!SLa%5g3A@fQ&UylZ%jYNRCF+X@VE-pJ+(CWB8 zDGB)@pZ0ionrZ=CrbK@;9R;^%B+8u4K>wYo&*H6Kdi+g)O1ODE*9`if$Qv)_#k`r8 Tu-(CC?xpL%<TJT_e17>4;b52`
delta 656 zcmXw!ODIHP6vw|?uE!njFhgTzOi3nYyasb;yv8sxo^!iK9tlwv3RzGrG(|QZDZa9j zB?~JZ+1My$i;az@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+QS`l^!nSw-~J;C6l%lwr~5F=qKz zmy?(Jy*yd_Wjye{4y`=ZDsxrI1-IXU5n+X*mIJivEJ(i7hzuVMUi3JcX#IAJ8&;aF zsBO3WN1EXwgprVHrN`B`5$D^R(R&&puNHN|Pe+lV@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@denx.de

Hi Tom,
On 08.12.2018 17:48, Lukasz Majewski wrote:
Hi Stefan,
From: Stefan Agner stefan.agner@toradex.com
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
I think all of those go directly into your tree?
-- Stefan
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 lukma@denx.de
Thanks Stefan :-)
Signed-off-by: Stefan Agner stefan.agner@toradex.com
tools/logos/toradex.bmp | Bin 24982 -> 23298 bytes 1 file changed, 0 insertions(+), 0 deletions(-)
diff --git a/tools/logos/toradex.bmp b/tools/logos/toradex.bmp index 3e2dcf23358dd46fc7b1bb0dae70d3ba985606ee..8b5cb18e8475b3eda72c9f0e821b2aad17bc1bae 100644 GIT binary patch delta 505 zcmYLEO-oxr6rCd>(HDKjZ>@>Jpis10rD|SMpO}a~;ztZ>#n#YvXDMP^LN{G@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@9V643tT)ro*yB*RRyf5r!m%wIs7(X6Z&}P4>2dpwK^!TCA z#vApR)u#7*|L~|!Y@L=^>|}#ZB|Th=)POrtDVzOa=q<<Og53sg4CGC*R@M$yfp)Br zTCNWr4?o94(4CgaqoH;%G%TNcHS!SLa%5g3A@fQ&UylZ%jYNRCF+X@VE-pJ+(CWB8 zDGB)@pZ0ionrZ=CrbK@;9R;^%B+8u4K>wYo&*H6Kdi+g)O1ODE*9`if$Qv)_#k`r8 Tu-(CC?xpL%<TJT_e17>4;b52`
delta 656 zcmXw!ODIHP6vw|?uE!njFhgTzOi3nYyasb;yv8sxo^!iK9tlwv3RzGrG(|QZDZa9j zB?~JZ+1My$i;az@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+QS`l^!nSw-~J;C6l%lwr~5F=qKz zmy?(Jy*yd_Wjye{4y`=ZDsxrI1-IXU5n+X*mIJivEJ(i7hzuVMUi3JcX#IAJ8&;aF zsBO3WN1EXwgprVHrN`B`5$D^R(R&&puNHN|Pe+lV@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@denx.de

On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner wrote:
Hi Tom,
On 08.12.2018 17:48, Lukasz Majewski wrote:
Hi Stefan,
From: Stefan Agner stefan.agner@toradex.com
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
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.

On 14.12.2018 14:20, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner wrote:
Hi Tom,
On 08.12.2018 17:48, Lukasz Majewski wrote:
Hi Stefan,
From: Stefan Agner stefan.agner@toradex.com
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
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

On Fri, Dec 14, 2018 at 02:26:22PM +0100, Stefan Agner wrote:
On 14.12.2018 14:20, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner wrote:
Hi Tom,
On 08.12.2018 17:48, Lukasz Majewski wrote:
Hi Stefan,
From: Stefan Agner stefan.agner@toradex.com
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
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.
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).

On 14/12/18 14:30, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:26:22PM +0100, Stefan Agner wrote:
On 14.12.2018 14:20, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner wrote:
Hi Tom,
On 08.12.2018 17:48, Lukasz Majewski wrote:
Hi Stefan,
From: Stefan Agner stefan.agner@toradex.com
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
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.
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.
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

Hi Stefano,
On 14.12.2018 14:40, Stefano Babic wrote:
On 14/12/18 14:30, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:26:22PM +0100, Stefan Agner wrote:
On 14.12.2018 14:20, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner wrote:
Hi Tom,
On 08.12.2018 17:48, Lukasz Majewski wrote:
Hi Stefan,
> From: Stefan Agner stefan.agner@toradex.com > > 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
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.
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
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

Hi Stefan, Lukasz,
On 04/01/19 16:05, Stefan Agner wrote:
Hi Stefano,
On 14.12.2018 14:40, Stefano Babic wrote:
On 14/12/18 14:30, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:26:22PM +0100, Stefan Agner wrote:
On 14.12.2018 14:20, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner wrote:
Hi Tom,
On 08.12.2018 17:48, Lukasz Majewski wrote: > Hi Stefan, > >> From: Stefan Agner stefan.agner@toradex.com >> >> 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.
> [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.
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.
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
Applied to u-boot-imx.
- 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

On Wed, 9 Jan 2019 16:41:40 +0100 Stefano Babic sbabic@denx.de wrote:
Hi Stefan, Lukasz,
On 04/01/19 16:05, Stefan Agner wrote:
Hi Stefano,
On 14.12.2018 14:40, Stefano Babic wrote:
On 14/12/18 14:30, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:26:22PM +0100, Stefan Agner wrote:
On 14.12.2018 14:20, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner wrote: > Hi Tom, > > On 08.12.2018 17:48, Lukasz Majewski wrote: >> Hi Stefan, >> >>> From: Stefan Agner stefan.agner@toradex.com >>> >>> 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.
>> [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.
> > 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.
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
Applied to u-boot-imx.
- 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/)
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
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@denx.de

On 09/01/19 22:38, Lukasz Majewski wrote:
On Wed, 9 Jan 2019 16:41:40 +0100 Stefano Babic sbabic@denx.de wrote:
Hi Stefan, Lukasz,
On 04/01/19 16:05, Stefan Agner wrote:
Hi Stefano,
On 14.12.2018 14:40, Stefano Babic wrote:
On 14/12/18 14:30, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:26:22PM +0100, Stefan Agner wrote:
On 14.12.2018 14:20, Tom Rini wrote: > On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner wrote: >> Hi Tom, >> >> On 08.12.2018 17:48, Lukasz Majewski wrote: >>> Hi Stefan, >>> >>>> From: Stefan Agner stefan.agner@toradex.com >>>> >>>> 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.
>>> [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.
>> >> 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.
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
Applied to u-boot-imx.
- 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 ?
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

Hi Stefano,
On 09/01/19 22:38, Lukasz Majewski wrote:
On Wed, 9 Jan 2019 16:41:40 +0100 Stefano Babic sbabic@denx.de wrote:
Hi Stefan, Lukasz,
On 04/01/19 16:05, Stefan Agner wrote:
Hi Stefano,
On 14.12.2018 14:40, Stefano Babic wrote:
On 14/12/18 14:30, Tom Rini wrote:
On Fri, Dec 14, 2018 at 02:26:22PM +0100, Stefan Agner wrote: > On 14.12.2018 14:20, Tom Rini wrote: >> On Fri, Dec 14, 2018 at 02:13:46PM +0100, Stefan Agner >> wrote: >>> Hi Tom, >>> >>> On 08.12.2018 17:48, Lukasz Majewski wrote: >>>> Hi Stefan, >>>> >>>>> From: Stefan Agner stefan.agner@toradex.com >>>>> >>>>> 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.
>>>> [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.
>>> >>> 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.
> 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
Applied to u-boot-imx.
- 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 ?
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...
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
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@denx.de

Hi Lukasz,
On 10/01/19 10:29, Lukasz Majewski wrote:
Hi Stefano,
[snip]
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 ?
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...
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

On Thu, 10 Jan 2019 10:34:21 +0100 Stefano Babic sbabic@denx.de wrote:
Hi Lukasz,
On 10/01/19 10:29, Lukasz Majewski wrote:
Hi Stefano,
[snip]
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 ?
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...
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
Regards, Stefano
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@denx.de
participants (4)
-
Lukasz Majewski
-
Stefan Agner
-
Stefano Babic
-
Tom Rini