Hi Tom, On 11 December 2014 at 08:58, Tom Warren <twar...@nvidia.com> wrote: > Simon, > > Feel free to do everything in the DM tree. Once DM makes it to mainline, and > hence to ARM and Tegra, we'll be better able to add Tegra things into the > Tegra repo.
OK, I'll pull in the nyan-big patches to DM then I2C and retest. - Simon > > Tom > >> -----Original Message----- >> From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass >> Sent: Thursday, December 11, 2014 7:22 AM >> To: Tom Warren >> Cc: U-Boot Mailing List; Stephen Warren; Allen Martin; Stephen Warren; >> Albert Aribaud >> Subject: Re: [PATCH v5 3/3] ARM: tegra: Add support for nyan-big board >> >> Hi Tom, >> >> On 10 December 2014 at 14:45, Simon Glass <s...@chromium.org> wrote: >> > >> > Hi Tom, >> > >> > On 10 December 2014 at 14:09, Tom Warren <twar...@nvidia.com> >> wrote: >> > > Simon, >> > > >> > >> -----Original Message----- >> > >> From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon >> > >> Glass >> > >> Sent: Wednesday, December 10, 2014 2:04 PM >> > >> To: Tom Warren >> > >> Cc: U-Boot Mailing List; Stephen Warren; Allen Martin; Stephen >> > >> Warren; Albert Aribaud >> > >> Subject: Re: [PATCH v5 3/3] ARM: tegra: Add support for nyan-big >> > >> board >> > >> >> > >> Hi Tom, >> > >> >> > >> On 10 December 2014 at 12:36, Simon Glass <s...@chromium.org> >> wrote: >> > >> > Hi Tom, >> > >> > >> > >> > On Dec 10, 2014 12:05 PM, "Tom Warren" <twar...@nvidia.com> >> wrote: >> > >> >> >> > >> >> Simon, >> > >> >> >> > >> >> > -----Original Message----- >> > >> >> > From: s...@google.com [mailto:s...@google.com] On Behalf Of >> > >> >> > Simon Glass >> > >> >> > Sent: Wednesday, December 10, 2014 10:50 AM >> > >> >> > To: Tom Warren >> > >> >> > Cc: U-Boot Mailing List; Allen Martin; Albert Aribaud; Stephen >> > >> >> > Warren; Stephen Warren >> > >> >> > Subject: Re: [PATCH v5 3/3] ARM: tegra: Add support for >> > >> >> > nyan-big board >> > >> >> > >> > >> >> > Hi Tom, >> > >> >> > >> > >> >> > >> > >> >> > On 10 December 2014 at 10:39, Tom Warren >> <twar...@nvidia.com> >> > >> wrote: >> > >> >> > > Simon, >> > >> >> > > >> > >> >> > > The v4 PCIE patch series fails to apply to >> > >> >> > > u-boot-tegra/master after >> > >> >> > rebasing my repo against both ARM master and U-Boot 'master' >> > >> >> > master (i.e. >> > >> >> > FETCH_HEAD = >> > >> >> > > 32fdf0e4d82bdca5d64d86330e461e59685f9959 branch >> 'master' of >> > >> >> > git://git.denx.de/u-boot). 3 - 4 patches fail when using git am. >> > >> >> > > >> > >> >> > > I worked thru the conflicts for both branches (my master >> > >> >> > > rebased against >> > >> >> > ARM and U-Boot TOT), and did a MAKEALL -s tegra, and >> > >> >> > jetson-tk1, beaver and cardhu fail to build (jetson err log >> > >> >> > below). Looks like you developed this with your DM I2C series in >> place? >> > >> >> > > >> > >> >> > > To submit a PR from the Tegra repo, I need to either rebase >> > >> >> > > first against >> > >> >> > ARM master (my SOP), or against U-Boot 'master'. So you >> > >> >> > patches, if they're bound for the Tegra repo, need to work in >> > >> >> > one of those scenarios. >> > >> >> > > >> > >> >> > > Tom >> > >> >> > >> > >> >> > Thanks for looking at it. This series depends on the I2C >> > >> >> > driver model series. >> > >> >> > It's available at u-boot-dm/i2c-working if you want to check >> > >> >> > it out. I plan to apply that very soon. >> > >> >> > >> > >> >> > I'm sorry I didn't make that clear in this series. I mentioned >> > >> >> > it on the list at one point but should have added it here. If >> > >> >> > you like, you could ack it and I can apply to u-boot-dm after >> > >> >> > I2C is in there. >> > >> >> I can't ACK something that I can't get to apply to my repo nor >> > >> >> build nor test. So you'll have to wait on that from Stephen, >> > >> >> Thierry (based on code >> > >> >> review) or someone that wants to add DM I2C to Tegra, apply your >> > >> >> PCIe patches, and test. That's not my normal workflow, though, >> > >> >> and I don't have the BW to devote to extra hurdles right now. >> > >> >> >> > >> > >> > >> > You could try applying to u-boot-dm/i2c-working. It only has the >> > >> > DM i2c series in it, plus Tegra stuff. I put a lot of time into >> > >> > cleaning up this Tegra series to save you and Thierry time. >> > >> >> > >> Also Thierry took a look the PCIe series and responded here: >> > >> >> > >> http://patchwork.ozlabs.org/patch/410175/ >> > >> >> > >> There were three nits which I have addressed. >> > >> >> > >> For this Nyan-big patch I really don't need a test credit (would be >> > >> nice, but not necessary). After all it's a new board and if >> > >> problems are found they can be fixed. But I do need to apply it before >> DM I2C. >> > >> Stephen has looked at a few revs. >> > >> >> > >> How about this for a plan: >> > >> >> > >> - Apply Nyan-big to DM >> > >> - Apply DM I2C series to DM >> > >> - Pull request to mainline (final for DM this release unless SPL is >> > >> resolved >> > >> somehow) >> > >> >> > >> Then if you have time: >> > >> - Tom - apply PCIe to Tegra tree >> > >> - Pull request to mainline >> > > The problem there is that I'd need to wait for the PR to mainline to go >> thru before I could rebase Tegra against mainline, and only then apply PCIe >> patches. Time-wise, it's obviously better for you to do this in one fell >> swoop >> from the DM repo. >> > > >> > > Does the nyan-big patchset apply/build cleanly in u-boot-tegra/master? If >> so, I can apply it and do a Tegra test build and then send a PR to Albert if >> all is >> clear. >> > >> > OK, true. >> > >> > The Tegra tree is too far behind master, but if you catch it up you >> > can make it work: >> > >> > (try-nyan=9e10b9: b/ common/ doc/ sandbox/) u> pwclient git-am 417781 >> > Applying patch #417781 using 'git am' >> > Description: [U-Boot,v5,1/3] dts: Bring in Chrome OS keyboard device >> > tree definition >> > Applying: dts: Bring in Chrome OS keyboard device tree definition >> > (try-nyan=ddebcc: b/ common/ doc/ sandbox/) u> pwclient git-am 417782 >> > Applying patch #417782 using 'git am' >> > Description: [U-Boot,v5,2/3] tegra: dts: Sync tegra124.dtsi with linux >> > kernel >> > Applying: tegra: dts: Sync tegra124.dtsi with linux kernel >> > ^[[A(try-nyan=9df313: b/ common/ doc/ sandbox/) u> pwclient git-am >> > 417783 Applying patch #417783 using 'git am' >> > Description: [U-Boot,v5,3/3] ARM: tegra: Add support for nyan-big >> > board >> > Applying: ARM: tegra: Add support for nyan-big board >> > error: patch failed: arch/arm/dts/Makefile:31 >> > error: arch/arm/dts/Makefile: patch does not apply Patch failed at >> > 0001 ARM: tegra: Add support for nyan-big board The copy of the patch >> > that failed is found in: >> > /usr/local/google/home/sjg/u/.git/rebase-apply/patch >> > When you have resolved this problem, run "git am --continue". >> > If you prefer to skip this patch, run "git am --skip" instead. >> > To restore the original branch and stop patching, run "git am --abort". >> > (try-nyan=9df313: b/ common/ doc/ sandbox/) u> patch -p1 < >> > /usr/local/google/home/sjg/u/.git/rebase-apply/patch >> > patching file arch/arm/cpu/armv7/tegra124/Kconfig >> > patching file arch/arm/dts/Makefile >> > Hunk #1 succeeded at 29 with fuzz 2 (offset -2 lines). >> > patching file arch/arm/dts/tegra124-nyan-big.dts >> > patching file board/nvidia/nyan-big/Kconfig patching file >> > board/nvidia/nyan-big/MAINTAINERS patching file >> > board/nvidia/nyan-big/Makefile patching file >> > board/nvidia/nyan-big/nyan-big.c patching file >> > board/nvidia/nyan-big/pinmux-config-nyan-big.h >> > patching file board/nvidia/venice2/as3722_init.h >> > patching file configs/nyan-big_defconfig patching file >> > include/configs/nyan-big.h >> > >> > Then: >> > >> > git add arch/arm/cpu/armv7/tegra124/Kconfig arch/arm/dts/Makefile >> > board/nvidia/venice2/as3722_init.h arch/arm/dts/tegra124-nyan-big.dts >> > board/nvidia/nyan-big/ configs/nyan-big_defconfig >> > include/configs/nyan-big.h git am --resolved >> > >> > I've pushed it to u-boot-dm/for-tom. >> > >> > It all builds cleanly for me. >> > >> > Regards, >> > Simon >> >> Further to this, the I2C patches are in u-boot-dm/master now, except for this >> Tegra I2C conversion, which depends on the nyan-big series: >> >> http://patchwork.ozlabs.org/patch/419730/ >> >> So I would like to bring in the nyan-big series to the DM tree, add this >> Tegra >> I2C patch and then we are ready to sort out the PCIe series. What do you >> think of that plan? Or if you prefer, I can issue a pull request to mainline >> for >> the DM stuff to date and you can do the rest? >> >> Regards, >> Simon >> >> > >> > > >> > >> >> > >> If you don't have time, I could do this last bit through DM. In any >> > >> case the problem is that it is getting late and we need to resolve this >> soon. >> > >> >> > >> Regards, >> > >> Simon >> > >> >> > >> > >> > >> > Regards, >> > >> > Simon >> > >> > >> > >> >> Tom >> > >> >> > >> > >> >> > Regards, >> > >> >> > Simon >> > >> >> > > >> > >> >> > > drivers/power/as3722.c: In function 'as3722_read': >> > >> >> > > drivers/power/as3722.c:34: warning: passing argument 1 of >> 'i2c_read' >> > >> >> > > makes integer from pointer without a cast >> > >> >> > > include/i2c.h:235: note: expected 'uint8_t' but argument is >> > >> >> > > of type 'struct >> > >> >> > udevice *' >> > >> >> > > drivers/power/as3722.c:34: warning: passing argument 3 of >> 'i2c_read' >> > >> >> > > makes integer from pointer without a cast >> > >> >> > > include/i2c.h:235: note: expected 'int' but argument is of type >> > >> >> > > 'u8 >> *' >> > >> >> > > drivers/power/as3722.c:34: warning: passing argument 4 of >> 'i2c_read' >> > >> >> > > makes pointer from integer without a cast >> > >> >> > > include/i2c.h:235: note: expected 'uint8_t *' but argument >> > >> >> > > is of type 'int' >> > >> >> > > drivers/power/as3722.c:34: error: too few arguments to >> > >> >> > > function 'i2c_read' >> > >> >> > > drivers/power/as3722.c: In function 'as3722_write': >> > >> >> > > drivers/power/as3722.c:45: warning: passing argument 1 of >> 'i2c_write' >> > >> >> > > makes integer from pointer without a cast >> > >> >> > > include/i2c.h:238: note: expected 'uint8_t' but argument is >> > >> >> > > of type 'struct >> > >> >> > udevice *' >> > >> >> > > drivers/power/as3722.c:45: warning: passing argument 3 of >> 'i2c_write' >> > >> >> > > makes integer from pointer without a cast >> > >> >> > > include/i2c.h:238: note: expected 'int' but argument is of type >> > >> >> > > 'u8 >> *' >> > >> >> > > drivers/power/as3722.c:45: warning: passing argument 4 of >> 'i2c_write' >> > >> >> > > makes pointer from integer without a cast >> > >> >> > > include/i2c.h:238: note: expected 'uint8_t *' but argument >> > >> >> > > is of type 'int' >> > >> >> > > drivers/power/as3722.c:45: error: too few arguments to >> > >> >> > > function >> > >> >> > 'i2c_write' >> > >> >> > > drivers/power/as3722.c: In function 'as3722_init': >> > >> >> > > drivers/power/as3722.c:245: warning: implicit declaration of >> > >> >> > > function >> > >> >> > 'i2c_get_chip_for_busnum' >> > >> >> > > make[1]: *** [drivers/power/as3722.o] Error 1 >> > >> >> > > make: *** [drivers/power] Error 2 >> > >> >> > > make: *** Waiting for unfinished jobs.... >> > >> >> > > >> > >> >> > >> -----Original Message----- >> > >> >> > >> From: Tom Warren >> > >> >> > >> Sent: Wednesday, December 10, 2014 8:46 AM >> > >> >> > >> To: Simon Glass; U-Boot Mailing List >> > >> >> > >> Cc: Allen Martin; Albert Aribaud; Stephen Warren; Stephen >> > >> >> > >> Warren >> > >> >> > >> Subject: RE: [PATCH v5 3/3] ARM: tegra: Add support for >> > >> >> > >> nyan-big board >> > >> >> > >> >> > >> >> > >> Simon, >> > >> >> > >> >> > >> >> > >> > -----Original Message----- >> > >> >> > >> > From: s...@google.com [mailto:s...@google.com] On Behalf Of >> > >> >> > >> > Simon Glass >> > >> >> > >> > Sent: Wednesday, December 10, 2014 8:01 AM >> > >> >> > >> > To: U-Boot Mailing List >> > >> >> > >> > Cc: Allen Martin; Simon Glass; Albert Aribaud; Tom >> > >> >> > >> > Warren; Stephen Warren; Stephen Warren >> > >> >> > >> > Subject: Re: [PATCH v5 3/3] ARM: tegra: Add support for >> > >> >> > >> > nyan-big board >> > >> >> > >> > >> > >> >> > >> > Hi, >> > >> >> > >> > >> > >> >> > >> > On 4 December 2014 at 06:36, Simon Glass >> > >> >> > >> > <s...@chromium.org> >> > >> wrote: >> > >> >> > >> > > From: Allen Martin <amar...@nvidia.com> >> > >> >> > >> > > >> > >> >> > >> > > Nyan-big is a Tegra124 clamshell board that is very >> > >> >> > >> > > similar to venice2, but it has a different panel, the >> > >> >> > >> > > sdcard cd and wp sense are flipped, and it has a >> > >> >> > >> > > different revision of the AS3722 >> > >> PMIC. >> > >> >> > >> > > >> > >> >> > >> > > This is the Acer Chromebook 13 CB5-311-T7NN (13.3-inch >> > >> >> > >> > > HD, NVIDIA Tegra K1, 2GB). The display is not currently >> > >> >> > >> > > supported, so it should boot on other nyan-based >> > >> >> > >> > > Chromebooks also, but only the device tree for nyan-big is >> provided here. >> > >> >> > >> > > >> > >> >> > >> > > The device tree file is from Linux but with features >> > >> >> > >> > > removed which are unlikely to be supported in U-Boot >> > >> >> > >> > > soon (regulators, >> > >> >> > pinmux). >> > >> >> > >> > > Also the addresses are updated to 32-bit. >> > >> >> > >> > > >> > >> >> > >> > > Signed-off-by: Allen Martin <amar...@nvidia.com> >> > >> >> > >> > > Signed-off-by: Simon Glass <s...@chromium.org> (rebase, >> > >> >> > >> > > change to 'nyan-big', fix pinmux that resets nyan-big) >> > >> >> > >> > > >> > >> >> > >> > > --- >> > >> >> > >> > > >> > >> >> > >> > > Changes in v5: >> > >> >> > >> > > - Remove call to pinmux_set_tristate_input_clamping() >> > >> >> > >> > > >> > >> >> > >> > >> > >> >> > >> > Any further comments on v5 please? >> > >> >> > >> > >> > >> >> > >> > I would like to apply this patch before the DM I2C >> > >> >> > >> > series, so that I2C works correct for nyan-big also. >> > >> >> > >> > >> > >> >> > >> > Tom are you looking at the rest of the PCI-e series? I >> > >> >> > >> > resent it so would like to get that in soon - the patches >> > >> >> > >> > originally date from August. >> > >> >> > >> I was waiting for Thierry or Stephen to Ack the series, or >> > >> >> > >> provide other feedback. I don't have time/boards to test >> > >> >> > >> this right >> > >> now. >> > >> >> > >> >> > >> >> > >> If someone can say it looks good, or has been tested on X >> board, etc. >> > >> >> > >> I can apply it. >> > >> >> > >> >> > >> >> > >> Tom >> > >> >> > >> > >> > >> >> > >> > Regards, >> > >> >> > >> > Simon >> > >> >> > >> -- >> > >> >> > >> nvpublic _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot