Subject: [PATCH 1/3] Documentation: bindings: brcmstb: Document write-pairing

From: [email-masked] (Florian Fainelli)
Date: Mon, 14 Sep 2015 14:21:53 -0700

On 14/09/15 13:36, Rob Herring wrote:
> On 09/14/2015 02:52 PM, Florian Fainelli wrote:
>> Document the hif-cpubiuctrl node a bit more, and add a documentation
>> entry for the optional "brcm,write-pairing" property.
>>
>> Signed-off-by: Florian Fainelli <[email-masked]>
>> ---
>>  .../devicetree/bindings/arm/bcm/brcm,brcmstb.txt          | 15 +++++++++++++++
>>  1 file changed, 15 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/bcm/brcm,brcmstb.txt b/Documentation/devicetree/bindings/arm/bcm/brcm,brcmstb.txt
>> index 430608ec09f0..602fa5f308bb 100644
>> --- a/Documentation/devicetree/bindings/arm/bcm/brcm,brcmstb.txt
>> +++ b/Documentation/devicetree/bindings/arm/bcm/brcm,brcmstb.txt
>> @@ -20,6 +20,20 @@ system control is required:
>>      - compatible: "brcm,bcm<chip_id>-hif-cpubiuctrl", "syscon"
> 
> This has multiple chips allowed, but...
> 
>>      - compatible: "brcm,bcm<chip_id>-hif-continuation", "syscon"
>>  
>> +hif-cpubiuctrl node
>> +-------------------
> 
> Perhaps it is outside this context, but a description of the block would
> be nice.

Sure.

> 
>> +
>> +Required properties:
>> +
>> +    - compatible
>> +	The string should be "brcm.bcm7445-hif-cpubiuctrl", "syscon"
> 
> This is just one.

Right, will fix that.

> 
>> +
>> +Optional properties:
>> +
>> +    - brcm,write-pairing:
>> +	Boolean property, which when present indicatse that the chip
> 
> s/indicatse/indicates/
> 
>> +	supports write-pairing.
> 
> What is write pairing? Please give brief description.

I will provide an explanation in v2 for what that is.
-- 
Florian