All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: tegra: Wrong hsp_aon reg property size
@ 2020-09-10  7:09 Dipen Patel
  2020-09-10  8:27 ` Mikko Perttunen
  0 siblings, 1 reply; 3+ messages in thread
From: Dipen Patel @ 2020-09-10  7:09 UTC (permalink / raw)
  To: robh+dt, thierry.reding, jonathanh, mperttunen; +Cc: linux-tegra

The hsp_aon node reg property size 0xa0000 will overlap with other
resources. This patch fixes that wrong value with correct size 0x90000.

Signed-off-by: Dipen Patel <dipenp@nvidia.com>

Fixes: a38570c22e9d ("arm64: tegra: Add nodes for TCU on Tegra194")
---
 arch/arm64/boot/dts/nvidia/tegra194.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/nvidia/tegra194.dtsi b/arch/arm64/boot/dts/nvidia/tegra194.dtsi
index e9c90f0f44ff..93438d2b9469 100644
--- a/arch/arm64/boot/dts/nvidia/tegra194.dtsi
+++ b/arch/arm64/boot/dts/nvidia/tegra194.dtsi
@@ -1161,7 +1161,7 @@
 
 		hsp_aon: hsp@c150000 {
 			compatible = "nvidia,tegra194-hsp", "nvidia,tegra186-hsp";
-			reg = <0x0c150000 0xa0000>;
+			reg = <0x0c150000 0x90000>;
 			interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>,
 			             <GIC_SPI 134 IRQ_TYPE_LEVEL_HIGH>,
 			             <GIC_SPI 135 IRQ_TYPE_LEVEL_HIGH>,
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: tegra: Wrong hsp_aon reg property size
  2020-09-10  7:09 [PATCH] arm64: tegra: Wrong hsp_aon reg property size Dipen Patel
@ 2020-09-10  8:27 ` Mikko Perttunen
  2020-09-11 21:40   ` Dipen Patel
  0 siblings, 1 reply; 3+ messages in thread
From: Mikko Perttunen @ 2020-09-10  8:27 UTC (permalink / raw)
  To: Dipen Patel, robh+dt, thierry.reding, jonathanh, mperttunen; +Cc: linux-tegra

On 9/10/20 10:09 AM, Dipen Patel wrote:
> The hsp_aon node reg property size 0xa0000 will overlap with other
> resources. This patch fixes that wrong value with correct size 0x90000.
> 
> Signed-off-by: Dipen Patel <dipenp@nvidia.com>
> 
> Fixes: a38570c22e9d ("arm64: tegra: Add nodes for TCU on Tegra194")

I think there shouldn't be a newline between Signed-off-by and Fixes; 
otherwise,

Reviewed-by: Mikko Perttunen <mperttunen@nvidia.com>

> ---
>   arch/arm64/boot/dts/nvidia/tegra194.dtsi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/nvidia/tegra194.dtsi b/arch/arm64/boot/dts/nvidia/tegra194.dtsi
> index e9c90f0f44ff..93438d2b9469 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra194.dtsi
> +++ b/arch/arm64/boot/dts/nvidia/tegra194.dtsi
> @@ -1161,7 +1161,7 @@
>   
>   		hsp_aon: hsp@c150000 {
>   			compatible = "nvidia,tegra194-hsp", "nvidia,tegra186-hsp";
> -			reg = <0x0c150000 0xa0000>;
> +			reg = <0x0c150000 0x90000>;
>   			interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>,
>   			             <GIC_SPI 134 IRQ_TYPE_LEVEL_HIGH>,
>   			             <GIC_SPI 135 IRQ_TYPE_LEVEL_HIGH>,
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: tegra: Wrong hsp_aon reg property size
  2020-09-10  8:27 ` Mikko Perttunen
@ 2020-09-11 21:40   ` Dipen Patel
  0 siblings, 0 replies; 3+ messages in thread
From: Dipen Patel @ 2020-09-11 21:40 UTC (permalink / raw)
  To: Mikko Perttunen, robh+dt, thierry.reding, jonathanh, mperttunen
  Cc: linux-tegra

On 9/10/20 1:27 AM, Mikko Perttunen wrote:
> On 9/10/20 10:09 AM, Dipen Patel wrote:
>> The hsp_aon node reg property size 0xa0000 will overlap with other
>> resources. This patch fixes that wrong value with correct size 0x90000.
>>
>> Signed-off-by: Dipen Patel <dipenp@nvidia.com>
>>
>> Fixes: a38570c22e9d ("arm64: tegra: Add nodes for TCU on Tegra194")
> 
> I think there shouldn't be a newline between Signed-off-by and Fixes; otherwise,
> 
> Reviewed-by: Mikko Perttunen <mperttunen@nvidia.com>
>

Ok, will fix it in next version.
 
>> ---
>>   arch/arm64/boot/dts/nvidia/tegra194.dtsi | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/nvidia/tegra194.dtsi b/arch/arm64/boot/dts/nvidia/tegra194.dtsi
>> index e9c90f0f44ff..93438d2b9469 100644
>> --- a/arch/arm64/boot/dts/nvidia/tegra194.dtsi
>> +++ b/arch/arm64/boot/dts/nvidia/tegra194.dtsi
>> @@ -1161,7 +1161,7 @@
>>             hsp_aon: hsp@c150000 {
>>               compatible = "nvidia,tegra194-hsp", "nvidia,tegra186-hsp";
>> -            reg = <0x0c150000 0xa0000>;
>> +            reg = <0x0c150000 0x90000>;
>>               interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>,
>>                            <GIC_SPI 134 IRQ_TYPE_LEVEL_HIGH>,
>>                            <GIC_SPI 135 IRQ_TYPE_LEVEL_HIGH>,
>>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-09-11 21:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-10  7:09 [PATCH] arm64: tegra: Wrong hsp_aon reg property size Dipen Patel
2020-09-10  8:27 ` Mikko Perttunen
2020-09-11 21:40   ` Dipen Patel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.