From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerry Van Baren Date: Sat, 15 Oct 2011 09:04:06 -0400 Subject: [U-Boot] [PATCH 1/2] fdt: Add new fdt_set_node_status & fdt_set_status_by_alias helpers In-Reply-To: <1318619444-2059-1-git-send-email-galak@kernel.crashing.org> References: <1318619444-2059-1-git-send-email-galak@kernel.crashing.org> Message-ID: <4E9984C6.5020004@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Kumar, Shengzhou, On 10/14/2011 03:10 PM, Kumar Gala wrote: > From: Shengzhou Liu > > Add common function fdt_set_node_status() to assist in various locations > that we set a nodes status. This function utilizes the status values > that are part of the EPAPR spec (on power.org). > > fdt_set_status_by_alias() is based on fdt_set_node_status() but uses an > alias string to identify the node to update. > > We also add some shortcut functions to help the common cases of setting > "okay" and "disabled": > > fdt_status_okay() > fdt_status_disabled() > fdt_status_okay_by_alias() > fdt_status_disabled_by_alias() > > Finally, we fixup the corenet_ds ethernet code which previously had > a function by the same name that can be replaced with the new helpers. > > Signed-off-by: Shengzhou Liu > Signed-off-by: Kumar Gala > --- > board/freescale/corenet_ds/eth_hydra.c | 26 ++------------ > board/freescale/corenet_ds/eth_p4080.c | 36 +++++-------------- > common/fdt_support.c | 60 +++++++++++++++++++++++++++++++- > include/fdt_support.h | 28 +++++++++++++++ > 4 files changed, 101 insertions(+), 49 deletions(-) While this touches fdt_support.[ch] which is nominally "mine", it is coupled with the Freescale support. I could make you break it into two, but then I would have to do more work and you (and wd) would have to make sure they went into u-boot in the right order. That sounds ugly, so I would prefer Kumar apply it to his subrepo to feed into u-boot. Acked-by: Gerald Van Baren Best regards, gvb [snip]