Re: [PATCH v2] arm64: dts: ti: k3-am625-sk: enable ramoops

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

 




On 16/05/22 11:06 pm, Guillaume LA ROQUE wrote:
> Le 16/05/2022 à 18:48, Nishanth Menon a écrit :
>> On 16:54-20220516, Guillaume La Roque wrote:
>>> Enable ramoops features to easily debug some issues
>>>
>>> Signed-off-by: Guillaume La Roque <glaroque@xxxxxxxxxxxx>
>>> Signed-off-by: Kevin Hilman <khilman@xxxxxxxxxxxx>
>>> ---
>>> Changes in v2:
>>> - Apply script create-mem_map.py  with args given by Nishanth Menon
>>> - Spelling fix
>>> ---
>>>   arch/arm64/boot/dts/ti/k3-am625-sk.dts | 9 +++++++++
>>>   1 file changed, 9 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/ti/k3-am625-sk.dts
>>> b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
>>> index 0de4113ccd5d..dfb16c29a000 100644
>>> --- a/arch/arm64/boot/dts/ti/k3-am625-sk.dts
>>> +++ b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
>>> @@ -36,6 +36,15 @@ reserved-memory {
>>>           #size-cells = <2>;
>>>           ranges;
>>>
>>> +        ramoops@0x9ca00000 {
>>> +            compatible = "ramoops";
>>> +            reg = <0x0 0xe0000000 0x0 0x00100000>;
>> I think you intended 0x9ca00000 here?
>>
>> Static checks should have caught this for you. please run them
>> prior to posting?
> sorry for that i forgot to run it and update line .
> 
> i will fix it .
> 
> sorry for noise...
> 

Also, convention in the file is to use 0x00 instead of 0x0. So,

		reg = <0x00 0x9ca00000 0x00 0x100000>;

>>> +            record-size = <0x8000>;
>>> +            console-size = <0x8000>;
>>> +            ftrace-size = <0x0>;
>>> +            pmsg-size = <0x8000>;
>>> +        };
>>> +
>>>           secure_tfa_ddr: tfa@9e780000 {
>>>               reg = <0x00 0x9e780000 0x00 0x80000>;
>>>               alignment = <0x1000>;
>>> -- 
>>> 2.25.1
>>>
> 



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux