* [PATCH 1/2] i2c: versatile: Allow compile test build
@ 2016-04-14 14:30 Axel Lin
2016-04-14 14:31 ` [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs Axel Lin
0 siblings, 1 reply; 7+ messages in thread
From: Axel Lin @ 2016-04-14 14:30 UTC (permalink / raw)
To: Wolfram Sang; +Cc: Pawel Moll, Russell King, linux-i2c
There is no build dependency for this driver, so enable COMPILE_TEST to get
better build coverage.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
drivers/i2c/busses/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index faa8e68..6e0bba0 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -924,7 +924,7 @@ config I2C_UNIPHIER_F
config I2C_VERSATILE
tristate "ARM Versatile/Realview I2C bus support"
- depends on ARCH_VERSATILE || ARCH_REALVIEW || ARCH_VEXPRESS
+ depends on ARCH_VERSATILE || ARCH_REALVIEW || ARCH_VEXPRESS || COMPILE_TEST
select I2C_ALGOBIT
help
Say yes if you want to support the I2C serial bus on ARMs Versatile
--
2.5.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs
2016-04-14 14:30 [PATCH 1/2] i2c: versatile: Allow compile test build Axel Lin
@ 2016-04-14 14:31 ` Axel Lin
2016-06-25 1:41 ` Axel Lin
0 siblings, 1 reply; 7+ messages in thread
From: Axel Lin @ 2016-04-14 14:31 UTC (permalink / raw)
To: Wolfram Sang; +Cc: Pawel Moll, Russell King, linux-i2c
Use devm_* APIs to simplify the code a bit.
This patch also fixes the memory leak when unload the module.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
Hi,
I don't have this h/w, I'd appreciate if someone can test this patch.
drivers/i2c/busses/i2c-versatile.c | 46 +++++++++++---------------------------
1 file changed, 13 insertions(+), 33 deletions(-)
diff --git a/drivers/i2c/busses/i2c-versatile.c b/drivers/i2c/busses/i2c-versatile.c
index 240637f..c73d2d2 100644
--- a/drivers/i2c/busses/i2c-versatile.c
+++ b/drivers/i2c/busses/i2c-versatile.c
@@ -70,28 +70,14 @@ static int i2c_versatile_probe(struct platform_device *dev)
struct resource *r;
int ret;
+ i2c = devm_kzalloc(&dev->dev, sizeof(struct i2c_versatile), GFP_KERNEL);
+ if (!i2c)
+ return -ENOMEM;
+
r = platform_get_resource(dev, IORESOURCE_MEM, 0);
- if (!r) {
- ret = -EINVAL;
- goto err_out;
- }
-
- if (!request_mem_region(r->start, resource_size(r), "versatile-i2c")) {
- ret = -EBUSY;
- goto err_out;
- }
-
- i2c = kzalloc(sizeof(struct i2c_versatile), GFP_KERNEL);
- if (!i2c) {
- ret = -ENOMEM;
- goto err_release;
- }
-
- i2c->base = ioremap(r->start, resource_size(r));
- if (!i2c->base) {
- ret = -ENOMEM;
- goto err_free;
- }
+ i2c->base = devm_ioremap_resource(&dev->dev, r);
+ if (IS_ERR(i2c->base))
+ return PTR_ERR(i2c->base);
writel(SCL | SDA, i2c->base + I2C_CONTROLS);
@@ -105,18 +91,12 @@ static int i2c_versatile_probe(struct platform_device *dev)
i2c->adap.nr = dev->id;
ret = i2c_bit_add_numbered_bus(&i2c->adap);
- if (ret >= 0) {
- platform_set_drvdata(dev, i2c);
- return 0;
- }
-
- iounmap(i2c->base);
- err_free:
- kfree(i2c);
- err_release:
- release_mem_region(r->start, resource_size(r));
- err_out:
- return ret;
+ if (ret < 0)
+ return ret;
+
+ platform_set_drvdata(dev, i2c);
+
+ return 0;
}
static int i2c_versatile_remove(struct platform_device *dev)
--
2.5.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs
2016-04-14 14:31 ` [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs Axel Lin
@ 2016-06-25 1:41 ` Axel Lin
2016-07-04 13:56 ` Pawel Moll
0 siblings, 1 reply; 7+ messages in thread
From: Axel Lin @ 2016-06-25 1:41 UTC (permalink / raw)
To: Wolfram Sang; +Cc: Pawel Moll, Russell King, linux-i2c
2016-04-14 22:31 GMT+08:00 Axel Lin <axel.lin@ingics.com>:
> Use devm_* APIs to simplify the code a bit.
> This patch also fixes the memory leak when unload the module.
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---
> Hi,
> I don't have this h/w, I'd appreciate if someone can test this patch.
Obviously the module unload path has memory leak.
Just wondering if anyone can review/test this patch.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs
2016-06-25 1:41 ` Axel Lin
@ 2016-07-04 13:56 ` Pawel Moll
0 siblings, 0 replies; 7+ messages in thread
From: Pawel Moll @ 2016-07-04 13:56 UTC (permalink / raw)
To: Liviu Dudau, Sudeep Holla, Lorenzo Pieralisi
Cc: Russell King, linux-i2c, Axel Lin, Wolfram Sang, linux-arm-kernel
Dnia 2016-06-25, Sat o godzinie 09:41 +0800, Axel Lin pisze:
> 2016-04-14 22:31 GMT+08:00 Axel Lin <axel.lin@ingics.com>:
> > Use devm_* APIs to simplify the code a bit.
> > This patch also fixes the memory leak when unload the module.
> >
> > Signed-off-by: Axel Lin <axel.lin@ingics.com>
> > ---
> > Hi,
> > I don't have this h/w, I'd appreciate if someone can test this
> > patch.
>
> Obviously the module unload path has memory leak.
> Just wondering if anyone can review/test this patch.
Gentlemen? (referring to the VE maintainers here)
Original thread: thread.gmane.org/gmane.linux.drivers.i2c/27277
Thanks!
Pawel
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs
@ 2016-07-04 13:56 ` Pawel Moll
0 siblings, 0 replies; 7+ messages in thread
From: Pawel Moll @ 2016-07-04 13:56 UTC (permalink / raw)
To: linux-arm-kernel
Dnia 2016-06-25, Sat o godzinie 09:41 +0800, Axel Lin pisze:
> 2016-04-14 22:31 GMT+08:00 Axel Lin <axel.lin@ingics.com>:
> > Use devm_* APIs to simplify the code a bit.
> > This patch also fixes the memory leak when unload the module.
> >
> > Signed-off-by: Axel Lin <axel.lin@ingics.com>
> > ---
> > Hi,
> > I don't have this h/w, I'd appreciate if someone can test this
> > patch.
>
> Obviously the module unload path has memory leak.
> Just wondering if anyone can review/test this patch.
Gentlemen? (referring to the VE maintainers here)
Original thread: thread.gmane.org/gmane.linux.drivers.i2c/27277
Thanks!
Pawel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs
2016-07-04 13:56 ` Pawel Moll
@ 2016-07-08 13:44 ` Lorenzo Pieralisi
-1 siblings, 0 replies; 7+ messages in thread
From: Lorenzo Pieralisi @ 2016-07-08 13:44 UTC (permalink / raw)
To: Pawel Moll
Cc: Liviu Dudau, Sudeep Holla, Russell King, linux-i2c, Axel Lin,
Wolfram Sang, linux-arm-kernel
On Mon, Jul 04, 2016 at 02:56:13PM +0100, Pawel Moll wrote:
> Dnia 2016-06-25, Sat o godzinie 09:41 +0800, Axel Lin pisze:
> > 2016-04-14 22:31 GMT+08:00 Axel Lin <axel.lin@ingics.com>:
> > > Use devm_* APIs to simplify the code a bit.
> > > This patch also fixes the memory leak when unload the module.
> > >
> > > Signed-off-by: Axel Lin <axel.lin@ingics.com>
> > > ---
> > > Hi,
> > > I don't have this h/w, I'd appreciate if someone can test this
> > > patch.
> >
> > Obviously the module unload path has memory leak.
> > Just wondering if anyone can review/test this patch.
>
> Gentlemen? (referring to the VE maintainers here)
>
> Original thread: thread.gmane.org/gmane.linux.drivers.i2c/27277
We weren't CC'ed, thanks for the heads-up anyway we will have a
look and test shortly (@Axel next time please CC us if you want
us to test your patches, thanks).
Lorenzo
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs
@ 2016-07-08 13:44 ` Lorenzo Pieralisi
0 siblings, 0 replies; 7+ messages in thread
From: Lorenzo Pieralisi @ 2016-07-08 13:44 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jul 04, 2016 at 02:56:13PM +0100, Pawel Moll wrote:
> Dnia 2016-06-25, Sat o godzinie 09:41 +0800, Axel Lin pisze:
> > 2016-04-14 22:31 GMT+08:00 Axel Lin <axel.lin@ingics.com>:
> > > Use devm_* APIs to simplify the code a bit.
> > > This patch also fixes the memory leak when unload the module.
> > >
> > > Signed-off-by: Axel Lin <axel.lin@ingics.com>
> > > ---
> > > Hi,
> > > I don't have this h/w, I'd appreciate if someone can test this
> > > patch.
> >
> > Obviously the module unload path has memory leak.
> > Just wondering if anyone can review/test this patch.
>
> Gentlemen? (referring to the VE maintainers here)
>
> Original thread: thread.gmane.org/gmane.linux.drivers.i2c/27277
We weren't CC'ed, thanks for the heads-up anyway we will have a
look and test shortly (@Axel next time please CC us if you want
us to test your patches, thanks).
Lorenzo
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-07-08 13:44 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-14 14:30 [PATCH 1/2] i2c: versatile: Allow compile test build Axel Lin
2016-04-14 14:31 ` [PATCH 2/2] i2c: versatile: Convert to use resource managed devm_* APIs Axel Lin
2016-06-25 1:41 ` Axel Lin
2016-07-04 13:56 ` Pawel Moll
2016-07-04 13:56 ` Pawel Moll
2016-07-08 13:44 ` Lorenzo Pieralisi
2016-07-08 13:44 ` Lorenzo Pieralisi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.