2023-12-05 10:13:04

by Yoshinori Sato

Subject: [DO NOT MERGE v5 04/37] dt-bindings: interrupt-controller: Add header for Renesas SH3/4 INTC.

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

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)
+
+#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
+
+#endif
--
2.39.2

2023-12-05 16:01:46

by Krzysztof Kozlowski

Subject: Re: [DO NOT MERGE v5 04/37] dt-bindings: interrupt-controller: Add header for Renesas SH3/4 INTC.

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