Hi Christoph, Your patch looks good to me, comments below :
Le lundi 21 janvier 2008, Christoph Muellner a écrit :
> +
> + /* Western Digital */
> + WDNetCenter,
> };
Please use upper case for the device enumeration.
>
> static void __init bcm4780_init(void) {
> @@ -134,6 +137,21 @@
> schedule_timeout(HZ * 5);
> }
>
> +static void __init NetCenter_init(void) {
Please name your function using lower case letters.
> + .leds = {
> + /* register_led changes values, so that we must
> exclude
> + * here to make NetCenter_init work */
> + //{ .name = "LED1", .gpio = 1 << 1},
> + //{ .name = "+5V", .gpio = 1 << 3},
> + //{ .name = "LED4", .gpio = 1 << 4},
> + //{ .name = "+12V", .gpio = 1 << 6},
Cannot the LEDs work after NetCenter_init was called ?
Thank you very much.
--
Best regards, Florian Fainelli
Email : [EMAIL PROTECTED]
http://openwrt.org
-------------------------------
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ openwrt-devel mailing list [email protected] http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
