Re: [PATCH 2/3] ECI: introducing ECI bus driver

From: Randy Dunlap
Date: Wed Dec 22 2010 - 12:18:38 EST


On Wed, 22 Dec 2010 14:20:34 +0200 tapio.vihuri@xxxxxxxxx wrote:

> From: Tapio Vihuri <tapio.vihuri@xxxxxxxxx>
>
> ECI bus controller is kind of bridge between host CPU I2C and ECI accessory
> ECI communication.
>
> Signed-off-by: Tapio Vihuri <tapio.vihuri@xxxxxxxxx>
> ---
> drivers/Kconfig | 2 +
> drivers/Makefile | 1 +
> drivers/ecibus/Kconfig | 46 ++++
> drivers/ecibus/Makefile | 10 +
> drivers/ecibus/ecibus.c | 583 +++++++++++++++++++++++++++++++++++++++++++++
> include/linux/input/eci.h | 8 +
> 6 files changed, 650 insertions(+), 0 deletions(-)
> create mode 100644 drivers/ecibus/Kconfig
> create mode 100644 drivers/ecibus/Makefile
> create mode 100644 drivers/ecibus/ecibus.c


> diff --git a/drivers/ecibus/Kconfig b/drivers/ecibus/Kconfig
> new file mode 100644
> index 0000000..27e5e36
> --- /dev/null
> +++ b/drivers/ecibus/Kconfig
> @@ -0,0 +1,46 @@
> +#
> +# ECI driver configuration
> +#
> +menuconfig ECI
> + bool "ECI support"
> + help
> + ECI (Enhancement Control Interface) accessory support
> +
> + The Enhancement Control Interface functionality
> + ECI is better known as Multimedia Headset for Nokia phones.
> + If headset has many buttons, like play, vol+, vol- etc. then
> + it is propably ECI accessory.
> + Among several buttons ECI accessory contains memory for storing
> + several parameters.
> +
> + Enable ECI support in terminal so that ECI input driver is able
> + to communicate with ECI accessory
> +
> +if ECI
> +
> +config ECI_DEBUG
> + boolean "Debug support for ECI drivers"
> + depends on DEBUG_KERNEL
> + help
> + Selects ECI driver debug messaging.
> +
> + Say "yes" to enable debug messaging (like dev_dbg and pr_debug),
> + sysfs, and debugfs support in ECI controller.
> +
> +comment "ECI Master Controller Drivers"
> +
> +config ECI_BUS
> + tristate "ECI bus controller driver"
> + select INPUT_ECI


'select' INPUT_ECI when INPUT and/or INPUT_MISC are not enabled causes this
kconfig warning:

warning: (ECI_BUS && ECI) selects INPUT_ECI which has unmet direct dependencies (!S390 && INPUT && INPUT_MISC)


> + help
> + This selects a driver for the ECI bus controller
> +
> + ECI bus controller is kind of bridge between host CPU I2C and
> + ECI accessory ECI communication.
> +
> + Say 'y' here to statically link this module into the kernel or 'm'
> + to build it as a dynamically loadable module. The module will be
> + called ecibus.ko
> +
> +
> +endif # ECI


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
desserts: http://www.xenotime.net/linux/recipes/
--
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/