Re: [patch 1/2] m68k: Atari EtherNAT (SMC91C111) driver (updated)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Interrupts are still missing, the driver operates from a timer routine
instead.

Thanks, updated. But...  it doesn't seem to compile?

drivers/net/atari_91C111.c: In function 'smc_rcv':
drivers/net/atari_91C111.c:584: error: implicit declaration of function 'readsw'

Odd - that should have been added by my previous io.h patch (the one that was too chatty in the comments, around July 20).

drivers/net/atari_91C111.c: In function 'smc_hardware_send_pkt':
drivers/net/atari_91C111.c:729: error: implicit declaration of function 'writew_be'

See patch 3 out of 2 (noticed that this morning only ...)

drivers/net/atari_91C111.c:729: error: implicit declaration of function 'writesw'

Same thing as the readsw function, was added by the July 20 patch.

drivers/net/atari_91C111.c: In function 'atari_ethernat_pdev_probe':
drivers/net/atari_91C111.c:2317: error: 'struct smc91x_platdata' has no member named 'irq_flags'

Duh - now where did that one come from? Seems someoene removed irq_flags from the platform data struct between 2.6.26 and 2.6.28 ... I'll check the driver changelog. Looks like I should work from a fresh tree anyway.

This is atari_defconfig with all `new' Atari drivers enabled.

BTW, can you please run scripts/checkpatch.pl on your patches?

| total: 63 errors, 87 warnings, 2621 lines checked

Duh again ... half the errors were in the original smc91x.c but I'll clean up
my mess now.

	Michael
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Video for Linux]     [Yosemite News]     [Linux S/390]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux