RE: [PATCH 6/10] omap mailbox: remove unnecessary arg for omap_mbox_msg_send

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

 



Hi Russell,
'arg' is used later in Patch 10, as part of the tasklet implementation, for writing messages to the mailbox. Should I be removing it in patch 6 and introduce it only for the tasklet implementation?

Thank you and regards
Subbu 

> -----Original Message-----
> From: Russell King [mailto:rmk@xxxxxxxxxxxxxxxx] 
> Sent: Monday, September 07, 2009 8:33 PM
> To: C.A, Subramaniam
> Cc: linux-omap@xxxxxxxxxxxxxxx; tony@xxxxxxxxxxx; 
> Hiroshi.DOYU@xxxxxxxxx; Kanigeri, Hari; Gupta, Ramesh
> Subject: Re: [PATCH 6/10] omap mailbox: remove unnecessary 
> arg for omap_mbox_msg_send
> 
> On Fri, Sep 04, 2009 at 05:18:04PM +0530, C.A, Subramaniam wrote:
> > @@ -114,7 +114,6 @@ int omap_mbox_msg_send(struct omap_mbox 
> *mbox, mbox_msg_t msg, void* arg)
> >  	}
> >  
> >  	tx_data->msg = msg;
> > -	tx_data->arg = arg;
> 
> What about also removing 'arg' from struct omap_msg_tx_data ?
> 
> --
> Russell King
>  Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
>  maintainer of:
> 
> --
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux