Re: [DO NOT MERGE v5 04/37] dt-bindings: interrupt-controller: Add header for Renesas SH3/4 INTC.
Krzysztof Kozlowski <[email protected]>
| Newsgroups | gmane.linux.ports.sh.devel,gmane.linux.ide,gmane.linux.drivers.devicetree,gmane.linux.kernel,gmane.linux.kernel.renesas-soc,gmane.linux.kernel.clk,gmane.comp.video.dri.devel,gmane.linux.kernel.pci,gmane.linux.serial |
|---|---|
| Message-ID | <[email protected]> |
On 05/12/2023 10:45, Yoshinori Sato wrote: > Renesas SH7751 Interrupt controller priority register define. > > Signed-off-by: Yoshinori Sato <[email protected]> > --- > .../interrupt-controller/sh7751-intc.h | 21 +++++++++++++++++++ > 1 file changed, 21 insertions(+) > create mode 100644 include/dt-bindings/interrupt-controller/sh7751-intc.h Still missing vendor prefix. This applies to all your bindings. > > diff --git a/include/dt-bindings/interrupt-controller/sh7751-intc.h b/include/dt-bindings/interrupt-controller/sh7751-intc.h > new file mode 100644 > index 000000000000..5783ec72d70f > --- /dev/null > +++ b/include/dt-bindings/interrupt-controller/sh7751-intc.h > @@ -0,0 +1,21 @@ > +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > + * > + * SH3/4 INTC EVT - IRQ conversion > + */ > + > +#ifndef __DT_BINDINGS_RENESAS_SH7751_INTC > +#define __DT_BINDINGS_RENESAS_SH7751_INTC > + > +#define evt2irq(evt) ((evt) >> 5) Still not a binding. > + > +#define IPRA 0 > +#define IPRB 4 > +#define IPRC 8 > +#define IPRD 12 > +#define INTPRI00 256 > +#define IPR_B12 12 > +#define IPR_B8 8 > +#define IPR_B4 4 > +#define IPR_B0 0 Neither these. Your commit msg says these are register values, so not bindings. Best regards, Krzysztof