Re: [PATCH] staging: alarm-dev: Support to Compile as Module

From: Greg Kroah-Hartman
Date: Thu Jun 12 2014 - 12:51:45 EST


On Thu, Jun 12, 2014 at 04:08:15PM +0530, pramod.gurav.etc@xxxxxxxxx wrote:
> From: Pramod Gurav <pramod.gurav.etc@xxxxxxxxx>
>
> Currently this alarm-dev can be compiles only as built in
> driver. This adds support to compile it as module as well.
> This has dependancy on:
> https://lkml.org/lkml/2014/6/12/180
>
> which exports symbol of alarmtimer function.
>
> CC: Greg Kroah-Hartman <greg@xxxxxxxxx>
> CC: Brian Swetland <swetland@xxxxxxxxxx>
> Signed-off-by: Pramod Gurav <pramod.gurav.etc@xxxxxxxxx>
> ---
> drivers/staging/android/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/android/Kconfig b/drivers/staging/android/Kconfig
> index 99e484f..c359317 100644
> --- a/drivers/staging/android/Kconfig
> +++ b/drivers/staging/android/Kconfig
> @@ -76,7 +76,7 @@ config ANDROID_LOW_MEMORY_KILLER
> Registers processes to be killed when memory is low
>
> config ANDROID_INTF_ALARM_DEV
> - bool "Android alarm driver"
> + tristate "Android alarm driver"
> depends on RTC_CLASS
> default n
> ---help---

Please resend the first patch (it's not a "trivial one") as part of this
patch series.

And do you ever really want this code as a module? What will auto-load
it if needed?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/