-lm3533.o
File size after:
text data bss dec hex filename
8368 4512 64 129443290 drivers/leds/leds-lm3533.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/leds/leds-lm3533.c | 2 +-
-blinkm.o
File size after:
text data bss dec hex filename
8753 3400 64 122172fb9 drivers/leds/leds-blinkm.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/leds/leds-blinkm.c | 2 +-
-blinkm.o
File size after:
text data bss dec hex filename
8753 3400 64 122172fb9 drivers/leds/leds-blinkm.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/leds/leds-blinkm.c | 2 +-
1 file changed, 1 insertion(+), 1
-lm3533.o
File size after:
text data bss dec hex filename
8368 4512 64 129443290 drivers/leds/leds-lm3533.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/leds/leds-lm3533.c | 2 +-
1 file changed, 1 insertion(+), 1
File size after:
text data bss dec hex filename
3717316584 960 54717d5bd drivers/media/rc/imon.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/media/rc/imon.c | 4 ++--
1 file chan
File size after:
text data bss dec hex filename
3717316584 960 54717d5bd drivers/media/rc/imon.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/media/rc/imon.c | 4 ++--
1 file changed, 2 insertions(+), 2
/switch.o
File size after:
text data bss dec hex filename
28661 7204 0 358658c19 drivers/thunderbolt/switch.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/thunderbolt/switch.c | 2 +-
/switch.o
File size after:
text data bss dec hex filename
28661 7204 0 358658c19 drivers/thunderbolt/switch.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/thunderbolt/switch.c | 2 +-
1 file changed, 1 insertion
File size after:
text data bss dec hex filename
28320 5421 480 3422185ad drivers/block/loop.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/block/loop.c | 2 +-
1 file changed, 1 ins
File size after:
text data bss dec hex filename
28320 5421 480 3422185ad drivers/block/loop.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/block/loop.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
File size after:
text data bss dec hex filename
7998 3432 64 114942ce6 drivers/acpi/dock.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/acpi/dock.c | 2 +-
1 file changed, 1 insertion
File size after:
text data bss dec hex filename
7998 3432 64 114942ce6 drivers/acpi/dock.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/acpi/dock.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
File size after:
text data bss dec hex filename
3593228232 832 64996fde4 drivers/atm/solos-pci.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/atm/solos-pci.c | 4 ++--
1 file chan
File size after:
text data bss dec hex filename
3593228232 832 64996fde4 drivers/atm/solos-pci.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/atm/solos-pci.c | 4 ++--
1 file changed, 2 insertions(+), 2
File size after:
text data bss dec hex filename
2129 1352 03481 d99 drivers/atm/adummy.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/atm/adummy.c | 2 +-
1 file changed, 1 ins
File size after:
text data bss dec hex filename
2129 1352 03481 d99 drivers/atm/adummy.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/atm/adummy.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
platform_device *pdev;
@@
- platform_set_drvdata(pdev, NULL);
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/staging/greybus/arche-platform.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/greybus/arche-platform.c
b/drivers/staging/greybus
platform_device *pdev;
@@
- platform_set_drvdata(pdev, NULL);
//
Signed-off-by: Amitoj Kaur Chawla
---
drivers/staging/greybus/arche-platform.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/greybus/arche-platform.c
b/drivers/staging/greybus/arche-platform.c
index eced2d2
/visorbus/visorchipset.o
File size after:
text data bss dec hex filename
242206120 448 307887844
drivers/staging/unisys/visorbus/visorchipset.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
d
/visorbus/visorchipset.o
File size after:
text data bss dec hex filename
242206120 448 307887844
drivers/staging/unisys/visorbus/visorchipset.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/staging/unisys/visorbus
/visorbus/visorbus_main.o
File size after:
text data bss dec hex filename
144087112 832 223525750
drivers/staging/unisys/visorbus/visorbus_main.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
d
/visorbus/visorbus_main.o
File size after:
text data bss dec hex filename
144087112 832 223525750
drivers/staging/unisys/visorbus/visorbus_main.o
This change was made with the help of Coccinelle.
Signed-off-by: Amitoj Kaur Chawla
---
drivers/staging/unisys/visorbus
:
//
@@
expression a;
@@
- if(...) { vfree(a); }
- else { kfree(a); }
+ kvfree(a);
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
Changes in v2:
-Remove wrapper function
.../staging/media/atomisp/pci/atomisp2/atomisp_cmd.c | 19 +++
.../staging/media/a
:
//
@@
expression a;
@@
- if(...) { vfree(a); }
- else { kfree(a); }
+ kvfree(a);
//
Signed-off-by: Amitoj Kaur Chawla
---
Changes in v2:
-Remove wrapper function
.../staging/media/atomisp/pci/atomisp2/atomisp_cmd.c | 19 +++
.../staging/media/atomisp/pci/atomisp2
On Sat, Jul 8, 2017 at 4:58 AM, Bernd Petrovitsch
<be...@petrovitsch.priv.at> wrote:
> On Fri, 2017-07-07 at 20:41 -0400, Amitoj Kaur Chawla wrote:
> [...]
>> --- a/drivers/staging/media/atomisp/pci/atomisp2/atomisp_cmd.c
>> +++ b/drivers/staging/media/atomisp/pc
On Sat, Jul 8, 2017 at 4:58 AM, Bernd Petrovitsch
wrote:
> On Fri, 2017-07-07 at 20:41 -0400, Amitoj Kaur Chawla wrote:
> [...]
>> --- a/drivers/staging/media/atomisp/pci/atomisp2/atomisp_cmd.c
>> +++ b/drivers/staging/media/atomisp/pci/atomisp2/atomisp_cmd.c
>> @
(a); }
- else { kfree(a); }
+ kvfree(a);
@@
expression a;
@@
- if(...) { kfree(a); }
- else { vfree(a); }
+ kvfree(a);
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/staging/media/atomisp/pci/atomisp2/atomisp_cmd.c | 6 +-
1 file changed, 1 insertion(+), 5 del
(a); }
- else { kfree(a); }
+ kvfree(a);
@@
expression a;
@@
- if(...) { kfree(a); }
- else { vfree(a); }
+ kvfree(a);
//
Signed-off-by: Amitoj Kaur Chawla
---
drivers/staging/media/atomisp/pci/atomisp2/atomisp_cmd.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers
(a);
@@
expression a;
@@
- if(...) { kfree(a); }
- else { vfree(a); }
+ kvfree(a);
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/staging/media/atomisp/pci/atomisp2/css2400/sh_css.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/staging/media/atomi
(a);
@@
expression a;
@@
- if(...) { kfree(a); }
- else { vfree(a); }
+ kvfree(a);
//
Signed-off-by: Amitoj Kaur Chawla
---
drivers/staging/media/atomisp/pci/atomisp2/css2400/sh_css.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/staging/media/atomisp/pci/atomisp2/css2400
Use offset_in_page macro instead of (var & ~PAGE_MASK)
The Coccinelle semantic patch used to make this change is as follows:
//
@@
unsigned long p;
@@
- p & ~PAGE_MASK
+ offset_in_page(p)
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/staging/comedi/co
Use offset_in_page macro instead of (var & ~PAGE_MASK)
The Coccinelle semantic patch used to make this change is as follows:
//
@@
unsigned long p;
@@
- p & ~PAGE_MASK
+ offset_in_page(p)
//
Signed-off-by: Amitoj Kaur Chawla
---
drivers/staging/comedi/comedi_buf.c | 2 +-
1 file ch
Return statement at the end of a void function is useless.
The Coccinelle semantic patch used to make this change is as follows:
//
@@
identifier f;
expression e;
@@
void f(...) {
<...
- return
e;
...>
}
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/s
Return statement at the end of a void function is useless.
The Coccinelle semantic patch used to make this change is as follows:
//
@@
identifier f;
expression e;
@@
void f(...) {
<...
- return
e;
...>
}
//
Signed-off-by: Amitoj Kaur Chawla
---
drivers/staging/media/atomisp/pci/at
Remove unnecassary casts in the argument to kfree.
Found using Coccinelle. The semantic patch used to find this is as
follows:
//
@@
type T;
expression *f;
@@
- kfree((T *)(f));
+ kfree(f);
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/staging/rtl8723bs/
Remove unnecassary casts in the argument to kfree.
Found using Coccinelle. The semantic patch used to find this is as
follows:
//
@@
type T;
expression *f;
@@
- kfree((T *)(f));
+ kfree(f);
//
Signed-off-by: Amitoj Kaur Chawla
---
drivers/staging/rtl8723bs/os_dep/ioctl_linux.c | 14
On Mon, Aug 1, 2016 at 4:53 PM, SF Markus Elfring
wrote:
>> +@vz1 depends on patch && !context && !org && !report@
>> +type T;
>> +T *d;
>> +statement S;
>> +@@
>> +
>> +d =
>> +-vmalloc
>> ++vzalloc
>> + (...);
>> +
On Mon, Aug 1, 2016 at 4:53 PM, SF Markus Elfring
wrote:
>> +@vz1 depends on patch && !context && !org && !report@
>> +type T;
>> +T *d;
>> +statement S;
>> +@@
>> +
>> +d =
>> +-vmalloc
>> ++vzalloc
>> + (...);
>> +if (!d) S
>> +-
urn
- ...
+ PTR_ERR(e)
;
...+>
}
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
Changes in v2:
-Correct typo
drivers/gpu/drm/tegra/dpaux.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/tegra/dpaux.c b/drivers/gpu/drm/t
urn
- ...
+ PTR_ERR(e)
;
...+>
}
Signed-off-by: Amitoj Kaur Chawla
---
Changes in v2:
-Correct typo
drivers/gpu/drm/tegra/dpaux.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/tegra/dpaux.c b/drivers/gpu/drm/tegra/dpaux.c
index 059f409..2
iommu_domain_alloc returns NULL on error so replace an incorrect
IS_ERR check with a NULL check.
The Coccinelle semantic patch used to find this issue is as follows:
@@
expression e;
statement S;
@@
*e = iommu_domain_alloc(...);
if (IS_ERR(e)) S
Signed-off-by: Amitoj Kaur Chawla <amit
iommu_domain_alloc returns NULL on error so replace an incorrect
IS_ERR check with a NULL check.
The Coccinelle semantic patch used to find this issue is as follows:
@@
expression e;
statement S;
@@
*e = iommu_domain_alloc(...);
if (IS_ERR(e)) S
Signed-off-by: Amitoj Kaur Chawla
---
drivers
urn
- ...
+ PTR_ERR(e)
;
...+>
}
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/gpu/drm/tegra/dpaux.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/tegra/dpaux.c b/drivers/gpu/drm/tegra/dpaux.c
index 059f409..2f
urn
- ...
+ PTR_ERR(e)
;
...+>
}
Signed-off-by: Amitoj Kaur Chawla
---
drivers/gpu/drm/tegra/dpaux.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/tegra/dpaux.c b/drivers/gpu/drm/tegra/dpaux.c
index 059f409..2fde44c 100644
--- a/drivers/gpu/drm
Kaur Chawla <amitoj1...@gmail.com>
---
arch/mips/ath79/clock.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/mips/ath79/clock.c b/arch/mips/ath79/clock.c
index 2e73784..cc3a1e3 100644
--- a/arch/mips/ath79/clock.c
+++ b/arch/mips/ath79/clock.c
@@ -96,7 +96,7 @@
Kaur Chawla
---
arch/mips/ath79/clock.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/mips/ath79/clock.c b/arch/mips/ath79/clock.c
index 2e73784..cc3a1e3 100644
--- a/arch/mips/ath79/clock.c
+++ b/arch/mips/ath79/clock.c
@@ -96,7 +96,7 @@ static struct clk * __init
The variable `err` is not used anywhere and just returns the
predefined value `0` at the end of the function. Hence, remove the
variable and return 0 explicitly.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
net/sunrpc/clnt.c | 3 +--
1 file changed, 1 insertion(+), 2 del
The variable `err` is not used anywhere and just returns the
predefined value `0` at the end of the function. Hence, remove the
variable and return 0 explicitly.
Signed-off-by: Amitoj Kaur Chawla
---
net/sunrpc/clnt.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/net
This script detects cases which have incorrect error handling for
devm_ioremap_resource function, employing a NULL test instead of an
IS_ERR() test.
Acked-by: Julia Lawall <julia.law...@lip6.fr>
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
Changes in v2:
-Ch
This script detects cases which have incorrect error handling for
devm_ioremap_resource function, employing a NULL test instead of an
IS_ERR() test.
Acked-by: Julia Lawall
Signed-off-by: Amitoj Kaur Chawla
---
Changes in v2:
-Changed script to correct error handling instead of just
;
@@
* e = debugfs_create_file(...);
... when != e == NULL
e->f
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
b/drivers/gpu/drm/a
;
@@
* e = debugfs_create_file(...);
... when != e == NULL
e->f
Signed-off-by: Amitoj Kaur Chawla
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
index 8a17
;
@@
* e = debugfs_create_file(...);
... when != e == NULL
e->f
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
b/drivers/gpu/drm/a
;
@@
* e = debugfs_create_file(...);
... when != e == NULL
e->f
Signed-off-by: Amitoj Kaur Chawla
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
in
)
{
<+...
return
- PTR_ERR(e)
+ -ENODEV
;
...+>
}
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/iommu/exynos-iommu.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c
index 97c41b8..28
)
{
<+...
return
- PTR_ERR(e)
+ -ENODEV
;
...+>
}
Signed-off-by: Amitoj Kaur Chawla
---
drivers/iommu/exynos-iommu.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c
index 97c41b8..2888bbf 100644
--- a/drivers/iommu/
This script finds instances of allocate and memset which can be
replaced with a direct call to zalloc equivalent of a function.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
Acked-by: Julia Lawall <julia.law...@lip6.fr>
---
Changes in v2:
-Modified commit message
This script finds instances of allocate and memset which can be
replaced with a direct call to zalloc equivalent of a function.
Signed-off-by: Amitoj Kaur Chawla
Acked-by: Julia Lawall
---
Changes in v2:
-Modified commit message and subject
Changes in v3:
-Modified comment
...;
+ return PTR_ERR(e);
}
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/media/i2c/adp1653.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/adp1653.c b/drivers/media/i2c/adp1653.c
index 7e9cbf7..54b355e 100644
--- a/drivers/med
...;
+ return PTR_ERR(e);
}
Signed-off-by: Amitoj Kaur Chawla
---
drivers/media/i2c/adp1653.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/adp1653.c b/drivers/media/i2c/adp1653.c
index 7e9cbf7..54b355e 100644
--- a/drivers/media/i2c/adp1653.c
+++ b/drivers
-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
arch/s390/pci/pci_debug.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/s390/pci/pci_debug.c b/arch/s390/pci/pci_debug.c
index 4129b0a..8a25b04 100644
--- a/arch/s390/pci/pci_debug.c
+++ b/arch/s390/pci/pci_debug.c
@@ -139,8 +139,6 @
-by: Amitoj Kaur Chawla
---
arch/s390/pci/pci_debug.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/s390/pci/pci_debug.c b/arch/s390/pci/pci_debug.c
index 4129b0a..8a25b04 100644
--- a/arch/s390/pci/pci_debug.c
+++ b/arch/s390/pci/pci_debug.c
@@ -139,8 +139,6 @@ void zpci_debug_init_device
ERR(e)
+ -ENOMEM
;
...+>
}
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
arch/mips/mm/sc-debugfs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/mips/mm/sc-debugfs.c b/arch/mips/mm/sc-debugfs.c
index 5eefe32..01f1154 100644
--- a/arc
ERR(e)
+ -ENOMEM
;
...+>
}
Signed-off-by: Amitoj Kaur Chawla
---
arch/mips/mm/sc-debugfs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/mips/mm/sc-debugfs.c b/arch/mips/mm/sc-debugfs.c
index 5eefe32..01f1154 100644
--- a/arch/mips/mm/sc-debugfs.c
+++ b/arc
This script finds instances of allocate and memset which can be
replaced with a direct call to zalloc equivalent of a function.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
Changes in v2:
-Modified commit message and subject
scripts/coccinelle/api/zalloc.cocci
This script finds instances of allocate and memset which can be
replaced with a direct call to zalloc equivalent of a function.
Signed-off-by: Amitoj Kaur Chawla
---
Changes in v2:
-Modified commit message and subject
scripts/coccinelle/api/zalloc.cocci | 556
This script detects cases which have incorrect error handling for
devm_ioremap_resource function, employing a NULL test instead of an
IS_ERR() test.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
Changes in v2:
-Changed script to correct error handling instead o
This script detects cases which have incorrect error handling for
devm_ioremap_resource function, employing a NULL test instead of an
IS_ERR() test.
Signed-off-by: Amitoj Kaur Chawla
---
Changes in v2:
-Changed script to correct error handling instead of just
detecting cases
This script finds instances of allocate and memcpy which can be
replaced with a direct call to zalloc equivalent of a function.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
scripts/coccinelle/api/zalloc.cocci | 556
1 file change
This script finds instances of allocate and memcpy which can be
replaced with a direct call to zalloc equivalent of a function.
Signed-off-by: Amitoj Kaur Chawla
---
scripts/coccinelle/api/zalloc.cocci | 556
1 file changed, 556 insertions(+)
create mode
Replace clk_disable and clk_unprepare with clk_disable_unprepare.
The Coccinelle semantic patch used to make this change is as follows:
@@
expression e;
@@
- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
d
Replace clk_disable and clk_unprepare with clk_disable_unprepare.
The Coccinelle semantic patch used to make this change is as follows:
@@
expression e;
@@
- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
Signed-off-by: Amitoj Kaur Chawla
---
drivers/clocksource/timer-sp804.c
Replace clk_disable and clk_unprepare with clk_disable_unprepare.
The Coccinelle semantic patch used to make this change is as follows:
@@
expression e;
@@
- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drive
Replace clk_disable and clk_unprepare with clk_disable_unprepare.
The Coccinelle semantic patch used to make this change is as follows:
@@
expression e;
@@
- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
Signed-off-by: Amitoj Kaur Chawla
---
drivers/gpu/drm/fsl-dcu
Replace clk_disable and clk_unprepare with clk_disable_unprepare.
The Coccinelle semantic patch used to make this change is as follows:
@@
expression e;
@@
- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
ar
Replace clk_disable and clk_unprepare with clk_disable_unprepare.
The Coccinelle semantic patch used to make this change is as follows:
@@
expression e;
@@
- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
Signed-off-by: Amitoj Kaur Chawla
---
arch/arm/common/sa.c | 3
;
@@
- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
Changes in v2:
-Rebased patch
drivers/i2c/busses/i2c-mv64xxx.c | 18 ++
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git a/drive
;
@@
- clk_disable(e);
- clk_unprepare(e);
+ clk_disable_unprepare(e);
Signed-off-by: Amitoj Kaur Chawla
---
Changes in v2:
-Rebased patch
drivers/i2c/busses/i2c-mv64xxx.c | 18 ++
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mv64xxx.c b
This script replaces manual calculations by using the predefined
macros in kernel.h, DIV_ROUND_UP and roundup for readability purposes.
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
scripts/coccinelle/api/roundup.cocci | 215 +++
1 file change
This script replaces manual calculations by using the predefined
macros in kernel.h, DIV_ROUND_UP and roundup for readability purposes.
Signed-off-by: Amitoj Kaur Chawla
---
scripts/coccinelle/api/roundup.cocci | 215 +++
1 file changed, 215 insertions(+)
create
Remove useless initialisation of variable whose value is reinitialised
later.
The Coccinelle semantic patch used to make this change is as follows:
@@
type T;
identifier x;
constant C;
expression e;
@@
T x
- = C
;
x = e;
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
C
Remove useless initialisation of variable whose value is reinitialised
later.
The Coccinelle semantic patch used to make this change is as follows:
@@
type T;
identifier x;
constant C;
expression e;
@@
T x
- = C
;
x = e;
Signed-off-by: Amitoj Kaur Chawla
---
Changes in v2:
-Modify
@@
type T;
identifier x;
constant C;
expression e;
@@
T x
- = C
;
x = e;
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
sound/oss/ad1848.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/oss/ad1848.c b/sound/oss/ad1848.c
index 10c8de1..6368e5c
@@
type T;
identifier x;
constant C;
expression e;
@@
T x
- = C
;
x = e;
Signed-off-by: Amitoj Kaur Chawla
---
sound/oss/ad1848.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/oss/ad1848.c b/sound/oss/ad1848.c
index 10c8de1..6368e5c 100644
--- a/sound/oss/ad1848.c
On Mon, Jul 4, 2016 at 8:36 PM, Amitoj Kaur Chawla <amitoj1...@gmail.com> wrote:
> On Mon, Jul 4, 2016 at 8:29 PM, Mark Brown <broo...@kernel.org> wrote:
>> On Mon, Jul 04, 2016 at 04:52:43PM +0200, Julia Lawall wrote:
>>> On Mon, 4 Jul 2016, Mark Brown wrote:
>
On Mon, Jul 4, 2016 at 8:36 PM, Amitoj Kaur Chawla wrote:
> On Mon, Jul 4, 2016 at 8:29 PM, Mark Brown wrote:
>> On Mon, Jul 04, 2016 at 04:52:43PM +0200, Julia Lawall wrote:
>>> On Mon, 4 Jul 2016, Mark Brown wrote:
>>
>>> > Please use subject lines
On Mon, Jul 4, 2016 at 8:29 PM, Mark Brown wrote:
> On Mon, Jul 04, 2016 at 04:52:43PM +0200, Julia Lawall wrote:
>> On Mon, 4 Jul 2016, Mark Brown wrote:
>
>> > Please use subject lines matching the style for the subsystem. This
>> > makes it easier for people to identify
On Mon, Jul 4, 2016 at 8:29 PM, Mark Brown wrote:
> On Mon, Jul 04, 2016 at 04:52:43PM +0200, Julia Lawall wrote:
>> On Mon, 4 Jul 2016, Mark Brown wrote:
>
>> > Please use subject lines matching the style for the subsystem. This
>> > makes it easier for people to identify relevant patches.
>
>>
= platform_get_resource(pdev, IORESOURCE_MEM, n);
... when != res
- if (res == NULL) { ... \(goto l;\|return ret;\) }
... when != res
+ res = platform_get_resource(pdev, IORESOURCE_MEM, n);
e = devm_ioremap_resource(e1, res);
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
sound/soc
= platform_get_resource(pdev, IORESOURCE_MEM, n);
... when != res
- if (res == NULL) { ... \(goto l;\|return ret;\) }
... when != res
+ res = platform_get_resource(pdev, IORESOURCE_MEM, n);
e = devm_ioremap_resource(e1, res);
//
Signed-off-by: Amitoj Kaur Chawla
---
sound/soc/atmel/atmel-pdmic.c | 5
= platform_get_resource(pdev, IORESOURCE_MEM, n);
... when != res
- if (res == NULL) { ... \(goto l;\|return ret;\) }
... when != res
+ res = platform_get_resource(pdev, IORESOURCE_MEM, n);
e = devm_ioremap_resource(e1, res);
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
sound/soc
= platform_get_resource(pdev, IORESOURCE_MEM, n);
... when != res
- if (res == NULL) { ... \(goto l;\|return ret;\) }
... when != res
+ res = platform_get_resource(pdev, IORESOURCE_MEM, n);
e = devm_ioremap_resource(e1, res);
//
Signed-off-by: Amitoj Kaur Chawla
---
sound/soc/atmel/atmel-classd.c | 5
= platform_get_resource(pdev, IORESOURCE_MEM, n);
... when != res
- if (res == NULL) { ... \(goto l;\|return ret;\) }
... when != res
+ res = platform_get_resource(pdev, IORESOURCE_MEM, n);
e = devm_ioremap_resource(e1, res);
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/
= platform_get_resource(pdev, IORESOURCE_MEM, n);
... when != res
- if (res == NULL) { ... \(goto l;\|return ret;\) }
... when != res
+ res = platform_get_resource(pdev, IORESOURCE_MEM, n);
e = devm_ioremap_resource(e1, res);
//
Signed-off-by: Amitoj Kaur Chawla
---
drivers/crypto/bfin_crc.c | 5 -
1
==
- PF_INET
+ AF_INET
|
sip.sin_family !=
- PF_INET
+ AF_INET
|
sip.sin_family =
- PF_INET
+ AF_INET
)
//
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/infiniband/hw/cxgb3/iwch_cm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/infi
==
- PF_INET
+ AF_INET
|
sip.sin_family !=
- PF_INET
+ AF_INET
|
sip.sin_family =
- PF_INET
+ AF_INET
)
//
Signed-off-by: Amitoj Kaur Chawla
---
drivers/infiniband/hw/cxgb3/iwch_cm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/infiniband/hw/cxgb3/iwch_cm.c
b
;
|
-init_timer();
+setup_timer(,f,d);
|
-init_timer_on_stack();
+setup_timer_on_stack(,f,d);
)
<... when != S1
t.x1 = e1;
...>
Also convert calls to add_timer and intialisation of timer's expire
field to a call to mod_timer which is more compact.
Signed-off-by: Amitoj Kaur Chawla &l
;
|
-init_timer();
+setup_timer(,f,d);
|
-init_timer_on_stack();
+setup_timer_on_stack(,f,d);
)
<... when != S1
t.x1 = e1;
...>
Also convert calls to add_timer and intialisation of timer's expire
field to a call to mod_timer which is more compact.
Signed-off-by: Amitoj Kaur Chawla
---
driv
;
|
-init_timer();
+setup_timer(,f,d);
|
-init_timer_on_stack();
+setup_timer_on_stack(,f,d);
)
<... when != S1
t.x1 = e1;
...>
Signed-off-by: Amitoj Kaur Chawla <amitoj1...@gmail.com>
---
drivers/atm/horizon.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/atm
;
|
-init_timer();
+setup_timer(,f,d);
|
-init_timer_on_stack();
+setup_timer_on_stack(,f,d);
)
<... when != S1
t.x1 = e1;
...>
Signed-off-by: Amitoj Kaur Chawla
---
drivers/atm/horizon.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/atm/horizon.c b/drivers/atm/hor
1 - 100 of 370 matches
Mail list logo