On 2015/3/28 1:35, Brian Norris wrote:
> This would go through your arm-soc submaintainer; i.e., Wei Xu
> 
> But FWIW...

Yes, I will ping him to check the patch.

> 
> On Wed, Mar 25, 2015 at 03:04:51PM +0800, Zhou Wang wrote:
>> This patch adds dts support for NAND flash controller of Hisilicon Soc Hip04.
>> Now it is based on v4.0-rc5
>>
>> Changes in v3:
>> - Change E-mail address in signed-off-by to "[email protected]"
>> Changes in v2:
>> - Base on v3.19-rc1
>> - Use nand-ecc-strength, nand-ecc-step-size to replace hisi,nand-ecc-bits
>> Changes in v1:
>> - Move partition and other board related information into board dts file:
>>   hip04-d01.dts
>>
>> Signed-off-by: Zhou Wang <[email protected]>
>> ---
>>  arch/arm/boot/dts/hip04-d01.dts | 27 +++++++++++++++++++++++++++
>>  arch/arm/boot/dts/hip04.dtsi    |  7 +++++++
>>  2 files changed, 34 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/hip04-d01.dts 
>> b/arch/arm/boot/dts/hip04-d01.dts
>> index 40a9e33..ba04dd5 100644
>> --- a/arch/arm/boot/dts/hip04-d01.dts
>> +++ b/arch/arm/boot/dts/hip04-d01.dts
>> @@ -28,5 +28,32 @@
>>              uart0: uart@4007000 {
>>                      status = "ok";
>>              };
>> +
>> +            nand: nand@4020000 {
>> +                    nand-bus-width = <8>;
>> +                    nand-ecc-mode = "hw";
>> +                    nand-ecc-strength = <16>;
>> +                    nand-ecc-step-size = <1024>;
>> +
>> +                    partition@0 {
>> +                            label = "nand_text";
>> +                            reg = <0x00000000 0x00400000>;
>> +                    };
>> +
>> +                    partition@00400000 {
>> +                            label = "nand_monitor";
>> +                            reg = <0x00400000 0x00400000>;
>> +                    };
>> +
>> +                    partition@00800000 {
>> +                            label = "nand_kernel";
>> +                            reg = <0x00800000 0x00800000>;
>> +                    };
>> +
>> +                    partition@01000000 {
>> +                            label = "nand_fs";
>> +                            reg = <0x01000000 0x1f000000>;
>> +                    };
>> +            };
>>      };
>>  };
>> diff --git a/arch/arm/boot/dts/hip04.dtsi b/arch/arm/boot/dts/hip04.dtsi
>> index 2388145..ac32fce 100644
>> --- a/arch/arm/boot/dts/hip04.dtsi
>> +++ b/arch/arm/boot/dts/hip04.dtsi
>> @@ -269,6 +269,13 @@
>>                      interrupts = <0 372 4>;
>>              };
>>  
>> +            nand: nand@4020000 {
>> +                    compatible = "hisilicon,504-nfc";
>> +                    reg = <0x4020000 0x10000>, <0x5000000 0x1000>;
>> +                    interrupts = <0 379 4>;
>> +                    #address-cells = <1>;
>> +                    #size-cells = <1>;
>> +            };
> 
> Seems reasonable and matches the binding we already accepted, so:
> 
> Reviewed-by: Brian Norris <[email protected]>
> 

Thanks,
Zhou

>>      };
>>  
>>      etb@0,e3c42000 {
>> -- 
>> 1.9.1
>>
> 
> .
> 


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

Reply via email to