Re: [PATCH] net/fec: fix compile error introduced by dt support

[Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>]

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.  :-)

g.

> 
> -- 
> Regards,
> Shawn
> 
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss