Re: [PATCH V2] watchdog: s3c2410_wdt: remove the global variables

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

 



Hi Leela,

On 22 July 2013 11:44, Leela Krishna Amudala <l.krishna@xxxxxxxxxxx> wrote:
> This patch removes the global variables in the driver file and
> group them into a structure.
>
> Signed-off-by: Leela Krishna Amudala <l.krishna@xxxxxxxxxxx>
> ---
>  Note: This patch is rebased on kgene's for-next branch and tested on SMDK5420.
>
>  Changes since V1:
>         - changed the patch subject.
>         - indentation correction in s3c2410_watchdog structure.
>
>  drivers/watchdog/s3c2410_wdt.c |  204 +++++++++++++++++++++++-----------------
>  1 file changed, 119 insertions(+), 85 deletions(-)
>
> diff --git a/drivers/watchdog/s3c2410_wdt.c b/drivers/watchdog/s3c2410_wdt.c
> index 6a22cf5..de679d0 100644
> --- a/drivers/watchdog/s3c2410_wdt.c
> +++ b/drivers/watchdog/s3c2410_wdt.c
> @@ -84,13 +84,20 @@ MODULE_PARM_DESC(soft_noboot, "Watchdog action, set to 1 to ignore reboots, "
>                         "0 to reboot (default 0)");
>  MODULE_PARM_DESC(debug, "Watchdog debug, set to >1 for debug (default 0)");
>
> -static struct device    *wdt_dev;      /* platform device attached to */
> -static struct resource *wdt_mem;
> -static struct resource *wdt_irq;
> -static struct clk      *wdt_clock;
> -static void __iomem    *wdt_base;
> -static unsigned int     wdt_count;
> -static DEFINE_SPINLOCK(wdt_lock);
> +struct s3c2410_watchdog {
> +       struct device           *dev;   /* platform device attached to */

Since you would anyway be respinning this patch to address comments
from Tomasz, please remove the above comment too.

-- 
With warm regards,
Sachin
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux