[PATCH 4/4] introduce-gta02-beachhead.patch

[Andy Green]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Somebody in the thread at some point said:

|> +	reg_init_data[PCF50633_REGULATOR_LDO6].constraints.min_uV = 3000000;
|> +	reg_init_data[PCF50633_REGULATOR_LDO6].constraints.max_uV = 3000000;
|> +	reg_init_data[PCF50633_REGULATOR_LDO6].constraints.apply_uV = 1;
|> +}
|
| It looks like the adjustments of the constraints could be removed here
| since you only support one possible configuration?

I just chopped down what we had already without much thought, but don't
these guys turn up in /sys?  We should constrain them if they're settable.

|> +static struct i2c_board_info gta02_i2c_devs[] __initdata = {
|> +	{
|> +		I2C_BOARD_INFO("pcf50633", 0x73),
|> +		.irq = GTA02_IRQ_PCF50633,
|> +		.platform_data = &gta02_pcf_pdata,
|> +	},
|
| You probably want to add the WM8753 here as well.  It should be in here
| and I expect to have the WM8753 driver updated to use it by the next
| merge window.

Ah we ran into trouble with that somehow when this stuff changed a
couple of Linux versions ago.  As it stands, we instantiate the audio
i2c device inside neo1973_gta02_wm8753.c.  If it's evil, it's fine to
change it since we're only using it because it "works".  But as it
stands it means including neo1973_gta02_wm8753.c in the patchset, is
that aligned to what your plans are in this area?

- -Andy
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iEYEARECAAYFAkmfdx0ACgkQOjLpvpq7dMqgNwCfWqLZhsmjA+vzg2VYyiOIQxD4
OCYAn06h4LvU0WP+ZHsLv/MpWyMir6GH
=Bx9d
-----END PGP SIGNATURE-----