From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Thu, 10 Jan 2019 10:29:18 +0100 Subject: [U-Boot] [PATCH v1] tools: logos: crop Toradex logo slightly In-Reply-To: <65e073a5-0508-a05d-efe4-765805322abd@denx.de> References: <20181206135302.5215-1-stefan@agner.ch> <20181208174859.5eb6e0d7@jawa> <6dca2ee67ee7f5d2c6824d745f60f06c@agner.ch> <20181214132039.GE8702@bill-the-cat> <20181214133010.GI8702@bill-the-cat> <96f905a79e1ddc187b371ce566948b6f@agner.ch> <20190109223818.2c590686@jawa> <65e073a5-0508-a05d-efe4-765805322abd@denx.de> Message-ID: <20190110102918.52383ba9@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Stefano, > On 09/01/19 22:38, Lukasz Majewski wrote: > > On Wed, 9 Jan 2019 16:41:40 +0100 > > Stefano Babic 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 > >>>>>>>>>> > >>>>>>>>>> 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-vf610.c;h=9a8e2f6aad10adc45fa791d24ffad52c42c5c492;hb=refs/heads/testing#l231 > > >> 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 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: