MPC5121 CAN and USB
Kári Davíðsson
kari.davidsson at marel.com
Sat Oct 17 00:53:45 EST 2009
Hello,
Wolfgang Denk wrote:
> Dear =?ISO-8859-1?Q?K=E1ri_Dav=ED=F0sson?=,
>
> In message <4AD70927.3030401 at marel.com> you wrote:
>> The kernel from the BSP on Freescale site is crashing on the CAN in my case
>> (might be a hardware bug).
>
> I don;t think so. There are some problems in this code, for example
> the clocks seem to be wrong. Not toi menthin that the whole code is
> hoplessly old and without chance of ever being merged into mainline.
Indeed. I had to disable the follwing clock stuff to prevent crashing. I am
running on Rev 3 chip.
diff --git a/drivers/net/can/mscan/mpc52xx_can.c b/drivers/net/can/mscan/mpc52xx_can.c
index e1d78b9..55def57 100644
--- a/drivers/net/can/mscan/mpc52xx_can.c
+++ b/drivers/net/can/mscan/mpc52xx_can.c
@@ -104,8 +104,8 @@ static int __devinit mpc52xx_can_probe(struct platform_device *pdev)
port_clk = clk_get(NULL, clk_name);
/* update clock rate for mpc5121e rev2 chip */
- if (port_clk)
- pdata->clock_frq = clk_get_rate(port_clk);
+// if (port_clk)
+// pdata->clock_frq = clk_get_rate(port_clk);
/* enable clock for mscan module */
clk_enable(mscan_clk);
>
>> I could not find the source for the kernel in the BSP or on the freescale site.
>
> Extracting just the sources from LTIB (without actually building all
> of it) is indeed not exactly easy.
>
>> I had not looked at http://git.denx.de/?p=linux-2.6-denx.git;a=shortlog;h=refs/heads/ltib-mpc5121ads-20090602
>
> Be careful. There are several branches that are intended for reference
> only, or reflect work in progress. These are usually not intended for
> use real use in a project:
>
> The "ltib-mpc5121ads-20090602" branch reflects the exact state of the
> kernel contained in the LTIB with this name (dated July 2009, despite
> the name; based at 2.6.24.6, i. e. 7+ kernel versions behind).
Yes this I found it odd that the mpc5121 work is so far behind.
We have two other mpc5200 based boards that I am running with 2.6.29.
They are running well.
>
> The "mpc512x" branch contains a much smaller subset of drivers, but is
> based on a more recent kernel tree (2.6.31-rc5) and is the base of our
> (currently dozing) attempts to push this code into mainline.
>
>> I am now in the progress of cloning the linux-mpc512x git from denx and building from it.
>> Hopefully that will help me.
>
> Do not do this. This old repository was intended for co-operation
> with John Rigby, when he was still working for Freescale. It is a
> dead end and totally unsupported. Guess I should remove it. Indeed.
> Removed now. Sorry for the confusion.
Already done it and I got it to compile but as you hint I am having other problems.
Serial ports and ethernet works though :)
>
> Best regards,
>
> Wolfgang Denk
>
rg
kd
More information about the Linuxppc-dev
mailing list