Hello.
On 10/11/10 21:08, Victor Rodriguez wrote:
From: Victor Rodriguez<[email protected]>
This patch adds MMC/SD support for the Hawkboard-L138 system
It is under the machine name "omapl138_hawkboard".
This system is based on the da850 davinci CPU architecture.
Signed-off-by: Victor Rodriguez<[email protected]>
[...]
diff --git a/arch/arm/mach-davinci/board-omapl138-hawk.c
b/arch/arm/mach-davinci/board-omapl138-hawk.c
index ba3718a..ae7f75c 100644
--- a/arch/arm/mach-davinci/board-omapl138-hawk.c
+++ b/arch/arm/mach-davinci/board-omapl138-hawk.c
[...]
@@ -180,6 +201,28 @@ static __init void omapl138_hawk_init(void)
"mcasp mux setup failed: %d\n", ret);
da8xx_register_mcasp(0,&omapl138_hawk_snd_data);
+ ret = davinci_cfg_reg_list(da850_mmcsd0_pins);
No, you can't use the generic pin list. You should add to it the two GPIOs
used as WP/CD inputs, and define the new list in the board file.
Ok but will have to also modify the more files
The other patch could be (please omit the board-omapl138-hawk.c
modification that will be changed in just one patch it is just for
compare previous and possible code)
---
arch/arm/mach-davinci/board-omapl138-hawk.c | 9 ++++++++-
arch/arm/mach-davinci/da850.c | 2 ++
arch/arm/mach-davinci/include/mach/mux.h | 2 ++
3 files changed, 12 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-davinci/board-omapl138-hawk.c
b/arch/arm/mach-davinci/board-omapl138-hawk.c
index 4a39249..8a9f9c0 100644
--- a/arch/arm/mach-davinci/board-omapl138-hawk.c
+++ b/arch/arm/mach-davinci/board-omapl138-hawk.c
@@ -151,6 +151,13 @@ static struct snd_platform_data omapl138_hawk_snd_data = {
.rxnumevt = 1,
};
+static const short hawk_mmcsd0_pins[] = {
+ DA850_MMCSD0_DAT_0, DA850_MMCSD0_DAT_1, DA850_MMCSD0_DAT_2,
+ DA850_MMCSD0_DAT_3, DA850_MMCSD0_CLK, DA850_MMCSD0_CMD,
+ DA850_GPIO3_12, DA850_GPIO3_13,
+ -1
+};
+
static int da850_hawk_mmc_get_ro(int index)
{
return gpio_get_value(DA850_HAWK_MMCSD_WP_PIN);
@@ -200,7 +207,7 @@ static __init void omapl138_hawk_init(void)
pr_warning("%s: mcasp mux setup failed: %d\n", __func__, ret);
da8xx_register_mcasp(0,&omapl138_hawk_snd_data);
- ret = davinci_cfg_reg_list(da850_mmcsd0_pins);
+ ret = davinci_cfg_reg_list(hawk_mmcsd0_pins);
if (ret)
pr_warning("%s: MMC/SD0 mux setup failed: %d\n",
__func__, ret);
diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c
index f033a0a..7787f5a 100644
--- a/arch/arm/mach-davinci/da850.c
+++ b/arch/arm/mach-davinci/da850.c
@@ -548,6 +548,8 @@ static const struct mux_config da850_pins[] = {
MUX_CFG(DA850, GPIO2_15, 5, 0, 15, 8, false)
MUX_CFG(DA850, GPIO4_0, 10, 28, 15, 8, false)
MUX_CFG(DA850, GPIO4_1, 10, 24, 15, 8, false)
+ MUX_CFG(DA850, GPIO3_12, 7, 12, 15, 8, false)
+ MUX_CFG(DA850, GPIO3_13, 7, 8, 15, 8, false)
MUX_CFG(DA850, RTC_ALARM, 0, 28, 15, 2, false)
#endif
};
diff --git a/arch/arm/mach-davinci/include/mach/mux.h
b/arch/arm/mach-davinci/include/mach/mux.h
index de11aac..e043f11 100644
--- a/arch/arm/mach-davinci/include/mach/mux.h
+++ b/arch/arm/mach-davinci/include/mach/mux.h
@@ -913,6 +913,8 @@ enum davinci_da850_index {
DA850_GPIO2_15,
DA850_GPIO4_0,
DA850_GPIO4_1,
+ DA850_GPIO3_12,
+ DA850_GPIO3_13,
DA850_RTC_ALARM,
};
I could make tow more pathces one for
arch/arm/mach-davinci/da850.c | 2 ++
and other for
arch/arm/mach-davinci/include/mach/mux.h | 2 ++
Is this Ok ?
No, this should be one patch. And you may also join it with the USB 1.1 GPIO
PinMux definitions patch.
WBR, Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
