Skip to content
Snippets Groups Projects
  1. Aug 25, 2008
    • Gerald Van Baren's avatar
      Add note on dereferencing /aliases pointers · 109c30fb
      Gerald Van Baren authored
      
      Replace the "must quote special characters" note with a hint on
      how to dereference /aliases pointers by omitting the leading "/".
      This feature was introduced by Kumar Gala as a libfdt enhancement
      in commit ed035708235332c3c117ee3bb1a426063f03cfcb.
      
      Example:
      
      => fdt print /aliases
      aliases {
              ethernet0 = "/qe@e0100000/ucc@2000";
              ethernet1 = "/qe@e0100000/ucc@3000";
              serial0 = "/soc8360@e0000000/serial@4500";
              serial1 = "/soc8360@e0000000/serial@4600";
              pci0 = "/pci@e0008500";
      };
      => fdt print ethernet0
      ucc@2000 {
              device_type = "network";
              compatible = "ucc_geth";
              cell-index = <0x1>;
              reg = <0x2000 0x200>;
              interrupts = <0x20>;
              interrupt-parent = <0x2>;
              local-mac-address = [00 00 00 00 00 00];
              rx-clock-name = "none";
              tx-clock-name = "clk9";
              phy-handle = <0x3>;
              phy-connection-type = "rgmii-id";
              pio-handle = <0x4>;
      };
      
      Signed-off-by: default avatarGerald Van Baren <vanbaren@cideas.com>
      109c30fb
    • Kumar Gala's avatar
      libfdt: Add support for using aliases in fdt_path_offset() · feeca3f5
      Kumar Gala authored
      
      If the path doesn't start with '/' check to see if it matches some alias
      under "/aliases" and substitute the matching alias value in the path
      and retry the lookup.
      
      Signed-off-by: default avatarKumar Gala <galak@kernel.crashing.org>
      Acked-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
      Acked-by: default avatarGerald Van Baren <vanbaren@cideas.com>
      feeca3f5
    • David Gibson's avatar
      libfdt: Implement fdt_get_property_namelen() and fdt_getprop_namelen() · 0219399a
      David Gibson authored
      
      As well as fdt_subnode_offset(), libfdt includes an
      fdt_subnode_offset_namelen() function that takes the subnode name to
      look up not as a NUL-terminated string, but as a string with an
      explicit length.  This can be useful when the caller has the name as
      part of a longer string, such as a full path.
      
      However, we don't have corresponding 'namelen' versions for
      fdt_get_property() and fdt_getprop().  There are less obvious use
      cases for these variants on property names, but there are
      circumstances where they can be useful e.g. looking up property names
      which need to be parsed from a longer string buffer such as user input
      or a configuration file, or looking up an alias in a path with
      IEEE1275 style aliases.
      
      So, since it's very easy to implement such variants, this patch does
      so.  The original NUL-terminated variants are, of course, implemented
      in terms of the namelen versions.
      
      Signed-off-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
      0219399a
    • David Gibson's avatar
      libfdt: Forgot one function when cleaning the namespace · f171746f
      David Gibson authored
      
      In commit b6d80a20fc293f3b995c3ce1a6744a5574192125, we renamed all
      libfdt functions to be prefixed with fdt_ or _fdt_ to minimise the
      chance of collisions with things from whatever package libfdt is
      embedded in, pulled into the libfdt build via that environment's
      libfdt_env.h.
      
      Except... I missed one.  This patch applies the same treatment to
      _stringlist_contains().  While we're at it, also make it static since
      it's only used in the same file.
      
      Signed-off-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
      f171746f
    • Wolfram Sang's avatar
      libfdt: Improve documentation in libfdt.h · 46390da1
      Wolfram Sang authored
      
      Fix a few typos and mistakes.
      
      Signed-off-by: default avatarWolfram Sang <w.sang@pengutronix.de>
      Acked-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
      46390da1
    • David Gibson's avatar
      libfdt: Increase namespace-pollution paranoia · fc7758ee
      David Gibson authored
      
      libfdt is supposed to easy to embed in projects all and sundry.
      Often, it won't be practical to separate the embedded libfdt's
      namespace from that of the surrounding project.  Which means there can
      be namespace conflicts between even libfdt's internal/static functions
      and functions or macros coming from the surrounding project's headers
      via libfdt_env.h.
      
      This patch, therefore, renames a bunch of libfdt internal functions
      and macros and makes a few other chances to reduce the chances of
      namespace collisions with embedding projects.  Specifically:
      	- Internal functions (even static ones) are now named _fdt_*()
      
      	- The type and (static) global for the error table in
                fdt_strerror() gain an fdt_ prefix
      
      	- The unused macro PALIGN is removed
      
      	- The memeq and streq macros are removed and open-coded in the
                users (they were only used once each)
      
      	- Other macros gain an FDT_ prefix
      
      	- To save some of the bulk from the previous change, an
                FDT_TAGALIGN() macro is introduced, where FDT_TAGALIGN(x) ==
                FDT_ALIGN(x, FDT_TAGSIZE)
      
      Signed-off-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
      fc7758ee
    • David Gibson's avatar
      dtc: Enable and fix -Wcast-qual warnings · c6683026
      David Gibson authored
      
      Enabling -Wcast-qual warnings in dtc shows up a number of places where
      we are incorrectly discarding a const qualification.  There are also
      some places where we are intentionally discarding the 'const', and we
      need an ugly cast through uintptr_t to suppress the warning.  However,
      most of these are pretty well isolated with the *_w() functions.  So
      in the interests of maximum safety with const qualifications, this
      patch enables the warnings and fixes the existing complaints.
      
      Signed-off-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
      Acked-by: default avatarGerald Van Baren <vanbaren@cideas.com>
      c6683026
    • David Gibson's avatar
      dtc: Enable and fix -Wpointer-arith warnings · ef4e8ce1
      David Gibson authored
      
      This patch turns on the -Wpointer-arith option in the dtc Makefile,
      and fixes the resulting warnings due to using (void *) in pointer
      arithmetic.  While convenient, pointer arithmetic on void * is not
      portable, so it's better that we avoid it, particularly in libfdt.
      
      Also add necessary definition of uintptr_t needed by David Gibson's
      changeset "dtc: Enable and fix -Wpointer-arith warnings" (the definition
      comes from stdint.h, which u-boot doesn't have). -- gvb
      
      Signed-off-by: default avatarDavid Gibson <david@gibson.dropbear.id.au>
      Signed-off-by: default avatarGerald Van Baren <vanbaren@cideas.com>
      ef4e8ce1
  2. Aug 22, 2008
  3. Aug 21, 2008
  4. Aug 20, 2008
Loading