On Wed, Oct 21, 2009 at 9:46 AM, Wolfgang Grandegger <[email protected]> 
wrote:
> Hi Christian,
>

Hi,

> are you heading for kernel inclusion? I browsed the mpc251x.c file and
> realized various issues which should be fixed for inclusion. How do you

Judging by the feedback on the list there are quite a lot of people
using it, so I think it's worth fixing all the problems and have the
driver included. I'll try to do so, just point me to the problematic
points.

> want to proceed? Fix SVN trunk first or directly prepare patches for
> inclusion (based on net-next-2.6) and do the backport thereafter?

If it's ok I will fix SVN trunk first. I have some customer using that
so it's easier for me have the changes tested there. Then I can rebase
the patch against:

http://git.kernel.org/?p=linux/kernel/git/davem/net-next-2.6.git;a=tree;f=drivers/net/can;h=baa1b9e29397740cdb0e1118db1cf11b1dddc5e0;hb=HEAD


>
> A RESTARTED error messages should be sent in case of automatic recovery.
> I don't find it in the code.
>

huh, forgot it.

>
> For all other issues I prefer to comment on mpc251x.c.
>

I'll collect all the comments and try to work out a patch ASAP.

Thanks,
Bye!


-- 
Christian Pellegrin, see http://www.evolware.org/chri/
"Real Programmers don't play tennis, or any other sport which requires
you to change clothes. Mountain climbing is OK, and Real Programmers
wear their climbing boots to work in case a mountain should suddenly
spring up in the middle of the computer room."
_______________________________________________
Socketcan-core mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/socketcan-core

Reply via email to