* [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* 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
* [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* 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
* [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