From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <512213FD.7010607@hilscher.com> Date: Mon, 18 Feb 2013 12:43:57 +0100 From: =?ISO-8859-1?Q?St=E9phane_LOS?= MIME-Version: 1.0 References: <511A2964.4080902@hilscher.com> <511A2CA4.1000408@siemens.com> <511B9E94.2030108@hilscher.com> <511CE85D.6070704@hilscher.com> <511CFC47.5000709@hilscher.com> <511E4C3B.2020407@siemens.com> In-Reply-To: <511E4C3B.2020407@siemens.com> Content-Type: text/plain; charset="iso-8859-1"; format="flowed" Content-Transfer-Encoding: quoted-printable Subject: Re: [Xenomai] Hilscher driver for cifX boards List-Id: Discussions about the Xenomai project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka Cc: Xenomai Le 15/02/2013 15:54, Jan Kiszka a =E9crit : > On 2013-02-14 16:01, St=E9phane LOS wrote: >> Hi, >> >> Please find attached the newest version that seems better to us. > Some remarks on the code first: > - Consider reformatting your code according to kernel style, it will > improve readability (no long lines, e.g.). > - rtdm_malloc is for the rare case that you allocate memory over RT > context. But probe is still Linux. > - If you register/free the IRQ handler on open/close, you must also > enable/disable the IRQ sources in the hardware in those functions. > - On removal, unregister the RTDM device first. That will synchronize > with remaining open instances. > - Minor: Avoid __devinit/exit, they were removed from recent kernels. > > So, what is the bug again? How does the oops message look like? When > will the device start generating IRQs, and do they arrive while rebooting? > > Jan > Thank you for your comments. I have forwarded them to the developer. Let's wait for his answer... Best Regards, Cordialement, St=E9phane LOS slos@hilscher.com Support technique ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Hilscher France 12, rue du 35=E8me R=E9giment d'Aviation Miniparc du Ch=EAne 69500 BRON France T=E9l. : +33 (0) 4 72 37 98 40 Fax : +33 (0) 4 78 26 83 27 http ://www.hilscher.fr ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~