On 12/19/2012 05:02 AM, Grant Likely wrote: > On Wed, Dec 19, 2012 at 10:58 AM, Grant Likely > <grant.lik...@secretlab.ca> wrote: >> Commit 50c8af4cf9, "of: introduce for_each_matching_node_and_match()" >> renamed of_find_matching_node() to of_find_matching_node_and_match() and >> created a new static inline of_find_matching_node() wrapper around the >> new name. However, the change neglected to change the EXPORT_SYMBOL() >> reference causing build errors for modules. >> >> This patch fixes the EXPORT_SYMBOL() statement. Discovered on a PowerPC >> Efika build with the mpc52xx_uart driver being built as a module. >> >> Reported-by: Benjamin Herrenschmidt <b...@kernel.crashing.org> >> Signed-off-by: Grant Likely <grant.lik...@secretlab.ca> >> Cc: Stephen Warren <swar...@nvidia.com> >> Cc: Rob Herring <rob.herr...@calxeda.com> >> Cc: Anatolij Gustschin <ag...@denx.de> > > Rob, I've just pushed this out to my devicetree/merge branch. If > you've got any fixes queued up for Linus, then please pull this in > before sending them on to him. Otherwise I'll send Linus a pull req > for this fix this evening. Ether way, please reply to let me know what > you're going to do. >
The only item on my todo is this one: "of: define struct device in of_platform.h if !OF_DEVICE and !OF_ADDRESS" But I'm not going to get to it today if you want to pick it up. Rob > g. > > The following changes since commit 752451f01c4567b506bf4343082682dbb8fb30dd: > > Merge branch 'i2c-embedded/for-next' of > git://git.pengutronix.de/git/wsa/linux (2012-12-18 16:51:10 -0800) > > are available in the git repository at: > > git://git.secretlab.ca/git/linux-2.6 devicetree/merge > > for you to fetch changes up to 80c2022e5645a1a789531d13010292c5c18bf1db: > > of: Fix export of of_find_matching_node_and_match() (2012-12-19 > 10:58:53 +0000) > > ---------------------------------------------------------------- > Grant Likely (1): > of: Fix export of of_find_matching_node_and_match() > > drivers/of/base.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >> --- >> >> I'll push this patch out to my tree ASAP. >> >> drivers/of/base.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/of/base.c b/drivers/of/base.c >> index db8d211..2390ddb 100644 >> --- a/drivers/of/base.c >> +++ b/drivers/of/base.c >> @@ -629,7 +629,7 @@ struct device_node >> *of_find_matching_node_and_match(struct device_node *from, >> read_unlock(&devtree_lock); >> return np; >> } >> -EXPORT_SYMBOL(of_find_matching_node); >> +EXPORT_SYMBOL(of_find_matching_node_and_match); >> >> /** >> * of_modalias_node - Lookup appropriate modalias for a device node >> -- >> 1.7.10.4 >> > > > > -- > Grant Likely, B.Sc., P.Eng. > Secret Lab Technologies Ltd. > _______________________________________________ > devicetree-discuss mailing list > devicetree-disc...@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/devicetree-discuss > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
