On Mon, Apr 04, 2011 at 10:54:00PM -0600, Grant Likely wrote:

On Fri, Apr 01, 2011 at 03:49:16PM +0800, Shawn Guo wrote:

...

On Thu, Mar 31, 2011 at 10:09:40AM -0600, Grant Likely wrote:

...

On Fri, Mar 25, 2011 at 03:13:58PM +0800, Shawn Guo wrote:

...

After fec dt support is added, the following compile error will be seen when building a pure non-dt kernel.

drivers/net/fec.c: In function ‘fec_probe’: drivers/net/fec.c:1383: error: implicit declaration of function ‘of_match_device’ drivers/net/fec.c:1383: warning: assignment makes pointer from integer without a cast

Earlier today I suggested fixing this by adding an empty implementation of of_match_device, but I forgot that an .of_match pointer has been added to struct device for exactly this purpose. You can use that instead.

That change is currently in mainline, but it has not been backported to the Linaro 2.6.38 tree (yet).

This simply is a fix to commit 54898b86fa9813313b3eb981c44610ff483b0067 "net/fec: add device tree matching support", which only sits on branch devicetree/test-2.6.38 right now. However, .of_match is not available on that tree yet. So I can not do anything until ether this patch shows on devicetree/test or .of_match is back ported on devicetree/test-2.6.38. Or you can simply ignore this patch since I just want let you know such a compile error.

Okay, of_match is in my devicetree/test branch now since I've rebased to 2.6.38-rc1. I'll happily take a fixup patch from you now. :-)

Sorry, Grant. I overlooked this reply, and noticed it when re-visiting the thread today. I just sent an updated patch. Please take a look.