* [PATCH net-next-2.6] can: mscan-mpc5xxx: fix broken support for the MPC5200
@ 2010-01-14 11:05 Wolfgang Grandegger
[not found] ` <4B4EFA8C.4010209-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>
0 siblings, 1 reply; 3+ messages in thread
From: Wolfgang Grandegger @ 2010-01-14 11:05 UTC (permalink / raw)
To: Netdev-u79uwXL29TY76Z2rM5mHXA; +Cc: SocketCAN Core Mailing List
From: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org>
Due to an invalid "#ifdef CONFIG_PPC_MPC5200", the real clock setup
function was not called for the MPC5200.
Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org>
---
drivers/net/can/mscan/mpc5xxx_can.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/can/mscan/mpc5xxx_can.c b/drivers/net/can/mscan/mpc5xxx_can.c
index f73487f..03e7c48 100644
--- a/drivers/net/can/mscan/mpc5xxx_can.c
+++ b/drivers/net/can/mscan/mpc5xxx_can.c
@@ -43,7 +43,7 @@ struct mpc5xxx_can_data {
int *mscan_clksrc);
};
-#ifdef CONFIG_PPC_MPC5200
+#ifdef CONFIG_PPC_MPC52xx
static struct of_device_id __devinitdata mpc52xx_cdm_ids[] = {
{ .compatible = "fsl,mpc5200-cdm", },
{}
@@ -84,7 +84,7 @@ static u32 __devinit mpc52xx_can_get_clock(struct of_device *ofdev,
/* Determine SYS_XTAL_IN frequency from the clock domain settings */
np_cdm = of_find_matching_node(NULL, mpc52xx_cdm_ids);
if (!np_cdm) {
- dev_err(&of->dev, "can't get clock node!\n");
+ dev_err(&ofdev->dev, "can't get clock node!\n");
return 0;
}
cdm = of_iomap(np_cdm, 0);
@@ -101,14 +101,14 @@ static u32 __devinit mpc52xx_can_get_clock(struct of_device *ofdev,
return freq;
}
-#else /* !CONFIG_PPC_MPC5200 */
+#else /* !CONFIG_PPC_MPC52xx */
static u32 __devinit mpc52xx_can_get_clock(struct of_device *ofdev,
const char *clock_name,
int *mscan_clksrc)
{
return 0;
}
-#endif /* CONFIG_PPC_MPC5200 */
+#endif /* CONFIG_PPC_MPC52xx */
#ifdef CONFIG_PPC_MPC512x
struct mpc512x_clockctl {
--
1.6.2.5
^ permalink raw reply related [flat|nested] 3+ messages in thread[parent not found: <4B4EFA8C.4010209-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>]
* Re: [PATCH net-next-2.6] can: mscan-mpc5xxx: fix broken support for the MPC5200 [not found] ` <4B4EFA8C.4010209-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org> @ 2010-01-14 11:15 ` Wolfram Sang [not found] ` <20100114111524.GA6041-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> 0 siblings, 1 reply; 3+ messages in thread From: Wolfram Sang @ 2010-01-14 11:15 UTC (permalink / raw) To: Wolfgang Grandegger Cc: SocketCAN Core Mailing List, Netdev-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1.1: Type: text/plain, Size: 562 bytes --] On Thu, Jan 14, 2010 at 12:05:48PM +0100, Wolfgang Grandegger wrote: > From: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> > > Due to an invalid "#ifdef CONFIG_PPC_MPC5200", the real clock setup > function was not called for the MPC5200. > > Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> Acked-by: Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [-- Attachment #1.2: Digital signature --] [-- Type: application/pgp-signature, Size: 197 bytes --] [-- Attachment #2: Type: text/plain, Size: 188 bytes --] _______________________________________________ Socketcan-core mailing list Socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org https://lists.berlios.de/mailman/listinfo/socketcan-core ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <20100114111524.GA6041-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>]
* Re: [PATCH net-next-2.6] can: mscan-mpc5xxx: fix broken support for the MPC5200 [not found] ` <20100114111524.GA6041-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> @ 2010-01-15 9:44 ` David Miller 0 siblings, 0 replies; 3+ messages in thread From: David Miller @ 2010-01-15 9:44 UTC (permalink / raw) To: w.sang-bIcnvbaLZ9MEGnE8C9+IrQ Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w, Netdev-u79uwXL29TY76Z2rM5mHXA, wg-5Yr1BZd7O62+XT7JhA+gdA From: Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Date: Thu, 14 Jan 2010 12:15:24 +0100 > On Thu, Jan 14, 2010 at 12:05:48PM +0100, Wolfgang Grandegger wrote: >> From: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> >> >> Due to an invalid "#ifdef CONFIG_PPC_MPC5200", the real clock setup >> function was not called for the MPC5200. >> >> Signed-off-by: Wolfgang Grandegger <wg-ynQEQJNshbs@public.gmane.org> > > Acked-by: Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Applied, thanks. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-01-15 9:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-14 11:05 [PATCH net-next-2.6] can: mscan-mpc5xxx: fix broken support for the MPC5200 Wolfgang Grandegger
[not found] ` <4B4EFA8C.4010209-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>
2010-01-14 11:15 ` Wolfram Sang
[not found] ` <20100114111524.GA6041-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2010-01-15 9:44 ` David Miller
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.