Re: [PATCH 07/10] pylibfdt: Add a method to access the device tree directly

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



Hi David,

On 20 August 2017 at 18:17, David Gibson <david@xxxxxxxxxxxxxxxxxxxxx> wrote:
> On Sat, Aug 19, 2017 at 11:18:00AM -0600, Simon Glass wrote:
>> When calling libfdt functions which are not supported by the Fdt class it
>> is necessary to get direct access to the device tree data. At present this
>> requries using the internal _fdt member. Add a new method to provide
>> public access to this.
>>
>> Signed-off-by: Simon Glass <sjg@xxxxxxxxxxxx>
>
> I've applied patches 1-6 already.
>
> This looks fine, except I'm dubious about the name.  "get_fdt" from an
> fdt object seems a bit.. tautological?  How about "as_bytestring"?

Well, sure it is.

It is the only way to get something that can be passed to the libfdt
function. So as_bytestring() seems a bit obscure, even though it is
more descriptive. I would suggest get_blob() except we don't really
use that name in libfdt.h.

I will have a think and then send v2.

Regards,
Simon
--
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