Re: [PATCH 0/2] mfd/stmpe: Add SPI interface support

From: Rabin Vincent
Date: Fri Nov 11 2011 - 04:44:30 EST


On Tue, Nov 8, 2011 at 12:35, Viresh Kumar <viresh.kumar@xxxxxx> wrote:
> Few STMPE controller can have register interface over SPI or I2C. Current
> implementation only supports I2C and all code is present in a single file
> stmpe.c.
>
> This patchset separates out code into three files
> - stmpe: specific to controller only
> - stmpe-i2c: specific to i2c interface
> - stmpe-spi: specific to spi interface
>
> In order to rebase it cleanly, please apply earlier floated patch
>        "mfd/stmpe: Add support for irq over gpio pin"
>
> Viresh Kumar (2):
>  mfd/stmpe: separate out controller and interface specific code
>  mfd/stmpe: Add support for SPI interface

Could you instead convert stmpe to use the register map API, and then
add the SPI interface? Converting to regmap should both simplify the
existing code as well as make it possible to add the second interface
with fewer lines of code.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/