From mboxrd@z Thu Jan  1 00:00:00 1970
From: Rajeshwari Birje <rajeshwari.birje@gmail.com>
Date: Wed, 23 Jan 2013 15:35:01 +0530
Subject: [U-Boot] [PATCH 4/7] EXYNOS5: GPIO to enable MAX98095
In-Reply-To: <CAPnjgZ0eQb7-YwHS2Ss11_zTZSLM98-S8eAE68E9-iqy2ygS4g@mail.gmail.com>
References: <1358765559-32709-1-git-send-email-rajeshwari.s@samsung.com>
	<1358765559-32709-5-git-send-email-rajeshwari.s@samsung.com>
	<CAPnjgZ0eQb7-YwHS2Ss11_zTZSLM98-S8eAE68E9-iqy2ygS4g@mail.gmail.com>
Message-ID: <CAPs=JDep4=KOWo7By4bq-x66wK-h4y5RnvRztHHZgo5HosU-mA@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Simon,

Thank you for comments.

On Tue, Jan 22, 2013 at 8:25 PM, Simon Glass <sjg@chromium.org> wrote:
> Hi Rajeshwari,
>
> On Mon, Jan 21, 2013 at 2:52 AM, Rajeshwari Shinde
> <rajeshwari.s@samsung.com> wrote:
>> This patch sets high a GPIO to enable the codec MAX98095
>>
>> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com>
>> ---
>>  board/samsung/smdk5250/smdk5250.c |   15 +++++++++++++++
>>  1 files changed, 15 insertions(+), 0 deletions(-)
>>
>> diff --git a/board/samsung/smdk5250/smdk5250.c b/board/samsung/smdk5250/smdk5250.c
>> index 12cc03e..6f2e067 100644
>> --- a/board/samsung/smdk5250/smdk5250.c
>> +++ b/board/samsung/smdk5250/smdk5250.c
>> @@ -56,6 +56,18 @@ int board_usb_vbus_init(void)
>>  }
>>  #endif
>>
>> +#ifdef CONFIG_SOUND_MAX98095
>> +static void  board_enable_audio_codec(void)
>> +{
>> +       struct exynos5_gpio_part1 *gpio1 = (struct exynos5_gpio_part1 *)
>> +                                               samsung_get_base_gpio_part1();
>> +
>> +       /* Enable MAX98095 Codec */
>> +       s5p_gpio_direction_output(&gpio1->x1, 7, 1);
>> +       s5p_gpio_set_pull(&gpio1->x1, 7, GPIO_PULL_NONE);
>
> This GPIO is hard-coded - does the FDT version of this file do this differently?
For FDT support of same we need gpio numbering feature which is still
not merged in the mainline code.
Hence currently the same has been hard coded for MAX98095 on Snow.
>
>> +}
>> +#endif
>> +
>>  int board_init(void)
>>  {
>>         gd->bd->bi_boot_params = (PHYS_SDRAM_1 + 0x100UL);
>> @@ -65,6 +77,9 @@ int board_init(void)
>>  #ifdef CONFIG_USB_EHCI_EXYNOS
>>         board_usb_vbus_init();
>>  #endif
>> +#ifdef CONFIG_SOUND_MAX98095
>> +       board_enable_audio_codec();
>> +#endif
>>         return 0;
>>  }
>>
>> --
>> 1.7.4.4
>>
>
> Regards,
> Simon
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-- 
Regards,
Rajeshwari Shinde