From: Rajeshwari Birje <rajeshwari.birje@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 7/7] EXYNOS5: Snow: Add a configuration file
Date: Wed, 23 Jan 2013 15:43:08 +0530 [thread overview]
Message-ID: <CAPs=JDdUJFz1AiKGbVj+GmyS-yKRPHkHObX=cjk-DLvUHpuOjg@mail.gmail.com> (raw)
In-Reply-To: <CAPnjgZ2E9oLegzvV9OpcBFHto0BxX4f9To0HBt2K9fiiOyp0Og@mail.gmail.com>
Hi Simon,
Thank you for comments.
On Tue, Jan 22, 2013 at 8:28 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 adds the configuration file for Snow Board and
>> defines the same in boards.cfg.
>> The Audio codec required for SMDK5250 and Snow are different
>> hence they are defined in the corresponding configuration files.
>>
>> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com>
>> ---
>> boards.cfg | 1 +
>> include/configs/exynos5250-dt.h | 1 -
>> include/configs/smdk5250.h | 1 +
>> include/configs/snow.h | 34 ++++++++++++++++++++++++++++++++++
>> 4 files changed, 36 insertions(+), 1 deletions(-)
>> create mode 100644 include/configs/snow.h
>>
>> diff --git a/boards.cfg b/boards.cfg
>> index e4b0d44..f247b03 100644
>> --- a/boards.cfg
>> +++ b/boards.cfg
>> @@ -283,6 +283,7 @@ s5p_goni arm armv7 goni samsung
>> smdkc100 arm armv7 smdkc100 samsung s5pc1xx
>> origen arm armv7 origen samsung exynos
>> s5pc210_universal arm armv7 universal_c210 samsung exynos
>> +snow arm armv7 smdk5250 samsung exynos
>> smdk5250 arm armv7 smdk5250 samsung exynos
>> smdkv310 arm armv7 smdkv310 samsung exynos
>> trats arm armv7 trats samsung exynos
>> diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h
>> index a01fb96..7b9c393 100644
>> --- a/include/configs/exynos5250-dt.h
>> +++ b/include/configs/exynos5250-dt.h
>> @@ -297,7 +297,6 @@
>> #ifdef CONFIG_CMD_SOUND
>> #define CONFIG_SOUND
>> #define CONFIG_I2S
>> -#define CONFIG_SOUND_WM8994
>> #endif
>>
>> /* Enable devicetree support */
>> diff --git a/include/configs/smdk5250.h b/include/configs/smdk5250.h
>> index 81f83a8..b23b5bc 100644
>> --- a/include/configs/smdk5250.h
>> +++ b/include/configs/smdk5250.h
>> @@ -30,4 +30,5 @@
>> #undef CONFIG_DEFAULT_DEVICE_TREE
>> #define CONFIG_DEFAULT_DEVICE_TREE exynos5250-smdk5250
>>
>> +#define CONFIG_SOUND_WM8994
>> #endif /* __CONFIG_SMDK_H */
>> diff --git a/include/configs/snow.h b/include/configs/snow.h
>> new file mode 100644
>> index 0000000..be38635
>> --- /dev/null
>> +++ b/include/configs/snow.h
>> @@ -0,0 +1,34 @@
>> +/*
>> + * Copyright (C) 2012 Samsung Electronics
>> + *
>> + * Configuration settings for the SAMSUNG SMDK5250 board.
>> + *
>> + * See file CREDITS for list of people who contributed to this
>> + * project.
>> + *
>> + * This program is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU General Public License as
>> + * published by the Free Software Foundation; either version 2 of
>> + * the License, or (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + *
>> + * You should have received a copy of the GNU General Public License
>> + * along with this program; if not, write to the Free Software
>> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
>> + * MA 02111-1307 USA
>> + */
>> +
>> +#ifndef __CONFIG_SMDK_H
>> +#define __CONFIG_SMDK_H
>> +
>> +#include <configs/exynos5250-dt.h>
>> +
>> +#undef CONFIG_DEFAULT_DEVICE_TREE
>> +#define CONFIG_DEFAULT_DEVICE_TREE exynos5250-snow
>> +
>> +#define CONFIG_SOUND_MAX98095
>
> Really it would be better if they could share the same config, with
> just the CONFIG_DEFAULT_DEVICE_TREE difference. Isn't there code to
> select the right codec based on the FDT nodes?
>
> Maybe I have this wrong, not sure.
Actually we do decode the codec from DTS file.
But we need to define this to compile the corresponding codec file.
And I felt since the codec is different for both the boards we can add
them in the respective config file.
Also in case of Non FDT we define the values based on this codec define.
>
>> +#endif /* __CONFIG_SMDK_H */
>> --
>> 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
next prev parent reply other threads:[~2013-01-23 10:13 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-21 10:52 [U-Boot] [PATCH 0/7] EXYNOS5: Add audio support for Snow Rajeshwari Shinde
2013-01-21 10:52 ` [U-Boot] [PATCH 1/7] EXYNOS5: Add function to enable XXTI clock source Rajeshwari Shinde
2013-01-22 14:51 ` Simon Glass
2013-01-21 10:52 ` [U-Boot] [PATCH 2/7] Sound: MAX98095: Add the driver for codec Rajeshwari Shinde
2013-01-21 10:52 ` [U-Boot] [PATCH 3/7] Sound: Support for MAX98095 codec in driver Rajeshwari Shinde
2013-01-21 10:52 ` [U-Boot] [PATCH 4/7] EXYNOS5: GPIO to enable MAX98095 Rajeshwari Shinde
2013-01-22 14:55 ` Simon Glass
2013-01-23 10:05 ` Rajeshwari Birje
2013-01-25 20:40 ` Simon Glass
2013-01-21 10:52 ` [U-Boot] [PATCH 5/7] EXYNOS5: FDT: Add compatible strings for MAX98095 Rajeshwari Shinde
2013-01-21 10:52 ` [U-Boot] [PATCH 6/7] EXYNOS5: Add initial DTS file for Snow Rajeshwari Shinde
2013-01-22 14:56 ` Simon Glass
2013-01-25 20:42 ` Simon Glass
2013-01-21 10:52 ` [U-Boot] [PATCH 7/7] EXYNOS5: Snow: Add a configuration file Rajeshwari Shinde
2013-01-22 14:58 ` Simon Glass
2013-01-23 10:13 ` Rajeshwari Birje [this message]
2013-01-25 21:03 ` Simon Glass
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='CAPs=JDdUJFz1AiKGbVj+GmyS-yKRPHkHObX=cjk-DLvUHpuOjg@mail.gmail.com' \
--to=rajeshwari.birje@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.