On 08/04/2011 09:54 PM, Marc Kleine-Budde wrote: > On 08/04/2011 09:15 PM, Robin Holt wrote: >> It looks like the of_device stuff got moved under the >> platform_device->driver and all we should need to do is define an >> of_match to get a .probe call out. Give this a try. > > have you tested the driver on PPC? what about the clock stuff, does it > work properly? > >> >> Signed-off-by: Robin Holt <[email protected]> >> To: Marc Kleine-Budde <[email protected]> >> To: Wolfgang Grandegger <[email protected]> >> Cc: [email protected] >> --- >> drivers/net/can/flexcan.c | 12 +++++++++++- >> 1 files changed, 11 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/net/can/flexcan.c b/drivers/net/can/flexcan.c >> index 74b1706..75e4d9c 100644 >> --- a/drivers/net/can/flexcan.c >> +++ b/drivers/net/can/flexcan.c >> @@ -1033,8 +1033,18 @@ static int __devexit flexcan_remove(struct >> platform_device *pdev) >> return 0; >> } >> >> +static struct of_device_id flexcan_of_match[] = { >> + { >> + .compatible = "fsl,flexcan-v1.0", >> + }, >> +}; > > I think you have to NULL-terminate that list.
And also please remove "-v1.0". We could add "fsl,flexcan-vx.x" later when it is really needed. Wolfgang. _______________________________________________ Socketcan-users mailing list [email protected] https://lists.berlios.de/mailman/listinfo/socketcan-users
