Skip to content

RFC PCI: tegra: Overhaul regulator usage

Bjorn Helgaas linux-arm-kernel linux-devicetree linux-pci linux-tegra 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest0 of 5 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 5 patches the cover letter announces are in this archive.

Reviews trailers per patch

PatchBjorn Helgaas
SeriesA-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [RFC 0/5] PCI: tegra: Overhaul regulator usage
Quoted 20 lines · Stephen Warren
Quoted 12 lines · Thierry Reding

The current device tree binding for the regulator setup on Tegra PCIe is not accurate. While it does work for current use-cases, that's likely by accident rather than design. This series replaces the existing set of power-supply properties with a new set that accurately describes the inputs of the IP block (depending on SoC generation).

As a heads-up, this breaks backwards compatibility with prior versions of the device tree bindings, but I don't see a reason why that should keep us from fixing this properly. Not many people are currently using these bindings and those who are are most likely tracking upstream development closely enough not to be impacted by this.

Quoted 6 lines

I've aimed to keep the series bisectible, which has the downside of interleaving patches to unrelated trees (ARM and PCI). I'm hoping that perhaps we can find a way to merge this as a whole to keep it possible to bisect across the series. Although again, I guess it wouldn't be all that bad if that wasn't the case, given how little PCIe is actually being used.

Quoted 2 lines

I'd be happy to put this series into a branch in the Tegra tree, given suitable acks from Bjorn on the PCI changes.

For 3/5 ("PCI: tegra: Implement accurate power supply scheme"),

I do have the "PCI: tegra: Use new OF interrupt mapping when possible" change from Lucas on my pci/host-tegra branch, but it shouldn't conflict with this.

Bjorn