Re: [v3 1/2] mmc: DT: sdhci-brcmstb: Bindings document for common sdhci-brcmstb

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Rob,

On Sep 2, 2016, at 11:55 PM, Rob Herring <robh@xxxxxxxxxx> wrote:
> 
> On Tue, Aug 30, 2016 at 03:30:09PM +0900, Jaedon Shin wrote:
>> - Renames the DT binding document to common for ARM and MIPS based SoCs
>> - Adds a compatible string "brcm,bcm7445-sdhci" that is representation
>>  for the Broadcom BCM7445 and family. It is initial and typical 28nm
>>  generation ARM based SoC.
>> 
>> Signed-off-by: Jaedon Shin <jaedon.shin@xxxxxxxxx>
>> ---
>> .../mmc/{brcm,bcm7425-sdhci.txt => brcm,sdhci-brcmstb.txt}        | 8 +++++---
>> 1 file changed, 5 insertions(+), 3 deletions(-)
>> rename Documentation/devicetree/bindings/mmc/{brcm,bcm7425-sdhci.txt => brcm,sdhci-brcmstb.txt} (81%)
>> 
>> diff --git a/Documentation/devicetree/bindings/mmc/brcm,bcm7425-sdhci.txt b/Documentation/devicetree/bindings/mmc/brcm,sdhci-brcmstb.txt
>> similarity index 81%
>> rename from Documentation/devicetree/bindings/mmc/brcm,bcm7425-sdhci.txt
>> rename to Documentation/devicetree/bindings/mmc/brcm,sdhci-brcmstb.txt
>> index 82847174c37d..555229367935 100644
>> --- a/Documentation/devicetree/bindings/mmc/brcm,bcm7425-sdhci.txt
>> +++ b/Documentation/devicetree/bindings/mmc/brcm,sdhci-brcmstb.txt
>> @@ -8,14 +8,16 @@ on Device Tree properties to enable them for SoC/Board combinations
>> that support them.
>> 
>> Required properties:
>> -- compatible: "brcm,bcm7425-sdhci"
>> +- compatible: should be one of the following
>> +  - "brcm,bcm7425-sdhci"
>> +  - "brcm,bcm7445-sdhci"
>> 
>> Refer to clocks/clock-bindings.txt for generic clock consumer properties.
>> 
>> Example:
>> 
>> 	sdhci@f03e0100 {
>> -		compatible = "brcm,bcm7425-sdhci";
>> +		compatible = "brcm,bcm7445-sdhci";
>> 		reg = <0xf03e0000 0x100>;
>> 		interrupts = <0x0 0x26 0x0>;
>> 		sdhci,auto-cmd12;
>> @@ -27,7 +29,7 @@ Example:
>> 	sdhci@f03e0300 {
>> 		non-removable;
>> 		bus-width = <0x8>;
>> -		compatible = "brcm,bcm7425-sdhci";
>> +		compatible = "brcm,bcm7445-sdhci";
> 
> These changes to the example seem pointless. Drop them.

Previously, the compatible string "brcm,bcm7425-sdhci" is representation all
MIPS and ARM SoCs including bcm7445. If compatible string is split two strings,
the example devicetree node is wrong. It is for bcm7445.

I'll be add commit message this or
Will I need to create a new patch to change example nodes?

Thanks,
Jaedon

> 
>> 		reg = <0xf03e0200 0x100>;
>> 		interrupts = <0x0 0x27 0x0>;
>> 		sdhci,auto-cmd12;
>> -- 
>> 2.9.3

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux