Re: [Patch v3 7/7] Onkey: DA9055 Onkey driver

On Thu, 2012-10-11 at 09:28 -0700, Dmitry Torokhov wrote:

Hi Ashish,

On Thu, Oct 11, 2012 at 04:15:16PM +0530, Ashish Jangam wrote:

quoted

This is the ONKEY driver of the Dialog DA9055 PMIC and depends on the DA9055 MFD
core driver.

This patch is functionally tested on SMDK6410 board.

Signed-off-by: David Dajun Chen <redacted>
Signed-off-by: Ashish Jangam <redacted>
---
changes since v3:
- used of module_platform_driver macro
- add regmap virtual irq function
changes since v2:
- add support for button release
- use of devm_request_threaded_irq API

This API does not make any sense for your driver as you are scheduling
delayed work from the IRQ handler. In such cases you need _first_ free
IRQ and _then_ cancel the work but with devm_* you do it in opposite
order.

I miss on this, will correct it.

Also, I think I mentioned this before - this driver looks like twin
brother for da9055_onkey.c. How many of such parts do you have (or will
you have)? Is there way to unify them?

Well, DA9052 and DA9055 register sets are entirely different even there
bit position are different and also few of its components have slight
different behaviour.  Device components functional difference can be
managed programmatic but, to unify these two different sets of register
will be impractical. In case of DA9052 I managed to club the DA9053
functionality within it but it was not possible for DA9055.

Thanks.