public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [RFC 01/21]arch:arm:mach Remove extra comma.
@ 2011-03-24 19:26 Justin P. Mattock
  2011-03-24 19:26 ` [RFC 02/21]arch:ia64 " Justin P. Mattock
                   ` (20 more replies)
  0 siblings, 21 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not. 

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 arch/arm/mach-at91/at91cap9_devices.c    |    2 +-
 arch/arm/mach-at91/at91sam9g45_devices.c |    2 +-
 arch/arm/mach-at91/at91sam9rl_devices.c  |    2 +-
 arch/arm/mach-tegra/tegra2_clocks.c      |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/mach-at91/at91cap9_devices.c b/arch/arm/mach-at91/at91cap9_devices.c
index d1f775e..308ce7a 100644
--- a/arch/arm/mach-at91/at91cap9_devices.c
+++ b/arch/arm/mach-at91/at91cap9_devices.c
@@ -171,7 +171,7 @@ void __init at91_add_device_usba(struct usba_platform_data *data)
 	 */
 	usba_udc_data.pdata.vbus_pin = -EINVAL;
 	usba_udc_data.pdata.num_ep = ARRAY_SIZE(usba_udc_ep);
-	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));;
+	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));
 
 	if (data && data->vbus_pin > 0) {
 		at91_set_gpio_input(data->vbus_pin, 0);
diff --git a/arch/arm/mach-at91/at91sam9g45_devices.c b/arch/arm/mach-at91/at91sam9g45_devices.c
index 1e8f275..5e9f8a4 100644
--- a/arch/arm/mach-at91/at91sam9g45_devices.c
+++ b/arch/arm/mach-at91/at91sam9g45_devices.c
@@ -256,7 +256,7 @@ void __init at91_add_device_usba(struct usba_platform_data *data)
 {
 	usba_udc_data.pdata.vbus_pin = -EINVAL;
 	usba_udc_data.pdata.num_ep = ARRAY_SIZE(usba_udc_ep);
-	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));;
+	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));
 
 	if (data && data->vbus_pin > 0) {
 		at91_set_gpio_input(data->vbus_pin, 0);
diff --git a/arch/arm/mach-at91/at91sam9rl_devices.c b/arch/arm/mach-at91/at91sam9rl_devices.c
index 53aaa94..c49262b 100644
--- a/arch/arm/mach-at91/at91sam9rl_devices.c
+++ b/arch/arm/mach-at91/at91sam9rl_devices.c
@@ -145,7 +145,7 @@ void __init at91_add_device_usba(struct usba_platform_data *data)
 	 */
 	usba_udc_data.pdata.vbus_pin = -EINVAL;
 	usba_udc_data.pdata.num_ep = ARRAY_SIZE(usba_udc_ep);
-	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));;
+	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));
 
 	if (data && data->vbus_pin > 0) {
 		at91_set_gpio_input(data->vbus_pin, 0);
diff --git a/arch/arm/mach-tegra/tegra2_clocks.c b/arch/arm/mach-tegra/tegra2_clocks.c
index 6d7c4ee..3b6f290 100644
--- a/arch/arm/mach-tegra/tegra2_clocks.c
+++ b/arch/arm/mach-tegra/tegra2_clocks.c
@@ -337,7 +337,7 @@ static int tegra2_super_clk_set_parent(struct clk *c, struct clk *p)
 	const struct clk_mux_sel *sel;
 	int shift;
 
-	val = clk_readl(c->reg + SUPER_CLK_MUX);;
+	val = clk_readl(c->reg + SUPER_CLK_MUX);
 	BUG_ON(((val & SUPER_STATE_MASK) != SUPER_STATE_RUN) &&
 		((val & SUPER_STATE_MASK) != SUPER_STATE_IDLE));
 	shift = ((val & SUPER_STATE_MASK) == SUPER_STATE_IDLE) ?
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 02/21]arch:ia64 Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 03/21]arch:um:drivers:mmapper_kern.c " Justin P. Mattock
                   ` (19 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 arch/ia64/include/asm/asmmacro.h |    4 ++--
 arch/ia64/include/asm/mca_asm.h  |    6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/ia64/include/asm/asmmacro.h b/arch/ia64/include/asm/asmmacro.h
index 3ab6d75..3c3904b 100644
--- a/arch/ia64/include/asm/asmmacro.h
+++ b/arch/ia64/include/asm/asmmacro.h
@@ -92,12 +92,12 @@ name:
 1:{ .mib;						\
 	nop.m 0;					\
 	mov r16=ar.pfs;					\
-	br.call.sptk.many b7=2f;;			\
+	br.call.sptk.many b7=2f;			\
   };							\
 2:{ .mib;						\
 	nop.m 0;					\
 	mov ar.pfs=r16;					\
-	br.ret.sptk.many b6;;				\
+	br.ret.sptk.many b6;				\
   }
 #else
 # define FSYS_RETURN	br.ret.sptk.many b6
diff --git a/arch/ia64/include/asm/mca_asm.h b/arch/ia64/include/asm/mca_asm.h
index dd2a5b1..d57d3c0 100644
--- a/arch/ia64/include/asm/mca_asm.h
+++ b/arch/ia64/include/asm/mca_asm.h
@@ -46,11 +46,11 @@
  *	1. Put 0x7 in bits 61 thru 63.
  */
 #define DATA_PA_TO_VA(addr,temp)							\
-	mov	temp	= 0x7	;;							\
+	mov	temp	= 0x7;							\
 	dep	addr	= temp, addr, 61, 3
 
 #define GET_THIS_PADDR(reg, var)		\
-	mov	reg = IA64_KR(PER_CPU_DATA);;	\
+	mov	reg = IA64_KR(PER_CPU_DATA);	\
         addl	reg = THIS_CPU(var), reg
 
 /*
@@ -82,7 +82,7 @@
 	;;										\
 	dep	old_psr = 0, old_psr, 32, 32;						\
 											\
-	mov	ar.rsc = 0 ;								\
+	mov	ar.rsc = 0;								\
 	;;										\
 	srlz.d;										\
 	mov	temp2 = ar.bspstore;							\
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 03/21]arch:um:drivers:mmapper_kern.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
  2011-03-24 19:26 ` [RFC 02/21]arch:ia64 " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 04/21]arch:s390:hypfs:hypfs.h " Justin P. Mattock
                   ` (18 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not. 

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 arch/um/drivers/mmapper_kern.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/um/drivers/mmapper_kern.c b/arch/um/drivers/mmapper_kern.c
index 7e0619c..c0ef803 100644
--- a/arch/um/drivers/mmapper_kern.c
+++ b/arch/um/drivers/mmapper_kern.c
@@ -116,7 +116,7 @@ static int __init mmapper_init(void)
 	if (err) {
 		printk(KERN_ERR "mmapper - misc_register failed, err = %d\n",
 		       err);
-		return err;;
+		return err;
 	}
 	return 0;
 }
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 04/21]arch:s390:hypfs:hypfs.h Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
  2011-03-24 19:26 ` [RFC 02/21]arch:ia64 " Justin P. Mattock
  2011-03-24 19:26 ` [RFC 03/21]arch:um:drivers:mmapper_kern.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 05/21]drivers:scsi " Justin P. Mattock
                   ` (17 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 arch/s390/hypfs/hypfs.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/s390/hypfs/hypfs.h b/arch/s390/hypfs/hypfs.h
index 80c1526..d9df5a0 100644
--- a/arch/s390/hypfs/hypfs.h
+++ b/arch/s390/hypfs/hypfs.h
@@ -47,7 +47,7 @@ struct hypfs_dbfs_data {
 	void			*buf;
 	void			*buf_free_ptr;
 	size_t			size;
-	struct hypfs_dbfs_file	*dbfs_file;;
+	struct hypfs_dbfs_file	*dbfs_file;
 	struct kref		kref;
 };
 
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 05/21]drivers:scsi Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (2 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 04/21]arch:s390:hypfs:hypfs.h " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 06/21]drivers:leds:leds-mc13783.c " Justin P. Mattock
                   ` (16 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/scsi/be2iscsi/be_main.c   |    4 ++--
 drivers/scsi/lpfc/lpfc_bsg.c      |    2 +-
 drivers/scsi/pm8001/pm8001_init.c |    2 +-
 drivers/scsi/qla2xxx/qla_isr.c    |    4 ++--
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/scsi/be2iscsi/be_main.c b/drivers/scsi/be2iscsi/be_main.c
index 24e20ba..44ac4ae 100644
--- a/drivers/scsi/be2iscsi/be_main.c
+++ b/drivers/scsi/be2iscsi/be_main.c
@@ -618,7 +618,7 @@ static void beiscsi_get_params(struct beiscsi_hba *phba)
 				    + BE2_NOPOUT_REQ));
 	phba->params.cxns_per_ctrl = phba->fw_config.iscsi_cid_count;
 	phba->params.asyncpdus_per_ctrl = phba->fw_config.iscsi_cid_count * 2;
-	phba->params.icds_per_ctrl = phba->fw_config.iscsi_icd_count;;
+	phba->params.icds_per_ctrl = phba->fw_config.iscsi_icd_count;
 	phba->params.num_sge_per_io = BE2_SGE;
 	phba->params.defpdu_hdr_sz = BE2_DEFPDU_HDR_SZ;
 	phba->params.defpdu_data_sz = BE2_DEFPDU_DATA_SZ;
@@ -781,7 +781,7 @@ static irqreturn_t be_isr(int irq, void *dev_id)
 	int isr;
 
 	phba = dev_id;
-	ctrl = &phba->ctrl;;
+	ctrl = &phba->ctrl;
 	isr = ioread32(ctrl->csr + CEV_ISR0_OFFSET +
 		       (PCI_FUNC(ctrl->pdev->devfn) * CEV_ISR_SIZE));
 	if (!isr)
diff --git a/drivers/scsi/lpfc/lpfc_bsg.c b/drivers/scsi/lpfc/lpfc_bsg.c
index 0dd43bb..04fef03 100644
--- a/drivers/scsi/lpfc/lpfc_bsg.c
+++ b/drivers/scsi/lpfc/lpfc_bsg.c
@@ -595,7 +595,7 @@ lpfc_bsg_rport_els(struct fc_bsg_job *job)
 	dd_data->context_un.iocb.cmdiocbq = cmdiocbq;
 	dd_data->context_un.iocb.rspiocbq = rspiocbq;
 	dd_data->context_un.iocb.set_job = job;
-	dd_data->context_un.iocb.bmp = NULL;;
+	dd_data->context_un.iocb.bmp = NULL;
 	dd_data->context_un.iocb.ndlp = ndlp;
 
 	if (phba->cfg_poll & DISABLE_FCP_RING_INT) {
diff --git a/drivers/scsi/pm8001/pm8001_init.c b/drivers/scsi/pm8001/pm8001_init.c
index 002360d..172cefb 100644
--- a/drivers/scsi/pm8001/pm8001_init.c
+++ b/drivers/scsi/pm8001/pm8001_init.c
@@ -160,7 +160,7 @@ static void pm8001_free(struct pm8001_hba_info *pm8001_ha)
 static void pm8001_tasklet(unsigned long opaque)
 {
 	struct pm8001_hba_info *pm8001_ha;
-	pm8001_ha = (struct pm8001_hba_info *)opaque;;
+	pm8001_ha = (struct pm8001_hba_info *)opaque;
 	if (unlikely(!pm8001_ha))
 		BUG_ON(1);
 	PM8001_CHIP_DISP->isr(pm8001_ha);
diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c
index d17ed9a..8b4d99a 100644
--- a/drivers/scsi/qla2xxx/qla_isr.c
+++ b/drivers/scsi/qla2xxx/qla_isr.c
@@ -1051,7 +1051,7 @@ qla2x00_ct_entry(scsi_qla_host_t *vha, struct req_que *req,
 		}
 		DEBUG2(qla2x00_dump_buffer((uint8_t *)pkt, sizeof(*pkt)));
 	} else {
-		bsg_job->reply->result =  DID_OK << 16;;
+		bsg_job->reply->result =  DID_OK << 16;
 		bsg_job->reply->reply_payload_rcv_len =
 		    bsg_job->reply_payload.payload_len;
 		bsg_job->reply_len = 0;
@@ -1146,7 +1146,7 @@ qla24xx_els_ct_entry(scsi_qla_host_t *vha, struct req_que *req,
 		DEBUG2(qla2x00_dump_buffer((uint8_t *)pkt, sizeof(*pkt)));
 	}
 	else {
-		bsg_job->reply->result =  DID_OK << 16;;
+		bsg_job->reply->result =  DID_OK << 16;
 		bsg_job->reply->reply_payload_rcv_len = bsg_job->reply_payload.payload_len;
 		bsg_job->reply_len = 0;
 	}
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 06/21]drivers:leds:leds-mc13783.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (3 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 05/21]drivers:scsi " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 07/21]drivers:xen " Justin P. Mattock
                   ` (15 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/leds/leds-mc13783.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/leds/leds-mc13783.c b/drivers/leds/leds-mc13783.c
index 06a5bb4..126ca79 100644
--- a/drivers/leds/leds-mc13783.c
+++ b/drivers/leds/leds-mc13783.c
@@ -235,7 +235,7 @@ static int __devinit mc13783_leds_prepare(struct platform_device *pdev)
 							MC13783_LED_Cx_PERIOD;
 
 	if (pdata->flags & MC13783_LED_TRIODE_TC3)
-		reg |= MC13783_LED_Cx_TRIODE_TC_BIT;;
+		reg |= MC13783_LED_Cx_TRIODE_TC_BIT;
 
 	ret = mc13783_reg_write(dev, MC13783_REG_LED_CONTROL_5, reg);
 	if (ret)
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 07/21]drivers:xen Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (4 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 06/21]drivers:leds:leds-mc13783.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-25 14:59   ` Konrad Rzeszutek Wilk
  2011-03-24 19:26 ` [RFC 08/21]drivers:tty:serial:mrst_max3110.c " Justin P. Mattock
                   ` (14 subsequent siblings)
  20 siblings, 1 reply; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/xen/evtchn.c      |    2 +-
 drivers/xen/swiotlb-xen.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/xen/evtchn.c b/drivers/xen/evtchn.c
index ef11daf..dbc13e9 100644
--- a/drivers/xen/evtchn.c
+++ b/drivers/xen/evtchn.c
@@ -470,7 +470,7 @@ static int evtchn_open(struct inode *inode, struct file *filp)
 
 	filp->private_data = u;
 
-	return nonseekable_open(inode, filp);;
+	return nonseekable_open(inode, filp);
 }
 
 static int evtchn_release(struct inode *inode, struct file *filp)
diff --git a/drivers/xen/swiotlb-xen.c b/drivers/xen/swiotlb-xen.c
index 54469c3..65ea21a 100644
--- a/drivers/xen/swiotlb-xen.c
+++ b/drivers/xen/swiotlb-xen.c
@@ -54,7 +54,7 @@ u64 start_dma_addr;
 
 static dma_addr_t xen_phys_to_bus(phys_addr_t paddr)
 {
-	return phys_to_machine(XPADDR(paddr)).maddr;;
+	return phys_to_machine(XPADDR(paddr)).maddr;
 }
 
 static phys_addr_t xen_bus_to_phys(dma_addr_t baddr)
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 08/21]drivers:tty:serial:mrst_max3110.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (5 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 07/21]drivers:xen " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 09/21]drivers:power:intel_mid_battery.c " Justin P. Mattock
                   ` (13 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/tty/serial/mrst_max3110.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/tty/serial/mrst_max3110.c b/drivers/tty/serial/mrst_max3110.c
index 37e13c3..8890612 100644
--- a/drivers/tty/serial/mrst_max3110.c
+++ b/drivers/tty/serial/mrst_max3110.c
@@ -56,7 +56,7 @@ struct uart_max3110 {
 	wait_queue_head_t wq;
 	struct task_struct *main_thread;
 	struct task_struct *read_thread;
-	struct mutex thread_mutex;;
+	struct mutex thread_mutex;
 
 	u32 baud;
 	u16 cur_conf;
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 09/21]drivers:power:intel_mid_battery.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (6 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 08/21]drivers:tty:serial:mrst_max3110.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 10/21]drivers:gpio:langwell_gpio.c " Justin P. Mattock
                   ` (12 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/power/intel_mid_battery.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/power/intel_mid_battery.c b/drivers/power/intel_mid_battery.c
index bce3a01..cffcb7c 100644
--- a/drivers/power/intel_mid_battery.c
+++ b/drivers/power/intel_mid_battery.c
@@ -522,7 +522,7 @@ static int pmic_battery_set_charger(struct pmic_power_module_info *pbi,
 	if (retval) {
 		dev_warn(pbi->dev, "%s(): ipc pmic read failed\n",
 								__func__);
-		return retval;;
+		return retval;
 	}
 
 	return 0;
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 10/21]drivers:gpio:langwell_gpio.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (7 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 09/21]drivers:power:intel_mid_battery.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 11/21]drivers:net " Justin P. Mattock
                   ` (11 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/gpio/langwell_gpio.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/gpio/langwell_gpio.c b/drivers/gpio/langwell_gpio.c
index 560ab64..1b06f67 100644
--- a/drivers/gpio/langwell_gpio.c
+++ b/drivers/gpio/langwell_gpio.c
@@ -122,7 +122,7 @@ static int lnw_gpio_direction_output(struct gpio_chip *chip,
 	lnw_gpio_set(chip, offset, value);
 	spin_lock_irqsave(&lnw->lock, flags);
 	value = readl(gpdr);
-	value |= BIT(offset % 32);;
+	value |= BIT(offset % 32);
 	writel(value, gpdr);
 	spin_unlock_irqrestore(&lnw->lock, flags);
 	return 0;
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 11/21]drivers:net Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (8 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 10/21]drivers:gpio:langwell_gpio.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 12/21]drivers:video:mxsfb.c " Justin P. Mattock
                   ` (10 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/net/can/softing/softing_main.c      |    2 +-
 drivers/net/wireless/ath/ath9k/phy.h        |    2 +-
 drivers/net/wireless/iwlwifi/iwl-agn.c      |    2 +-
 drivers/net/wireless/rtlwifi/core.c         |    2 +-
 drivers/net/wireless/rtlwifi/pci.c          |    2 +-
 drivers/net/wireless/rtlwifi/rtl8192ce/hw.c |    4 ++--
 6 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/net/can/softing/softing_main.c b/drivers/net/can/softing/softing_main.c
index aeea9f9..8d75953 100644
--- a/drivers/net/can/softing/softing_main.c
+++ b/drivers/net/can/softing/softing_main.c
@@ -797,7 +797,7 @@ static __devinit int softing_pdev_probe(struct platform_device *pdev)
 	ret = -EINVAL;
 	pres = platform_get_resource(pdev, IORESOURCE_MEM, 0);
 	if (!pres)
-		goto platform_resource_failed;;
+		goto platform_resource_failed;
 	card->dpram_phys = pres->start;
 	card->dpram_size = pres->end - pres->start + 1;
 	card->dpram = ioremap_nocache(card->dpram_phys, card->dpram_size);
diff --git a/drivers/net/wireless/ath/ath9k/phy.h b/drivers/net/wireless/ath/ath9k/phy.h
index 5e3d749..3a7cd75 100644
--- a/drivers/net/wireless/ath/ath9k/phy.h
+++ b/drivers/net/wireless/ath/ath9k/phy.h
@@ -54,7 +54,7 @@
 #define RF_BANK_SETUP(_bank, _iniarray, _col) do {			\
 		int i;							\
 		for (i = 0; i < (_iniarray)->ia_rows; i++)		\
-			(_bank)[i] = INI_RA((_iniarray), i, _col);;	\
+			(_bank)[i] = INI_RA((_iniarray), i, _col);	\
 	} while (0)
 
 #define	AR_PHY_TIMING11_SPUR_FREQ_SD		0x3FF00000
diff --git a/drivers/net/wireless/iwlwifi/iwl-agn.c b/drivers/net/wireless/iwlwifi/iwl-agn.c
index 581dc9f..31233a3 100644
--- a/drivers/net/wireless/iwlwifi/iwl-agn.c
+++ b/drivers/net/wireless/iwlwifi/iwl-agn.c
@@ -2062,7 +2062,7 @@ static const char *desc_lookup(u32 num)
 	max = ARRAY_SIZE(advanced_lookup) - 1;
 	for (i = 0; i < max; i++) {
 		if (advanced_lookup[i].num == num)
-			break;;
+			break;
 	}
 	return advanced_lookup[i].name;
 }
diff --git a/drivers/net/wireless/rtlwifi/core.c b/drivers/net/wireless/rtlwifi/core.c
index e4f4aee..d52fa27 100644
--- a/drivers/net/wireless/rtlwifi/core.c
+++ b/drivers/net/wireless/rtlwifi/core.c
@@ -719,7 +719,7 @@ static void rtl_op_set_tsf(struct ieee80211_hw *hw, u64 tsf)
 {
 	struct rtl_priv *rtlpriv = rtl_priv(hw);
 	struct rtl_mac *mac = rtl_mac(rtl_priv(hw));
-	u8 bibss = (mac->opmode == NL80211_IFTYPE_ADHOC) ? 1 : 0;;
+	u8 bibss = (mac->opmode == NL80211_IFTYPE_ADHOC) ? 1 : 0;
 
 	mac->tsf = tsf;
 	rtlpriv->cfg->ops->set_hw_reg(hw, HW_VAR_CORRECT_TSF, (u8 *) (&bibss));
diff --git a/drivers/net/wireless/rtlwifi/pci.c b/drivers/net/wireless/rtlwifi/pci.c
index 9cd7703..baa2efe 100644
--- a/drivers/net/wireless/rtlwifi/pci.c
+++ b/drivers/net/wireless/rtlwifi/pci.c
@@ -1226,7 +1226,7 @@ static unsigned int _rtl_mac_to_hwqueue(__le16 fc,
 		hw_queue_index = VI_QUEUE;
 		break;
 	case 2:
-		hw_queue_index = BE_QUEUE;;
+		hw_queue_index = BE_QUEUE;
 		break;
 	case 3:
 		hw_queue_index = BK_QUEUE;
diff --git a/drivers/net/wireless/rtlwifi/rtl8192ce/hw.c b/drivers/net/wireless/rtlwifi/rtl8192ce/hw.c
index 05477f4..7432c85 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192ce/hw.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192ce/hw.c
@@ -729,7 +729,7 @@ static bool _rtl92ce_init_mac(struct ieee80211_hw *hw)
 	rtl_write_word(rtlpriv, REG_CR, 0x2ff);
 
 	if (_rtl92ce_llt_table_init(hw) == false)
-		return false;;
+		return false;
 
 	rtl_write_dword(rtlpriv, REG_HISR, 0xffffffff);
 	rtl_write_byte(rtlpriv, REG_HISRE, 0xff);
@@ -786,7 +786,7 @@ static bool _rtl92ce_init_mac(struct ieee80211_hw *hw)
 
 	rtl_write_dword(rtlpriv, REG_MCUTST_1, 0x0);
 
-	return true;;
+	return true;
 }
 
 static void _rtl92ce_hw_configure(struct ieee80211_hw *hw)
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 12/21]drivers:video:mxsfb.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (9 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 11/21]drivers:net " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 13/21]drivers:i2c:busses:i2c-designware.c " Justin P. Mattock
                   ` (9 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/video/mxsfb.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/video/mxsfb.c b/drivers/video/mxsfb.c
index 7d02848..0b2f2dd 100644
--- a/drivers/video/mxsfb.c
+++ b/drivers/video/mxsfb.c
@@ -401,7 +401,7 @@ static int mxsfb_set_par(struct fb_info *fb_info)
 	writel(CTRL1_FIFO_CLEAR, host->base + LCDC_CTRL1 + REG_SET);
 
 	ctrl = CTRL_BYPASS_COUNT | CTRL_MASTER |
-		CTRL_SET_BUS_WIDTH(host->ld_intf_width);;
+		CTRL_SET_BUS_WIDTH(host->ld_intf_width);
 
 	switch (fb_info->var.bits_per_pixel) {
 	case 16:
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 13/21]drivers:i2c:busses:i2c-designware.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (10 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 12/21]drivers:video:mxsfb.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 14/21]drivers:staging " Justin P. Mattock
                   ` (8 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/i2c/busses/i2c-designware.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/i2c/busses/i2c-designware.c b/drivers/i2c/busses/i2c-designware.c
index b664ed8..a93922d 100644
--- a/drivers/i2c/busses/i2c-designware.c
+++ b/drivers/i2c/busses/i2c-designware.c
@@ -390,7 +390,7 @@ i2c_dw_xfer_msg(struct dw_i2c_dev *dev)
 	int tx_limit, rx_limit;
 	u32 addr = msgs[dev->msg_write_idx].addr;
 	u32 buf_len = dev->tx_buf_len;
-	u8 *buf = dev->tx_buf;;
+	u8 *buf = dev->tx_buf;
 
 	intr_mask = DW_IC_INTR_DEFAULT_MASK;
 
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 14/21]drivers:staging Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (11 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 13/21]drivers:i2c:busses:i2c-designware.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 23:20   ` Alan Cox
  2011-03-24 19:26 ` [RFC 15/21]drivers:target " Justin P. Mattock
                   ` (7 subsequent siblings)
  20 siblings, 1 reply; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/gma500/psb_buffer.c     |    4 ++--
 drivers/staging/gma500/psb_fb.c         |    2 +-
 drivers/staging/iio/addac/adt7316-i2c.c |    2 +-
 drivers/staging/iio/dds/ad9832.c        |    2 +-
 drivers/staging/iio/dds/ad9834.c        |    2 +-
 drivers/staging/rt2860/rtmp.h           |    2 +-
 6 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/gma500/psb_buffer.c b/drivers/staging/gma500/psb_buffer.c
index 3077f6a..4f60850 100644
--- a/drivers/staging/gma500/psb_buffer.c
+++ b/drivers/staging/gma500/psb_buffer.c
@@ -384,12 +384,12 @@ static int psb_ttm_io_mem_reserve(struct ttm_bo_device *bdev,
 		break;
 	case TTM_PL_CI:
 		mem->bus.offset = mm_node->start << PAGE_SHIFT;
-		mem->bus.base = dev_priv->ci_region_start;;
+		mem->bus.base = dev_priv->ci_region_start;
 		mem->bus.is_iomem = true;
 		break;
 	case TTM_PL_RAR:
 		mem->bus.offset = mm_node->start << PAGE_SHIFT;
-		mem->bus.base = dev_priv->rar_region_start;;
+		mem->bus.base = dev_priv->rar_region_start;
 		mem->bus.is_iomem = true;
 		break;
 	default:
diff --git a/drivers/staging/gma500/psb_fb.c b/drivers/staging/gma500/psb_fb.c
index f67f53b..6c2b06e 100644
--- a/drivers/staging/gma500/psb_fb.c
+++ b/drivers/staging/gma500/psb_fb.c
@@ -194,7 +194,7 @@ static int psbfb_vm_fault(struct vm_area_struct *vma, struct vm_fault *vmf)
 	struct drm_device *dev = psbfb->base.dev;
 	struct drm_psb_private *dev_priv = dev->dev_private;
 	struct psb_gtt *pg = dev_priv->pg;
-	unsigned long phys_addr = (unsigned long)pg->stolen_base;;
+	unsigned long phys_addr = (unsigned long)pg->stolen_base;
 
 	page_num = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
 
diff --git a/drivers/staging/iio/addac/adt7316-i2c.c b/drivers/staging/iio/addac/adt7316-i2c.c
index 52d1ea3..1c86cf1 100644
--- a/drivers/staging/iio/addac/adt7316-i2c.c
+++ b/drivers/staging/iio/addac/adt7316-i2c.c
@@ -109,7 +109,7 @@ static int __devinit adt7316_i2c_probe(struct i2c_client *client,
 
 static int __devexit adt7316_i2c_remove(struct i2c_client *client)
 {
-	return adt7316_remove(&client->dev);;
+	return adt7316_remove(&client->dev);
 }
 
 static const struct i2c_device_id adt7316_i2c_id[] = {
diff --git a/drivers/staging/iio/dds/ad9832.c b/drivers/staging/iio/dds/ad9832.c
index 3e8491f..554441d 100644
--- a/drivers/staging/iio/dds/ad9832.c
+++ b/drivers/staging/iio/dds/ad9832.c
@@ -52,7 +52,7 @@ static int ad9832_write_frequency(struct ad9832_state *st,
 					((addr - 3) << ADD_SHIFT) |
 					((regval >> 0) & 0xFF));
 
-	return spi_sync(st->spi, &st->freq_msg);;
+	return spi_sync(st->spi, &st->freq_msg);
 }
 
 static int ad9832_write_phase(struct ad9832_state *st,
diff --git a/drivers/staging/iio/dds/ad9834.c b/drivers/staging/iio/dds/ad9834.c
index eb1a681..5086f9d 100644
--- a/drivers/staging/iio/dds/ad9834.c
+++ b/drivers/staging/iio/dds/ad9834.c
@@ -47,7 +47,7 @@ static int ad9834_write_frequency(struct ad9834_state *st,
 				       (AD9834_FREQ_BITS / 2)) &
 				       RES_MASK(AD9834_FREQ_BITS / 2)));
 
-	return spi_sync(st->spi, &st->freq_msg);;
+	return spi_sync(st->spi, &st->freq_msg);
 }
 
 static int ad9834_write_phase(struct ad9834_state *st,
diff --git a/drivers/staging/rt2860/rtmp.h b/drivers/staging/rt2860/rtmp.h
index d16b06a..afdf0bf 100644
--- a/drivers/staging/rt2860/rtmp.h
+++ b/drivers/staging/rt2860/rtmp.h
@@ -1683,7 +1683,7 @@ struct rt_rtmp_chip_op {
 	/* MCU related callback functions */
 	int (*loadFirmware) (struct rt_rtmp_adapter *pAd);	/* int (*loadFirmware)(struct rt_rtmp_adapter *pAd); */
 	int (*eraseFirmware) (struct rt_rtmp_adapter *pAd);	/* int (*eraseFirmware)(struct rt_rtmp_adapter *pAd); */
-	int (*sendCommandToMcu) (struct rt_rtmp_adapter *pAd, u8 cmd, u8 token, u8 arg0, u8 arg1);;	/* int (*sendCommandToMcu)(struct rt_rtmp_adapter *pAd, u8 cmd, u8 token, u8 arg0, u8 arg1); */
+	int (*sendCommandToMcu) (struct rt_rtmp_adapter *pAd, u8 cmd, u8 token, u8 arg0, u8 arg1);	/* int (*sendCommandToMcu)(struct rt_rtmp_adapter *pAd, u8 cmd, u8 token, u8 arg0, u8 arg1); */
 
 	/* RF access related callback functions */
 	struct rt_reg_pair *pRFRegTable;
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 15/21]drivers:target Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (12 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 14/21]drivers:staging " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 16/21]fs:logfs " Justin P. Mattock
                   ` (6 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/target/target_core_alua.c      |    4 ++--
 drivers/target/target_core_transport.c |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/target/target_core_alua.c b/drivers/target/target_core_alua.c
index 2c5fcfe..e7dddaf 100644
--- a/drivers/target/target_core_alua.c
+++ b/drivers/target/target_core_alua.c
@@ -1036,7 +1036,7 @@ core_alua_allocate_lu_gp(const char *name, int def_group)
 	lu_gp = kmem_cache_zalloc(t10_alua_lu_gp_cache, GFP_KERNEL);
 	if (!(lu_gp)) {
 		printk(KERN_ERR "Unable to allocate struct t10_alua_lu_gp\n");
-		return ERR_PTR(-ENOMEM);;
+		return ERR_PTR(-ENOMEM);
 	}
 	INIT_LIST_HEAD(&lu_gp->lu_gp_list);
 	INIT_LIST_HEAD(&lu_gp->lu_gp_mem_list);
@@ -1044,7 +1044,7 @@ core_alua_allocate_lu_gp(const char *name, int def_group)
 	atomic_set(&lu_gp->lu_gp_ref_cnt, 0);
 
 	if (def_group) {
-		lu_gp->lu_gp_id = se_global->alua_lu_gps_counter++;;
+		lu_gp->lu_gp_id = se_global->alua_lu_gps_counter++;
 		lu_gp->lu_gp_valid_id = 1;
 		se_global->alua_lu_gps_count++;
 	}
diff --git a/drivers/target/target_core_transport.c b/drivers/target/target_core_transport.c
index ff9ace0..39ac190 100644
--- a/drivers/target/target_core_transport.c
+++ b/drivers/target/target_core_transport.c
@@ -2131,7 +2131,7 @@ static void transport_failure_reset_queue_depth(struct se_device *dev)
 {
 	unsigned long flags;
 
-	spin_lock_irqsave(&SE_HBA(dev)->hba_queue_lock, flags);;
+	spin_lock_irqsave(&SE_HBA(dev)->hba_queue_lock, flags);
 	atomic_inc(&dev->depth_left);
 	atomic_inc(&SE_HBA(dev)->left_queue_depth);
 	spin_unlock_irqrestore(&SE_HBA(dev)->hba_queue_lock, flags);
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 16/21]fs:logfs Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (13 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 15/21]drivers:target " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 17/21]kernel:pm_qos_params.c " Justin P. Mattock
                   ` (5 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 fs/logfs/readwrite.c    |    2 +-
 fs/ocfs2/refcounttree.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs/logfs/readwrite.c b/fs/logfs/readwrite.c
index ee99a9f..3dcb3a6 100644
--- a/fs/logfs/readwrite.c
+++ b/fs/logfs/readwrite.c
@@ -481,7 +481,7 @@ static int inode_write_alias(struct super_block *sb,
 			val = inode_val0(inode);
 			break;
 		case INODE_USED_OFS:
-			val = cpu_to_be64(li->li_used_bytes);;
+			val = cpu_to_be64(li->li_used_bytes);
 			break;
 		case INODE_SIZE_OFS:
 			val = cpu_to_be64(i_size_read(inode));
diff --git a/fs/ocfs2/refcounttree.c b/fs/ocfs2/refcounttree.c
index c384d63..c4feced 100644
--- a/fs/ocfs2/refcounttree.c
+++ b/fs/ocfs2/refcounttree.c
@@ -3707,7 +3707,7 @@ int ocfs2_refcount_cow_xattr(struct inode *inode,
 	context->cow_start = cow_start;
 	context->cow_len = cow_len;
 	context->ref_tree = ref_tree;
-	context->ref_root_bh = ref_root_bh;;
+	context->ref_root_bh = ref_root_bh;
 	context->cow_object = xv;
 
 	context->cow_duplicate_clusters = ocfs2_duplicate_clusters_by_jbd;
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 17/21]kernel:pm_qos_params.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (14 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 16/21]fs:logfs " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 18/21]mm:hugetlb.c " Justin P. Mattock
                   ` (4 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 kernel/pm_qos_params.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/kernel/pm_qos_params.c b/kernel/pm_qos_params.c
index 0da058b..beb1846 100644
--- a/kernel/pm_qos_params.c
+++ b/kernel/pm_qos_params.c
@@ -385,7 +385,7 @@ static ssize_t pm_qos_power_read(struct file *filp, char __user *buf,
 	s32 value;
 	unsigned long flags;
 	struct pm_qos_object *o;
-	struct pm_qos_request_list *pm_qos_req = filp->private_data;;
+	struct pm_qos_request_list *pm_qos_req = filp->private_data;
 
 	if (!pm_qos_req)
 		return -EINVAL;
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 18/21]mm:hugetlb.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (15 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 17/21]kernel:pm_qos_params.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 19/21]net remove " Justin P. Mattock
                   ` (3 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 mm/hugetlb.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/mm/hugetlb.c b/mm/hugetlb.c
index 06de5aa..51e49fe 100644
--- a/mm/hugetlb.c
+++ b/mm/hugetlb.c
@@ -475,7 +475,7 @@ static struct page *dequeue_huge_page_vma(struct hstate *h,
 
 	/* If reserves cannot be used, ensure enough pages are in the pool */
 	if (avoid_reserve && h->free_huge_pages - h->resv_huge_pages == 0)
-		goto err;;
+		goto err;
 
 	for_each_zone_zonelist_nodemask(zone, z, zonelist,
 						MAX_NR_ZONES - 1, nodemask) {
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 19/21]net remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (16 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 18/21]mm:hugetlb.c " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 20/21]security:selinux:selinuxfs.c Remove " Justin P. Mattock
                   ` (2 subsequent siblings)
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 net/netfilter/ipset/ip_set_bitmap_ip.c |    2 +-
 net/sunrpc/addr.c                      |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/netfilter/ipset/ip_set_bitmap_ip.c b/net/netfilter/ipset/ip_set_bitmap_ip.c
index bca9699..cc29922 100644
--- a/net/netfilter/ipset/ip_set_bitmap_ip.c
+++ b/net/netfilter/ipset/ip_set_bitmap_ip.c
@@ -293,7 +293,7 @@ bitmap_ip_uadt(struct ip_set *set, struct nlattr *tb[],
 
 	for (; !before(ip_to, ip); ip += map->hosts) {
 		id = ip_to_id(map, ip);
-		ret = adtfn(set, &id, timeout);;
+		ret = adtfn(set, &id, timeout);
 
 		if (ret && !ip_set_eexist(ret, flags))
 			return ret;
diff --git a/net/sunrpc/addr.c b/net/sunrpc/addr.c
index 1419d0c..4195233 100644
--- a/net/sunrpc/addr.c
+++ b/net/sunrpc/addr.c
@@ -151,7 +151,7 @@ static size_t rpc_pton4(const char *buf, const size_t buflen,
 		return 0;
 
 	sin->sin_family = AF_INET;
-	return sizeof(struct sockaddr_in);;
+	return sizeof(struct sockaddr_in);
 }
 
 #if defined(CONFIG_IPV6) || defined(CONFIG_IPV6_MODULE)
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 20/21]security:selinux:selinuxfs.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (17 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 19/21]net remove " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 19:26 ` [RFC 21/21]tools:power:x86:turbostat:turbostat.c " Justin P. Mattock
  2011-03-24 20:16 ` [RFC 01/21]arch:arm:mach " Nick Bowler
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 security/selinux/selinuxfs.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/security/selinux/selinuxfs.c b/security/selinux/selinuxfs.c
index ea39cb7..47b7d62 100644
--- a/security/selinux/selinuxfs.c
+++ b/security/selinux/selinuxfs.c
@@ -280,7 +280,7 @@ static ssize_t sel_write_disable(struct file *file, const char __user *buf,
 
 	length = -ENOMEM;
 	if (count >= PAGE_SIZE)
-		goto out;;
+		goto out;
 
 	/* No partial writes. */
 	length = -EINVAL;
@@ -876,12 +876,12 @@ static ssize_t sel_write_user(struct file *file, char *buf, size_t size)
 
 	length = task_has_security(current, SECURITY__COMPUTE_USER);
 	if (length)
-		goto out;;
+		goto out;
 
 	length = -ENOMEM;
 	con = kzalloc(size + 1, GFP_KERNEL);
 	if (!con)
-		goto out;;
+		goto out;
 
 	length = -ENOMEM;
 	user = kzalloc(size + 1, GFP_KERNEL);
@@ -941,7 +941,7 @@ static ssize_t sel_write_member(struct file *file, char *buf, size_t size)
 	length = -ENOMEM;
 	scon = kzalloc(size + 1, GFP_KERNEL);
 	if (!scon)
-		goto out;;
+		goto out;
 
 	length = -ENOMEM;
 	tcon = kzalloc(size + 1, GFP_KERNEL);
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* [RFC 21/21]tools:power:x86:turbostat:turbostat.c Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (18 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 20/21]security:selinux:selinuxfs.c Remove " Justin P. Mattock
@ 2011-03-24 19:26 ` Justin P. Mattock
  2011-03-24 20:16 ` [RFC 01/21]arch:arm:mach " Nick Bowler
  20 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 19:26 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock

The patch below removes an extra comma from various parts 
of the kernel. Please have a look when you have time, and let
me know if its legit or not.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 tools/power/x86/turbostat/turbostat.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c
index 362a0cb..6d8ef4a 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -990,7 +990,7 @@ int fork_it(char **argv)
 	if (!retval)
 		print_counters(cnt_delta);
 
-	fprintf(stderr, "%.6f sec\n", tv_delta.tv_sec + tv_delta.tv_usec/1000000.0);;
+	fprintf(stderr, "%.6f sec\n", tv_delta.tv_sec + tv_delta.tv_usec/1000000.0);
 
 	return 0;
 }
-- 
1.7.4.1


^ permalink raw reply related	[flat|nested] 26+ messages in thread

* Re: [RFC 01/21]arch:arm:mach Remove extra comma.
  2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
                   ` (19 preceding siblings ...)
  2011-03-24 19:26 ` [RFC 21/21]tools:power:x86:turbostat:turbostat.c " Justin P. Mattock
@ 2011-03-24 20:16 ` Nick Bowler
  2011-03-24 20:17   ` Justin P. Mattock
  20 siblings, 1 reply; 26+ messages in thread
From: Nick Bowler @ 2011-03-24 20:16 UTC (permalink / raw)
  To: Justin P. Mattock; +Cc: trivial, linux-kernel

On 2011-03-24 12:26 -0700, Justin P. Mattock wrote:
> The patch below removes an extra comma from various parts 
> of the kernel. Please have a look when you have time, and let
> me know if its legit or not. 

You say here that you are removing commas...

> diff --git a/arch/arm/mach-at91/at91cap9_devices.c b/arch/arm/mach-at91/at91cap9_devices.c
[...]
> -	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));;
> +	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));

...yet you are actually removing semicolons.

-- 
Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [RFC 01/21]arch:arm:mach Remove extra comma.
  2011-03-24 20:16 ` [RFC 01/21]arch:arm:mach " Nick Bowler
@ 2011-03-24 20:17   ` Justin P. Mattock
  0 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 20:17 UTC (permalink / raw)
  To: Nick Bowler; +Cc: trivial, linux-kernel

On 03/24/2011 01:16 PM, Nick Bowler wrote:
> On 2011-03-24 12:26 -0700, Justin P. Mattock wrote:
>> The patch below removes an extra comma from various parts
>> of the kernel. Please have a look when you have time, and let
>> me know if its legit or not.
>
> You say here that you are removing commas...

I know!! just got an email from randy dunlop on that..(pretty bad) I am 
resending..

>
>> diff --git a/arch/arm/mach-at91/at91cap9_devices.c b/arch/arm/mach-at91/at91cap9_devices.c
> [...]
>> -	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));;
>> +	memcpy(usba_udc_data.ep, usba_udc_ep, sizeof(usba_udc_ep));
>
> ...yet you are actually removing semicolons.
>


^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [RFC 14/21]drivers:staging Remove extra comma.
  2011-03-24 19:26 ` [RFC 14/21]drivers:staging " Justin P. Mattock
@ 2011-03-24 23:20   ` Alan Cox
  2011-03-24 23:26     ` Justin P. Mattock
  0 siblings, 1 reply; 26+ messages in thread
From: Alan Cox @ 2011-03-24 23:20 UTC (permalink / raw)
  To: Justin P. Mattock; +Cc: trivial, linux-kernel

On Thu, 24 Mar 2011 12:26:52 -0700
"Justin P. Mattock" <justinmattock@gmail.com> wrote:

> The patch below removes an extra comma from various parts 
> of the kernel. Please have a look when you have time, and let
> me know if its legit or not.
> 
> Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

Can you split the gma500 one out and send it my way instead of via
trivial/Greg otherwise it'll collide with some of the big cleanup work
going on I fear.

And ack for the 3 intel ones (langwell_gpio, max3110, intel_mid_battery)
to go in directly.

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [RFC 14/21]drivers:staging Remove extra comma.
  2011-03-24 23:20   ` Alan Cox
@ 2011-03-24 23:26     ` Justin P. Mattock
  0 siblings, 0 replies; 26+ messages in thread
From: Justin P. Mattock @ 2011-03-24 23:26 UTC (permalink / raw)
  To: Alan Cox; +Cc: trivial, linux-kernel

On 03/24/2011 04:20 PM, Alan Cox wrote:
> On Thu, 24 Mar 2011 12:26:52 -0700
> "Justin P. Mattock"<justinmattock@gmail.com>  wrote:
>
>> The patch below removes an extra comma from various parts
>> of the kernel. Please have a look when you have time, and let
>> me know if its legit or not.
>>
>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com>
>
> Can you split the gma500 one out and send it my way instead of via
> trivial/Greg otherwise it'll collide with some of the big cleanup work
> going on I fear.
>
> And ack for the 3 intel ones (langwell_gpio, max3110, intel_mid_battery)
> to go in directly.
>

sure... finishing up on some SELinux stuff, then I will resend for you..

Justin P. Mattock

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [RFC 07/21]drivers:xen Remove extra comma.
  2011-03-24 19:26 ` [RFC 07/21]drivers:xen " Justin P. Mattock
@ 2011-03-25 14:59   ` Konrad Rzeszutek Wilk
  0 siblings, 0 replies; 26+ messages in thread
From: Konrad Rzeszutek Wilk @ 2011-03-25 14:59 UTC (permalink / raw)
  To: Justin P. Mattock; +Cc: trivial, linux-kernel

On Thu, Mar 24, 2011 at 12:26:45PM -0700, Justin P. Mattock wrote:
> The patch below removes an extra comma from various parts 
> of the kernel. Please have a look when you have time, and let
> me know if its legit or not.

Looks ok.
> 
> Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
> 
> ---
>  drivers/xen/evtchn.c      |    2 +-
>  drivers/xen/swiotlb-xen.c |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/xen/evtchn.c b/drivers/xen/evtchn.c
> index ef11daf..dbc13e9 100644
> --- a/drivers/xen/evtchn.c
> +++ b/drivers/xen/evtchn.c
> @@ -470,7 +470,7 @@ static int evtchn_open(struct inode *inode, struct file *filp)
>  
>  	filp->private_data = u;
>  
> -	return nonseekable_open(inode, filp);;
> +	return nonseekable_open(inode, filp);
>  }
>  
>  static int evtchn_release(struct inode *inode, struct file *filp)
> diff --git a/drivers/xen/swiotlb-xen.c b/drivers/xen/swiotlb-xen.c
> index 54469c3..65ea21a 100644
> --- a/drivers/xen/swiotlb-xen.c
> +++ b/drivers/xen/swiotlb-xen.c
> @@ -54,7 +54,7 @@ u64 start_dma_addr;
>  
>  static dma_addr_t xen_phys_to_bus(phys_addr_t paddr)
>  {
> -	return phys_to_machine(XPADDR(paddr)).maddr;;
> +	return phys_to_machine(XPADDR(paddr)).maddr;
>  }
>  
>  static phys_addr_t xen_bus_to_phys(dma_addr_t baddr)
> -- 
> 1.7.4.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

^ permalink raw reply	[flat|nested] 26+ messages in thread

end of thread, other threads:[~2011-03-25 15:00 UTC | newest]

Thread overview: 26+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-24 19:26 [RFC 01/21]arch:arm:mach Remove extra comma Justin P. Mattock
2011-03-24 19:26 ` [RFC 02/21]arch:ia64 " Justin P. Mattock
2011-03-24 19:26 ` [RFC 03/21]arch:um:drivers:mmapper_kern.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 04/21]arch:s390:hypfs:hypfs.h " Justin P. Mattock
2011-03-24 19:26 ` [RFC 05/21]drivers:scsi " Justin P. Mattock
2011-03-24 19:26 ` [RFC 06/21]drivers:leds:leds-mc13783.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 07/21]drivers:xen " Justin P. Mattock
2011-03-25 14:59   ` Konrad Rzeszutek Wilk
2011-03-24 19:26 ` [RFC 08/21]drivers:tty:serial:mrst_max3110.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 09/21]drivers:power:intel_mid_battery.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 10/21]drivers:gpio:langwell_gpio.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 11/21]drivers:net " Justin P. Mattock
2011-03-24 19:26 ` [RFC 12/21]drivers:video:mxsfb.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 13/21]drivers:i2c:busses:i2c-designware.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 14/21]drivers:staging " Justin P. Mattock
2011-03-24 23:20   ` Alan Cox
2011-03-24 23:26     ` Justin P. Mattock
2011-03-24 19:26 ` [RFC 15/21]drivers:target " Justin P. Mattock
2011-03-24 19:26 ` [RFC 16/21]fs:logfs " Justin P. Mattock
2011-03-24 19:26 ` [RFC 17/21]kernel:pm_qos_params.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 18/21]mm:hugetlb.c " Justin P. Mattock
2011-03-24 19:26 ` [RFC 19/21]net remove " Justin P. Mattock
2011-03-24 19:26 ` [RFC 20/21]security:selinux:selinuxfs.c Remove " Justin P. Mattock
2011-03-24 19:26 ` [RFC 21/21]tools:power:x86:turbostat:turbostat.c " Justin P. Mattock
2011-03-24 20:16 ` [RFC 01/21]arch:arm:mach " Nick Bowler
2011-03-24 20:17   ` Justin P. Mattock

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox