On 09/02/2023 14:26, Keguang Zhang wrote: > Add devicetree binding document and related header file > for the Loongson-1 clock. > > Signed-off-by: Keguang Zhang <keguang.zhang@xxxxxxxxx> > --- > V1 -> V2: Change to one clock controller > Add clock-related dt-binding header file > Fix the warning of dt_binding_check > --- Thank you for your patch. There is something to discuss/improve. > diff --git a/include/dt-bindings/clock/loongson,ls1x-clk.h b/include/dt-bindings/clock/loongson,ls1x-clk.h > new file mode 100644 > index 000000000000..579552c5f14b > --- /dev/null > +++ b/include/dt-bindings/clock/loongson,ls1x-clk.h > @@ -0,0 +1,19 @@ > +/* SPDX-License-Identifier: GPL-2.0-or-later */ Same license as binding please. > +/* > + * Loongson-1 clock tree IDs > + * > + * Copyright (C) 2023 Keguang Zhang <keguang.zhang@xxxxxxxxx> > + */ Best regards, Krzysztof