[PATCH v2] ARM: S3C64XX: Use module identification for Craggamore system builds

Kukjin Kim kgene.kim at samsung.com
Fri Aug 19 09:52:26 EDT 2011


Mark Brown wrote:
> 
> The Cragganmore system is modular with I2C based identification chips on
> the system allowing identification of the system build. Provide a stub I2C
> driver which parses the module IDs and uses them to select the appropriate
> audio subsystem components to register. To avoid confusion due to having
> the mini-driver in the system the driver is placed in a separate file.
> 
> Signed-off-by: Mark Brown <broonie at opensource.wolfsonmicro.com>
> ---
> 
> Add a bunch more modules to the table.  No devices are registered for
> most of the modules but the text is useful for diagnostics if nothing
> else.
> 
>  arch/arm/mach-s3c64xx/Kconfig                 |    1 +
>  arch/arm/mach-s3c64xx/Makefile                |    2 +-
>  arch/arm/mach-s3c64xx/include/mach/crag6410.h |   23 ++++
>  arch/arm/mach-s3c64xx/mach-crag6410-module.c  |  173
> +++++++++++++++++++++++++
>  arch/arm/mach-s3c64xx/mach-crag6410.c         |   85 +------------
>  5 files changed, 203 insertions(+), 81 deletions(-)
>  create mode 100644 arch/arm/mach-s3c64xx/include/mach/crag6410.h
>  create mode 100644 arch/arm/mach-s3c64xx/mach-crag6410-module.c
> 
> diff --git a/arch/arm/mach-s3c64xx/Kconfig b/arch/arm/mach-s3c64xx/Kconfig
> index f057b6a..5552e04 100644
> --- a/arch/arm/mach-s3c64xx/Kconfig
> +++ b/arch/arm/mach-s3c64xx/Kconfig
> @@ -288,5 +288,6 @@ config MACH_WLF_CRAGG_6410
>  	select S3C_DEV_RTC
>  	select S3C64XX_DEV_SPI
>  	select S3C24XX_GPIO_EXTRA128
> +	select I2C
>  	help
>  	  Machine support for the Wolfson Cragganmore S3C6410 variant.
> diff --git a/arch/arm/mach-s3c64xx/Makefile
b/arch/arm/mach-s3c64xx/Makefile
> index 61b4034..5fdea6a 100644
> --- a/arch/arm/mach-s3c64xx/Makefile
> +++ b/arch/arm/mach-s3c64xx/Makefile
> @@ -55,7 +55,7 @@ obj-$(CONFIG_MACH_HMT)		+= mach-hmt.o
>  obj-$(CONFIG_MACH_SMARTQ)	+= mach-smartq.o
>  obj-$(CONFIG_MACH_SMARTQ5)	+= mach-smartq5.o
>  obj-$(CONFIG_MACH_SMARTQ7)	+= mach-smartq7.o
> -obj-$(CONFIG_MACH_WLF_CRAGG_6410) += mach-crag6410.o
> +obj-$(CONFIG_MACH_WLF_CRAGG_6410) += mach-crag6410.o mach-crag6410-
> module.o
> 
>  # device support
> 
> diff --git a/arch/arm/mach-s3c64xx/include/mach/crag6410.h
b/arch/arm/mach-
> s3c64xx/include/mach/crag6410.h
> new file mode 100644
> index 0000000..be9074e
> --- /dev/null
> +++ b/arch/arm/mach-s3c64xx/include/mach/crag6410.h
> @@ -0,0 +1,23 @@
> +/* Cragganmore 6410 shared definitions
> + *
> + * Copyright 2011 Wolfson Microelectronics plc
> + *	Mark Brown <broonie at opensource.wolfsonmicro.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#ifndef MACH_CRAG6410_H
> +#define MACH_CRAG6410_H
> +
> +#include <linux/gpio.h>
> +
> +#define BANFF_PMIC_IRQ_BASE		IRQ_BOARD_START
> +#define GLENFARCLAS_PMIC_IRQ_BASE	(IRQ_BOARD_START + 64)
> +
> +#define PCA935X_GPIO_BASE		GPIO_BOARD_START
> +#define CODEC_GPIO_BASE		(GPIO_BOARD_START + 8)
> +#define GLENFARCLAS_PMIC_GPIO_BASE	(GPIO_BOARD_START + 16)
> +
> +#endif
> diff --git a/arch/arm/mach-s3c64xx/mach-crag6410-module.c b/arch/arm/mach-
> s3c64xx/mach-crag6410-module.c
> new file mode 100644
> index 0000000..0708fa0
> --- /dev/null
> +++ b/arch/arm/mach-s3c64xx/mach-crag6410-module.c
> @@ -0,0 +1,173 @@
> +/* Speyside modules for Cragganmore - board data probing
> + *
> + * Copyright 2011 Wolfson Microelectronics plc
> + *	Mark Brown <broonie at opensource.wolfsonmicro.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/module.h>
> +#include <linux/interrupt.h>
> +#include <linux/i2c.h>
> +
> +#include <linux/mfd/wm831x/irq.h>
> +#include <linux/mfd/wm831x/gpio.h>
> +
> +#include <sound/wm8996.h>
> +#include <sound/wm8962.h>
> +#include <sound/wm9081.h>
> +
> +#include <mach/crag6410.h>
> +
> +static struct wm8996_retune_mobile_config wm8996_retune[] = {
> +	{
> +		.name = "Sub LPF",
> +		.rate = 48000,
> +		.regs = {
> +			0x6318, 0x6300, 0x1000, 0x0000, 0x0004, 0x2000,
> 0xF000,
> +			0x0000, 0x0004, 0x2000, 0xF000, 0x0000, 0x0004,
> 0x2000,
> +			0xF000, 0x0000, 0x0004, 0x1000, 0x0800, 0x4000
> +		},
> +	},
> +	{
> +		.name = "Sub HPF",
> +		.rate = 48000,
> +		.regs = {
> +			0x000A, 0x6300, 0x1000, 0x0000, 0x0004, 0x2000,
> 0xF000,
> +			0x0000, 0x0004, 0x2000, 0xF000, 0x0000, 0x0004,
> 0x2000,
> +			0xF000, 0x0000, 0x0004, 0x1000, 0x0800, 0x4000
> +		},
> +	},
> +};
> +
> +static struct wm8996_pdata wm8996_pdata __initdata = {
> +	.ldo_ena = S3C64XX_GPN(7),
> +	.gpio_base = CODEC_GPIO_BASE,
> +	.micdet_def = 1,
> +	.inl_mode = WM8996_DIFFERRENTIAL_1,
> +	.inr_mode = WM8996_DIFFERRENTIAL_1,
> +
> +	.irq_flags = IRQF_TRIGGER_RISING,
> +
> +	.gpio_default = {
> +		0x8001, /* GPIO1 == ADCLRCLK1 */
> +		0x8001, /* GPIO2 == ADCLRCLK2, input due to CPU */
> +		0x0141, /* GPIO3 == HP_SEL */
> +		0x0002, /* GPIO4 == IRQ */
> +		0x020e, /* GPIO5 == CLKOUT */
> +	},
> +
> +	.retune_mobile_cfgs = wm8996_retune,
> +	.num_retune_mobile_cfgs = ARRAY_SIZE(wm8996_retune),
> +};
> +
> +static struct wm8962_pdata wm8962_pdata __initdata = {
> +	.gpio_init = {
> +		0,
> +		WM8962_GPIO_FN_OPCLK,
> +		WM8962_GPIO_FN_DMICCLK,
> +		0,
> +		0x8000 | WM8962_GPIO_FN_DMICDAT,
> +		WM8962_GPIO_FN_IRQ,    /* Open drain mode */
> +	},
> +	.irq_active_low = true,
> +};
> +
> +static struct wm9081_pdata wm9081_pdata __initdata = {
> +	.irq_high = false,
> +	.irq_cmos = false,
> +};
> +
> +static const struct i2c_board_info wm1254_devs[] = {
> +	{ I2C_BOARD_INFO("wm8996", 0x1a),
> +	  .platform_data = &wm8996_pdata,
> +	  .irq = GLENFARCLAS_PMIC_IRQ_BASE + WM831X_IRQ_GPIO_2,
> +	},
> +	{ I2C_BOARD_INFO("wm9081", 0x6c),
> +	  .platform_data = &wm9081_pdata, },
> +};
> +
> +static const struct i2c_board_info wm1259_devs[] = {
> +	{ I2C_BOARD_INFO("wm8962", 0x1a),
> +	  .platform_data = &wm8962_pdata,
> +	  .irq = GLENFARCLAS_PMIC_IRQ_BASE + WM831X_IRQ_GPIO_2,
> +	},
> +};
> +
> +
> +static __devinitdata const struct {
> +	u8 id;
> +	const char *name;
> +	const struct i2c_board_info *i2c_devs;
> +	int num_i2c_devs;
> +} gf_mods[] = {
> +	{ .id = 0x01, .name = "1250-EV1 Springbank" },
> +	{ .id = 0x02, .name = "1251-EV1 Jura" },
> +	{ .id = 0x03, .name = "1252-EV1 Glenlivet" },
> +	{ .id = 0x11, .name = "6249-EV2 Glenfarclas", },
> +	{ .id = 0x21, .name = "1275-EV1 Mortlach" },
> +	{ .id = 0x25, .name = "1274-EV1 Glencadam" },
> +	{ .id = 0x31, .name = "1253-EV1 Tomatin", },
> +	{ .id = 0x39, .name = "1254-EV1 Dallas Dhu",
> +	  .i2c_devs = wm1254_devs, .num_i2c_devs =
> ARRAY_SIZE(wm1254_devs) },
> +	{ .id = 0x3a, .name = "1259-EV1 Tobermory",
> +	  .i2c_devs = wm1259_devs, .num_i2c_devs =
> ARRAY_SIZE(wm1259_devs) },
> +	{ .id = 0x3b, .name = "1255-EV1 Kilchoman" },
> +	{ .id = 0x3c, .name = "1273-EV1 Longmorn" },
> +};
> +
> +static __devinit int wlf_gf_module_probe(struct i2c_client *i2c,
> +					 const struct i2c_device_id *i2c_id)
> +{
> +	int ret, i, j, id, rev;
> +
> +	ret = i2c_smbus_read_byte_data(i2c, 0);
> +	if (ret < 0) {
> +		dev_err(&i2c->dev, "Failed to read ID: %d\n", ret);
> +		return ret;
> +	}
> +
> +	id = (ret & 0xfe) >> 2;
> +	rev = ret & 0x3;
> +	for (i = 0; i < ARRAY_SIZE(gf_mods); i++)
> +		if (id == gf_mods[i].id)
> +			break;
> +
> +	if (i < ARRAY_SIZE(gf_mods)) {
> +		dev_info(&i2c->dev, "%s revision %d\n",
> +			 gf_mods[i].name, rev + 1);
> +		for (j = 0; j < gf_mods[i].num_i2c_devs; j++) {
> +			if (!i2c_new_device(i2c->adapter,
> +					    &(gf_mods[i].i2c_devs[j])))
> +				dev_err(&i2c->dev,
> +					"Failed to register dev: %d\n",
ret);
> +		}
> +	} else {
> +		dev_warn(&i2c->dev, "Unknown module ID %d revision %d\n",
> +			 id, rev);
> +	}
> +
> +	return 0;
> +}
> +
> +static const struct i2c_device_id wlf_gf_module_id[] = {
> +	{ "wlf-gf-module", 0 },
> +	{ }
> +};
> +
> +static struct i2c_driver wlf_gf_module_driver = {
> +	.driver = {
> +		.name = "wlf-gf-module",
> +		.owner = THIS_MODULE,
> +	},
> +	.probe = wlf_gf_module_probe,
> +	.id_table = wlf_gf_module_id,
> +};
> +
> +static int __init wlf_gf_module_register(void)
> +{
> +	return i2c_add_driver(&wlf_gf_module_driver);
> +}
> +module_init(wlf_gf_module_register);
> diff --git a/arch/arm/mach-s3c64xx/mach-crag6410.c b/arch/arm/mach-
> s3c64xx/mach-crag6410.c
> index 90baf1c..f74a811 100644
> --- a/arch/arm/mach-s3c64xx/mach-crag6410.c
> +++ b/arch/arm/mach-s3c64xx/mach-crag6410.c
> @@ -47,6 +47,7 @@
>  #include <mach/regs-sys.h>
>  #include <mach/regs-gpio.h>
>  #include <mach/regs-modem.h>
> +#include <mach/crag6410.h>
> 
>  #include <mach/regs-gpio-memport.h>
> 
> @@ -65,17 +66,6 @@
>  #include <plat/iic.h>
>  #include <plat/pm.h>
> 
> -#include <sound/wm8996.h>

Hmm...Why is this <sound/wm8915.h> in my tree?


> -#include <sound/wm8962.h>
> -#include <sound/wm9081.h>
> -
> -#define BANFF_PMIC_IRQ_BASE		IRQ_BOARD_START
> -#define GLENFARCLAS_PMIC_IRQ_BASE	(IRQ_BOARD_START + 64)
> -
> -#define PCA935X_GPIO_BASE		GPIO_BOARD_START
> -#define CODEC_GPIO_BASE		(GPIO_BOARD_START + 8)
> -#define GLENFARCLAS_PMIC_GPIO_BASE	(GPIO_BOARD_START + 16)
> -
>  /* serial port setup */
> 
>  #define UCON (S3C2410_UCON_DEFAULT | S3C2410_UCON_UCLK)
> @@ -620,81 +610,16 @@ static struct wm831x_pdata glenfarclas_pmic_pdata
> __initdata = {
>  	.disable_touch = true,
>  };
> 
> -static struct wm8996_retune_mobile_config wm8996_retune[] = {
> -	{
> -		.name = "Sub LPF",
> -		.rate = 48000,
> -		.regs = {
> -			0x6318, 0x6300, 0x1000, 0x0000, 0x0004, 0x2000,
> 0xF000,
> -			0x0000, 0x0004, 0x2000, 0xF000, 0x0000, 0x0004,
> 0x2000,
> -			0xF000, 0x0000, 0x0004, 0x1000, 0x0800, 0x4000
> -		},
> -	},
> -	{
> -		.name = "Sub HPF",
> -		.rate = 48000,
> -		.regs = {
> -			0x000A, 0x6300, 0x1000, 0x0000, 0x0004, 0x2000,
> 0xF000,
> -			0x0000, 0x0004, 0x2000, 0xF000, 0x0000, 0x0004,
> 0x2000,
> -			0xF000, 0x0000, 0x0004, 0x1000, 0x0800, 0x4000
> -		},
> -	},
> -};
> -
> -static struct wm8996_pdata wm8996_pdata __initdata = {
> -	.ldo_ena = S3C64XX_GPN(7),
> -	.gpio_base = CODEC_GPIO_BASE,
> -	.micdet_def = 1,
> -	.inl_mode = WM8996_DIFFERRENTIAL_1,
> -	.inr_mode = WM8996_DIFFERRENTIAL_1,
> -
> -	.irq_flags = IRQF_TRIGGER_RISING,
> -
> -	.gpio_default = {
> -		0x8001, /* GPIO1 == ADCLRCLK1 */
> -		0x8001, /* GPIO2 == ADCLRCLK2, input due to CPU */
> -		0x0141, /* GPIO3 == HP_SEL */
> -		0x0002, /* GPIO4 == IRQ */
> -		0x020e, /* GPIO5 == CLKOUT */
> -	},
> -
> -	.retune_mobile_cfgs = wm8996_retune,
> -	.num_retune_mobile_cfgs = ARRAY_SIZE(wm8996_retune),
> -};
> -
> -static struct wm8962_pdata wm8962_pdata __initdata = {
> -	.gpio_init = {
> -		0,
> -		WM8962_GPIO_FN_OPCLK,
> -		WM8962_GPIO_FN_DMICCLK,
> -		0,
> -		0x8000 | WM8962_GPIO_FN_DMICDAT,
> -		WM8962_GPIO_FN_IRQ,    /* Open drain mode */
> -	},
> -	.irq_active_low = true,
> -};
> -
> -static struct wm9081_pdata wm9081_pdata __initdata = {
> -	.irq_high = false,
> -	.irq_cmos = false,
> -};
> -
>  static struct i2c_board_info i2c_devs1[] __initdata = {
>  	{ I2C_BOARD_INFO("wm8311", 0x34),
>  	  .irq = S3C_EINT(0),
>  	  .platform_data = &glenfarclas_pmic_pdata },
> 
> +	{ I2C_BOARD_INFO("wlf-gf-module", 0x24) },
> +	{ I2C_BOARD_INFO("wlf-gf-module", 0x25) },
> +	{ I2C_BOARD_INFO("wlf-gf-module", 0x26) },
> +
>  	{ I2C_BOARD_INFO("wm1250-ev1", 0x27) },
> -	{ I2C_BOARD_INFO("wm8996", 0x1a),
> -	  .platform_data = &wm8996_pdata,
> -	  .irq = GLENFARCLAS_PMIC_IRQ_BASE + WM831X_IRQ_GPIO_2,
> -	},
> -	{ I2C_BOARD_INFO("wm9081", 0x6c),
> -	  .platform_data = &wm9081_pdata, },
> -	{ I2C_BOARD_INFO("wm8962", 0x1a),
> -	  .platform_data = &wm8962_pdata,
> -	  .irq = GLENFARCLAS_PMIC_IRQ_BASE + WM831X_IRQ_GPIO_2,
> -	},
>  };
> 
>  static void __init crag6410_map_io(void)
> --
> 1.7.5.4

Hi Mark,

Probably, this is v3 not v2 ;)
And my tree should be against on -rc2 not -rc1.

Anyway, applied.
Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.




More information about the linux-arm-kernel mailing list