Re: [OE-core] [PATCH v2] device-tree.bbclass: Add support to compile overlays separately

2019-02-11 Thread Jaewon Lee
Ping

-Original Message-
From: Jaewon Lee 
Sent: Monday, January 28, 2019 3:19 PM
To: Jaewon Lee ; openembedded-core@lists.openembedded.org; 
nat...@nathanrossi.com
Cc: Alejandro Enedino Hernandez Samaniego ; Manjukumar 
Harthikote Matha 
Subject: RE: [OE-core][PATCH v2] device-tree.bbclass: Add support to compile 
overlays separately

ping

-Original Message-
From: Jaewon Lee [mailto:jaewon@xilinx.com] 
Sent: Friday, December 14, 2018 9:54 AM
To: openembedded-core@lists.openembedded.org; nat...@nathanrossi.com
Cc: Jaewon Lee ; Alejandro Enedino Hernandez Samaniego 
; Manjukumar Harthikote Matha 
Subject: [OE-core][PATCH v2] device-tree.bbclass: Add support to compile 
overlays separately

Currently only dts files are considered when looping through files to compile. 
Modifying the loop to compile other files that are overlays.
Also surrounding this check with a try block as the function to find overlays 
parses the file for a '/plugin/' tag, and there may be files in the 
DT_FILES_PATH directory that are not parseable.

Signed-off-by: Jaewon Lee 
Signed-off-by: Alejandro Enedino Hernandez Samaniego 
Signed-off-by: Manjukumar Matha 
---
 meta/classes/devicetree.bbclass | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/meta/classes/devicetree.bbclass b/meta/classes/devicetree.bbclass 
index 8fe5a5e..d785285 100644
--- a/meta/classes/devicetree.bbclass
+++ b/meta/classes/devicetree.bbclass
@@ -120,9 +120,12 @@ python devicetree_do_compile() {
 includes = expand_includes("DT_INCLUDE", d)
 listpath = d.getVar("DT_FILES_PATH")
 for dts in os.listdir(listpath):
-if not dts.endswith(".dts"):
-continue # skip non-.dts files
 dtspath = os.path.join(listpath, dts)
+try:
+if not(os.path.isfile(dtspath)) or not(dts.endswith(".dts") or 
devicetree_source_is_overlay(dtspath)):
+continue # skip non-.dts files and non-overlay files
+except:
+continue # skip if can't determine if overlay
 devicetree_compile(dtspath, includes, d)  }
 
--
2.7.5

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v2] device-tree.bbclass: Add support to compile overlays separately

2019-01-28 Thread Jaewon Lee
ping

-Original Message-
From: Jaewon Lee [mailto:jaewon@xilinx.com] 
Sent: Friday, December 14, 2018 9:54 AM
To: openembedded-core@lists.openembedded.org; nat...@nathanrossi.com
Cc: Jaewon Lee ; Alejandro Enedino Hernandez Samaniego 
; Manjukumar Harthikote Matha 
Subject: [OE-core][PATCH v2] device-tree.bbclass: Add support to compile 
overlays separately

Currently only dts files are considered when looping through files to compile. 
Modifying the loop to compile other files that are overlays.
Also surrounding this check with a try block as the function to find overlays 
parses the file for a '/plugin/' tag, and there may be files in the 
DT_FILES_PATH directory that are not parseable.

Signed-off-by: Jaewon Lee 
Signed-off-by: Alejandro Enedino Hernandez Samaniego 
Signed-off-by: Manjukumar Matha 
---
 meta/classes/devicetree.bbclass | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/meta/classes/devicetree.bbclass b/meta/classes/devicetree.bbclass 
index 8fe5a5e..d785285 100644
--- a/meta/classes/devicetree.bbclass
+++ b/meta/classes/devicetree.bbclass
@@ -120,9 +120,12 @@ python devicetree_do_compile() {
 includes = expand_includes("DT_INCLUDE", d)
 listpath = d.getVar("DT_FILES_PATH")
 for dts in os.listdir(listpath):
-if not dts.endswith(".dts"):
-continue # skip non-.dts files
 dtspath = os.path.join(listpath, dts)
+try:
+if not(os.path.isfile(dtspath)) or not(dts.endswith(".dts") or 
devicetree_source_is_overlay(dtspath)):
+continue # skip non-.dts files and non-overlay files
+except:
+continue # skip if can't determine if overlay
 devicetree_compile(dtspath, includes, d)  }
 
--
2.7.5

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2] device-tree.bbclass: Add support to compile overlays separately

2018-12-15 Thread Jaewon Lee
Currently only dts files are considered when looping through files to
compile. Modifying the loop to compile other files that are overlays.
Also surrounding this check with a try block as the function to find
overlays parses the file for a '/plugin/' tag, and there may be files in
the DT_FILES_PATH directory that are not parseable.

Signed-off-by: Jaewon Lee 
Signed-off-by: Alejandro Enedino Hernandez Samaniego 
Signed-off-by: Manjukumar Matha 
---
 meta/classes/devicetree.bbclass | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/meta/classes/devicetree.bbclass b/meta/classes/devicetree.bbclass
index 8fe5a5e..d785285 100644
--- a/meta/classes/devicetree.bbclass
+++ b/meta/classes/devicetree.bbclass
@@ -120,9 +120,12 @@ python devicetree_do_compile() {
 includes = expand_includes("DT_INCLUDE", d)
 listpath = d.getVar("DT_FILES_PATH")
 for dts in os.listdir(listpath):
-if not dts.endswith(".dts"):
-continue # skip non-.dts files
 dtspath = os.path.join(listpath, dts)
+try:
+if not(os.path.isfile(dtspath)) or not(dts.endswith(".dts") or 
devicetree_source_is_overlay(dtspath)):
+continue # skip non-.dts files and non-overlay files
+except:
+continue # skip if can't determine if overlay
 devicetree_compile(dtspath, includes, d)
 }
 
-- 
2.7.5

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core