Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hi Albert, On 26 October 2014 13:07, Albert ARIBAUD wrote: > Le Tue, 26 Aug 2014 17:33:48 +0200, > Thierry Reding a écrit : > >> From: Thierry Reding >> >> This series adds PCIe support for Tegra20, Tegra30 and Tegra124. The >> size is mostly due to the large number of infrastructure that's added >> (libfdt, Tegra specific drivers required by the PCIe driver). In this >> version I've included all patches that were previously split into >> three separate series. Spreading them over multiple series is >> probably not worth it since there might be some dependencies between >> them and only the end result gives a completely working setup. >> >> Patch 1 adds a %pa modifier to printf() to print physical addresses. >> This is required to allow code to output such addresses irrespective >> of whether a 64 bit or 32 bit architecture is built for. >> >> Patches 2-7 add various FDT helpers to make it easier to parse >> complex device trees. >> >> Patch 8 is a minor cleanup to the PCI command that prevents a spew of >> error messages if a bus does not exist. Patch 9 modifies the PCI >> enumeration code to honor theh restrictions encoded within a host >> controller driver's pci_ski_dev() implementation. This is required to >> prevent exceptions from the NVIDIA Tegra PCIe controller. >> >> Patch 10 imports the pr_fmt() macro used within the Linux kernel to >> reduce the number of characters consumed by literal strings by >> allowing a source file to specify a prefix or suffix that should be >> applied to all format strings in the file. It is currently used by >> the debug() and error() macros. >> >> Patches 11-14 are some cleanup and refactoring of I2C core code, the >> addition of a higher level API that makes it easier for I2C client >> drivers to talk to devices. The Tegra I2C driver now implements >> i2c_get_bus_num_fdt() to obtain bus numbers corresponding to a DT >> node. >> >> Patch 15 implements a driver for the AS3722 PMIC used on the Venice2 >> and Jetson TK1 boards. >> >> Patches 16-21 are preparatory work for the Tegra PCIe controller. >> They add missing clock driver functionality as well as drivers for >> the Tegra powergate and XUSB pad controller blocks. >> >> Patch 22 adds the PCIe controller driver for Tegra20, Tegra30 and >> Tegra124. >> >> Device tree nodes and configurations options to enable PCIe on the >> TrimSlice (Tegra20), Beaver, Cardhu (Tegra30) and Jetson TK1 >> (Tegra124) boards are added in patches 23-31. >> >> Patches 32-35 implement non-cached memory support that will be used >> in the last batch of patches to implement more reliable packet >> transfers in the r8169 driver. Patch 36 enables non-cached memory >> support on Tegra. >> >> Finally, patches 37-40 implement non-cached memory support and >> various fixes in the r8169 driver and add support for the revision of >> the NIC found on the Jetson TK1. >> >> The above boards all have an ethernet NIC connected to PCIe, which is >> what I tested with. >> >> Thierry Reding (40): >> vsprintf: Add modifier for phys_addr_t >> fdt: Add a function to count strings >> fdt: Add a function to get the index of a string >> fdt: Add functions to retrieve strings >> fdt: Add resource parsing functions >> fdt: Add a function to return PCI BDF triplet >> fdt: Add a subnodes iterator macro >> pci: Abort early if bus does not exist >> pci: Honour pci_skip_dev() >> Add pr_fmt() macro >> i2c: Initialize the correct bus >> i2c: Refactor adapter initialization >> i2c: Add high-level API >> i2c: tegra: Implement i2c_get_bus_num_fdt() >> power: Add AMS AS3722 PMIC support >> ARM: tegra: Implement tegra_plle_enable() >> ARM: tegra: Provide PCIEXCLK reset ID >> ARM: tegra: Implement powergate support >> ARM: tegra: Implement XUSB pad controller >> ARM: tegra: Add XUSB pad controller on Tegra124 >> ARM: tegra: Enable XUSB pad controller on Jetson TK1 >> pci: tegra: Add Tegra PCIe driver >> ARM: tegra: Add Tegra20 PCIe device tree node >> ARM: tegra: Enable PCIe on TrimSlice >> ARM: tegra: Add GIC for Tegra30 >> ARM: tegra: Add Tegra30 PCIe device tree node >> ARM: tegra: Enable PCIe on Cardhu >> ARM: tegra: Enable PCIe on Beaver >> ARM: tegra: Add GIC for Tegra124 >> ARM: tegra: Add Tegra124 PCIe device tree node >> ARM: tegra: Enable PCIe on Jetson TK1 >> ARM: cache_v7: Various minor cleanups >> ARM: cache-cp15: Use more accurate types >> malloc: Output region when debugging >> ARM: Implement non-cached memory support >> ARM: tegra: Enable non-cached memory >> net: rtl8169: Honor CONFIG_SYS_RX_ETH_BUFFER >> net: rtl8169: Properly align buffers >> net: rtl8169: Use non-cached memory if available >> net: rtl8169: Add support for RTL-8168/8111g >> >> README | 19 + >> arch/arm/cpu/armv7/cache_v7.c| 14 +- >> arch/arm/cpu/tegra-common/Makefile |2 + >> arch/arm/cpu/tegra-common/powergate.c
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
On Sun, 26 Oct 2014 20:07:10 +0100, Albert ARIBAUD a écrit : > delegated to a single custodian -- right now it is split over five > custodians (although part of it -- so that all of it is applied in Sorry for the "(although part of it" which I should have edited out. Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Le Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding a écrit : > From: Thierry Reding > > This series adds PCIe support for Tegra20, Tegra30 and Tegra124. The > size is mostly due to the large number of infrastructure that's added > (libfdt, Tegra specific drivers required by the PCIe driver). In this > version I've included all patches that were previously split into > three separate series. Spreading them over multiple series is > probably not worth it since there might be some dependencies between > them and only the end result gives a completely working setup. > > Patch 1 adds a %pa modifier to printf() to print physical addresses. > This is required to allow code to output such addresses irrespective > of whether a 64 bit or 32 bit architecture is built for. > > Patches 2-7 add various FDT helpers to make it easier to parse > complex device trees. > > Patch 8 is a minor cleanup to the PCI command that prevents a spew of > error messages if a bus does not exist. Patch 9 modifies the PCI > enumeration code to honor theh restrictions encoded within a host > controller driver's pci_ski_dev() implementation. This is required to > prevent exceptions from the NVIDIA Tegra PCIe controller. > > Patch 10 imports the pr_fmt() macro used within the Linux kernel to > reduce the number of characters consumed by literal strings by > allowing a source file to specify a prefix or suffix that should be > applied to all format strings in the file. It is currently used by > the debug() and error() macros. > > Patches 11-14 are some cleanup and refactoring of I2C core code, the > addition of a higher level API that makes it easier for I2C client > drivers to talk to devices. The Tegra I2C driver now implements > i2c_get_bus_num_fdt() to obtain bus numbers corresponding to a DT > node. > > Patch 15 implements a driver for the AS3722 PMIC used on the Venice2 > and Jetson TK1 boards. > > Patches 16-21 are preparatory work for the Tegra PCIe controller. > They add missing clock driver functionality as well as drivers for > the Tegra powergate and XUSB pad controller blocks. > > Patch 22 adds the PCIe controller driver for Tegra20, Tegra30 and > Tegra124. > > Device tree nodes and configurations options to enable PCIe on the > TrimSlice (Tegra20), Beaver, Cardhu (Tegra30) and Jetson TK1 > (Tegra124) boards are added in patches 23-31. > > Patches 32-35 implement non-cached memory support that will be used > in the last batch of patches to implement more reliable packet > transfers in the r8169 driver. Patch 36 enables non-cached memory > support on Tegra. > > Finally, patches 37-40 implement non-cached memory support and > various fixes in the r8169 driver and add support for the revision of > the NIC found on the Jetson TK1. > > The above boards all have an ethernet NIC connected to PCIe, which is > what I tested with. > > Thierry Reding (40): > vsprintf: Add modifier for phys_addr_t > fdt: Add a function to count strings > fdt: Add a function to get the index of a string > fdt: Add functions to retrieve strings > fdt: Add resource parsing functions > fdt: Add a function to return PCI BDF triplet > fdt: Add a subnodes iterator macro > pci: Abort early if bus does not exist > pci: Honour pci_skip_dev() > Add pr_fmt() macro > i2c: Initialize the correct bus > i2c: Refactor adapter initialization > i2c: Add high-level API > i2c: tegra: Implement i2c_get_bus_num_fdt() > power: Add AMS AS3722 PMIC support > ARM: tegra: Implement tegra_plle_enable() > ARM: tegra: Provide PCIEXCLK reset ID > ARM: tegra: Implement powergate support > ARM: tegra: Implement XUSB pad controller > ARM: tegra: Add XUSB pad controller on Tegra124 > ARM: tegra: Enable XUSB pad controller on Jetson TK1 > pci: tegra: Add Tegra PCIe driver > ARM: tegra: Add Tegra20 PCIe device tree node > ARM: tegra: Enable PCIe on TrimSlice > ARM: tegra: Add GIC for Tegra30 > ARM: tegra: Add Tegra30 PCIe device tree node > ARM: tegra: Enable PCIe on Cardhu > ARM: tegra: Enable PCIe on Beaver > ARM: tegra: Add GIC for Tegra124 > ARM: tegra: Add Tegra124 PCIe device tree node > ARM: tegra: Enable PCIe on Jetson TK1 > ARM: cache_v7: Various minor cleanups > ARM: cache-cp15: Use more accurate types > malloc: Output region when debugging > ARM: Implement non-cached memory support > ARM: tegra: Enable non-cached memory > net: rtl8169: Honor CONFIG_SYS_RX_ETH_BUFFER > net: rtl8169: Properly align buffers > net: rtl8169: Use non-cached memory if available > net: rtl8169: Add support for RTL-8168/8111g > > README | 19 + > arch/arm/cpu/armv7/cache_v7.c| 14 +- > arch/arm/cpu/tegra-common/Makefile |2 + > arch/arm/cpu/tegra-common/powergate.c| 102 ++ > arch/arm/cpu/tegra-common/xusb-padctl.c | 39 + > arch/arm/cpu/tegra124-common/Makefile|1 + > arch/arm/cpu/tegra124-c
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hi Thierry, On 23 October 2014 02:11, Thierry Reding wrote: > On Wed, Oct 22, 2014 at 09:07:40PM -0600, Simon Glass wrote: >> Hi, >> >> On 11 September 2014 10:00, Albert ARIBAUD wrote: >> > Hi Thierry, >> > >> > On Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding >> > wrote: >> > >> >> From: Thierry Reding >> > >> > Note: this series was split over several custodians (including myself). >> > It might thus get applied piecewise... Shouldn't it rather be assigned >> > a single custodian -with others giving their Ack) and be applied as a >> > whole? And yes, I'm ok with getting all the pieces. >> >> As this series still need some minor work, and the DT patches are >> separate, I have pushed them to u-boot-fdt/master and issued a pull >> request. >> >> Thierry are you going to pick this up again soon? We should try to get >> this series applied in the next few weeks. > > I'm completely swamped right now, but I definitely plan on getting back > to this at some point. But I can't really say for sure how soon that'll > be. Since the changes are fairly trivial and I happen to have some of the platforms, I could respin if you like? Would be great to it into this merge window, and I have patches that it might affect. Let me know what you think. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
On Wed, Oct 22, 2014 at 09:07:40PM -0600, Simon Glass wrote: > Hi, > > On 11 September 2014 10:00, Albert ARIBAUD wrote: > > Hi Thierry, > > > > On Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding > > wrote: > > > >> From: Thierry Reding > > > > Note: this series was split over several custodians (including myself). > > It might thus get applied piecewise... Shouldn't it rather be assigned > > a single custodian -with others giving their Ack) and be applied as a > > whole? And yes, I'm ok with getting all the pieces. > > As this series still need some minor work, and the DT patches are > separate, I have pushed them to u-boot-fdt/master and issued a pull > request. > > Thierry are you going to pick this up again soon? We should try to get > this series applied in the next few weeks. I'm completely swamped right now, but I definitely plan on getting back to this at some point. But I can't really say for sure how soon that'll be. Thierry pgp5mX71fqp7u.pgp Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hi, On 11 September 2014 10:00, Albert ARIBAUD wrote: > Hi Thierry, > > On Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding > wrote: > >> From: Thierry Reding > > Note: this series was split over several custodians (including myself). > It might thus get applied piecewise... Shouldn't it rather be assigned > a single custodian -with others giving their Ack) and be applied as a > whole? And yes, I'm ok with getting all the pieces. As this series still need some minor work, and the DT patches are separate, I have pushed them to u-boot-fdt/master and issued a pull request. Thierry are you going to pick this up again soon? We should try to get this series applied in the next few weeks. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hello Thierry, Am 29.09.2014 10:11, schrieb Thierry Reding: On Sun, Sep 28, 2014 at 04:48:47PM -0600, Simon Glass wrote: Hi Thierry, On 26 August 2014 09:33, Thierry Reding wrote: From: Thierry Reding This series adds PCIe support for Tegra20, Tegra30 and Tegra124. The size is mostly due to the large number of infrastructure that's added (libfdt, Tegra specific drivers required by the PCIe driver). In this version I've included all patches that were previously split into three separate series. Spreading them over multiple series is probably not worth it since there might be some dependencies between them and only the end result gives a completely working setup. FYI I have applied this series on top of driver model - it is at u-boot-dm.git branch 'norrin-working'. There were a quite few conflicts which I resolved (mostly with mainline I think). I tested it on beaver, trimslice and jetson-tk1. Are you going to respin it? I started looking at DM for I2C as Heiko suggested but it became too complex and time consuming, so what I'm planning to do is drop the I2C "enhancements" and use plain i2c_set_bus_num() and friends. If there's any interest I can push a branch with the work-in-progress I2C DM patches, perhaps it could serve as the basis if someone else wanted to continue with that effort. Yes please, maybe its worth to use it as a base ... bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hi Thierry, On 29 September 2014 02:11, Thierry Reding wrote: > On Sun, Sep 28, 2014 at 04:48:47PM -0600, Simon Glass wrote: > > Hi Thierry, > > > > On 26 August 2014 09:33, Thierry Reding > wrote: > > > > > From: Thierry Reding > > > > > > This series adds PCIe support for Tegra20, Tegra30 and Tegra124. The > size > > > is > > > mostly due to the large number of infrastructure that's added (libfdt, > > > Tegra > > > specific drivers required by the PCIe driver). In this version I've > > > included > > > all patches that were previously split into three separate series. > > > Spreading > > > them over multiple series is probably not worth it since there might be > > > some > > > dependencies between them and only the end result gives a completely > > > working > > > setup. > > > > > > > FYI I have applied this series on top of driver model - it is at > > u-boot-dm.git branch 'norrin-working'. There were a quite few conflicts > > which I resolved (mostly with mainline I think). I tested it on beaver, > > trimslice and jetson-tk1. > > > > Are you going to respin it? > > I started looking at DM for I2C as Heiko suggested but it became too > complex and time consuming, so what I'm planning to do is drop the I2C > "enhancements" and use plain i2c_set_bus_num() and friends. > OK. I looked at the series and you should really be using the pmic framework. I've pushed a branch to 'dm/norrin-working' that changes it (see the as3722 patch near the top). See tps65090 for an example. It should avoid any messing with I2C. Then, when the new PMIC framework rolls out your code will ride along for free. Plus you can drop the i2c additions that way. Also, there is a clock function that seems to bash various registers - tegra_plle_enable. I think this should use clock_set_rate() - i.e. build in support for the new clock rather than creating a parallel function. The power gate stuff looks good. Are you planning to replace the power gate code in arch/arm/cpu/arm720t/tegra124/cpu.c and the like? The only minor issue I see is the availability of get_timer() - but in tegra_powergate_set() 25ms seems like a very long delay to me anyway. > If there's any interest I can push a branch with the work-in-progress > I2C DM patches, perhaps it could serve as the basis if someone else > wanted to continue with that effort. > Yes I think that would be useful. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
On Sun, Sep 28, 2014 at 04:48:47PM -0600, Simon Glass wrote: > Hi Thierry, > > On 26 August 2014 09:33, Thierry Reding wrote: > > > From: Thierry Reding > > > > This series adds PCIe support for Tegra20, Tegra30 and Tegra124. The size > > is > > mostly due to the large number of infrastructure that's added (libfdt, > > Tegra > > specific drivers required by the PCIe driver). In this version I've > > included > > all patches that were previously split into three separate series. > > Spreading > > them over multiple series is probably not worth it since there might be > > some > > dependencies between them and only the end result gives a completely > > working > > setup. > > > > FYI I have applied this series on top of driver model - it is at > u-boot-dm.git branch 'norrin-working'. There were a quite few conflicts > which I resolved (mostly with mainline I think). I tested it on beaver, > trimslice and jetson-tk1. > > Are you going to respin it? I started looking at DM for I2C as Heiko suggested but it became too complex and time consuming, so what I'm planning to do is drop the I2C "enhancements" and use plain i2c_set_bus_num() and friends. If there's any interest I can push a branch with the work-in-progress I2C DM patches, perhaps it could serve as the basis if someone else wanted to continue with that effort. Thierry pgpuuLQR2zSIL.pgp Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hi Thierry, On 26 August 2014 09:33, Thierry Reding wrote: > From: Thierry Reding > > This series adds PCIe support for Tegra20, Tegra30 and Tegra124. The size > is > mostly due to the large number of infrastructure that's added (libfdt, > Tegra > specific drivers required by the PCIe driver). In this version I've > included > all patches that were previously split into three separate series. > Spreading > them over multiple series is probably not worth it since there might be > some > dependencies between them and only the end result gives a completely > working > setup. > FYI I have applied this series on top of driver model - it is at u-boot-dm.git branch 'norrin-working'. There were a quite few conflicts which I resolved (mostly with mainline I think). I tested it on beaver, trimslice and jetson-tk1. Are you going to respin it? Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
HI Albert, On 18 September 2014 02:43, Albert ARIBAUD wrote: > Hi Simon, > > On Thu, 11 Sep 2014 13:20:03 -0600, Simon Glass > wrote: > > > Hi, > > > > On 11 September 2014 10:17, Stephen Warren > wrote: > > > On 09/11/2014 10:00 AM, Albert ARIBAUD wrote: > > >> > > >> Hi Thierry, > > >> > > >> On Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding > > >> wrote: > > >> > > >>> From: Thierry Reding > > >> > > >> > > >> Note: this series was split over several custodians (including > myself). > > >> It might thus get applied piecewise... Shouldn't it rather be assigned > > >> a single custodian -with others giving their Ack) and be applied as a > > >> whole? And yes, I'm ok with getting all the pieces. > > > > > > > > > Yes, applying it all in one place would probably make sense. > > > > > > IIRC, Simon Glass already applied some of the DT patches early in the > > > series? I CC'd him to check, > > > > Yes I applied the fdt patches to u-boot-fdt/next. I can do a pull > > request to ARM if that helps, or you can just grab them. > > Well, the idea was applying it all in one place :) and I don't like the > idea of pulling the fdt tree into the arm tree, so I'll pick them up, > but please give your Acked-by to them, so that PW sees it and puts it in > when I apply the patches. > They should all be acked already (I did that before I pulled them in) so please go ahead. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hi Simon, On Thu, 11 Sep 2014 13:20:03 -0600, Simon Glass wrote: > Hi, > > On 11 September 2014 10:17, Stephen Warren wrote: > > On 09/11/2014 10:00 AM, Albert ARIBAUD wrote: > >> > >> Hi Thierry, > >> > >> On Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding > >> wrote: > >> > >>> From: Thierry Reding > >> > >> > >> Note: this series was split over several custodians (including myself). > >> It might thus get applied piecewise... Shouldn't it rather be assigned > >> a single custodian -with others giving their Ack) and be applied as a > >> whole? And yes, I'm ok with getting all the pieces. > > > > > > Yes, applying it all in one place would probably make sense. > > > > IIRC, Simon Glass already applied some of the DT patches early in the > > series? I CC'd him to check, > > Yes I applied the fdt patches to u-boot-fdt/next. I can do a pull > request to ARM if that helps, or you can just grab them. Well, the idea was applying it all in one place :) and I don't like the idea of pulling the fdt tree into the arm tree, so I'll pick them up, but please give your Acked-by to them, so that PW sees it and puts it in when I apply the patches. > > BTW, Thierry is on vacation at the moment, so I don't expect he'll respond. > > Regards, > Simon Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hi, On 11 September 2014 10:17, Stephen Warren wrote: > On 09/11/2014 10:00 AM, Albert ARIBAUD wrote: >> >> Hi Thierry, >> >> On Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding >> wrote: >> >>> From: Thierry Reding >> >> >> Note: this series was split over several custodians (including myself). >> It might thus get applied piecewise... Shouldn't it rather be assigned >> a single custodian -with others giving their Ack) and be applied as a >> whole? And yes, I'm ok with getting all the pieces. > > > Yes, applying it all in one place would probably make sense. > > IIRC, Simon Glass already applied some of the DT patches early in the > series? I CC'd him to check, Yes I applied the fdt patches to u-boot-fdt/next. I can do a pull request to ARM if that helps, or you can just grab them. > > BTW, Thierry is on vacation at the moment, so I don't expect he'll respond. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
On 09/11/2014 10:00 AM, Albert ARIBAUD wrote: Hi Thierry, On Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding wrote: From: Thierry Reding Note: this series was split over several custodians (including myself). It might thus get applied piecewise... Shouldn't it rather be assigned a single custodian -with others giving their Ack) and be applied as a whole? And yes, I'm ok with getting all the pieces. Yes, applying it all in one place would probably make sense. IIRC, Simon Glass already applied some of the DT patches early in the series? I CC'd him to check, BTW, Thierry is on vacation at the moment, so I don't expect he'll respond. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
Hi Thierry, On Tue, 26 Aug 2014 17:33:48 +0200, Thierry Reding wrote: > From: Thierry Reding Note: this series was split over several custodians (including myself). It might thus get applied piecewise... Shouldn't it rather be assigned a single custodian -with others giving their Ack) and be applied as a whole? And yes, I'm ok with getting all the pieces. Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH v2 00/40] ARM: tegra: Add PCIe support
From: Thierry Reding This series adds PCIe support for Tegra20, Tegra30 and Tegra124. The size is mostly due to the large number of infrastructure that's added (libfdt, Tegra specific drivers required by the PCIe driver). In this version I've included all patches that were previously split into three separate series. Spreading them over multiple series is probably not worth it since there might be some dependencies between them and only the end result gives a completely working setup. Patch 1 adds a %pa modifier to printf() to print physical addresses. This is required to allow code to output such addresses irrespective of whether a 64 bit or 32 bit architecture is built for. Patches 2-7 add various FDT helpers to make it easier to parse complex device trees. Patch 8 is a minor cleanup to the PCI command that prevents a spew of error messages if a bus does not exist. Patch 9 modifies the PCI enumeration code to honor theh restrictions encoded within a host controller driver's pci_ski_dev() implementation. This is required to prevent exceptions from the NVIDIA Tegra PCIe controller. Patch 10 imports the pr_fmt() macro used within the Linux kernel to reduce the number of characters consumed by literal strings by allowing a source file to specify a prefix or suffix that should be applied to all format strings in the file. It is currently used by the debug() and error() macros. Patches 11-14 are some cleanup and refactoring of I2C core code, the addition of a higher level API that makes it easier for I2C client drivers to talk to devices. The Tegra I2C driver now implements i2c_get_bus_num_fdt() to obtain bus numbers corresponding to a DT node. Patch 15 implements a driver for the AS3722 PMIC used on the Venice2 and Jetson TK1 boards. Patches 16-21 are preparatory work for the Tegra PCIe controller. They add missing clock driver functionality as well as drivers for the Tegra powergate and XUSB pad controller blocks. Patch 22 adds the PCIe controller driver for Tegra20, Tegra30 and Tegra124. Device tree nodes and configurations options to enable PCIe on the TrimSlice (Tegra20), Beaver, Cardhu (Tegra30) and Jetson TK1 (Tegra124) boards are added in patches 23-31. Patches 32-35 implement non-cached memory support that will be used in the last batch of patches to implement more reliable packet transfers in the r8169 driver. Patch 36 enables non-cached memory support on Tegra. Finally, patches 37-40 implement non-cached memory support and various fixes in the r8169 driver and add support for the revision of the NIC found on the Jetson TK1. The above boards all have an ethernet NIC connected to PCIe, which is what I tested with. Thierry Reding (40): vsprintf: Add modifier for phys_addr_t fdt: Add a function to count strings fdt: Add a function to get the index of a string fdt: Add functions to retrieve strings fdt: Add resource parsing functions fdt: Add a function to return PCI BDF triplet fdt: Add a subnodes iterator macro pci: Abort early if bus does not exist pci: Honour pci_skip_dev() Add pr_fmt() macro i2c: Initialize the correct bus i2c: Refactor adapter initialization i2c: Add high-level API i2c: tegra: Implement i2c_get_bus_num_fdt() power: Add AMS AS3722 PMIC support ARM: tegra: Implement tegra_plle_enable() ARM: tegra: Provide PCIEXCLK reset ID ARM: tegra: Implement powergate support ARM: tegra: Implement XUSB pad controller ARM: tegra: Add XUSB pad controller on Tegra124 ARM: tegra: Enable XUSB pad controller on Jetson TK1 pci: tegra: Add Tegra PCIe driver ARM: tegra: Add Tegra20 PCIe device tree node ARM: tegra: Enable PCIe on TrimSlice ARM: tegra: Add GIC for Tegra30 ARM: tegra: Add Tegra30 PCIe device tree node ARM: tegra: Enable PCIe on Cardhu ARM: tegra: Enable PCIe on Beaver ARM: tegra: Add GIC for Tegra124 ARM: tegra: Add Tegra124 PCIe device tree node ARM: tegra: Enable PCIe on Jetson TK1 ARM: cache_v7: Various minor cleanups ARM: cache-cp15: Use more accurate types malloc: Output region when debugging ARM: Implement non-cached memory support ARM: tegra: Enable non-cached memory net: rtl8169: Honor CONFIG_SYS_RX_ETH_BUFFER net: rtl8169: Properly align buffers net: rtl8169: Use non-cached memory if available net: rtl8169: Add support for RTL-8168/8111g README | 19 + arch/arm/cpu/armv7/cache_v7.c| 14 +- arch/arm/cpu/tegra-common/Makefile |2 + arch/arm/cpu/tegra-common/powergate.c| 102 ++ arch/arm/cpu/tegra-common/xusb-padctl.c | 39 + arch/arm/cpu/tegra124-common/Makefile|1 + arch/arm/cpu/tegra124-common/clock.c | 109 +++ arch/arm/cpu/tegra124-common/xusb-padctl.c | 716 ++ arch/arm/cpu/tegra20-common/clock.c | 141 ++- arch/arm/cpu/tegra30-common/clock.c | 155 +++ arch/arm/dts/tegra124-jetson-tk1.dts |