Hello Vipul,

can you please remove all the (rtems_interrupt_handler) casts.

On 2013-07-23 17:03, Vipul Nayyar wrote:
[...]
@@ -746,8 +681,8 @@ int rtems_3c509_driver_attach (struct rtems_bsdnet_ifconfig 
*config )
         */
        overrun = 0;
        resend = 0;
-    ep_unit = 0;
-    ep_boards = 0;
+  ep_unit = 0;
+  ep_boards = 0;

Here the indentation is wrong, please check other parts of the patch as well.

[...]
@@ -326,7 +309,7 @@ void Clock_driver_support_initialize_hardware(void)

    if ( use_8254 ) {
      /* printk( "Use 8254\n" ); */
-    Clock_driver_support_at_tick = Clock_driver_support_at_tick_empty;
+    Clock_driver_support_at_tick = NULL;

What is the reason for this change?

[...]
@@ -381,7 +381,7 @@ ne_check_status (struct ne_softc *sc, int from_irq_handler)
  /* Handle an NE2000 interrupt.  */

  static void
-ne_interrupt_handler (rtems_irq_hdl_param cdata)
+ne_interrupt_handler (void * cdata)
  {
    rtems_vector_number v = (rtems_vector_number) cdata;
    struct ne_softc *sc;

I think you can get rid of ne_device_for_irno if you use the right cdata value (e.g. pass directly sc).

[...]

--
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.hu...@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
_______________________________________________
rtems-devel mailing list
rtems-devel@rtems.org
http://www.rtems.org/mailman/listinfo/rtems-devel

Reply via email to