Hi Sergey, On Thu, Oct 28, 2021 at 12:26 AM Serge Semin <fancer.lancer@xxxxxxxxx> wrote: > > Hello Brad > > The patch name "dt-bindings: spi: Add compatible for Pensando Elba > SoC" doesn't mention to what bindings it is referring to. For the sake > of having a more representative git log I'd suggest at least to add > "cdns" vendor name in the title, like: "dt-bindings: spi: cdns: Add ..." > Otherwise it's impossible to understand to what bindings you're adding > a new compatibility especially seeing you are doing the similar thing > for the DW SPI in the next patch. I'll add "cdns" to the git log for the re-spin. Thanks Brad