On 07/02/2022 02:46, Li-hao Kuo wrote: > Remove the include path and modify parameters for fix error for bt binding test 1. No trailing dots in commit title. Space after each ":". 2. s/bt/dt/ 3. Wrap the commit msg: https://elixir.bootlin.com/linux/v5.13/source/Documentation/process/submitting-patches.rst#L124 4. What is exactly the error you are fixing? > > Fixes: a708078eeb99 ("spi: Add Sunplus SP7021 schema") 5. Only one space after SHA. 6. Wasn't it reported to you? Looks like it was, so you need to add "Reported-by" tag: https://lore.kernel.org/all/CAL_JsqKwg_bzJmehTG8tR2cpr+HwwvpF_cpoESidZouc_SpKcQ@xxxxxxxxxxxxxx/ > Signed-off-by: Li-hao Kuo <lhjeff911@xxxxxxxxx> > --- > Documentation/devicetree/bindings/spi/spi-sunplus-sp7021.yaml | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/spi/spi-sunplus-sp7021.yaml b/Documentation/devicetree/bindings/spi/spi-sunplus-sp7021.yaml > index 38589fd..298eac2 100644 > --- a/Documentation/devicetree/bindings/spi/spi-sunplus-sp7021.yaml > +++ b/Documentation/devicetree/bindings/spi/spi-sunplus-sp7021.yaml > @@ -59,8 +59,6 @@ unevaluatedProperties: false > > examples: > - | > - #include <dt-bindings/clock/sp-sp7021.h> > - #include <dt-bindings/reset/sp-sp7021.h> > #include <dt-bindings/interrupt-controller/irq.h> > spi@9C002D80 { > compatible = "sunplus,sp7021-spi"; > @@ -73,8 +71,8 @@ examples: > interrupts = <144 IRQ_TYPE_LEVEL_HIGH>, > <146 IRQ_TYPE_LEVEL_HIGH>, > <145 IRQ_TYPE_LEVEL_HIGH>; > - clocks = <&clkc SPI_COMBO_0>; > - resets = <&rstc RST_SPI_COMBO_0>; > + clocks = <&clkc 0x32>; > + resets = <&rstc 0x22>; > pinctrl-names = "default"; > pinctrl-0 = <&pins_spi0>; > }; Best regards, Krzysztof
