Messages in this thread |  | | From | Chuanhong Guo <> | Date | Sun, 3 Apr 2022 23:45:44 +0800 | Subject | Re: [PATCH 3/4] dt-bindings: spi: add binding doc for spi-mtk-snfi |
| |
Hi!
On Sun, Apr 3, 2022 at 11:37 PM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > [...] > > + compatible: > > + enum: > > + - mediatek,mt7622-snand > > + - mediatek,mt7629-snand > > One blank line, please.
I'll fix this in the next version.
> > > + reg: > > + items: > > + - description: core registers > > + > > + interrupts: > > + items: > > + - description: NFI interrupt > > + > > + clocks: > > + items: > > + - description: clock used for the controller > > + - description: clock used for the SPI bus > > + > > + clock-names: > > + items: > > + - const: nfi_clk > > + - const: pad_clk > > + > > + ecc-engine: > > The nand-chip.yaml defines a nand-ecc-engine, so how about using that > one? I know mtk-nand.txt uses ecc-engine, but for new schema better to > use properties from existing YAML.
The ecc-engine code is shared between mtk_nand.c and this driver, and the property name is defined in the shared part. I left it as-is so that existing dt for mtk_nand doesn't break.
-- Regards, Chuanhong Guo
|  |