From: cw00.choi@samsung•com (Chanwoo Choi)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH 1/2] ARM: S5PV210: Add audio support to Aquila
Date: Fri, 30 Jul 2010 16:15:29 +0900 [thread overview]
Message-ID: <4C527C11.5090409@samsung.com> (raw)
In-Reply-To: <20100729173825.GE24781@opensource.wolfsonmicro.com>
Mark Brown wrote:
> On Wed, Jul 28, 2010 at 12:04:44PM +0900, Chanwoo Choi wrote:
>
>> +static struct regulator_consumer_supply wm8994_fixed_voltage0_supplies[] = {
>> + {
>> + .dev_name = "5-001a",
>> + .supply = "DBVDD",
>> + }, {
>> + .dev_name = "5-001a",
>> + .supply = "AVDD2",
>> + }, {
>> + .dev_name = "5-001a",
>> + .supply = "CPVDD",
>> + },
>> +
>> +};
>> +
>> +static struct regulator_consumer_supply wm8994_fixed_voltage1_supplies[] = {
>
> All these fixed voltage regulators seem a bit suspicous for a mobile
> phone - I'd have expected that the supplies would all be being provided
> by your PMIC except for things taken directly from the battery supply
> (like the speakers tend to be, for example)? There's no problem with
> the code itself, it just looks a bit odd.
>
All these consumer supply of WM8994 codec connected the below regulator(VCC_1.8V) on
a circuit diagram. "VCC_1.8V" regualtor is always enabled, because it is used to many devices.
Then I haven't connected all these consumer supply of WM8994 codec to "VCC_1.8V" regulator.
I will modify that the consumer supply would be provided by PMIC.
static struct regulator_init_data aquila_buck3_data = {
.constraints = {
.name = "VCC_1.8V",
.min_uV = 1800000,
.max_uV = 1800000,
.apply_uV = 1,
.state_mem = {
.enabled = 1,
},
},
};
>> +static struct i2c_board_info i2c_gpio5_devs[] __initdata = {
>> + {
>> + /* CS/ADDR = low 0x34 (FYI: high = 0x36) */
>> + I2C_BOARD_INFO("wm8994", 0x34 >> 1),
>> + .platform_data = &wm8994_platform_data,
>> + },
>> +};
>
> Probably clearer for generic Linux use to specify the address as 0x1a
> directly.
Ok, I will do.
>
>> +static void __init aquila_sound_init(void)
>> +{
>> + unsigned int gpio;
>> +
>> + /* CODEC_XTAL_EN */
>> + gpio = S5PV210_GPH3(2); /* XEINT_26 */
>> + gpio_request(gpio, "CODEC_XTAL_EN");
>> + s3c_gpio_cfgpin(gpio, S3C_GPIO_OUTPUT);
>> + s3c_gpio_setpull(gpio, S3C_GPIO_PULL_NONE);
>> + gpio_direction_output(gpio, 1);
>
> Might be as well to provide some or all this stuff in your audio machine
> driver?
The Aquila board have a oscillator which provide main clock to
WM8994 audio codec. The oscillator provide 24MHz clock to WM8994 audio codec
(MCLK1 pin). I set gpio setting of "CODEC_XTAL_EN" to enable a oscillator.
>
>> + /* MICBIAS_EN */
>> + gpio = S5PV210_GPJ4(2); /* XMSMRN */
>> + gpio_request(gpio, "MICBIAS_EN");
>> + s3c_gpio_cfgpin(gpio, S3C_GPIO_OUTPUT);
>> + gpio_direction_output(gpio, 1);
>
> This in particular would benefit from keeping the request of the GPIO
> joined up with the driver that uses it.
Ok, I will move this code to machine driver(sound/soc/s3c24xxx/aquila_wm8994.c).
>> + /* ADC_EN */
>> + gpio = S5PV210_GPJ3(2);
>> + gpio_request(gpio, "ADC_EN");
>> + s3c_gpio_cfgpin(gpio, S3C_GPIO_OUTPUT);
>> + gpio_direction_output(gpio, 1);
>
> I'm not sure what this does?
>
I explained below description about "ADC_EN" :
"ADC_EN : This gpio enable the ADC device which is used to detect
the kind of jack. (SND_JACK_HEADPHONE/HEADSET/MECHANICAL/AVOUT)
According to the kind of jack, an electric current is changed.
(Only used on Aquila board) "
When inserting the jack to Aquila board, I used ADC driver so that, detecting
the kind of jack(SND_JACK_HEADPHONE/HEADSET/MECHANICAL/AVOUT).
I will separately make the another function to initialize ADC driver.
Thank you for your comment.
Chanwoo Choi
next prev parent reply other threads:[~2010-07-30 7:15 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-28 3:04 [PATCH 1/2] ARM: S5PV210: Add audio support to Aquila Chanwoo Choi
2010-07-29 17:38 ` Mark Brown
2010-07-30 7:15 ` Chanwoo Choi [this message]
2010-07-30 7:34 ` MyungJoo Ham
2010-08-03 8:59 ` Mark Brown
2010-08-04 6:16 ` [alsa-devel] " Chanwoo Choi
2010-08-04 8:19 ` Mark Brown
2010-09-29 12:16 ` Kukjin Kim
2010-09-30 1:23 ` Chanwoo Choi
2010-09-30 1:50 ` Kukjin Kim
2010-09-30 4:55 ` Chanwoo Choi
2010-07-29 22:48 ` Ben Dooks
2010-07-30 0:14 ` Kyungmin Park
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4C527C11.5090409@samsung.com \
--to=cw00.choi@samsung$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox