From: edwin_rong <edwin_rong@realsil.com.cn>
To: "Németh Márton" <nm127@freemail.hu>
Cc: 王炜 <wei_wang@realsil.com.cn>,
"Greg Kroah-Hartman" <gregkh@suse.de>,
"Devendra Naga" <devendra.aaru@gmail.com>,
"devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] rts5139: fix pointer coding style
Date: Sun, 29 Jan 2012 08:52:31 +0800 [thread overview]
Message-ID: <4F24984F.4030801@realsil.com.cn> (raw)
In-Reply-To: <4F1C83E3.7090901@freemail.hu>
Dear Marton,
Thanks for the fix.
BRs
Edwin
On 01/23/2012 05:47 AM, Németh Márton wrote:
> From: Márton Németh<nm127@freemail.hu>
>
> This will remove the following checkpatch.pl error:
> ERROR: "foo * bar" should be "foo *bar"
>
> Signed-off-by: Márton Németh<nm127@freemail.hu>
> ---
> drivers/staging/rts5139/ms.h | 4 ++--
> drivers/staging/rts5139/rts51x_chip.c | 14 +++++++-------
> drivers/staging/rts5139/rts51x_chip.h | 6 +++---
> drivers/staging/rts5139/rts51x_fop.h | 2 +-
> drivers/staging/rts5139/rts51x_transport.c | 2 +-
> drivers/staging/rts5139/rts51x_transport.h | 2 +-
> drivers/staging/rts5139/sd_cprm.c | 2 +-
> 7 files changed, 16 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/staging/rts5139/ms.h b/drivers/staging/rts5139/ms.h
> index f9d46d2..3ce1dc9 100644
> --- a/drivers/staging/rts5139/ms.h
> +++ b/drivers/staging/rts5139/ms.h
> @@ -249,9 +249,9 @@ int ms_delay_write(struct rts51x_chip *chip);
> #ifdef SUPPORT_MAGIC_GATE
>
> int ms_switch_clock(struct rts51x_chip *chip);
> -int ms_write_bytes(struct rts51x_chip *chip, u8 tpc, u8 cnt, u8 cfg, u8 * data,
> +int ms_write_bytes(struct rts51x_chip *chip, u8 tpc, u8 cnt, u8 cfg, u8 *data,
> int data_len);
> -int ms_read_bytes(struct rts51x_chip *chip, u8 tpc, u8 cnt, u8 cfg, u8 * data,
> +int ms_read_bytes(struct rts51x_chip *chip, u8 tpc, u8 cnt, u8 cfg, u8 *data,
> int data_len);
> int ms_set_rw_reg_addr(struct rts51x_chip *chip, u8 read_start, u8 read_cnt,
> u8 write_start, u8 write_cnt);
> diff --git a/drivers/staging/rts5139/rts51x_chip.c b/drivers/staging/rts5139/rts51x_chip.c
> index adc0d00..b3e0bb2 100644
> --- a/drivers/staging/rts5139/rts51x_chip.c
> +++ b/drivers/staging/rts5139/rts51x_chip.c
> @@ -541,7 +541,7 @@ int rts51x_get_rsp(struct rts51x_chip *chip, int rsp_len, int timeout)
> return STATUS_SUCCESS;
> }
>
> -int rts51x_get_card_status(struct rts51x_chip *chip, u16 * status)
> +int rts51x_get_card_status(struct rts51x_chip *chip, u16 *status)
> {
> int retval;
> u16 val;
> @@ -577,7 +577,7 @@ int rts51x_write_register(struct rts51x_chip *chip, u16 addr, u8 mask, u8 data)
> return STATUS_SUCCESS;
> }
>
> -int rts51x_read_register(struct rts51x_chip *chip, u16 addr, u8 * data)
> +int rts51x_read_register(struct rts51x_chip *chip, u16 addr, u8 *data)
> {
> int retval;
>
> @@ -620,7 +620,7 @@ int rts51x_ep0_write_register(struct rts51x_chip *chip, u16 addr, u8 mask,
> return STATUS_SUCCESS;
> }
>
> -int rts51x_ep0_read_register(struct rts51x_chip *chip, u16 addr, u8 * data)
> +int rts51x_ep0_read_register(struct rts51x_chip *chip, u16 addr, u8 *data)
> {
> int retval;
> u16 value = 0;
> @@ -720,7 +720,7 @@ int rts51x_seq_read_register(struct rts51x_chip *chip, u16 addr, u16 len,
> return STATUS_SUCCESS;
> }
>
> -int rts51x_read_ppbuf(struct rts51x_chip *chip, u8 * buf, int buf_len)
> +int rts51x_read_ppbuf(struct rts51x_chip *chip, u8 *buf, int buf_len)
> {
> int retval;
>
> @@ -735,7 +735,7 @@ int rts51x_read_ppbuf(struct rts51x_chip *chip, u8 * buf, int buf_len)
> return STATUS_SUCCESS;
> }
>
> -int rts51x_write_ppbuf(struct rts51x_chip *chip, u8 * buf, int buf_len)
> +int rts51x_write_ppbuf(struct rts51x_chip *chip, u8 *buf, int buf_len)
> {
> int retval;
>
> @@ -776,7 +776,7 @@ int rts51x_write_phy_register(struct rts51x_chip *chip, u8 addr, u8 val)
> return STATUS_SUCCESS;
> }
>
> -int rts51x_read_phy_register(struct rts51x_chip *chip, u8 addr, u8 * val)
> +int rts51x_read_phy_register(struct rts51x_chip *chip, u8 addr, u8 *val)
> {
> int retval;
>
> @@ -921,7 +921,7 @@ void rts51x_trace_msg(struct rts51x_chip *chip, unsigned char *buf, int clear)
> }
> #endif
>
> -void rts51x_pp_status(struct rts51x_chip *chip, unsigned int lun, u8 * status,
> +void rts51x_pp_status(struct rts51x_chip *chip, unsigned int lun, u8 *status,
> u8 status_len)
> {
> struct sd_info *sd_card =&(chip->sd_card);
> diff --git a/drivers/staging/rts5139/rts51x_chip.h b/drivers/staging/rts5139/rts51x_chip.h
> index 321ece7..13fc2a4 100644
> --- a/drivers/staging/rts5139/rts51x_chip.h
> +++ b/drivers/staging/rts5139/rts51x_chip.h
> @@ -857,12 +857,12 @@ static inline u8 *rts51x_get_rsp_data(struct rts51x_chip *chip)
> return chip->rsp_buf;
> }
>
> -int rts51x_get_card_status(struct rts51x_chip *chip, u16 * status);
> +int rts51x_get_card_status(struct rts51x_chip *chip, u16 *status);
> int rts51x_write_register(struct rts51x_chip *chip, u16 addr, u8 mask, u8 data);
> -int rts51x_read_register(struct rts51x_chip *chip, u16 addr, u8 * data);
> +int rts51x_read_register(struct rts51x_chip *chip, u16 addr, u8 *data);
> int rts51x_ep0_write_register(struct rts51x_chip *chip, u16 addr, u8 mask,
> u8 data);
> -int rts51x_ep0_read_register(struct rts51x_chip *chip, u16 addr, u8 * data);
> +int rts51x_ep0_read_register(struct rts51x_chip *chip, u16 addr, u8 *data);
> int rts51x_seq_write_register(struct rts51x_chip *chip, u16 addr, u16 len,
> u8 *data);
> int rts51x_seq_read_register(struct rts51x_chip *chip, u16 addr, u16 len,
> diff --git a/drivers/staging/rts5139/rts51x_fop.h b/drivers/staging/rts5139/rts51x_fop.h
> index 0453f57..94d75f0 100644
> --- a/drivers/staging/rts5139/rts51x_fop.h
> +++ b/drivers/staging/rts5139/rts51x_fop.h
> @@ -48,7 +48,7 @@ int rts51x_open(struct inode *inode, struct file *filp);
> int rts51x_release(struct inode *inode, struct file *filp);
> ssize_t rts51x_read(struct file *filp, char __user *buf, size_t count,
> loff_t *f_pos);
> -ssize_t rts51x_write(struct file *filp, const char __user * buf, size_t count,
> +ssize_t rts51x_write(struct file *filp, const char __user *buf, size_t count,
> loff_t *f_pos);
> #if 0 /* LINUX_VERSION_CODE< KERNEL_VERSION(2, 6, 36) */
> int rts51x_ioctl(struct inode *inode, struct file *filp, unsigned int cmd,
> diff --git a/drivers/staging/rts5139/rts51x_transport.c b/drivers/staging/rts5139/rts51x_transport.c
> index e11467a..da9c83b 100644
> --- a/drivers/staging/rts5139/rts51x_transport.c
> +++ b/drivers/staging/rts5139/rts51x_transport.c
> @@ -883,7 +883,7 @@ int rts51x_transfer_data_partial(struct rts51x_chip *chip, unsigned int pipe,
> return result;
> }
>
> -int rts51x_get_epc_status(struct rts51x_chip *chip, u16 * status)
> +int rts51x_get_epc_status(struct rts51x_chip *chip, u16 *status)
> {
> unsigned int pipe = RCV_INTR_PIPE(chip);
> struct usb_host_endpoint *ep;
> diff --git a/drivers/staging/rts5139/rts51x_transport.h b/drivers/staging/rts5139/rts51x_transport.h
> index 8464c48..9dd556e 100644
> --- a/drivers/staging/rts5139/rts51x_transport.h
> +++ b/drivers/staging/rts5139/rts51x_transport.h
> @@ -73,7 +73,7 @@ int rts51x_start_epc_transfer(struct rts51x_chip *chip);
> void rts51x_cancel_epc_transfer(struct rts51x_chip *chip);
> #endif
>
> -int rts51x_get_epc_status(struct rts51x_chip *chip, u16 * status);
> +int rts51x_get_epc_status(struct rts51x_chip *chip, u16 *status);
> void rts51x_invoke_transport(struct scsi_cmnd *srb, struct rts51x_chip *chip);
>
> #endif /* __RTS51X_TRANSPORT_H */
> diff --git a/drivers/staging/rts5139/sd_cprm.c b/drivers/staging/rts5139/sd_cprm.c
> index 407cd43..d5969d9 100644
> --- a/drivers/staging/rts5139/sd_cprm.c
> +++ b/drivers/staging/rts5139/sd_cprm.c
> @@ -233,7 +233,7 @@ RTY_SEND_CMD:
> return STATUS_SUCCESS;
> }
>
> -int ext_sd_get_rsp(struct rts51x_chip *chip, int len, u8 * rsp, u8 rsp_type)
> +int ext_sd_get_rsp(struct rts51x_chip *chip, int len, u8 *rsp, u8 rsp_type)
> {
> int retval, rsp_len;
> u16 reg_addr;
prev parent reply other threads:[~2012-01-29 1:08 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-22 21:47 [PATCH] rts5139: fix pointer coding style Németh Márton
2012-01-29 0:52 ` edwin_rong [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F24984F.4030801@realsil.com.cn \
--to=edwin_rong@realsil.com.cn \
--cc=devel@driverdev.osuosl.org \
--cc=devendra.aaru@gmail.com \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=nm127@freemail.hu \
--cc=wei_wang@realsil.com.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.