[RFC PATCH 10/13] of: Update comments to reflect changes and increase clarity

2016-10-25 Thread frowand . list
From: Frank Rowand Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 51 --- 1 file changed, 32 insertions(+), 19 deletions(-) diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index 76c09cb57eae..4e6df385118b 100644 ---

[RFC PATCH 07/13] of: Rename variables to better reflect purpose or follow convention

2016-10-25 Thread frowand . list
From: Frank Rowand Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 172 +- 1 file changed, 85 insertions(+), 87 deletions(-) diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index 0ce38aa0ed3c..0778747cdd58 100644 ---

[RFC PATCH 08/13] of: Update structure of code, remove BUG_ON()

2016-10-25 Thread frowand . list
From: Frank Rowand Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 48 +--- 1 file changed, 17 insertions(+), 31 deletions(-) diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index 0778747cdd58..708daca1d522 100644 ---

[RFC PATCH 06/13] of: Remove prefix "__of_" and prefix "__" from local function names

2016-10-25 Thread frowand . list
From: Frank Rowand Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index 3d123b612789..0ce38aa0ed3c 100644 ---

[RFC PATCH 06/13] of: Remove prefix "__of_" and prefix "__" from local function names

2016-10-25 Thread frowand . list
From: Frank Rowand Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index 3d123b612789..0ce38aa0ed3c 100644 --- a/drivers/of/resolver.c +++ b/drivers/of/resolver.c

[RFC PATCH 04/13] of: Convert comparisons to zero or NULL to simplify logical expressions

2016-10-25 Thread frowand . list
From: Frank Rowand A small number of such comparisons remain where they provide more clarity of the numeric nature of a variable. Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 42 -- 1 file

[RFC PATCH 12/13] of: Move setting of pointer to beside test for non-null

2016-10-25 Thread frowand . list
From: Frank Rowand Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index 664c97e1ecb4..3f7cf569c7ea 100644 ---

[RFC PATCH 13/13] of: Remove unused variable overlay_symbols

2016-10-25 Thread frowand . list
From: Frank Rowand This unused variable is a reminder that symbols in overlays are not available to subsequent overlays. If such a feature is desired then there are several ways it could be implemented. Signed-off-by: Frank Rowand ---

[RFC PATCH 12/13] of: Move setting of pointer to beside test for non-null

2016-10-25 Thread frowand . list
From: Frank Rowand Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index 664c97e1ecb4..3f7cf569c7ea 100644 --- a/drivers/of/resolver.c +++ b/drivers/of/resolver.c @@ -305,8

[RFC PATCH 13/13] of: Remove unused variable overlay_symbols

2016-10-25 Thread frowand . list
From: Frank Rowand This unused variable is a reminder that symbols in overlays are not available to subsequent overlays. If such a feature is desired then there are several ways it could be implemented. Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 5 + 1 file changed, 1

[RFC PATCH 04/13] of: Convert comparisons to zero or NULL to simplify logical expressions

2016-10-25 Thread frowand . list
From: Frank Rowand A small number of such comparisons remain where they provide more clarity of the numeric nature of a variable. Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 42 -- 1 file changed, 20 insertions(+), 22 deletions(-) diff

[PATCH] of: Add missing space at end of pr_fmt().

2016-10-17 Thread frowand . list
From: Frank Rowand Make pr_fmt() in fdt.c consistent with all other files in drivers/of/ Signed-off-by: Frank Rowand --- drivers/of/fdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c

[PATCH] of: Add missing space at end of pr_fmt().

2016-10-17 Thread frowand . list
From: Frank Rowand Make pr_fmt() in fdt.c consistent with all other files in drivers/of/ Signed-off-by: Frank Rowand --- drivers/of/fdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c index c89d5d231a0e..9f98e437e1f1 100644 ---

[PATCH] of: make of_device_make_bus_id() static

2016-10-17 Thread frowand . list
From: Frank Rowand of_device_make_bus_id() was changed to non-static by commit c66012253800 ("of/device: Make of_device_make_bus_id() usable by other code") more than 6 years ago, but there are no users of it outside of platform.c. Make the function static again.

[PATCH] of: make of_device_make_bus_id() static

2016-10-17 Thread frowand . list
From: Frank Rowand of_device_make_bus_id() was changed to non-static by commit c66012253800 ("of/device: Make of_device_make_bus_id() usable by other code") more than 6 years ago, but there are no users of it outside of platform.c. Make the function static again. Signed-off-by: Frank Rowand

[PATCH] make of_device_make_bus_id() static

2016-10-17 Thread frowand . list
From: Frank Rowand of_device_make_bus_id() was changed to non-static by commit c66012253800 ("of/device: Make of_device_make_bus_id() usable by other code") more than 6 years ago, but there are no users of it outside of platform.c. Make the function static again.

[PATCH] make of_device_make_bus_id() static

2016-10-17 Thread frowand . list
From: Frank Rowand of_device_make_bus_id() was changed to non-static by commit c66012253800 ("of/device: Make of_device_make_bus_id() usable by other code") more than 6 years ago, but there are no users of it outside of platform.c. Make the function static again. Signed-off-by: Frank Rowand

[PATCH] Add missing space at end of pr_fmt().

2016-10-17 Thread frowand . list
From: Frank Rowand Make pr_fmt() in fdt.c consistent with all other files in drivers/of/ Signed-off-by: Frank Rowand --- drivers/of/fdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c

[PATCH] Add missing space at end of pr_fmt().

2016-10-17 Thread frowand . list
From: Frank Rowand Make pr_fmt() in fdt.c consistent with all other files in drivers/of/ Signed-off-by: Frank Rowand --- drivers/of/fdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c index c89d5d231a0e..9f98e437e1f1 100644 ---

[RFC PATCH 1/1] device tree connectors, using plugs and sockets.

2016-07-02 Thread frowand . list
From: Frank Rowand This patch has been compiled but has not been booted. It is likely to contain bugs. Problem: mother boards may contain multiple connectors that daughter boards may be attached to. If two of the daughter boards can be described by the same .dtsi

[RFC PATCH 1/1] device tree connectors, using plugs and sockets.

2016-07-02 Thread frowand . list
From: Frank Rowand This patch has been compiled but has not been booted. It is likely to contain bugs. Problem: mother boards may contain multiple connectors that daughter boards may be attached to. If two of the daughter boards can be described by the same .dtsi file, then it should be

[RFC PATCH 0/1] Portable Device Tree Connector -- conceptual

2016-07-02 Thread frowand . list
From: Frank Rowand Hi All, This is version 2 of this email. Changes from version 1: - some rewording of the text - removed new (theoretical) dtc directive "/connector/" - added compatibility between mother board and daughter board - added info on applying a

[RFC PATCH 0/1] Portable Device Tree Connector -- conceptual

2016-07-02 Thread frowand . list
From: Frank Rowand Hi All, This is version 2 of this email. Changes from version 1: - some rewording of the text - removed new (theoretical) dtc directive "/connector/" - added compatibility between mother board and daughter board - added info on applying a single .dtbo to different

<    1   2   3   4   5   6