* [PATCH repost] ARM: shmobile: lager: correct memory map
@ 2014-09-03 0:49 ` Simon Horman
0 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2014-09-03 0:49 UTC (permalink / raw)
To: linux-arm-kernel
The base address of the second memory region on the lager
board is 0x140000000. Update the tag used in the dts file accordingly.
This is a documentation fix and should have no run-time affect.
This problem was introduced when the second memory region
was added to the lager dts file by 62bc32a2573c4219
("ARM: shmobile: Include all 4 GiB of memory on Lager)"
in v3.14.
Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
I am reposting this as Olof originally requested a syntax change,
which I followed up on, however that change no longer seems appropriate.
This this patch seems to be correct.
diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index 0118cbf..84dcafa 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -32,7 +32,7 @@
reg = <0 0x40000000 0 0x40000000>;
};
- memory@180000000 {
+ memory@140000000 {
device_type = "memory";
reg = <1 0x40000000 0 0xc0000000>;
};
--
2.0.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH repost] ARM: shmobile: lager: correct memory map
@ 2014-09-03 0:49 ` Simon Horman
0 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2014-09-03 0:49 UTC (permalink / raw)
To: linux-arm-kernel
The base address of the second memory region on the lager
board is 0x140000000. Update the tag used in the dts file accordingly.
This is a documentation fix and should have no run-time affect.
This problem was introduced when the second memory region
was added to the lager dts file by 62bc32a2573c4219
("ARM: shmobile: Include all 4 GiB of memory on Lager)"
in v3.14.
Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
I am reposting this as Olof originally requested a syntax change,
which I followed up on, however that change no longer seems appropriate.
This this patch seems to be correct.
diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index 0118cbf..84dcafa 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -32,7 +32,7 @@
reg = <0 0x40000000 0 0x40000000>;
};
- memory at 180000000 {
+ memory at 140000000 {
device_type = "memory";
reg = <1 0x40000000 0 0xc0000000>;
};
--
2.0.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH repost] ARM: shmobile: lager: correct memory map
2014-09-03 0:49 ` Simon Horman
@ 2014-09-03 2:42 ` Khiem Nguyen
-1 siblings, 0 replies; 10+ messages in thread
From: Khiem Nguyen @ 2014-09-03 2:42 UTC (permalink / raw)
To: linux-arm-kernel
Dear Simon-san, Shiiba-san,
Thanks for the patch.
On 9/3/2014 9:49 AM, Simon Horman wrote:
> The base address of the second memory region on the lager
> board is 0x140000000. Update the tag used in the dts file accordingly.
>
> This is a documentation fix and should have no run-time affect.
>
> This problem was introduced when the second memory region
> was added to the lager dts file by 62bc32a2573c4219
> ("ARM: shmobile: Include all 4 GiB of memory on Lager)"
> in v3.14.
>
> Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> I am reposting this as Olof originally requested a syntax change,
> which I followed up on, however that change no longer seems appropriate.
> This this patch seems to be correct.
[...]
> - memory@180000000 {
> + memory@140000000 {
Could you give more information about not using new syntax ?
I checked memblock information in both 2 ways
(i.e memory@140000000 and memory@1,40000000)
and I got same result.
root@lager:~# cat /sys/kernel/debug/memblock/memory
0: 0x0000000040000000..0x000000007fffffff
1: 0x0000000140000000..0x00000001ffffffff
So, what does 'no longer seems appropriate" mean ?
> device_type = "memory";
> reg = <1 0x40000000 0 0xc0000000>;
> };
>
Thanks.
--
Best regards,
KHIEM Nguyen
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH repost] ARM: shmobile: lager: correct memory map
@ 2014-09-03 2:42 ` Khiem Nguyen
0 siblings, 0 replies; 10+ messages in thread
From: Khiem Nguyen @ 2014-09-03 2:42 UTC (permalink / raw)
To: linux-arm-kernel
Dear Simon-san, Shiiba-san,
Thanks for the patch.
On 9/3/2014 9:49 AM, Simon Horman wrote:
> The base address of the second memory region on the lager
> board is 0x140000000. Update the tag used in the dts file accordingly.
>
> This is a documentation fix and should have no run-time affect.
>
> This problem was introduced when the second memory region
> was added to the lager dts file by 62bc32a2573c4219
> ("ARM: shmobile: Include all 4 GiB of memory on Lager)"
> in v3.14.
>
> Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> I am reposting this as Olof originally requested a syntax change,
> which I followed up on, however that change no longer seems appropriate.
> This this patch seems to be correct.
[...]
> - memory at 180000000 {
> + memory at 140000000 {
Could you give more information about not using new syntax ?
I checked memblock information in both 2 ways
(i.e memory at 140000000 and memory at 1,40000000)
and I got same result.
root at lager:~# cat /sys/kernel/debug/memblock/memory
0: 0x0000000040000000..0x000000007fffffff
1: 0x0000000140000000..0x00000001ffffffff
So, what does 'no longer seems appropriate" mean ?
> device_type = "memory";
> reg = <1 0x40000000 0 0xc0000000>;
> };
>
Thanks.
--
Best regards,
KHIEM Nguyen
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH repost] ARM: shmobile: lager: correct memory map
2014-09-03 2:42 ` Khiem Nguyen
@ 2014-09-03 5:48 ` Simon Horman
-1 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2014-09-03 5:48 UTC (permalink / raw)
To: linux-arm-kernel
Hi Khiem-san,
On Wed, Sep 03, 2014 at 11:42:57AM +0900, Khiem Nguyen wrote:
> Dear Simon-san, Shiiba-san,
>
> Thanks for the patch.
>
> On 9/3/2014 9:49 AM, Simon Horman wrote:
> > The base address of the second memory region on the lager
> > board is 0x140000000. Update the tag used in the dts file accordingly.
> >
> > This is a documentation fix and should have no run-time affect.
> >
> > This problem was introduced when the second memory region
> > was added to the lager dts file by 62bc32a2573c4219
> > ("ARM: shmobile: Include all 4 GiB of memory on Lager)"
> > in v3.14.
> >
> > Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > I am reposting this as Olof originally requested a syntax change,
> > which I followed up on, however that change no longer seems appropriate.
> > This this patch seems to be correct.
> [...]
> > - memory@180000000 {
> > + memory@140000000 {
>
> Could you give more information about not using new syntax ?
>
> I checked memblock information in both 2 ways
> (i.e memory@140000000 and memory@1,40000000)
> and I got same result.
> root@lager:~# cat /sys/kernel/debug/memblock/memory
> 0: 0x0000000040000000..0x000000007fffffff
> 1: 0x0000000140000000..0x00000001ffffffff
>
> So, what does 'no longer seems appropriate" mean ?
Sorry for being vague.
I am specifically referring to the conversation in
the thread "[PATCH 11/14] arm64: dts: Add initial device tree support for
EXYNOS7" and in particular the following contribution to that discussion by
Olof:
"Ok, I'm happily proven wrong here, also by confirming how this is
done on "real" OF.
According to benh:
15:20 <benh> ojn: 0,0 is not quite right, it's supposed to be used
when the two numbers are different things, like device,fn on PCI
The same is true for >2^32 unit addresses, they just use the one
integer instead of x,y.
So, I take back all I've said on this in the last 72 hours. :) It
looks like we might need to revisit some of the 32-bit DTs. Simon,
drop the series you had. :)"
An archive of the message in question is available at:
https://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg36123.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH repost] ARM: shmobile: lager: correct memory map
@ 2014-09-03 5:48 ` Simon Horman
0 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2014-09-03 5:48 UTC (permalink / raw)
To: linux-arm-kernel
Hi Khiem-san,
On Wed, Sep 03, 2014 at 11:42:57AM +0900, Khiem Nguyen wrote:
> Dear Simon-san, Shiiba-san,
>
> Thanks for the patch.
>
> On 9/3/2014 9:49 AM, Simon Horman wrote:
> > The base address of the second memory region on the lager
> > board is 0x140000000. Update the tag used in the dts file accordingly.
> >
> > This is a documentation fix and should have no run-time affect.
> >
> > This problem was introduced when the second memory region
> > was added to the lager dts file by 62bc32a2573c4219
> > ("ARM: shmobile: Include all 4 GiB of memory on Lager)"
> > in v3.14.
> >
> > Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > I am reposting this as Olof originally requested a syntax change,
> > which I followed up on, however that change no longer seems appropriate.
> > This this patch seems to be correct.
> [...]
> > - memory at 180000000 {
> > + memory at 140000000 {
>
> Could you give more information about not using new syntax ?
>
> I checked memblock information in both 2 ways
> (i.e memory at 140000000 and memory at 1,40000000)
> and I got same result.
> root at lager:~# cat /sys/kernel/debug/memblock/memory
> 0: 0x0000000040000000..0x000000007fffffff
> 1: 0x0000000140000000..0x00000001ffffffff
>
> So, what does 'no longer seems appropriate" mean ?
Sorry for being vague.
I am specifically referring to the conversation in
the thread "[PATCH 11/14] arm64: dts: Add initial device tree support for
EXYNOS7" and in particular the following contribution to that discussion by
Olof:
"Ok, I'm happily proven wrong here, also by confirming how this is
done on "real" OF.
According to benh:
15:20 <benh> ojn: 0,0 is not quite right, it's supposed to be used
when the two numbers are different things, like device,fn on PCI
The same is true for >2^32 unit addresses, they just use the one
integer instead of x,y.
So, I take back all I've said on this in the last 72 hours. :) It
looks like we might need to revisit some of the 32-bit DTs. Simon,
drop the series you had. :)"
An archive of the message in question is available at:
https://www.mail-archive.com/linux-samsung-soc at vger.kernel.org/msg36123.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH repost] ARM: shmobile: lager: correct memory map
2014-09-03 5:48 ` Simon Horman
@ 2014-09-03 6:22 ` Khiem Nguyen
-1 siblings, 0 replies; 10+ messages in thread
From: Khiem Nguyen @ 2014-09-03 6:22 UTC (permalink / raw)
To: linux-arm-kernel
Dear Simon-san,
On 9/3/2014 2:48 PM, Simon Horman wrote:
> Hi Khiem-san,
>
> On Wed, Sep 03, 2014 at 11:42:57AM +0900, Khiem Nguyen wrote:
>> Dear Simon-san, Shiiba-san,
>>
>> Thanks for the patch.
>>
>> On 9/3/2014 9:49 AM, Simon Horman wrote:
>>> The base address of the second memory region on the lager
>>> board is 0x140000000. Update the tag used in the dts file accordingly.
>>>
>>> This is a documentation fix and should have no run-time affect.
>>>
>>> This problem was introduced when the second memory region
>>> was added to the lager dts file by 62bc32a2573c4219
>>> ("ARM: shmobile: Include all 4 GiB of memory on Lager)"
>>> in v3.14.
>>>
>>> Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
>>> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
>>> ---
>>> arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> I am reposting this as Olof originally requested a syntax change,
>>> which I followed up on, however that change no longer seems appropriate.
>>> This this patch seems to be correct.
>> [...]
>>> - memory@180000000 {
>>> + memory@140000000 {
>>
>> Could you give more information about not using new syntax ?
>>
>> I checked memblock information in both 2 ways
>> (i.e memory@140000000 and memory@1,40000000)
>> and I got same result.
>> root@lager:~# cat /sys/kernel/debug/memblock/memory
>> 0: 0x0000000040000000..0x000000007fffffff
>> 1: 0x0000000140000000..0x00000001ffffffff
>>
>> So, what does 'no longer seems appropriate" mean ?
>
> Sorry for being vague.
>
> I am specifically referring to the conversation in
> the thread "[PATCH 11/14] arm64: dts: Add initial device tree support for
> EXYNOS7" and in particular the following contribution to that discussion by
> Olof:
>
> "Ok, I'm happily proven wrong here, also by confirming how this is
> done on "real" OF.
>
> According to benh:
>
> 15:20 <benh> ojn: 0,0 is not quite right, it's supposed to be used
> when the two numbers are different things, like device,fn on PCI
>
> The same is true for >2^32 unit addresses, they just use the one
> integer instead of x,y.
>
> So, I take back all I've said on this in the last 72 hours. :) It
> looks like we might need to revisit some of the 32-bit DTs. Simon,
> drop the series you had. :)"
>
> An archive of the message in question is available at:
> https://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg36123.html
OK. I understood.
Let's follow the right way. :)
--
Best regards,
KHIEM Nguyen
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH repost] ARM: shmobile: lager: correct memory map
@ 2014-09-03 6:22 ` Khiem Nguyen
0 siblings, 0 replies; 10+ messages in thread
From: Khiem Nguyen @ 2014-09-03 6:22 UTC (permalink / raw)
To: linux-arm-kernel
Dear Simon-san,
On 9/3/2014 2:48 PM, Simon Horman wrote:
> Hi Khiem-san,
>
> On Wed, Sep 03, 2014 at 11:42:57AM +0900, Khiem Nguyen wrote:
>> Dear Simon-san, Shiiba-san,
>>
>> Thanks for the patch.
>>
>> On 9/3/2014 9:49 AM, Simon Horman wrote:
>>> The base address of the second memory region on the lager
>>> board is 0x140000000. Update the tag used in the dts file accordingly.
>>>
>>> This is a documentation fix and should have no run-time affect.
>>>
>>> This problem was introduced when the second memory region
>>> was added to the lager dts file by 62bc32a2573c4219
>>> ("ARM: shmobile: Include all 4 GiB of memory on Lager)"
>>> in v3.14.
>>>
>>> Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
>>> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
>>> ---
>>> arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> I am reposting this as Olof originally requested a syntax change,
>>> which I followed up on, however that change no longer seems appropriate.
>>> This this patch seems to be correct.
>> [...]
>>> - memory at 180000000 {
>>> + memory at 140000000 {
>>
>> Could you give more information about not using new syntax ?
>>
>> I checked memblock information in both 2 ways
>> (i.e memory at 140000000 and memory at 1,40000000)
>> and I got same result.
>> root at lager:~# cat /sys/kernel/debug/memblock/memory
>> 0: 0x0000000040000000..0x000000007fffffff
>> 1: 0x0000000140000000..0x00000001ffffffff
>>
>> So, what does 'no longer seems appropriate" mean ?
>
> Sorry for being vague.
>
> I am specifically referring to the conversation in
> the thread "[PATCH 11/14] arm64: dts: Add initial device tree support for
> EXYNOS7" and in particular the following contribution to that discussion by
> Olof:
>
> "Ok, I'm happily proven wrong here, also by confirming how this is
> done on "real" OF.
>
> According to benh:
>
> 15:20 <benh> ojn: 0,0 is not quite right, it's supposed to be used
> when the two numbers are different things, like device,fn on PCI
>
> The same is true for >2^32 unit addresses, they just use the one
> integer instead of x,y.
>
> So, I take back all I've said on this in the last 72 hours. :) It
> looks like we might need to revisit some of the 32-bit DTs. Simon,
> drop the series you had. :)"
>
> An archive of the message in question is available at:
> https://www.mail-archive.com/linux-samsung-soc at vger.kernel.org/msg36123.html
OK. I understood.
Let's follow the right way. :)
--
Best regards,
KHIEM Nguyen
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH repost] ARM: shmobile: lager: correct memory map
2014-09-03 6:22 ` Khiem Nguyen
@ 2014-09-04 1:10 ` Simon Horman
-1 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2014-09-04 1:10 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Sep 03, 2014 at 03:22:14PM +0900, Khiem Nguyen wrote:
> Dear Simon-san,
>
> On 9/3/2014 2:48 PM, Simon Horman wrote:
> > Hi Khiem-san,
> >
> > On Wed, Sep 03, 2014 at 11:42:57AM +0900, Khiem Nguyen wrote:
> >> Dear Simon-san, Shiiba-san,
> >>
> >> Thanks for the patch.
> >>
> >> On 9/3/2014 9:49 AM, Simon Horman wrote:
> >>> The base address of the second memory region on the lager
> >>> board is 0x140000000. Update the tag used in the dts file accordingly.
> >>>
> >>> This is a documentation fix and should have no run-time affect.
> >>>
> >>> This problem was introduced when the second memory region
> >>> was added to the lager dts file by 62bc32a2573c4219
> >>> ("ARM: shmobile: Include all 4 GiB of memory on Lager)"
> >>> in v3.14.
> >>>
> >>> Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
> >>> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >>> ---
> >>> arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
> >>> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> I am reposting this as Olof originally requested a syntax change,
> >>> which I followed up on, however that change no longer seems appropriate.
> >>> This this patch seems to be correct.
> >> [...]
> >>> - memory@180000000 {
> >>> + memory@140000000 {
> >>
> >> Could you give more information about not using new syntax ?
> >>
> >> I checked memblock information in both 2 ways
> >> (i.e memory@140000000 and memory@1,40000000)
> >> and I got same result.
> >> root@lager:~# cat /sys/kernel/debug/memblock/memory
> >> 0: 0x0000000040000000..0x000000007fffffff
> >> 1: 0x0000000140000000..0x00000001ffffffff
> >>
> >> So, what does 'no longer seems appropriate" mean ?
> >
> > Sorry for being vague.
> >
> > I am specifically referring to the conversation in
> > the thread "[PATCH 11/14] arm64: dts: Add initial device tree support for
> > EXYNOS7" and in particular the following contribution to that discussion by
> > Olof:
> >
> > "Ok, I'm happily proven wrong here, also by confirming how this is
> > done on "real" OF.
> >
> > According to benh:
> >
> > 15:20 <benh> ojn: 0,0 is not quite right, it's supposed to be used
> > when the two numbers are different things, like device,fn on PCI
> >
> > The same is true for >2^32 unit addresses, they just use the one
> > integer instead of x,y.
> >
> > So, I take back all I've said on this in the last 72 hours. :) It
> > looks like we might need to revisit some of the 32-bit DTs. Simon,
> > drop the series you had. :)"
> >
> > An archive of the message in question is available at:
> > https://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg36123.html
>
> OK. I understood.
> Let's follow the right way. :)
Thanks. I have queued up this patch.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH repost] ARM: shmobile: lager: correct memory map
@ 2014-09-04 1:10 ` Simon Horman
0 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2014-09-04 1:10 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Sep 03, 2014 at 03:22:14PM +0900, Khiem Nguyen wrote:
> Dear Simon-san,
>
> On 9/3/2014 2:48 PM, Simon Horman wrote:
> > Hi Khiem-san,
> >
> > On Wed, Sep 03, 2014 at 11:42:57AM +0900, Khiem Nguyen wrote:
> >> Dear Simon-san, Shiiba-san,
> >>
> >> Thanks for the patch.
> >>
> >> On 9/3/2014 9:49 AM, Simon Horman wrote:
> >>> The base address of the second memory region on the lager
> >>> board is 0x140000000. Update the tag used in the dts file accordingly.
> >>>
> >>> This is a documentation fix and should have no run-time affect.
> >>>
> >>> This problem was introduced when the second memory region
> >>> was added to the lager dts file by 62bc32a2573c4219
> >>> ("ARM: shmobile: Include all 4 GiB of memory on Lager)"
> >>> in v3.14.
> >>>
> >>> Reported-by: NAOYA SHIIBA <naoya.shiiba.nx@renesas.com>
> >>> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >>> ---
> >>> arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
> >>> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> I am reposting this as Olof originally requested a syntax change,
> >>> which I followed up on, however that change no longer seems appropriate.
> >>> This this patch seems to be correct.
> >> [...]
> >>> - memory at 180000000 {
> >>> + memory at 140000000 {
> >>
> >> Could you give more information about not using new syntax ?
> >>
> >> I checked memblock information in both 2 ways
> >> (i.e memory at 140000000 and memory at 1,40000000)
> >> and I got same result.
> >> root at lager:~# cat /sys/kernel/debug/memblock/memory
> >> 0: 0x0000000040000000..0x000000007fffffff
> >> 1: 0x0000000140000000..0x00000001ffffffff
> >>
> >> So, what does 'no longer seems appropriate" mean ?
> >
> > Sorry for being vague.
> >
> > I am specifically referring to the conversation in
> > the thread "[PATCH 11/14] arm64: dts: Add initial device tree support for
> > EXYNOS7" and in particular the following contribution to that discussion by
> > Olof:
> >
> > "Ok, I'm happily proven wrong here, also by confirming how this is
> > done on "real" OF.
> >
> > According to benh:
> >
> > 15:20 <benh> ojn: 0,0 is not quite right, it's supposed to be used
> > when the two numbers are different things, like device,fn on PCI
> >
> > The same is true for >2^32 unit addresses, they just use the one
> > integer instead of x,y.
> >
> > So, I take back all I've said on this in the last 72 hours. :) It
> > looks like we might need to revisit some of the 32-bit DTs. Simon,
> > drop the series you had. :)"
> >
> > An archive of the message in question is available at:
> > https://www.mail-archive.com/linux-samsung-soc at vger.kernel.org/msg36123.html
>
> OK. I understood.
> Let's follow the right way. :)
Thanks. I have queued up this patch.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-09-04 1:10 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-03 0:49 [PATCH repost] ARM: shmobile: lager: correct memory map Simon Horman
2014-09-03 0:49 ` Simon Horman
2014-09-03 2:42 ` Khiem Nguyen
2014-09-03 2:42 ` Khiem Nguyen
2014-09-03 5:48 ` Simon Horman
2014-09-03 5:48 ` Simon Horman
2014-09-03 6:22 ` Khiem Nguyen
2014-09-03 6:22 ` Khiem Nguyen
2014-09-04 1:10 ` Simon Horman
2014-09-04 1:10 ` Simon Horman
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.