drivers: interrupt-controller: Add TI VIM Interrupt Controller support

[slpp95prashanth]

Add VIM (Vectored Interrupt Manager) interrupt controller support

TRM for J721e https://www.ti.com/lit/zip/spruil1
File: spruil1c.pdf
VIM: section 6.3.3.6

BeagleBone AI_64 https://beagleboard.org/ai-64

[galak]

Please have devicetree binding match https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/tree/Bindings/interrupt-controller/arm,vic.yaml

[slpp95prashanth]

Please have devicetree binding match https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/tree/Bindings/interrupt-controller/arm,vic.yaml

@galak I have added compatible property in arm,vim.yaml for devicetree binding match. Am I missing something?

[vaishnavachath]

Please have devicetree binding match https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/tree/Bindings/interrupt-controller/arm,vic.yaml

@galak Does the current binding look okay to you now? when this PR was created initially, the naming, compatible were chosen in a confusing manner which created the impression that the support being added here was for standard ARM Vectored Interrupt Controller while the support being added here is for a TI specific interrupt controller (VIM) which is R5F VIC port compatible but not equivalent.

Also the arm,vic.yaml binding in kernel has:

  "#interrupt-cells":
    const: 1
    description:
      The number of cells to define the interrupts.  It must be 1 as the
      VIC has no configuration options for interrupt sources. The single
      cell defines the interrupt number.

But in our case there is support for configuration for interrupt sources (flags, priority .etc).

Please let know if changes needs to be made.

[vaishnavachath]

Please have devicetree binding match https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/tree/Bindings/interrupt-controller/arm,vic.yaml

@galak Does the current binding look okay to you now? when this PR was created initially, the naming, compatible were chosen in a confusing manner which created the impression that the support being added here was for standard ARM Vectored Interrupt Controller while the support being added here is for a TI specific interrupt controller (VIM) which is R5F VIC port compatible but not equivalent.

Also the arm,vic.yaml binding in kernel has:

  "#interrupt-cells":
    const: 1
    description:
      The number of cells to define the interrupts.  It must be 1 as the
      VIC has no configuration options for interrupt sources. The single
      cell defines the interrupt number.

But in our case there is support for configuration for interrupt sources (flags, priority .etc).

Please let know if changes needs to be made.

@galak , can you please let know if this is still relevant/if not can you remove the change requested label?

stale

[cfriedt]

Sorry - this comment was left in pending for a while.