lkml.org 
[lkml]   [2009]   [Oct]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] leds-alix2: add support for button connected to J15
Hi Daniel,

On Tue, Oct 20, 2009 at 12:13:57PM +0200, Daniel Mack wrote:
> +
> + ret = input_register_polled_device(ipdev);
> +
> + if (ret) {
> + input_free_polled_device(ipdev);
> + ipdev = NULL;
> + }
> +
> + return ret;
> +}

Ah, don't be lazy ;)

static void alix_button_unregister(void)
{
if (ipdev) {
input_unregister_polled_device(ipdev);
/* Yes, polled devices need to be freed */
input_free_polled_device(ipdev);
ipdev = NULL;
}
}

> +#else
> +#define alix_button_register() (0)

static inline int alix_button_register(void) { return 0; }
static inline void alix_button_unregister(void) { }

So you get typechecking (if any).

> +#endif
> +
> static int __init alix_led_probe(struct platform_device *pdev)
> {
> int i;
> @@ -89,6 +139,11 @@ static int __init alix_led_probe(struct platform_device *pdev)
> if (ret < 0)
> goto fail;
> }
> +
> + ret = alix_button_register();
> + if (ret)
> + goto fail;
> +
> return 0;
>
> fail:
> @@ -103,6 +158,12 @@ static int alix_led_remove(struct platform_device *pdev)
>
> for (i = 0; i < ARRAY_SIZE(alix_leds); i++)
> led_classdev_unregister(&alix_leds[i].cdev);
> +
> +#ifdef CONFIG_LEDS_ALIX2_BUTTON
> + if (ipdev)
> + input_unregister_polled_device(ipdev);
> +#endif

alix_button_unregister();

> +
> return 0;
> }
>

--
Dmitry

\
 
 \ /
  Last update: 2009-10-21 06:39    [W:0.135 / U:0.596 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site