All of lore.kernel.org
 help / color / mirror / Atom feed
* Xilinx Zynq CAN driver
@ 2014-01-03 11:26 Oliver Hartkopp
  2014-01-03 13:40 ` Michal Simek
  0 siblings, 1 reply; 3+ messages in thread
From: Oliver Hartkopp @ 2014-01-03 11:26 UTC (permalink / raw)
  To: Michal Simek; +Cc: Soren Brinkmann, Kedareswara rao Appana, linux-can

Hello Michal,

your update patch for the MAINTAINERS file ("MAINTAINERS: set up proper
record for Xilinx Zynq")

http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=c2fd4e38032239746badb0e21d72c3a00438feac

pointed me to your Xilinx repository hosted at github.

There I've seen that there's already a netdevice driver for your Zynq CAN
controller: 

https://github.com/Xilinx/linux-xlnx/blob/master/drivers/net/can/xilinx_can.c

IMHO the commit is already in pretty good shape: 

https://github.com/Xilinx/linux-xlnx/commit/cca204594df616b02c63f91a8b176c2665be57ea

Probably there are only some smaller formatting things to fix, e.g. remove
some debug stuff ...

Btw. is there anything that prevents you from posting your existing driver on
the linux-can mailing list for mainline upstream?

Having this driver in Linux mainline would at least reduce your out-of-tree
maintenance.

Best regards,
Oliver

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Xilinx Zynq CAN driver
  2014-01-03 11:26 Xilinx Zynq CAN driver Oliver Hartkopp
@ 2014-01-03 13:40 ` Michal Simek
  2014-01-03 14:10   ` Oliver Hartkopp
  0 siblings, 1 reply; 3+ messages in thread
From: Michal Simek @ 2014-01-03 13:40 UTC (permalink / raw)
  To: Oliver Hartkopp
  Cc: Michal Simek, Soren Brinkmann, Kedareswara rao Appana, linux-can

Hi Oliver,

On 01/03/2014 12:26 PM, Oliver Hartkopp wrote:
> Hello Michal,
> 
> your update patch for the MAINTAINERS file ("MAINTAINERS: set up proper
> record for Xilinx Zynq")
> 
> http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=c2fd4e38032239746badb0e21d72c3a00438feac
> 
> pointed me to your Xilinx repository hosted at github.
> 
> There I've seen that there's already a netdevice driver for your Zynq CAN
> controller: 
> 
> https://github.com/Xilinx/linux-xlnx/blob/master/drivers/net/can/xilinx_can.c
> 
> IMHO the commit is already in pretty good shape: 
> 
> https://github.com/Xilinx/linux-xlnx/commit/cca204594df616b02c63f91a8b176c2665be57ea
> 
> Probably there are only some smaller formatting things to fix, e.g. remove
> some debug stuff ...
> 
> Btw. is there anything that prevents you from posting your existing driver on
> the linux-can mailing list for mainline upstream?
> 
> Having this driver in Linux mainline would at least reduce your out-of-tree
> maintenance.

We are in process of validation of our drivers and they will be send to mainline
soon. I was in touch with Marc regarding can driver testing which we need to do.

Thanks,
Michal



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Xilinx Zynq CAN driver
  2014-01-03 13:40 ` Michal Simek
@ 2014-01-03 14:10   ` Oliver Hartkopp
  0 siblings, 0 replies; 3+ messages in thread
From: Oliver Hartkopp @ 2014-01-03 14:10 UTC (permalink / raw)
  To: Michal Simek; +Cc: Soren Brinkmann, Kedareswara rao Appana, linux-can

On 03.01.2014 14:40, Michal Simek wrote:

> 
> We are in process of validation of our drivers and they will be send to mainline
> soon. I was in touch with Marc regarding can driver testing which we need to do.
> 

Great!

Thanks in advance for your contribution.

Best regards,
Oliver

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-01-03 14:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-03 11:26 Xilinx Zynq CAN driver Oliver Hartkopp
2014-01-03 13:40 ` Michal Simek
2014-01-03 14:10   ` Oliver Hartkopp

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.