On Wed, Jan 28, 2015 at 5:49 PM, Paul Walmsley <[email protected]> wrote: > > Add compatible strings for the PCIe IP blocks present on several Tegra > chips. The primary objective here is to avoid checkpatch warnings, > per: > > http://marc.info/?l=linux-tegra&m=142201349727836&w=2 > > Signed-off-by: Paul Walmsley <[email protected]> > Cc: Thierry Reding <[email protected]> > Cc: Rob Herring <[email protected]> > Cc: Pawel Moll <[email protected]> > Cc: Mark Rutland <[email protected]> > Cc: Ian Campbell <[email protected]> > Cc: Kumar Gala <[email protected]> > Cc: Stephen Warren <[email protected]> > Cc: Alexandre Courbot <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > --- > .../bindings/pci/nvidia,tegra20-pcie.txt | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/pci/nvidia,tegra20-pcie.txt > b/Documentation/devicetree/bindings/pci/nvidia,tegra20-pcie.txt > index d763e047c6ae..e772884f1c33 100644 > --- a/Documentation/devicetree/bindings/pci/nvidia,tegra20-pcie.txt > +++ b/Documentation/devicetree/bindings/pci/nvidia,tegra20-pcie.txt > @@ -5,6 +5,8 @@ Required properties: > - "nvidia,tegra20-pcie" > - "nvidia,tegra30-pcie" > - "nvidia,tegra124-pcie" > + - "nvidia,tegra132-pcie" (not yet matched in the driver) > + - "nvidia,tegra210-pcie" (not yet matched in the driver)
Whether the driver matches or not is irrelevant to the binding and may change over time. Does this mean the driver matches on something else or Tegra132 is not yet supported in the driver? If the former, what is important is what are the valid combinations of compatible properties and that is not captured here. In other words, what is the fallback compatible string for each chip? The same comments apply to the rest of the series. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
