[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <89d49eada6411c99fdf3ca389b1ff01c2694f954.camel@iscas.ac.cn>
Date: Tue, 10 Feb 2026 14:33:04 +0800
From: Icenowy Zheng <zhengxingda@...as.ac.cn>
To: Rob Herring <robh@...nel.org>
Cc: Thomas Gleixner <tglx@...nel.org>, Krzysztof Kozlowski
<krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Thomas
Bogendoerfer <tsbogend@...ha.franken.de>, Huacai Chen
<chenhuacai@...nel.org>, Jiaxun Yang <jiaxun.yang@...goat.com>,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
linux-mips@...r.kernel.org
Subject: Re: [PATCH 2/8] dt-bindings: interrupt-controller: add LS7A PCH LPC
在 2026-02-09星期一的 17:48 -0600,Rob Herring写道:
> On Sat, Jan 31, 2026 at 05:45:41PM +0800, Icenowy Zheng wrote:
> > Loongson 7A series PCH contains an LPC controller with an interrupt
> > controller.
> >
> > Add the device tree binding for the interrupt controller.
> >
> > Signed-off-by: Icenowy Zheng <zhengxingda@...as.ac.cn>
> > ---
> > .../loongson,pch-lpc.yaml | 52
> > +++++++++++++++++++
> > 1 file changed, 52 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/interrupt-
> > controller/loongson,pch-lpc.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/interrupt-
> > controller/loongson,pch-lpc.yaml
> > b/Documentation/devicetree/bindings/interrupt-
> > controller/loongson,pch-lpc.yaml
> > new file mode 100644
> > index 0000000000000..c00fbf31f47f0
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/interrupt-
> > controller/loongson,pch-lpc.yaml
> > @@ -0,0 +1,52 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id:
> > http://devicetree.org/schemas/interrupt-controller/loongson,pch-lpc.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Loongson PCH LPC Controller
> > +
> > +maintainers:
> > + - Jiaxun Yang <jiaxun.yang@...goat.com>
> > +
> > +description:
> > + This interrupt controller is found in the Loongson LS7A family
> > of PCH for
> > + accepting interrupts sent by LPC-connected peripherals and
> > signalling PIC
> > + via a single interrupt line when interrupts are available.
> > +
> > +properties:
> > + compatible:
> > + const: loongson,pch-lpc-1.0
>
> Where does 1.0 come from? We don't do version numbers generally
> unless
> you define where the versions come from (e.g. Soft IP releases for
> FPGAs). I would have expected "ls7a" in the compatible instead.
Well this replicates what's set on PCH PIC, although I am okay to
change it to `7a1000-pch-lpc` .
Thanks
Icenowy
>
> Rob
>
Powered by blists - more mailing lists