From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Subject: [PATCH v5 0/5] Add fdtget and fdtput for access to fdt from build system Date: Thu, 22 Sep 2011 10:11:01 -0700 Message-ID: <1316711466-14199-1-git-send-email-sjg@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Devicetree Discuss List-Id: devicetree@vger.kernel.org This patch set adds two new utilities: fdtget for reading properties from a device tree binary file fdtput for updating the device tree binary file These are useful in scripts where configuration or other information must be passed to the running system from the build system or vice versa. For example, fdtget can be used to obtain the LCD screen width/height for use with preparing custom bitmap images for display on the screen. Going the other way, fdtput can be used to set configuration parameters known only to the build or manufacturing test system, for example a hardware ID or the particular type of boot EEPROM used on this PCB. It is desirable to use these utilities rather than parsing or updating the device tree source file, since this parsing is already implemented in dtc and it makes no sense to reimplement it in a script. It also means that the build system and the run-time environment see the same device tree values, so that parsing or updating bugs do not creap in. I believe that these utilities belong with dtc rather than libfdt since they are more useful in the build system context rather than at run-time. At run-time it is easier and better to use the provided library rather than these utiltiies. But for build scripts and production automation, these uitilites are better. Some effort has been made to add tests and the parameter format has been revised after previous feedback. Changes in v2: - Separate arguments for node and property - Remove limits on data size of property writting to fdt - Remove use of getopt_long() - Adjust tests for new fdtput arguments - Add test for multiple strings - Add test for exhausting fdt space - Merge two related commits into this one - Use structure for storing display options - Adjust tests for new fdtget arguments - Remove util_decode_key - Add utilfdt_decode_type to be used by fdtget/put - Remove limits on device tree binary size Changes in v3: - Squash fdtput tests into fdtput commit - Squash fdtget test commit into fdtget - Add a few more tests for 1- and 2-byte formats - Change format of -t argument to be more like printf - Change help string and make part of it common - Add error checking for the property not being a multiple of value size - Make testutils.c use utilfdt for load/save blobs - Move utilfdt into its own directory and make it a library - Use code closer to testutils.c implementation - Use open/close instead of fopen/fclose - Add tests for utilfdt functions Changes in v4: - Rebase to ToT - Squash utilfdt functions into util.c - Squash utilfdt tests into main commit (this is getting very squashed) - Remove xmalloc() etc. from tests.h since they are in util.h Changes in v5: - Tidy up commit message - Rebase to ToT Simon Glass (5): Add fdt read/write utility functions Make testutils use utilfdt ftdump: use utilfdt to read blob Add fdtget utility to read property values from a device tree Add fdtput utility to write property values to a device tree .gitignore | 2 + Makefile | 10 ++- Makefile.utils | 14 +++ fdtget.c | 226 +++++++++++++++++++++++++++++++++++++++++++++ fdtput.c | 235 +++++++++++++++++++++++++++++++++++++++++++++++ ftdump.c | 32 +------ tests/Makefile.tests | 10 +- tests/fdtget-runtest.sh | 35 +++++++ tests/fdtput-runtest.sh | 55 +++++++++++ tests/run_tests.sh | 121 ++++++++++++++++++++++++- tests/tests.h | 18 +--- tests/tests.sh | 2 + tests/testutils.c | 59 ++---------- tests/utilfdt_test.c | 128 +++++++++++++++++++++++++ util.c | 142 ++++++++++++++++++++++++++++ util.h | 78 ++++++++++++++++ 16 files changed, 1069 insertions(+), 98 deletions(-) create mode 100644 fdtget.c create mode 100644 fdtput.c create mode 100755 tests/fdtget-runtest.sh create mode 100644 tests/fdtput-runtest.sh create mode 100644 tests/utilfdt_test.c -- 1.7.3.1