On 07/09/2018 08:03 PM, Dinh Nguyen wrote:
> 
> 
> On 05/31/2018 03:08 AM, tien.fong.c...@intel.com wrote:
>> From: Tien Fong Chee <tien.fong.c...@intel.com>
>>
>> Update pdma properties for Stratix 10
>>
>> Signed-off-by: Tien Fong Chee <tien.fong.c...@intel.com>
>> ---
>>  arch/arm/dts/socfpga_stratix10.dtsi | 20 ++++++++++++++++++++
>>  1 file changed, 20 insertions(+)
>>
>> diff --git a/arch/arm/dts/socfpga_stratix10.dtsi 
>> b/arch/arm/dts/socfpga_stratix10.dtsi
>> index ccd3f32..311ba09 100644
>> --- a/arch/arm/dts/socfpga_stratix10.dtsi
>> +++ b/arch/arm/dts/socfpga_stratix10.dtsi
>> @@ -82,6 +82,26 @@
>>              ranges = <0 0 0 0xffffffff>;
>>              u-boot,dm-pre-reloc;
>>  
>> +            amba {
>> +                    u-boot,dm-pre-reloc;
>> +                    compatible = "arm,amba-bus";
>> +                    #address-cells = <1>;
>> +                    #size-cells = <1>;
>> +                    ranges;
>> +
>> +                    pdma: pdma@ffda0000 {
>> +                                    u-boot,dm-pre-reloc;
>> +                                    compatible = "arm,pl330", "arm,dma330";
> 
> I think you got "arm,dma330" binding wrong. I don't see any binding with
> that name.

I think the whole idea of using pl330 to scrub ECC DRAM is wrong. It
adds massive amount of code while a CPU can do the same and faster, cfr
arria10.

-- 
Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to