* [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
@ 2016-03-02 22:37 Stuart Yoder
2016-03-03 2:44 ` Prabhakar Kushwaha
2016-03-29 15:40 ` York Sun
0 siblings, 2 replies; 6+ messages in thread
From: Stuart Yoder @ 2016-03-02 22:37 UTC (permalink / raw)
To: u-boot
From: Stuart Yoder <stuart.yoder@nxp.com>
The fsl-mc node has been moved under /soc, so update
the path references accordingly. Backwards compatibility
is retained for /fsl-mc.
Delete backwards compatibility for the completely obsolete
/fsl,dprc at 0.
Signed-off-by: Stuart Yoder <stuart.yoder@nxp.com>
---
board/freescale/ls2080a/ls2080a.c | 6 +++---
board/freescale/ls2080aqds/ls2080aqds.c | 4 ++--
board/freescale/ls2080ardb/ls2080ardb.c | 4 ++--
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/board/freescale/ls2080a/ls2080a.c b/board/freescale/ls2080a/ls2080a.c
index 7bce8b0..ace0d79 100644
--- a/board/freescale/ls2080a/ls2080a.c
+++ b/board/freescale/ls2080a/ls2080a.c
@@ -87,14 +87,14 @@ void fdt_fixup_board_enet(void *fdt)
{
int offset;
- offset = fdt_path_offset(fdt, "/fsl-mc");
+ offset = fdt_path_offset(fdt, "/soc/fsl-mc");
/*
* TODO: Remove this when backward compatibility
- * with old DT node (fsl,dprc at 0) is no longer needed.
+ * with old DT node (/fsl-mc) is no longer needed.
*/
if (offset < 0)
- offset = fdt_path_offset(fdt, "/fsl,dprc at 0");
+ offset = fdt_path_offset(fdt, "/fsl-mc");
if (offset < 0) {
printf("%s: ERROR: fsl-mc node not found in device tree (error %d)\n",
diff --git a/board/freescale/ls2080aqds/ls2080aqds.c b/board/freescale/ls2080aqds/ls2080aqds.c
index aa256a2..f3925e2 100644
--- a/board/freescale/ls2080aqds/ls2080aqds.c
+++ b/board/freescale/ls2080aqds/ls2080aqds.c
@@ -258,10 +258,10 @@ void fdt_fixup_board_enet(void *fdt)
{
int offset;
- offset = fdt_path_offset(fdt, "/fsl-mc");
+ offset = fdt_path_offset(fdt, "/soc/fsl-mc");
if (offset < 0)
- offset = fdt_path_offset(fdt, "/fsl,dprc at 0");
+ offset = fdt_path_offset(fdt, "/fsl-mc");
if (offset < 0) {
printf("%s: ERROR: fsl-mc node not found in device tree (error %d)\n",
diff --git a/board/freescale/ls2080ardb/ls2080ardb.c b/board/freescale/ls2080ardb/ls2080ardb.c
index c63b639..6f518b2 100644
--- a/board/freescale/ls2080ardb/ls2080ardb.c
+++ b/board/freescale/ls2080ardb/ls2080ardb.c
@@ -224,10 +224,10 @@ void fdt_fixup_board_enet(void *fdt)
{
int offset;
- offset = fdt_path_offset(fdt, "/fsl-mc");
+ offset = fdt_path_offset(fdt, "/soc/fsl-mc");
if (offset < 0)
- offset = fdt_path_offset(fdt, "/fsl,dprc@0");
+ offset = fdt_path_offset(fdt, "/fsl-mc");
if (offset < 0) {
printf("%s: ERROR: fsl-mc node not found in device tree (error %d)\n",
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
2016-03-02 22:37 [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node Stuart Yoder
@ 2016-03-03 2:44 ` Prabhakar Kushwaha
2016-03-03 3:01 ` Stuart Yoder
2016-03-29 15:40 ` York Sun
1 sibling, 1 reply; 6+ messages in thread
From: Prabhakar Kushwaha @ 2016-03-03 2:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Stuart Yoder [mailto:stuart.yoder at nxp.com]
> Sent: Thursday, March 03, 2016 4:07 AM
> To: u-boot at lists.denx.de
> Cc: york sun <york.sun@nxp.com>; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; Yang-Leo Li <leoyang.li@nxp.com>;
> Stuart Yoder <stuart.yoder@nxp.com>
> Subject: [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
>
> From: Stuart Yoder <stuart.yoder@nxp.com>
>
> The fsl-mc node has been moved under /soc, so update the path references
> accordingly. Backwards compatibility is retained for /fsl-mc.
>
> Delete backwards compatibility for the completely obsolete /fsl,dprc at 0.
>
> Signed-off-by: Stuart Yoder <stuart.yoder@nxp.com>
How will backward compatibility maintained for older Linux version which has previous device tree format..
--prabhakar
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
2016-03-03 2:44 ` Prabhakar Kushwaha
@ 2016-03-03 3:01 ` Stuart Yoder
2016-03-22 19:04 ` york sun
0 siblings, 1 reply; 6+ messages in thread
From: Stuart Yoder @ 2016-03-03 3:01 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Prabhakar Kushwaha
> Sent: Wednesday, March 02, 2016 8:45 PM
> To: Stuart Yoder <stuart.yoder@nxp.com>; u-boot at lists.denx.de
> Cc: york sun <york.sun@nxp.com>; Yang-Leo Li <leoyang.li@nxp.com>; Stuart Yoder
> <stuart.yoder@nxp.com>
> Subject: RE: [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
>
>
> > -----Original Message-----
> > From: Stuart Yoder [mailto:stuart.yoder at nxp.com]
> > Sent: Thursday, March 03, 2016 4:07 AM
> > To: u-boot at lists.denx.de
> > Cc: york sun <york.sun@nxp.com>; Prabhakar Kushwaha
> > <prabhakar.kushwaha@nxp.com>; Yang-Leo Li <leoyang.li@nxp.com>;
> > Stuart Yoder <stuart.yoder@nxp.com>
> > Subject: [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
> >
> > From: Stuart Yoder <stuart.yoder@nxp.com>
> >
> > The fsl-mc node has been moved under /soc, so update the path references
> > accordingly. Backwards compatibility is retained for /fsl-mc.
> >
> > Delete backwards compatibility for the completely obsolete /fsl,dprc at 0.
> >
> > Signed-off-by: Stuart Yoder <stuart.yoder@nxp.com>
>
> How will backward compatibility maintained for older Linux version which has previous
> device tree format..
As you can see in the patch, we first check /soc/fsl-mc. If that is not
there we check /fsl-mc. So, both will work.
Use of /fsl,dprc at 0 I think was Freescale-internal only and there is
no actual use of it remaining.
Stuart
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
2016-03-03 3:01 ` Stuart Yoder
@ 2016-03-22 19:04 ` york sun
2016-03-23 8:12 ` Prabhakar Kushwaha
0 siblings, 1 reply; 6+ messages in thread
From: york sun @ 2016-03-22 19:04 UTC (permalink / raw)
To: u-boot
On 03/02/2016 07:01 PM, Stuart Yoder wrote:
>
>
>> -----Original Message-----
>> From: Prabhakar Kushwaha
>> Sent: Wednesday, March 02, 2016 8:45 PM
>> To: Stuart Yoder <stuart.yoder@nxp.com>; u-boot at lists.denx.de
>> Cc: york sun <york.sun@nxp.com>; Yang-Leo Li <leoyang.li@nxp.com>; Stuart Yoder
>> <stuart.yoder@nxp.com>
>> Subject: RE: [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
>>
>>
>>> -----Original Message-----
>>> From: Stuart Yoder [mailto:stuart.yoder at nxp.com]
>>> Sent: Thursday, March 03, 2016 4:07 AM
>>> To: u-boot at lists.denx.de
>>> Cc: york sun <york.sun@nxp.com>; Prabhakar Kushwaha
>>> <prabhakar.kushwaha@nxp.com>; Yang-Leo Li <leoyang.li@nxp.com>;
>>> Stuart Yoder <stuart.yoder@nxp.com>
>>> Subject: [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
>>>
>>> From: Stuart Yoder <stuart.yoder@nxp.com>
>>>
>>> The fsl-mc node has been moved under /soc, so update the path references
>>> accordingly. Backwards compatibility is retained for /fsl-mc.
>>>
>>> Delete backwards compatibility for the completely obsolete /fsl,dprc at 0.
>>>
>>> Signed-off-by: Stuart Yoder <stuart.yoder@nxp.com>
>>
>> How will backward compatibility maintained for older Linux version which has previous
>> device tree format..
>
> As you can see in the patch, we first check /soc/fsl-mc. If that is not
> there we check /fsl-mc. So, both will work.
>
> Use of /fsl,dprc at 0 I think was Freescale-internal only and there is
> no actual use of it remaining.
>
Prabhakar,
Are you OK with Stuart's answer?
York
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
2016-03-22 19:04 ` york sun
@ 2016-03-23 8:12 ` Prabhakar Kushwaha
0 siblings, 0 replies; 6+ messages in thread
From: Prabhakar Kushwaha @ 2016-03-23 8:12 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: york sun
> Sent: Wednesday, March 23, 2016 12:34 AM
> To: Stuart Yoder <stuart.yoder@nxp.com>; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; u-boot at lists.denx.de
> Cc: Yang-Leo Li <leoyang.li@nxp.com>
> Subject: Re: [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
>
> On 03/02/2016 07:01 PM, Stuart Yoder wrote:
> >
> >
> >> -----Original Message-----
> >> From: Prabhakar Kushwaha
> >> Sent: Wednesday, March 02, 2016 8:45 PM
> >> To: Stuart Yoder <stuart.yoder@nxp.com>; u-boot at lists.denx.de
> >> Cc: york sun <york.sun@nxp.com>; Yang-Leo Li <leoyang.li@nxp.com>;
> >> Stuart Yoder <stuart.yoder@nxp.com>
> >> Subject: RE: [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
> >>
> >>
> >>> -----Original Message-----
> >>> From: Stuart Yoder [mailto:stuart.yoder at nxp.com]
> >>> Sent: Thursday, March 03, 2016 4:07 AM
> >>> To: u-boot at lists.denx.de
> >>> Cc: york sun <york.sun@nxp.com>; Prabhakar Kushwaha
> >>> <prabhakar.kushwaha@nxp.com>; Yang-Leo Li <leoyang.li@nxp.com>;
> >>> Stuart Yoder <stuart.yoder@nxp.com>
> >>> Subject: [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
> >>>
> >>> From: Stuart Yoder <stuart.yoder@nxp.com>
> >>>
> >>> The fsl-mc node has been moved under /soc, so update the path
> >>> references accordingly. Backwards compatibility is retained for /fsl-mc.
> >>>
> >>> Delete backwards compatibility for the completely obsolete /fsl,dprc at 0.
> >>>
> >>> Signed-off-by: Stuart Yoder <stuart.yoder@nxp.com>
> >>
> >> How will backward compatibility maintained for older Linux version
> >> which has previous device tree format..
> >
> > As you can see in the patch, we first check /soc/fsl-mc. If that is
> > not there we check /fsl-mc. So, both will work.
> >
> > Use of /fsl,dprc at 0 I think was Freescale-internal only and there is no
> > actual use of it remaining.
> >
>
> Prabhakar,
>
> Are you OK with Stuart's answer?
>
Yes York.
--prabhakar
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node
2016-03-02 22:37 [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node Stuart Yoder
2016-03-03 2:44 ` Prabhakar Kushwaha
@ 2016-03-29 15:40 ` York Sun
1 sibling, 0 replies; 6+ messages in thread
From: York Sun @ 2016-03-29 15:40 UTC (permalink / raw)
To: u-boot
On 03/02/2016 02:58 PM, Stuart Yoder wrote:
> From: Stuart Yoder <stuart.yoder@nxp.com>
>
> The fsl-mc node has been moved under /soc, so update
> the path references accordingly. Backwards compatibility
> is retained for /fsl-mc.
>
> Delete backwards compatibility for the completely obsolete
> /fsl,dprc at 0.
>
> Signed-off-by: Stuart Yoder <stuart.yoder@nxp.com>
> ---
> board/freescale/ls2080a/ls2080a.c | 6 +++---
> board/freescale/ls2080aqds/ls2080aqds.c | 4 ++--
> board/freescale/ls2080ardb/ls2080ardb.c | 4 ++--
> 3 files changed, 7 insertions(+), 7 deletions(-)
Applied to u-boot-fsl-qoriq master. Awaiting upstream.
Thanks.
York
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-03-29 15:40 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-02 22:37 [U-Boot] [PATCH] armv8: ls2080a: update fdt path for fsl-mc node Stuart Yoder
2016-03-03 2:44 ` Prabhakar Kushwaha
2016-03-03 3:01 ` Stuart Yoder
2016-03-22 19:04 ` york sun
2016-03-23 8:12 ` Prabhakar Kushwaha
2016-03-29 15:40 ` York Sun
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox