Messages in this thread | | | From | Grant Likely <> | Date | Thu, 23 Jun 2011 22:05:00 -0600 | Subject | Re: [PATCH 1/3] serial/imx: add device tree support |
| |
On Thu, Jun 23, 2011 at 9:49 PM, Shawn Guo <shawn.guo@freescale.com> wrote: > On Thu, Jun 23, 2011 at 05:11:54PM -0600, Grant Likely wrote: >> > + int devid = get_alias_name_id(pp->name, devname); >> > + >> > + /* We do not want to proceed this sentinel one */ >> > + if (!strcmp(pp->name, "name") && !strcmp(pp->value, "aliases")) >> > + break; >> >> Skipping the 'name' property is good, but I don't think you need to >> check the value. You should also skip the "phandle" property. >> > Ok. I have not seen "phandle" property in aliases node though. Can > you give me an example, so that I can make one up for testing?
It's a standard property that shows up automatically if any node happens to have a phandle reference to another node. The name will be either "phandle" or "linux,phandle". Search for "linux,phandle" in drivers/of/fdt.c to see how that property is processed.
g. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |