Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] arm64: dts: ti: k3-am65: Add MSMC RAM ranges in interconnect node
@ 2019-05-29 21:13 Suman Anna
  2019-05-30 10:49 ` Nishanth Menon
  0 siblings, 1 reply; 6+ messages in thread
From: Suman Anna @ 2019-05-29 21:13 UTC (permalink / raw)
  To: Tero Kristo, Nishanth Menon
  Cc: devicetree, Rob Herring, linux-arm-kernel, Roger Quadros

From: Roger Quadros <rogerq@ti.com>

Add the MSCM RAM address space to the ranges property of the cbass_main
interconnect node so that the addresses can be translated properly.

This fixes the probe failure in the sram driver for the MSMC RAM node.

Signed-off-by: Roger Quadros <rogerq@ti.com>
Signed-off-by: Suman Anna <s-anna@ti.com>
---

The following error message is seen without this:
[    0.480261] sram interconnect@100000:sram@70000000: found no memory resource
[    0.487497] sram: probe of interconnect@100000:sram@70000000 failed with error -22

regards
Suman

 arch/arm64/boot/dts/ti/k3-am65.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am65.dtsi b/arch/arm64/boot/dts/ti/k3-am65.dtsi
index 50f4be2047a9..68b3f954f1d1 100644
--- a/arch/arm64/boot/dts/ti/k3-am65.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65.dtsi
@@ -68,6 +68,7 @@
 			 <0x00 0x00900000 0x00 0x00900000 0x00 0x00012000>, /* serdes */
 			 <0x00 0x01000000 0x00 0x01000000 0x00 0x0af02400>, /* Most peripherals */
 			 <0x00 0x30800000 0x00 0x30800000 0x00 0x0bc00000>, /* MAIN NAVSS */
+			 <0x00 0x70000000 0x00 0x70000000 0x00 0x00200000>, /* MSMC SRAM */
 			 /* MCUSS Range */
 			 <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>,
 			 <0x00 0x40200000 0x00 0x40200000 0x00 0x00900100>,
-- 
2.21.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] arm64: dts: ti: k3-am65: Add MSMC RAM ranges in interconnect node
  2019-05-29 21:13 [PATCH] arm64: dts: ti: k3-am65: Add MSMC RAM ranges in interconnect node Suman Anna
@ 2019-05-30 10:49 ` Nishanth Menon
  2019-06-17 14:54   ` Tero Kristo
  0 siblings, 1 reply; 6+ messages in thread
From: Nishanth Menon @ 2019-05-30 10:49 UTC (permalink / raw)
  To: Suman Anna
  Cc: Tero Kristo, devicetree, Rob Herring, linux-arm-kernel, Roger Quadros

On 16:13-20190529, Suman Anna wrote:
> From: Roger Quadros <rogerq@ti.com>
> 
> Add the MSCM RAM address space to the ranges property of the cbass_main
> interconnect node so that the addresses can be translated properly.
> 
> This fixes the probe failure in the sram driver for the MSMC RAM node.
> 
> Signed-off-by: Roger Quadros <rogerq@ti.com>
> Signed-off-by: Suman Anna <s-anna@ti.com>
> ---
> 
> The following error message is seen without this:
> [    0.480261] sram interconnect@100000:sram@70000000: found no memory resource
> [    0.487497] sram: probe of interconnect@100000:sram@70000000 failed with error -22
> 
> regards
> Suman
> 
>  arch/arm64/boot/dts/ti/k3-am65.dtsi | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am65.dtsi b/arch/arm64/boot/dts/ti/k3-am65.dtsi
> index 50f4be2047a9..68b3f954f1d1 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65.dtsi
> @@ -68,6 +68,7 @@
>  			 <0x00 0x00900000 0x00 0x00900000 0x00 0x00012000>, /* serdes */
>  			 <0x00 0x01000000 0x00 0x01000000 0x00 0x0af02400>, /* Most peripherals */
>  			 <0x00 0x30800000 0x00 0x30800000 0x00 0x0bc00000>, /* MAIN NAVSS */
> +			 <0x00 0x70000000 0x00 0x70000000 0x00 0x00200000>, /* MSMC SRAM */
>  			 /* MCUSS Range */
>  			 <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>,
>  			 <0x00 0x40200000 0x00 0x40200000 0x00 0x00900100>,
> -- 
> 2.21.0
> 

Arrgh.. Nice catch. Thanks. We should consider pulling this in as part
of early rc fixes please.

Acked-by: Nishanth Menon <nm@ti.com>

-- 
Regards,
Nishanth Menon

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] arm64: dts: ti: k3-am65: Add MSMC RAM ranges in interconnect node
  2019-05-30 10:49 ` Nishanth Menon
@ 2019-06-17 14:54   ` Tero Kristo
  2019-06-17 15:42     ` Nishanth Menon
  2019-06-17 15:53     ` Suman Anna
  0 siblings, 2 replies; 6+ messages in thread
From: Tero Kristo @ 2019-06-17 14:54 UTC (permalink / raw)
  To: Nishanth Menon, Suman Anna
  Cc: devicetree, Rob Herring, linux-arm-kernel, Roger Quadros

On 30/05/2019 13:49, Nishanth Menon wrote:
> On 16:13-20190529, Suman Anna wrote:
>> From: Roger Quadros <rogerq@ti.com>
>>
>> Add the MSCM RAM address space to the ranges property of the cbass_main
>> interconnect node so that the addresses can be translated properly.
>>
>> This fixes the probe failure in the sram driver for the MSMC RAM node.
>>
>> Signed-off-by: Roger Quadros <rogerq@ti.com>
>> Signed-off-by: Suman Anna <s-anna@ti.com>
>> ---
>>
>> The following error message is seen without this:
>> [    0.480261] sram interconnect@100000:sram@70000000: found no memory resource
>> [    0.487497] sram: probe of interconnect@100000:sram@70000000 failed with error -22
>>
>> regards
>> Suman
>>
>>   arch/arm64/boot/dts/ti/k3-am65.dtsi | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am65.dtsi b/arch/arm64/boot/dts/ti/k3-am65.dtsi
>> index 50f4be2047a9..68b3f954f1d1 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am65.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am65.dtsi
>> @@ -68,6 +68,7 @@
>>   			 <0x00 0x00900000 0x00 0x00900000 0x00 0x00012000>, /* serdes */
>>   			 <0x00 0x01000000 0x00 0x01000000 0x00 0x0af02400>, /* Most peripherals */
>>   			 <0x00 0x30800000 0x00 0x30800000 0x00 0x0bc00000>, /* MAIN NAVSS */
>> +			 <0x00 0x70000000 0x00 0x70000000 0x00 0x00200000>, /* MSMC SRAM */
>>   			 /* MCUSS Range */
>>   			 <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>,
>>   			 <0x00 0x40200000 0x00 0x40200000 0x00 0x00900100>,
>> -- 
>> 2.21.0
>>
> 
> Arrgh.. Nice catch. Thanks. We should consider pulling this in as part
> of early rc fixes please.
> 
> Acked-by: Nishanth Menon <nm@ti.com>

Hmm, what is the failure this causes, except the probe message? SRAM 
driver obviously won't work but is it used for anything at the moment 
anyways?

I would rather queue this towards 5.3.

-Tero
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] arm64: dts: ti: k3-am65: Add MSMC RAM ranges in interconnect node
  2019-06-17 14:54   ` Tero Kristo
@ 2019-06-17 15:42     ` Nishanth Menon
  2019-06-17 15:53     ` Suman Anna
  1 sibling, 0 replies; 6+ messages in thread
From: Nishanth Menon @ 2019-06-17 15:42 UTC (permalink / raw)
  To: Tero Kristo; +Cc: devicetree, linux-arm-kernel, Rob Herring, Roger Quadros

On 17:54-20190617, Tero Kristo wrote:
> On 30/05/2019 13:49, Nishanth Menon wrote:
> > On 16:13-20190529, Suman Anna wrote:
> > > From: Roger Quadros <rogerq@ti.com>
> > > 
> > > Add the MSCM RAM address space to the ranges property of the cbass_main
> > > interconnect node so that the addresses can be translated properly.
> > > 
> > > This fixes the probe failure in the sram driver for the MSMC RAM node.
> > > 
> > > Signed-off-by: Roger Quadros <rogerq@ti.com>
> > > Signed-off-by: Suman Anna <s-anna@ti.com>
> > > ---
> > > 
> > > The following error message is seen without this:
> > > [    0.480261] sram interconnect@100000:sram@70000000: found no memory resource
> > > [    0.487497] sram: probe of interconnect@100000:sram@70000000 failed with error -22
> > > 
> > > regards
> > > Suman
> > > 
> > >   arch/arm64/boot/dts/ti/k3-am65.dtsi | 1 +
> > >   1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/arch/arm64/boot/dts/ti/k3-am65.dtsi b/arch/arm64/boot/dts/ti/k3-am65.dtsi
> > > index 50f4be2047a9..68b3f954f1d1 100644
> > > --- a/arch/arm64/boot/dts/ti/k3-am65.dtsi
> > > +++ b/arch/arm64/boot/dts/ti/k3-am65.dtsi
> > > @@ -68,6 +68,7 @@
> > >   			 <0x00 0x00900000 0x00 0x00900000 0x00 0x00012000>, /* serdes */
> > >   			 <0x00 0x01000000 0x00 0x01000000 0x00 0x0af02400>, /* Most peripherals */
> > >   			 <0x00 0x30800000 0x00 0x30800000 0x00 0x0bc00000>, /* MAIN NAVSS */
> > > +			 <0x00 0x70000000 0x00 0x70000000 0x00 0x00200000>, /* MSMC SRAM */
> > >   			 /* MCUSS Range */
> > >   			 <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>,
> > >   			 <0x00 0x40200000 0x00 0x40200000 0x00 0x00900100>,
> > > -- 
> > > 2.21.0
> > > 
> > 
> > Arrgh.. Nice catch. Thanks. We should consider pulling this in as part
> > of early rc fixes please.
> > 
> > Acked-by: Nishanth Menon <nm@ti.com>
> 
> Hmm, what is the failure this causes, except the probe message? SRAM driver
> obviously won't work but is it used for anything at the moment anyways?
> 
> I would rather queue this towards 5.3.

Yes, this should be good enough for 5.3

-- 
Regards,
Nishanth Menon

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] arm64: dts: ti: k3-am65: Add MSMC RAM ranges in interconnect node
  2019-06-17 14:54   ` Tero Kristo
  2019-06-17 15:42     ` Nishanth Menon
@ 2019-06-17 15:53     ` Suman Anna
  2019-06-17 19:42       ` Tero Kristo
  1 sibling, 1 reply; 6+ messages in thread
From: Suman Anna @ 2019-06-17 15:53 UTC (permalink / raw)
  To: Tero Kristo, Nishanth Menon
  Cc: devicetree, Rob Herring, linux-arm-kernel, Roger Quadros

On 6/17/19 9:54 AM, Tero Kristo wrote:
> On 30/05/2019 13:49, Nishanth Menon wrote:
>> On 16:13-20190529, Suman Anna wrote:
>>> From: Roger Quadros <rogerq@ti.com>
>>>
>>> Add the MSCM RAM address space to the ranges property of the cbass_main
>>> interconnect node so that the addresses can be translated properly.
>>>
>>> This fixes the probe failure in the sram driver for the MSMC RAM node.
>>>
>>> Signed-off-by: Roger Quadros <rogerq@ti.com>
>>> Signed-off-by: Suman Anna <s-anna@ti.com>
>>> ---
>>>
>>> The following error message is seen without this:
>>> [    0.480261] sram interconnect@100000:sram@70000000: found no
>>> memory resource
>>> [    0.487497] sram: probe of interconnect@100000:sram@70000000
>>> failed with error -22
>>>
>>> regards
>>> Suman
>>>
>>>   arch/arm64/boot/dts/ti/k3-am65.dtsi | 1 +
>>>   1 file changed, 1 insertion(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/ti/k3-am65.dtsi
>>> b/arch/arm64/boot/dts/ti/k3-am65.dtsi
>>> index 50f4be2047a9..68b3f954f1d1 100644
>>> --- a/arch/arm64/boot/dts/ti/k3-am65.dtsi
>>> +++ b/arch/arm64/boot/dts/ti/k3-am65.dtsi
>>> @@ -68,6 +68,7 @@
>>>                <0x00 0x00900000 0x00 0x00900000 0x00 0x00012000>, /*
>>> serdes */
>>>                <0x00 0x01000000 0x00 0x01000000 0x00 0x0af02400>, /*
>>> Most peripherals */
>>>                <0x00 0x30800000 0x00 0x30800000 0x00 0x0bc00000>, /*
>>> MAIN NAVSS */
>>> +             <0x00 0x70000000 0x00 0x70000000 0x00 0x00200000>, /*
>>> MSMC SRAM */
>>>                /* MCUSS Range */
>>>                <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>,
>>>                <0x00 0x40200000 0x00 0x40200000 0x00 0x00900100>,
>>> -- 
>>> 2.21.0
>>>
>>
>> Arrgh.. Nice catch. Thanks. We should consider pulling this in as part
>> of early rc fixes please.
>>
>> Acked-by: Nishanth Menon <nm@ti.com>
> 
> Hmm, what is the failure this causes, except the probe message? SRAM
> driver obviously won't work but is it used for anything at the moment
> anyways?
> 
> I would rather queue this towards 5.3.

OK, I am fine either way.

regards
Suman

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] arm64: dts: ti: k3-am65: Add MSMC RAM ranges in interconnect node
  2019-06-17 15:53     ` Suman Anna
@ 2019-06-17 19:42       ` Tero Kristo
  0 siblings, 0 replies; 6+ messages in thread
From: Tero Kristo @ 2019-06-17 19:42 UTC (permalink / raw)
  To: Suman Anna, Nishanth Menon
  Cc: devicetree, Rob Herring, linux-arm-kernel, Roger Quadros

On 17/06/2019 18:53, Suman Anna wrote:
> On 6/17/19 9:54 AM, Tero Kristo wrote:
>> On 30/05/2019 13:49, Nishanth Menon wrote:
>>> On 16:13-20190529, Suman Anna wrote:
>>>> From: Roger Quadros <rogerq@ti.com>
>>>>
>>>> Add the MSCM RAM address space to the ranges property of the cbass_main
>>>> interconnect node so that the addresses can be translated properly.
>>>>
>>>> This fixes the probe failure in the sram driver for the MSMC RAM node.
>>>>
>>>> Signed-off-by: Roger Quadros <rogerq@ti.com>
>>>> Signed-off-by: Suman Anna <s-anna@ti.com>
>>>> ---
>>>>
>>>> The following error message is seen without this:
>>>> [    0.480261] sram interconnect@100000:sram@70000000: found no
>>>> memory resource
>>>> [    0.487497] sram: probe of interconnect@100000:sram@70000000
>>>> failed with error -22
>>>>
>>>> regards
>>>> Suman
>>>>
>>>>    arch/arm64/boot/dts/ti/k3-am65.dtsi | 1 +
>>>>    1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/arch/arm64/boot/dts/ti/k3-am65.dtsi
>>>> b/arch/arm64/boot/dts/ti/k3-am65.dtsi
>>>> index 50f4be2047a9..68b3f954f1d1 100644
>>>> --- a/arch/arm64/boot/dts/ti/k3-am65.dtsi
>>>> +++ b/arch/arm64/boot/dts/ti/k3-am65.dtsi
>>>> @@ -68,6 +68,7 @@
>>>>                 <0x00 0x00900000 0x00 0x00900000 0x00 0x00012000>, /*
>>>> serdes */
>>>>                 <0x00 0x01000000 0x00 0x01000000 0x00 0x0af02400>, /*
>>>> Most peripherals */
>>>>                 <0x00 0x30800000 0x00 0x30800000 0x00 0x0bc00000>, /*
>>>> MAIN NAVSS */
>>>> +             <0x00 0x70000000 0x00 0x70000000 0x00 0x00200000>, /*
>>>> MSMC SRAM */
>>>>                 /* MCUSS Range */
>>>>                 <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>,
>>>>                 <0x00 0x40200000 0x00 0x40200000 0x00 0x00900100>,
>>>> -- 
>>>> 2.21.0
>>>>
>>>
>>> Arrgh.. Nice catch. Thanks. We should consider pulling this in as part
>>> of early rc fixes please.
>>>
>>> Acked-by: Nishanth Menon <nm@ti.com>
>>
>> Hmm, what is the failure this causes, except the probe message? SRAM
>> driver obviously won't work but is it used for anything at the moment
>> anyways?
>>
>> I would rather queue this towards 5.3.
> 
> OK, I am fine either way.
> 
> regards
> Suman
> 

Ok thanks, queued towards 5.3.

That basically saves me one pull-request. :)

-Tero
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

end of thread, back to index

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-29 21:13 [PATCH] arm64: dts: ti: k3-am65: Add MSMC RAM ranges in interconnect node Suman Anna
2019-05-30 10:49 ` Nishanth Menon
2019-06-17 14:54   ` Tero Kristo
2019-06-17 15:42     ` Nishanth Menon
2019-06-17 15:53     ` Suman Anna
2019-06-17 19:42       ` Tero Kristo

Linux-ARM-Kernel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-arm-kernel/0 linux-arm-kernel/git/0.git
	git clone --mirror https://lore.kernel.org/linux-arm-kernel/1 linux-arm-kernel/git/1.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-arm-kernel linux-arm-kernel/ https://lore.kernel.org/linux-arm-kernel \
		linux-arm-kernel@lists.infradead.org infradead-linux-arm-kernel@archiver.kernel.org
	public-inbox-index linux-arm-kernel


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.infradead.lists.linux-arm-kernel


AGPL code for this site: git clone https://public-inbox.org/ public-inbox