Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Jan Kiszka
Matthias Fuchs wrote: > On Tuesday 12 September 2006 18:46, Wolfgang Grandegger wrote: >> Matthias Fuchs wrote: >>> Hi, >>> >>> here's the 2nd try: >>> - updated external clock frequency handling >>> - use request_mem_region() before remapping memory >>> - renamed local variable mem to vmem in clea

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Matthias Fuchs
On Tuesday 12 September 2006 18:46, Wolfgang Grandegger wrote: > Matthias Fuchs wrote: > > Hi, > > > > here's the 2nd try: > > - updated external clock frequency handling > > - use request_mem_region() before remapping memory > > - renamed local variable mem to vmem in clean loop because it shadowe

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Jan Kiszka
Wolfgang Grandegger wrote: > Matthias Fuchs wrote: >> Hi, >> >> here's the 2nd try: >> - updated external clock frequency handling >> - use request_mem_region() before remapping memory >> - renamed local variable mem to vmem in clean loop because it shadowed >> a >> module parameter > > I have

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Wolfgang Grandegger
Matthias Fuchs wrote: Hi, here's the 2nd try: - updated external clock frequency handling - use request_mem_region() before remapping memory - renamed local variable mem to vmem in clean loop because it shadowed a module parameter I have one more comment, sorry. I think you should use:

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Matthias Fuchs
Hi, here's the 2nd try: - updated external clock frequency handling - use request_mem_region() before remapping memory - renamed local variable mem to vmem in clean loop because it shadowed a module parameter Now it's on you to decide if we should merge and/or rename. I wouldn't merge be

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Jan Kiszka
Wolfgang Grandegger wrote: > Matthias Fuchs wrote: >> Hi Wolfgang, >> >> On Tuesday 12 September 2006 14:22, Wolfgang Grandegger wrote: >>> What about using request_mem_region()? While looking to the driver I now >> ... will be added, of course. >>> realize, that it's mainly duplicated code. Does

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Wolfgang Grandegger
Matthias Fuchs wrote: Hi Wolfgang, On Tuesday 12 September 2006 14:22, Wolfgang Grandegger wrote: What about using request_mem_region()? While looking to the driver I now ... will be added, of course. realize, that it's mainly duplicated code. Does it not make more sense to make a combined io

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Wolfgang Grandegger
Jan Kiszka wrote: Wolfgang Grandegger wrote: Hi Matthias, Matthias Fuchs wrote: Hi, attached you will find a patch that adds support for memory mapped SJA1000 CAN controllers as they often can be found on embedded boards. The driver is based on the rtmen_isa driver. What about using request_m

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Matthias Fuchs
Hi Wolfgang, On Tuesday 12 September 2006 14:22, Wolfgang Grandegger wrote: > What about using request_mem_region()? While looking to the driver I now ... will be added, of course. > realize, that it's mainly duplicated code. Does it not make more sense > to make a combined io/mem driver. If io

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Jan Kiszka
Wolfgang Grandegger wrote: > Hi Matthias, > > Matthias Fuchs wrote: >> Hi, >> >> attached you will find a patch that adds support for memory mapped >> SJA1000 CAN controllers as they often can be found on embedded boards. >> The driver is based on the rtmen_isa driver. > > What about using reques

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Jan Kiszka
Matthias Fuchs wrote: > Hi, > > attached you will find a patch that adds support for memory mapped SJA1000 > CAN > controllers as they often can be found on embedded boards. The driver is > based on the rtmen_isa driver. Thanks for your patch! Below are only cleanup comments, not all directed

Re: [Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Wolfgang Grandegger
Hi Matthias, Matthias Fuchs wrote: Hi, attached you will find a patch that adds support for memory mapped SJA1000 CAN controllers as they often can be found on embedded boards. The driver is based on the rtmen_isa driver. What about using request_mem_region()? While looking to the driver I

[Xenomai-core] [PATCH] rtcan_mem - CAN driver for memory mapped SJA1000 controllers

2006-09-12 Thread Matthias Fuchs
Hi, attached you will find a patch that adds support for memory mapped SJA1000 CAN controllers as they often can be found on embedded boards. The driver is based on the rtmen_isa driver. The driver has been tested on esd's embedded PowerPC boards with AMCC PPC405 CPUs. Thanks to Jan for givin