| Message ID | 20220722184138.2666241-3-f.fainelli@gmail.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | Add Broadcom STB memory controller driver | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success |
On 22/07/2022 20:41, Florian Fainelli wrote: > Remove the list of existing compatible strings and prefer to use the > YAML binding which more complete and up to date. Ah, so this is partial conversion from TXT? Then please squash it with previous commit and extend the commit msg. > > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> > --- > .../devicetree/bindings/arm/bcm/brcm,brcmstb.txt | 11 ++--------- > 1 file changed, 2 insertions(+), 9 deletions(-) > Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/arm/bcm/brcm,brcmstb.txt b/Documentation/devicetree/bindings/arm/bcm/brcm,brcmstb.txt index 104cc9b41df4..4db7209d5be3 100644 --- a/Documentation/devicetree/bindings/arm/bcm/brcm,brcmstb.txt +++ b/Documentation/devicetree/bindings/arm/bcm/brcm,brcmstb.txt @@ -187,15 +187,8 @@ Required properties: Sequencer DRAM parameters and control registers. Used for Self-Refresh Power-Down (SRPD), among other things. -Required properties: -- compatible : should contain one of these - "brcm,brcmstb-memc-ddr-rev-b.2.1" - "brcm,brcmstb-memc-ddr-rev-b.2.2" - "brcm,brcmstb-memc-ddr-rev-b.2.3" - "brcm,brcmstb-memc-ddr-rev-b.3.0" - "brcm,brcmstb-memc-ddr-rev-b.3.1" - "brcm,brcmstb-memc-ddr" -- reg : the MEMC DDR register range +See Documentation/devicetree/bindings/memory-controller/brcm,memc.yaml for +a full list of supported compatible strings. Example:
Remove the list of existing compatible strings and prefer to use the YAML binding which more complete and up to date. Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> --- .../devicetree/bindings/arm/bcm/brcm,brcmstb.txt | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-)