Re: [PATCH v8 4/6] PCI: tegra: Add Tegra264 support

[Thierry Reding <thierry.reding@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1469 bytes --]

On Wed, Jul 22, 2026 at 11:44:42AM -0500, Bjorn Helgaas wrote:
> On Thu, Jul 16, 2026 at 07:07:15PM +0200, Thierry Reding wrote:
> > From: Thierry Reding <treding@nvidia.com>
> > 
> > Add a driver for the PCIe controller found on NVIDIA Tegra264 SoCs. The
> > driver is very small, with its main purpose being to set up the address
> > translation registers and then creating a standard PCI host using ECAM.
> > ...
> 
> > + * The various memory regions used by the controller (I/O, memory, ECAM) are
> > + * set up during early boot and have hardware-level protections in place. If
> > + * the DT ranges don't match what's been setup, the controller won't be able
> > + * to write the address endpoints properly, so make sure to validate that DT
> > + * and firmware programming agree on these ranges.
> > + */
> > +static bool tegra264_pcie_check_ranges(struct platform_device *pdev)
> 
> Drive-by comment: I have a bit of an aversion to boolean functions
> named "check" because "check" is not a predicate with a true/false or
> yes/no answer.
> 
> "true" usually means success, which is the opposite of functions that
> return 0 for success.
> 
> In this case, it looks like "if (tegra264_pcie_check_ranges())" means
> the DT ranges match the hardware configuration.  If it were named
> "tegra264_pcie_valid_ranges()" or similar, an "if" test like that
> would read better.

Indeed, "valid ranges" is superior in this case.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]