- Oct 10, 2017
-
-
Juan David González Cobas authored
It turns out that trans_start (see 7499f529) was deprecated in RHEL kernel versions earlier than in mainstream. This adds logic to take care of all cases.
-
- Jun 27, 2017
-
-
- Feb 28, 2017
-
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
The trans_start field from net_device has been removed. ---------------- commit 9b36627acecd5792e81daf1a3bff8eab39ed45fb Author: Florian Westphal <fw@strlen.de> Date: Tue May 3 16:33:14 2016 +0200 net: remove dev->trans_start ---------------- commit 860e9538a9482bb84589f7d0718a7e6d0a944d58 Author: Florian Westphal <fw@strlen.de> Date: Tue May 3 16:33:13 2016 +0200 treewide: replace dev->trans_start update with helper ---------------- Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
The following patch deprecated the macro DEFINE_PCI_DEVICE_TABLE in favor of using directly the structure type ------------------ commit 92e112fdbb3cb55b43390426501a7efacd893b96 Author: Joe Perches <joe@perches.com> Date: Fri Dec 13 11:36:22 2013 -0700 PCI/checkpatch: Deprecate DEFINE_PCI_DEVICE_TABLE ------------------ Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
It should not have any effect on previous kernels Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Jul 17, 2016
-
-
Dimitris Lampridis authored
Compiling on kernel >=4.5 produces the following error: CC [M] spec-sw/kernel/wr-nic-gpio.o spec-sw/kernel/wr-nic-gpio.c: In function ‘gc_to_fmc’: spec-sw/kernel/wr-nic-gpio.c:20:25: error: ‘struct gpio_chip’ has no member named ‘dev’ struct device *dev = gc->dev; ^ spec-sw/kernel/wr-nic-gpio.c: In function ‘wrn_gpio_init’: spec-sw/kernel/wr-nic-gpio.c:75:4: error: ‘struct gpio_chip’ has no member named ‘dev’ gc->dev = &fmc->dev; ^ This is due to a change in struct gpio_chip introduced in 4.5, where the "dev" field was renamed to "parent". 58383c7 gpio: change member .dev to .parent This patch makes our code compatible with modern kernel versions Signed-off-by:
Federico Vaga <federico.vaga@cern.ch> Signed-off-by:
Dimitris Lampridis <Dimitris.Lampridis@cern.ch>
-
- Mar 03, 2016
-
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@gmail.com>
-
Federico Vaga authored
This reverts commit 0598de40. This was not a memory leakage because the fmc-bus release that pointer for us. Is it good? We allocate and it is someone else us free, it lacks of simmetry. Probably the register function should be called devm_ like all the others in the kernel who relase for your the memory. In this way it is clear the behaviour.
-
- Feb 10, 2016
-
-
Federico Vaga authored
We ack the GN4124 **before** the VIC. The other way around makes the interrupt handling timing dependent. For instance. When the VIC VIC_CTL_EMU_LEN_W is too short it may happens that the VIC asserts the interrupt **before** the GN4124 is acked; then, so do not receive interrupts anymore from the GN4124 because it misses the VIC edge. Signed-off-by:
Federico Vaga <federico.vaga@cern.ch> Signed-off-by:
Michel Arruat <michel.arruat@cern.ch>
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Nov 20, 2015
-
-
Adam Wujek authored
Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
- Nov 13, 2015
-
-
Adam Wujek authored
Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
Adam Wujek authored
Use _ABS in Makefiles wherever possible. Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
Adam Wujek authored
FMC_BUS_ABS is passed to Kbuild, which uses different current directory. It is not trivial to handle FMC_BUS to be absolute or relative path. Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
Adam Wujek authored
Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
Adam Wujek authored
files wr_nic/.*.o.cmd were not removed Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
Adam Wujek authored
Include parent_common.mk from up directory. Which may contain various environment definitions such compiler, linker, etc. If such file is not present it will not affect build. Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
Adam Wujek authored
--export FMC_DRV --correct passing GIT_VERSION Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
Adam Wujek authored
Signed-off-by:
Adam Wujek <adam.wujek@cern.ch>
-
Federico Vaga authored
We were using vmalloc() assunign that its library was already included by some other libraries. Today, this is not the case anymore, so we must include it Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Aug 25, 2015
-
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- May 07, 2015
-
-
Federico Vaga authored
Mainly for compatibility (simplification) with the CERN super module. Insted of using a script to guess which fmc-bus define, assume always that parent project define the FMC_BUS to use. Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
For the time being you must write the bitstream in a single write. For example: dd if=/home/fvaga/spec-fmc-adc-v4.0.bin of=/dev/spec-0700 obs=5000000 Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Feb 24, 2015
-
-
Federico Vaga authored
Since kernel 3.18 the gpiochip_remove function return void and not an error code as before. In case the gpiochip is removed while there is a requested gpio, the kernel will print an error message. By applying this patch without any control on the kernel version we are going to loose the error message in case of trouble with old kernels Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Jan 20, 2015
-
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Sep 18, 2014
-
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Aug 13, 2014
-
-
Federico Vaga authored
There should not be any side effect on resetting the FGPA every time we program it. Confirmed by Tomasz Wlostowski <tomasz.wlostowski@cern.ch> Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-nof -by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Aug 07, 2014
-
-
Federico Vaga authored
With some kernel configuration the previous locking policy can generate a deadlock on the spec->irq_lock spinlock. For example, this will happen sistematically if the option CONFIG_DEBUG_SHIRQ_FIXME is enabled. In this case, the handler is called immediatly an it is not a separeted thread. This patch move out from the locking section what doesn't need protection and is 'cause' of the deadlock. The deadlock was introduced with patch d4bbcc73 by using a large locking section. In realty the PCI request_irq is protected anyway thanks to the proctection of the variable 'first_time' and the 'spec_vic_irq_request' function. [more explaination about patch d4bbcc73 ] The purpose of that patch was to protect the initialization of the first VIC handler and the release of the last one. It was happening that while we are releasing the last VIC handler, some one was registering a new one but the spec->vic is still there. The result was that sometimes the PCI handler was removed (last VIC handler) but not re-requested for the incoming request. Signen-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Aug 01, 2014
-
-
Federico Vaga authored
Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
- Jul 14, 2014
-
-
Alessandro Rubini authored
Signed-off-by:
Alessandro Rubini <rubini@gnudd.com>
-
- Jul 08, 2014
-
-
Federico Vaga authored
Protecting the handlers' vector is not enough. Managing the interrupts involves other tasks that must be serialized. Mainly to handle the first request and the last free of a VIC handler. The VIC internal locking (vec_lock) is not useful anymore, but I do not remove it because theoretically the VIC should be an independent bunch of code. Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
When possible, avoid statement-scope variable for a better maintanability of the code. Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
If there are no reasons for a *_free() function to return an error, change the return type to void Signed-off-by:
Federico Vaga <federico.vaga@cern.ch>
-