[PATCH v3 3/4] pylibfdt: Return string instead of bytearray from getprop()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



The return value is not actually mutable, so it seems more correct to
return a string rather than a bytearray.

Suggested-by: David Gibson <david@xxxxxxxxxxxxxxxxxxxxx>
Signed-off-by: Simon Glass <sjg@xxxxxxxxxxxx>
---

Changes in v3: None
Changes in v2: None

 pylibfdt/libfdt.i | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pylibfdt/libfdt.i b/pylibfdt/libfdt.i
index 2c104ac..7e915cf 100644
--- a/pylibfdt/libfdt.i
+++ b/pylibfdt/libfdt.i
@@ -365,7 +365,7 @@ class Fdt:
             quiet: Errors to ignore (empty to raise on all errors)
 
         Returns:
-            Value of property as a bytearray, or -ve error number
+            Value of property as a string, or -ve error number
 
         Raises:
             FdtError if any error occurs (e.g. the property is not found)
@@ -374,7 +374,7 @@ class Fdt:
                                quiet)
         if isinstance(pdata, (int)):
             return pdata
-        return bytearray(pdata[0])
+        return str(pdata[0])
 
     def get_phandle(self, nodeoffset):
         """Get the phandle of a node
-- 
2.17.0.441.gb46fe60e1d-goog

--
To unsubscribe from this list: send the line "unsubscribe devicetree-compiler" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux