Re: [PATCH 1/1] of: Move OF property and graph API from base.c to property.c

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

 




Hi Frank,

Thank you for the review.

Frank Rowand wrote:
Hi Sakari,

On 03/24/17 03:59, Sakari Ailus wrote:
base.c contains both core OF functions and increasingly other
functionality such as accessing properties and graphs, including
convenience functions. In the near future this would also include OF
specific implementation of the fwnode property and graph APIs. Prepare for
this by moving the functionality to property.c.

For future me, for when I look back and read this commit, please add something
to the effect of:

  Create driver/of/property.c to contain procedures for accessing and
  interpreting device tree properties.  The procedures are moved from
  drivers/of/base.c, with no changes other than copying only the
  includes required by the moved procedures.

How about this as the commit message combining the two and dropping the last sentence from the original description:

base.c contains both core OF functions and increasingly other
functionality such as accessing properties and graphs, including
convenience functions. In the near future this would also include OF
specific implementation of the fwnode property and graph APIs.

Create driver/of/property.c to contain procedures for accessing and
interpreting device tree properties. The procedures are moved from
drivers/of/base.c, with no changes other than copying only the
includes required by the moved procedures.

...

diff --git a/drivers/of/property.c b/drivers/of/property.c
new file mode 100644
index 0000000..afcf00a
--- /dev/null
+++ b/drivers/of/property.c
@@ -0,0 +1,727 @@
+/*
+ * drivers/of/property.c - Devicetree properties and graphs.

    * drivers/of/property.c - Procedures for accessing and interpreting
    * Devicetree properties and graphs.
    *
    * Initially created by copying procedures from drivers/of/base.c

The reference to base.c provides a documentation trail to all of the
earlier git commits that created and modified the procedures.

I'll add that.


It would also be useful to say what procedures do and do not belong
in this file.  My understanding is that the intent is for
drivers/of/property.c and drivers/acpi/property.c to provide the
respective functionality that is used by drivers/base/property.c
and that drivers also may directly access the procedures in
drivers/of/property.c.

I'll add something along those lines. The fwnode op struct and the operations in that will be in this file as well.

--
Kind regards,

Sakari Ailus
sakari.ailus@xxxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe devicetree" 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 Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux