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