On 29/04/2024 16:38, Eddie James wrote: > > On 4/28/24 11:39, Krzysztof Kozlowski wrote: >> On 26/04/2024 16:49, Eddie James wrote: >>> On 4/26/24 01:15, Krzysztof Kozlowski wrote: >>>> On 25/04/2024 23:36, Eddie James wrote: >>>>> The IBM Power chips have a basic SPI controller. Document it. >>>> Please use subject prefixes matching the subsystem. You can get them for >>>> example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory >>>> your patch is touching. >>> >>> Isn't spi the right subsystem here? >> And what prefix shall be for SPI bindings? Did you run the command or >> just replying to disagree with me? > > > The correct prefix is either dt-bindings: spi or spi: dt-bindings. I > used the former. Would you prefer I use the latter? I followed your The correct is the latter, although I see now Mark stopped stripping dt-bindings in such case. Log is still suggesting that (git log --oneline -- Documentation/devicetree/bindings/spi/ | grep "^[0-9a-f]\+ spi:" | wc -l gives 370 vs 130), although I understand now different approach. It's also mentioned here: https://www.kernel.org/doc/html/latest/devicetree/bindings/submitting-patches.html Best regards, Krzysztof