2014-04-29 08:24:45

by Jingoo Han

Subject: [PATCH 0/4] clocksource: remove unnecessary OOM messages

The site-specific OOM messages are unnecessary, because they
duplicate the MM subsystem generic OOM message. For example,
k.alloc and v.alloc failures use dump_stack().

Jingoo Han (4):
clocksource: em_sti: remove unnecessary OOM messages
clocksource: sh_cmt: remove unnecessary OOM messages
clocksource: sh_mtu2: remove unnecessary OOM messages
clocksource: sh_tmu: remove unnecessary OOM messages

---
drivers/clocksource/em_sti.c | 4 +---
drivers/clocksource/sh_cmt.c | 4 +---
drivers/clocksource/sh_mtu2.c | 4 +---
drivers/clocksource/sh_tmu.c | 4 +---
4 files changed, 4 insertions(+), 12 deletions(-)

2014-04-29 08:26:08

by Jingoo Han

Subject: [PATCH 1/4] clocksource: em_sti: remove unnecessary OOM messages

The site-specific OOM messages are unnecessary, because they
duplicate the MM subsystem generic OOM message.

Signed-off-by: Jingoo Han <[email protected]>
---
drivers/clocksource/em_sti.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clocksource/em_sti.c b/drivers/clocksource/em_sti.c
index 9d17083..d0a7bd6 100644
--- a/drivers/clocksource/em_sti.c
+++ b/drivers/clocksource/em_sti.c
@@ -318,10 +318,8 @@ static int em_sti_probe(struct platform_device *pdev)
int irq;

p = devm_kzalloc(&pdev->dev, sizeof(*p), GFP_KERNEL);
- if (p == NULL) {
- dev_err(&pdev->dev, "failed to allocate driver data\n");
+ if (p == NULL)
return -ENOMEM;
- }

p->pdev = pdev;
platform_set_drvdata(pdev, p);
--
1.7.10.4

2014-04-29 08:27:10

by Jingoo Han

Subject: [PATCH 2/4] clocksource: sh_cmt: remove unnecessary OOM messages

The site-specific OOM messages are unnecessary, because they
duplicate the MM subsystem generic OOM message.

Signed-off-by: Jingoo Han <[email protected]>
---
drivers/clocksource/sh_cmt.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clocksource/sh_cmt.c b/drivers/clocksource/sh_cmt.c
index 0b1836a..d2aa209 100644
--- a/drivers/clocksource/sh_cmt.c
+++ b/drivers/clocksource/sh_cmt.c
@@ -824,10 +824,8 @@ static int sh_cmt_probe(struct platform_device *pdev)
}

p = kmalloc(sizeof(*p), GFP_KERNEL);
- if (p == NULL) {
- dev_err(&pdev->dev, "failed to allocate driver data\n");
+ if (p == NULL)
return -ENOMEM;
- }

ret = sh_cmt_setup(p, pdev);
if (ret) {
--
1.7.10.4

2014-04-29 08:27:50

by Jingoo Han

Subject: [PATCH 3/4] clocksource: sh_mtu2: remove unnecessary OOM messages

The site-specific OOM messages are unnecessary, because they
duplicate the MM subsystem generic OOM message.

Signed-off-by: Jingoo Han <[email protected]>
---
drivers/clocksource/sh_mtu2.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clocksource/sh_mtu2.c b/drivers/clocksource/sh_mtu2.c
index e30d76e..e6bee62 100644
--- a/drivers/clocksource/sh_mtu2.c
+++ b/drivers/clocksource/sh_mtu2.c
@@ -349,10 +349,8 @@ static int sh_mtu2_probe(struct platform_device *pdev)
}

p = kmalloc(sizeof(*p), GFP_KERNEL);
- if (p == NULL) {
- dev_err(&pdev->dev, "failed to allocate driver data\n");
+ if (p == NULL)
return -ENOMEM;
- }

ret = sh_mtu2_setup(p, pdev);
if (ret) {
--
1.7.10.4

2014-04-29 08:28:22

by Jingoo Han

Subject: [PATCH 4/4] clocksource: sh_tmu: remove unnecessary OOM messages

The site-specific OOM messages are unnecessary, because they
duplicate the MM subsystem generic OOM message.

Signed-off-by: Jingoo Han <[email protected]>
---
drivers/clocksource/sh_tmu.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clocksource/sh_tmu.c b/drivers/clocksource/sh_tmu.c
index ecd7b60..b62f28c 100644
--- a/drivers/clocksource/sh_tmu.c
+++ b/drivers/clocksource/sh_tmu.c
@@ -514,10 +514,8 @@ static int sh_tmu_probe(struct platform_device *pdev)
}

p = kmalloc(sizeof(*p), GFP_KERNEL);
- if (p == NULL) {
- dev_err(&pdev->dev, "failed to allocate driver data\n");
+ if (p == NULL)
return -ENOMEM;
- }

ret = sh_tmu_setup(p, pdev);
if (ret) {
--
1.7.10.4