Re: [PATCH] [LEDS] Amstrad Delta LED support

[Ben Dooks <ben@fluff.org.uk>]

On Sat, Apr 22, 2006 at 02:08:23PM +0100, Jonathan McDowell wrote:
> [Which tree should I be trying to submit this to? The patch is against
> and works fine with 2.6.17-rc2]

I prefer the following PM defines, so there is only
one block of CONFIG_PM

#ifdef CONFIG_PM
int foo_suspend(...)
{
}

int foo_resume(...)
{
}

#else
#define foo_suspend NULL
#define foo_resume NULL
#endif 

 
> +static struct platform_driver foo_driver = {
> +	.probe		= foo_probe,
> +	.remove		= foo_remove,
> +	.suspend	= foo_suspend,
> +	.resume		= foo_resume,
> +};

-- 
Ben (ben@fluff.org, http://www.fluff.org/)

  'a smiley only costs 4 bytes'
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/