Skip to content

Commit

Permalink
Input: omap4-keypad - use platform device helpers
Browse files Browse the repository at this point in the history
Get mem and irq resources using platform helpers

 - platform_get_base
 - platform_get_irq

Signed-off-by: Abraham Arce <x0066660@ti.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
  • Loading branch information
Abraham Arce authored and Dmitry Torokhov committed Oct 13, 2010
1 parent 4c61c84 commit f3a1ba6
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 8 deletions.
4 changes: 1 addition & 3 deletions arch/arm/plat-omap/include/plat/omap4-keypad.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,7 @@ struct omap4_keypad_platform_data {

u8 rows;
u8 cols;

u16 irq;
void __iomem *base;
};

extern int omap4_keyboard_init(struct omap4_keypad_platform_data *);
#endif
40 changes: 35 additions & 5 deletions drivers/input/keyboard/omap4-keypad.c
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,10 @@ static int __devinit omap4_keypad_probe(struct platform_device *pdev)
const struct omap4_keypad_platform_data *pdata;
struct omap4_keypad *keypad_data;
struct input_dev *input_dev;
struct resource *res;
resource_size_t size;
unsigned int row_shift, max_keys;
int irq;
int error;

/* platform data */
Expand All @@ -158,12 +161,14 @@ static int __devinit omap4_keypad_probe(struct platform_device *pdev)
return -EINVAL;
}

if (!pdata->base) {
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
if (!res) {
dev_err(&pdev->dev, "no base address specified\n");
return -EINVAL;
}

if (!pdata->irq) {
irq = platform_get_irq(pdev, 0);
if (!irq) {
dev_err(&pdev->dev, "no keyboard irq assigned\n");
return -EINVAL;
}
Expand All @@ -184,9 +189,23 @@ static int __devinit omap4_keypad_probe(struct platform_device *pdev)
return -ENOMEM;
}

keypad_data->base = pdata->base;
keypad_data->irq = pdata->irq;
size = resource_size(res);

res = request_mem_region(res->start, size, pdev->name);
if (!res) {
dev_err(&pdev->dev, "can't request mem region\n");
error = -EBUSY;
goto err_free_keypad;
}

keypad_data->base = ioremap(res->start, resource_size(res));
if (!keypad_data->base) {
dev_err(&pdev->dev, "can't ioremap mem resource\n");
error = -ENOMEM;
goto err_release_mem;
}

keypad_data->irq = irq;
keypad_data->row_shift = row_shift;
keypad_data->rows = pdata->rows;
keypad_data->cols = pdata->cols;
Expand All @@ -195,7 +214,7 @@ static int __devinit omap4_keypad_probe(struct platform_device *pdev)
keypad_data->input = input_dev = input_allocate_device();
if (!input_dev) {
error = -ENOMEM;
goto err_free_keypad;
goto err_unmap;
}

input_dev->name = pdev->name;
Expand Down Expand Up @@ -243,6 +262,10 @@ static int __devinit omap4_keypad_probe(struct platform_device *pdev)
free_irq(keypad_data->irq, keypad_data);
err_free_input:
input_free_device(input_dev);
err_unmap:
iounmap(keypad_data->base);
err_release_mem:
release_mem_region(res->start, size);
err_free_keypad:
kfree(keypad_data);
return error;
Expand All @@ -251,9 +274,16 @@ static int __devinit omap4_keypad_probe(struct platform_device *pdev)
static int __devexit omap4_keypad_remove(struct platform_device *pdev)
{
struct omap4_keypad *keypad_data = platform_get_drvdata(pdev);
struct resource *res;

free_irq(keypad_data->irq, keypad_data);
input_unregister_device(keypad_data->input);

iounmap(keypad_data->base);

res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
release_mem_region(res->start, resource_size(res));

kfree(keypad_data);
platform_set_drvdata(pdev, NULL);

Expand Down

0 comments on commit f3a1ba6

Please sign in to comment.