On Thu, Sep 25, 2008 at 12:31:38PM +0530, ext Gadiyar, Anand wrote:
> From: Madhusudhan Chikkature <[EMAIL PROTECTED]>
> 
> Make hdq_reg_out as void and remove suspend, resume from the driver structure.
> 
> Signed-off-by: Madhusudhan Chikkature <[EMAIL PROTECTED]>
Acked-by: Felipe Balbi <[EMAIL PROTECTED]>
> ---
>  drivers/w1/masters/omap_hdq.c |   12 +++++-------
>  1 files changed, 5 insertions(+), 7 deletions(-)
> 
> Index: linux-omap-2.6/drivers/w1/masters/omap_hdq.c
> ===================================================================
> --- linux-omap-2.6.orig/drivers/w1/masters/omap_hdq.c 2008-09-24 
> 15:37:18.000000000 +0530
> +++ linux-omap-2.6/drivers/w1/masters/omap_hdq.c      2008-09-25 
> 10:19:22.000000000 +0530
> @@ -55,11 +55,13 @@ static int w1_id;
>  struct hdq_data {
>       struct device           *dev;
>       void __iomem            *hdq_base;
> +     /* lock status update */
>       struct  mutex           hdq_mutex;
>       int                     hdq_usecount;
>       struct  clk             *hdq_ick;
>       struct  clk             *hdq_fck;
>       u8                      hdq_irqstatus;
> +     /* device lock */
>       spinlock_t              hdq_spinlock;
>       /*
>        * Used to control the call to omap_hdq_get and omap_hdq_put.
> @@ -79,8 +81,6 @@ static int omap_hdq_remove(struct platfo
>  static struct platform_driver omap_hdq_driver = {
>       .probe =        omap_hdq_probe,
>       .remove =       omap_hdq_remove,
> -     .suspend =      NULL,
> -     .resume =       NULL,
>       .driver =       {
>               .name = "omap_hdq",
>       },
> @@ -106,11 +106,9 @@ static inline u8 hdq_reg_in(struct hdq_d
>       return __raw_readb(hdq_data->hdq_base + offset);
>  }
>  
> -static inline u8 hdq_reg_out(struct hdq_data *hdq_data, u32 offset, u8 val)
> +static inline void hdq_reg_out(struct hdq_data *hdq_data, u32 offset, u8 val)
>  {
>       __raw_writeb(val, hdq_data->hdq_base + offset);
> -
> -     return val;
>  }
>  
>  static inline u8 hdq_reg_merge(struct hdq_data *hdq_data, u32 offset,
> @@ -715,17 +713,17 @@ omap_hdq_init(void)
>  {
>       return platform_driver_register(&omap_hdq_driver);
>  }
> +module_init(omap_hdq_init);
>  
>  static void __exit
>  omap_hdq_exit(void)
>  {
>       platform_driver_unregister(&omap_hdq_driver);
>  }
> -
> -module_init(omap_hdq_init);
>  module_exit(omap_hdq_exit);
>  
>  module_param(w1_id, int, S_IRUSR);
> +MODULE_PARM_DESC(w1_id, "1-wire id for the slave detection");
>  
>  MODULE_AUTHOR("Texas Instruments");
>  MODULE_DESCRIPTION("HDQ driver Library");
-- 
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to