Re: [PATCH v6 2/2] PCI: altera: support dt binding update

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Thu, 13 Jun 2024, Rob Herring wrote:

On Wed, Jun 12, 2024 at 08:12:05AM -0700, matthew.gerlach@xxxxxxxxxxxxxxx wrote:


On Tue, 11 Jun 2024, Bjorn Helgaas wrote:

On Tue, Jun 11, 2024 at 11:35:25AM -0500, matthew.gerlach@xxxxxxxxxxxxxxx wrote:
From: Matthew Gerlach <matthew.gerlach@xxxxxxxxxxxxxxx>

Add support for the device tree binding update. As part of
converting the binding document from text to yaml, with schema
validation, a device tree subnode was added to properly map
legacy interrupts. Maintain backward compatibility with previous binding.

If something was *added* to the binding, I think it would be helpful
to split that into two patches: (1) convert to YAML with zero
functional changes, (2) add the new stuff.  Adding something at the
same time as changing the format makes it hard to review.

The policy for conversions is changes to match reality are fine, just
need to be noted in the commit message. That generally implies no driver
or dts changes which is not the case here.

Thanks for feedback. It was during the conversion to YAML that a problem
with the original binding was discovered. As Rob Herring pointed out in
https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240513205913.313592-1-matthew.gerlach@xxxxxxxxxxxxxxx/

"Making the PCI host the interrupt parent didn't even work in the kernel
 until somewhat recently (maybe a few years now). That's why a bunch of PCI
 hosts have an interrupt-controller child node."

This was an attempt to fix the problem. I can resubmit a conversion to YAML
with zero functional changes.

I wasn't suggesting you fix it. Just something I noticed looking at
the other issue. If no one noticed or cared, why bother? It should work
fine for recent kernels.

Thanks for the feedback. I can resubmit a single conversion commit that passes the schema check by adding 3 address fields as you suggested. I will also mention this slight modification in the commit message.

Matthew


Rob





[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux