* [PATCH v3] omap: beagle: add support for wl1271 on the board file
@ 2010-09-23 10:03 Luciano Coelho
[not found] ` <1285236215-21136-1-git-send-email-luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
0 siblings, 1 reply; 10+ messages in thread
From: Luciano Coelho @ 2010-09-23 10:03 UTC (permalink / raw)
To: linux-omap; +Cc: tony, linux-wireless, Ido Yariv, Ohad Ben-Cohen
Add board configuration for the wl1271 daughter board. This patch is based
on Ohad Ben-Cohen's patches for Zoom boards.
Cc: Ohad Ben-Cohen <ohad@wizery.com>
Signed-off-by: Luciano Coelho <luciano.coelho@nokia.com>
---
from v2 to v3: removed accidental change to wl1271.h
arch/arm/mach-omap2/board-omap3beagle.c | 69 +++++++++++++++++++++++++++++++
1 files changed, 69 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index 87969c7..755df29 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -29,7 +29,9 @@
#include <linux/mtd/nand.h>
#include <linux/regulator/machine.h>
+#include <linux/regulator/fixed.h>
#include <linux/i2c/twl.h>
+#include <linux/wl12xx.h>
#include <mach/hardware.h>
#include <asm/mach-types.h>
@@ -48,6 +50,9 @@
#include "mux.h"
#include "hsmmc.h"
+#define OMAP_BEAGLE_WLAN_EN_GPIO (139)
+#define OMAP_BEAGLE_WLAN_IRQ_GPIO (137)
+
#define NAND_BLOCK_SIZE SZ_128K
static struct mtd_partition omap3beagle_nand_partitions[] = {
@@ -163,12 +168,25 @@ static void __init beagle_display_init(void)
#include "sdram-micron-mt46h32m32lf-6.h"
+struct wl12xx_platform_data omap_beagle_wlan_data __initdata = {
+ .irq = OMAP_GPIO_IRQ(OMAP_BEAGLE_WLAN_IRQ_GPIO),
+ .board_ref_clock = 2, /* 38.4 MHz */
+};
+
static struct omap2_hsmmc_info mmc[] = {
{
.mmc = 1,
.wires = 8,
.gpio_wp = 29,
},
+ {
+ .name = "wl1271",
+ .mmc = 2,
+ .wires = 4,
+ .gpio_wp = -EINVAL,
+ .gpio_cd = -EINVAL,
+ .nonremovable = true,
+ },
{} /* Terminator */
};
@@ -176,10 +194,43 @@ static struct regulator_consumer_supply beagle_vmmc1_supply = {
.supply = "vmmc",
};
+static struct regulator_consumer_supply beagle_vmmc2_supply = {
+ .supply = "vmmc",
+ .dev_name = "mmci-omap-hs.1",
+};
+
+
static struct regulator_consumer_supply beagle_vsim_supply = {
.supply = "vmmc_aux",
};
+
+static struct regulator_init_data beagle_vmmc2 = {
+ .constraints = {
+ .valid_ops_mask = REGULATOR_CHANGE_STATUS,
+ },
+ .num_consumer_supplies = 1,
+ .consumer_supplies = &beagle_vmmc2_supply,
+};
+
+static struct fixed_voltage_config beagle_vwlan = {
+ .supply_name = "vwl1271",
+ .microvolts = 1800000, /* 1.8V */
+ .gpio = OMAP_BEAGLE_WLAN_EN_GPIO,
+ .startup_delay = 70000, /* 70ms */
+ .enable_high = 1,
+ .enabled_at_boot = 0,
+ .init_data = &beagle_vmmc2,
+};
+
+static struct platform_device omap_vwlan_device = {
+ .name = "reg-fixed-voltage",
+ .id = 1,
+ .dev = {
+ .platform_data = &beagle_vwlan,
+ },
+};
+
static struct gpio_led gpio_leds[];
static int beagle_twl_gpio_setup(struct device *dev,
@@ -449,6 +500,19 @@ static const struct ehci_hcd_omap_platform_data ehci_pdata __initconst = {
#ifdef CONFIG_OMAP_MUX
static struct omap_board_mux board_mux[] __initdata = {
+ /* WLAN IRQ - GPIO 137 */
+ OMAP3_MUX(SDMMC2_DAT5, OMAP_MUX_MODE4 | OMAP_PIN_INPUT_PULLUP),
+ /* WLAN POWER ENABLE - GPIO 139 */
+ OMAP3_MUX(SDMMC2_DAT7, OMAP_MUX_MODE4 | OMAP_PIN_OUTPUT),
+ /* WLAN SDIO: MMC2 CMD */
+ OMAP3_MUX(SDMMC2_CMD, OMAP_MUX_MODE0 | OMAP_PIN_INPUT_PULLUP),
+ /* WLAN SDIO: MMC2 CLK */
+ OMAP3_MUX(SDMMC2_CLK, OMAP_MUX_MODE0 | OMAP_PIN_INPUT_PULLUP),
+ /* WLAN SDIO: MMC2 DAT[0-3] */
+ OMAP3_MUX(SDMMC2_DAT0, OMAP_MUX_MODE0 | OMAP_PIN_INPUT_PULLUP),
+ OMAP3_MUX(SDMMC2_DAT1, OMAP_MUX_MODE0 | OMAP_PIN_INPUT_PULLUP),
+ OMAP3_MUX(SDMMC2_DAT2, OMAP_MUX_MODE0 | OMAP_PIN_INPUT_PULLUP),
+ OMAP3_MUX(SDMMC2_DAT3, OMAP_MUX_MODE0 | OMAP_PIN_INPUT_PULLUP),
{ .reg_offset = OMAP_MUX_TERMINATOR },
};
#else
@@ -464,9 +528,14 @@ static struct omap_musb_board_data musb_board_data = {
static void __init omap3_beagle_init(void)
{
omap3_mux_init(board_mux, OMAP_PACKAGE_CBB);
+ if (wl12xx_set_platform_data(&omap_beagle_wlan_data))
+ pr_err("error setting wl12xx data\n");
+
omap3_beagle_i2c_init();
platform_add_devices(omap3_beagle_devices,
ARRAY_SIZE(omap3_beagle_devices));
+ platform_device_register(&omap_vwlan_device);
+
omap_serial_init();
omap_mux_init_gpio(170, OMAP_PIN_INPUT);
--
1.6.3.3
^ permalink raw reply related [flat|nested] 10+ messages in thread[parent not found: <1285236215-21136-1-git-send-email-luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>]
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file [not found] ` <1285236215-21136-1-git-send-email-luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> @ 2010-09-24 18:03 ` John W. Linville 2010-09-24 18:50 ` Luciano Coelho 0 siblings, 1 reply; 10+ messages in thread From: John W. Linville @ 2010-09-24 18:03 UTC (permalink / raw) To: Luciano Coelho Cc: linux-omap-u79uwXL29TY76Z2rM5mHXA, tony-4v6yS6AI5VpBDgjK7y7TUQ, linux-wireless-u79uwXL29TY76Z2rM5mHXA, Ido Yariv, Ohad Ben-Cohen On Thu, Sep 23, 2010 at 01:03:35PM +0300, Luciano Coelho wrote: > Add board configuration for the wl1271 daughter board. This patch is based > on Ohad Ben-Cohen's patches for Zoom boards. > > Cc: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> > Signed-off-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> > --- > from v2 to v3: removed accidental change to wl1271.h I presume that this should come through my tree as well (since it contains the move of wl12xx.h)? John -- John W. Linville Someday the world will need a hero, and you linville-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org might be all we have. Be ready. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file 2010-09-24 18:03 ` John W. Linville @ 2010-09-24 18:50 ` Luciano Coelho 2010-09-24 19:11 ` Tony Lindgren 0 siblings, 1 reply; 10+ messages in thread From: Luciano Coelho @ 2010-09-24 18:50 UTC (permalink / raw) To: ext John W. Linville Cc: linux-omap@vger.kernel.org, tony@atomide.com, linux-wireless@vger.kernel.org, Ido Yariv, Ohad Ben-Cohen On Fri, 2010-09-24 at 20:03 +0200, ext John W. Linville wrote: > On Thu, Sep 23, 2010 at 01:03:35PM +0300, Luciano Coelho wrote: > > Add board configuration for the wl1271 daughter board. This patch is based > > on Ohad Ben-Cohen's patches for Zoom boards. > > > > Cc: Ohad Ben-Cohen <ohad@wizery.com> > > Signed-off-by: Luciano Coelho <luciano.coelho@nokia.com> > > --- > > from v2 to v3: removed accidental change to wl1271.h > > I presume that this should come through my tree as well (since it > contains the move of wl12xx.h)? Yes, this is based on your tree. But please hold on, because there were some comments (namely, the expansion board should be detected, not configured for wl1271 by default), which I still have to fix. I'm waiting for the manufacturer of the expansion board to provide me with some more info for proper detection. Also, I know that there are some other work being done in the beagle board file, so we may need to rethink how to sync this. So, for now, let's skip this patch and I'll provide another one when these issues are addressed. -- Cheers, Luca. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file 2010-09-24 18:50 ` Luciano Coelho @ 2010-09-24 19:11 ` Tony Lindgren 2010-09-24 19:21 ` Gadiyar, Anand [not found] ` <20100924191139.GI4211-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> 0 siblings, 2 replies; 10+ messages in thread From: Tony Lindgren @ 2010-09-24 19:11 UTC (permalink / raw) To: Luciano Coelho Cc: ext John W. Linville, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Ido Yariv, Ohad Ben-Cohen * Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> [100924 11:42]: > On Fri, 2010-09-24 at 20:03 +0200, ext John W. Linville wrote: > > On Thu, Sep 23, 2010 at 01:03:35PM +0300, Luciano Coelho wrote: > > > Add board configuration for the wl1271 daughter board. This patch is based > > > on Ohad Ben-Cohen's patches for Zoom boards. > > > > > > Cc: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> > > > Signed-off-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> > > > --- > > > from v2 to v3: removed accidental change to wl1271.h > > > > I presume that this should come through my tree as well (since it > > contains the move of wl12xx.h)? > > Yes, this is based on your tree. But please hold on, because there were > some comments (namely, the expansion board should be detected, not > configured for wl1271 by default), which I still have to fix. I'm > waiting for the manufacturer of the expansion board to provide me with > some more info for proper detection. > > Also, I know that there are some other work being done in the beagle > board file, so we may need to rethink how to sync this. > > So, for now, let's skip this patch and I'll provide another one when > these issues are addressed. You could also do it based on some kernel cmdline option. Of course detection during the runtime would be better. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file 2010-09-24 19:11 ` Tony Lindgren @ 2010-09-24 19:21 ` Gadiyar, Anand 2010-09-24 21:08 ` Tony Lindgren [not found] ` <20100924191139.GI4211-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> 1 sibling, 1 reply; 10+ messages in thread From: Gadiyar, Anand @ 2010-09-24 19:21 UTC (permalink / raw) To: Tony Lindgren Cc: Luciano Coelho, ext John W. Linville, linux-omap@vger.kernel.org, linux-wireless@vger.kernel.org, Ido Yariv, Ohad Ben-Cohen On Sat, Sep 25, 2010 at 12:41 AM, Tony Lindgren <tony@atomide.com> wrote: > * Luciano Coelho <luciano.coelho@nokia.com> [100924 11:42]: >> On Fri, 2010-09-24 at 20:03 +0200, ext John W. Linville wrote: >> > On Thu, Sep 23, 2010 at 01:03:35PM +0300, Luciano Coelho wrote: >> > > Add board configuration for the wl1271 daughter board. This patch is based >> > > on Ohad Ben-Cohen's patches for Zoom boards. >> > > >> > > Cc: Ohad Ben-Cohen <ohad@wizery.com> >> > > Signed-off-by: Luciano Coelho <luciano.coelho@nokia.com> >> > > --- >> > > from v2 to v3: removed accidental change to wl1271.h >> > >> > I presume that this should come through my tree as well (since it >> > contains the move of wl12xx.h)? >> >> Yes, this is based on your tree. But please hold on, because there were >> some comments (namely, the expansion board should be detected, not >> configured for wl1271 by default), which I still have to fix. I'm >> waiting for the manufacturer of the expansion board to provide me with >> some more info for proper detection. >> >> Also, I know that there are some other work being done in the beagle >> board file, so we may need to rethink how to sync this. >> >> So, for now, let's skip this patch and I'll provide another one when >> these issues are addressed. > > You could also do it based on some kernel cmdline option. Of course > detection during the runtime would be better. > This seems to be a generic problem that many boards with potential for adding expansion boards seem to have. How about one of these options: (a) having u-boot modify ATAGS_REVISION based on an environment variable or (b) add a generic board_revision kernel command line parameter that each such board file can parse By default - with no special command line options - peripherals for the most common board revision could be picked. Anyway a user that uses such a daughterboard would know to modify the kernel command line, or set an environment variable in u-boot, (or use a custom u-boot) so this could scale well. What do you think? - Anand -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file 2010-09-24 19:21 ` Gadiyar, Anand @ 2010-09-24 21:08 ` Tony Lindgren 0 siblings, 0 replies; 10+ messages in thread From: Tony Lindgren @ 2010-09-24 21:08 UTC (permalink / raw) To: Gadiyar, Anand Cc: Luciano Coelho, ext John W. Linville, linux-omap@vger.kernel.org, linux-wireless@vger.kernel.org, Ido Yariv, Ohad Ben-Cohen * Gadiyar, Anand <gadiyar@ti.com> [100924 12:13]: > On Sat, Sep 25, 2010 at 12:41 AM, Tony Lindgren <tony@atomide.com> wrote: > > This seems to be a generic problem that many boards with potential for > adding expansion boards seem to have. How about one of these options: > (a) having u-boot modify ATAGS_REVISION based on an environment variable > or > (b) add a generic board_revision kernel command line parameter that > each such board file can parse There is already system_rev that can be used for that and has an ARM generic atag for it. Just needs to be passed from u-boot, and should have setenv option for it in u-boot also. Tony ^ permalink raw reply [flat|nested] 10+ messages in thread
[parent not found: <20100924191139.GI4211-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>]
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file [not found] ` <20100924191139.GI4211-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> @ 2010-09-24 19:42 ` Koen Kooi [not found] ` <8004B807-6120-41C0-979B-214BFB75E5CA-QLwJDigV5abLmq1fohREcCpxlwaOVQ5f@public.gmane.org> 2010-09-24 19:55 ` Luciano Coelho 1 sibling, 1 reply; 10+ messages in thread From: Koen Kooi @ 2010-09-24 19:42 UTC (permalink / raw) To: Tony Lindgren Cc: Luciano Coelho, ext John W. Linville, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Ido Yariv, Ohad Ben-Cohen Op 24 sep 2010, om 21:11 heeft Tony Lindgren het volgende geschreven: > * Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> [100924 11:42]: >> On Fri, 2010-09-24 at 20:03 +0200, ext John W. Linville wrote: >>> On Thu, Sep 23, 2010 at 01:03:35PM +0300, Luciano Coelho wrote: >>>> Add board configuration for the wl1271 daughter board. This patch is based >>>> on Ohad Ben-Cohen's patches for Zoom boards. >>>> >>>> Cc: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> >>>> Signed-off-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> >>>> --- >>>> from v2 to v3: removed accidental change to wl1271.h >>> >>> I presume that this should come through my tree as well (since it >>> contains the move of wl12xx.h)? >> >> Yes, this is based on your tree. But please hold on, because there were >> some comments (namely, the expansion board should be detected, not >> configured for wl1271 by default), which I still have to fix. I'm >> waiting for the manufacturer of the expansion board to provide me with >> some more info for proper detection. >> >> Also, I know that there are some other work being done in the beagle >> board file, so we may need to rethink how to sync this. >> >> So, for now, let's skip this patch and I'll provide another one when >> these issues are addressed. > > You could also do it based on some kernel cmdline option. Of course > detection during the runtime would be better. That's what we're currently doing in the beagle kernels: http://gitorious.org/beagleboard-validation/linux/commit/17c1b214b918c6f9efc5579134a7ff71d51b60bf That needs forward porting to .37 before I can submit is as RFC here :)-- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 10+ messages in thread
[parent not found: <8004B807-6120-41C0-979B-214BFB75E5CA-QLwJDigV5abLmq1fohREcCpxlwaOVQ5f@public.gmane.org>]
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file [not found] ` <8004B807-6120-41C0-979B-214BFB75E5CA-QLwJDigV5abLmq1fohREcCpxlwaOVQ5f@public.gmane.org> @ 2010-09-24 20:00 ` Luciano Coelho 2010-09-24 20:21 ` Koen Kooi 0 siblings, 1 reply; 10+ messages in thread From: Luciano Coelho @ 2010-09-24 20:00 UTC (permalink / raw) To: ext Koen Kooi Cc: Tony Lindgren, ext John W. Linville, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Ido Yariv, Ohad Ben-Cohen On Fri, 2010-09-24 at 21:42 +0200, ext Koen Kooi wrote: > Op 24 sep 2010, om 21:11 heeft Tony Lindgren het volgende geschreven: > > > * Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> [100924 11:42]: > >> On Fri, 2010-09-24 at 20:03 +0200, ext John W. Linville wrote: > >>> On Thu, Sep 23, 2010 at 01:03:35PM +0300, Luciano Coelho wrote: > >>>> Add board configuration for the wl1271 daughter board. This patch is based > >>>> on Ohad Ben-Cohen's patches for Zoom boards. > >>>> > >>>> Cc: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> > >>>> Signed-off-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> > >>>> --- > >>>> from v2 to v3: removed accidental change to wl1271.h > >>> > >>> I presume that this should come through my tree as well (since it > >>> contains the move of wl12xx.h)? > >> > >> Yes, this is based on your tree. But please hold on, because there were > >> some comments (namely, the expansion board should be detected, not > >> configured for wl1271 by default), which I still have to fix. I'm > >> waiting for the manufacturer of the expansion board to provide me with > >> some more info for proper detection. > >> > >> Also, I know that there are some other work being done in the beagle > >> board file, so we may need to rethink how to sync this. > >> > >> So, for now, let's skip this patch and I'll provide another one when > >> these issues are addressed. > > > > You could also do it based on some kernel cmdline option. Of course > > detection during the runtime would be better. > > That's what we're currently doing in the beagle kernels: http://gitorious.org/beagleboard-validation/linux/commit/17c1b214b918c6f9efc5579134a7ff71d51b60bf > > That needs forward porting to .37 before I can submit is as RFC here :) Yes! I have seen that as pointed out by Robert. It would be nice if you could port this soon(ish) so that others can start using it (ok, maybe not many people, but at least me :P) What would you think about having the actual probing moved from u-boot to the kernel? Do you think that would make sense or are there some other reasons for having it in the bootloader? -- Cheers, Luca. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file 2010-09-24 20:00 ` Luciano Coelho @ 2010-09-24 20:21 ` Koen Kooi 0 siblings, 0 replies; 10+ messages in thread From: Koen Kooi @ 2010-09-24 20:21 UTC (permalink / raw) To: Luciano Coelho Cc: Tony Lindgren, ext John W. Linville, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Ido Yariv, Ohad Ben-Cohen Op 24 sep 2010, om 22:00 heeft Luciano Coelho het volgende geschreven: > On Fri, 2010-09-24 at 21:42 +0200, ext Koen Kooi wrote: >> Op 24 sep 2010, om 21:11 heeft Tony Lindgren het volgende geschreven: >> >>> * Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> [100924 11:42]: >>>> On Fri, 2010-09-24 at 20:03 +0200, ext John W. Linville wrote: >>>>> On Thu, Sep 23, 2010 at 01:03:35PM +0300, Luciano Coelho wrote: >>>>>> Add board configuration for the wl1271 daughter board. This patch is based >>>>>> on Ohad Ben-Cohen's patches for Zoom boards. >>>>>> >>>>>> Cc: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> >>>>>> Signed-off-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> >>>>>> --- >>>>>> from v2 to v3: removed accidental change to wl1271.h >>>>> >>>>> I presume that this should come through my tree as well (since it >>>>> contains the move of wl12xx.h)? >>>> >>>> Yes, this is based on your tree. But please hold on, because there were >>>> some comments (namely, the expansion board should be detected, not >>>> configured for wl1271 by default), which I still have to fix. I'm >>>> waiting for the manufacturer of the expansion board to provide me with >>>> some more info for proper detection. >>>> >>>> Also, I know that there are some other work being done in the beagle >>>> board file, so we may need to rethink how to sync this. >>>> >>>> So, for now, let's skip this patch and I'll provide another one when >>>> these issues are addressed. >>> >>> You could also do it based on some kernel cmdline option. Of course >>> detection during the runtime would be better. >> >> That's what we're currently doing in the beagle kernels: http://gitorious.org/beagleboard-validation/linux/commit/17c1b214b918c6f9efc5579134a7ff71d51b60bf >> >> That needs forward porting to .37 before I can submit is as RFC here :) > > Yes! I have seen that as pointed out by Robert. It would be nice if you > could port this soon(ish) so that others can start using it (ok, maybe > not many people, but at least me :P) > > What would you think about having the actual probing moved from u-boot > to the kernel? Do you think that would make sense or are there some > other reasons for having it in the bootloader? It would make a lot of sense, but when we did the code the in-kernel muxing was pretty much unusable for this goal. I'd prefer to get the uboot based detection way into the kernel first before embarking on the bigger fully in-kernel stuff. Note that on beagleboard-xM we have an additional problem with the camera modules, the different sensors use the same i2c address, which means it completely falls down with the current i2c implementation. Does anyone have any hints on how to read a register over i2c before registering the i2c platform data? regards, Koen-- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] omap: beagle: add support for wl1271 on the board file [not found] ` <20100924191139.GI4211-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> 2010-09-24 19:42 ` Koen Kooi @ 2010-09-24 19:55 ` Luciano Coelho 1 sibling, 0 replies; 10+ messages in thread From: Luciano Coelho @ 2010-09-24 19:55 UTC (permalink / raw) To: ext Tony Lindgren Cc: ext John W. Linville, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Ido Yariv, Ohad Ben-Cohen, ext Robert Nelson On Fri, 2010-09-24 at 21:11 +0200, ext Tony Lindgren wrote: > * Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> [100924 11:42]: > > On Fri, 2010-09-24 at 20:03 +0200, ext John W. Linville wrote: > > > On Thu, Sep 23, 2010 at 01:03:35PM +0300, Luciano Coelho wrote: > > > > Add board configuration for the wl1271 daughter board. This patch is based > > > > on Ohad Ben-Cohen's patches for Zoom boards. > > > > > > > > Cc: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> > > > > Signed-off-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> > > > > --- > > > > from v2 to v3: removed accidental change to wl1271.h > > > > > > I presume that this should come through my tree as well (since it > > > contains the move of wl12xx.h)? > > > > Yes, this is based on your tree. But please hold on, because there were > > some comments (namely, the expansion board should be detected, not > > configured for wl1271 by default), which I still have to fix. I'm > > waiting for the manufacturer of the expansion board to provide me with > > some more info for proper detection. > > > > Also, I know that there are some other work being done in the beagle > > board file, so we may need to rethink how to sync this. > > > > So, for now, let's skip this patch and I'll provide another one when > > these issues are addressed. > > You could also do it based on some kernel cmdline option. Of course > detection during the runtime would be better. Yes, this is more or less what Openembedded (?) guys seem to be doing. You can check Robert Nelson's email in the "v2" thread. The already have some changes in u-boot (which are apparently not in its mainline yet) that does exactly that. The bootloader probes the expansion board and passes the result in a kernel cmdline option. This is "runtime", but I think the best solution would be in the end to let the kernel do the probing itself. As far as I could tell after some googling, having the bootloader probe this kind of stuff is legacy because of problems with muxing in older kernels, so the omap muxing had to be done in the bootloader. Now that muxing is working fine in newer kernel, this kind of thing should be moved to the kernel, or what do you think? -- Cheers, Luca. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2010-09-24 21:08 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-23 10:03 [PATCH v3] omap: beagle: add support for wl1271 on the board file Luciano Coelho
[not found] ` <1285236215-21136-1-git-send-email-luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-09-24 18:03 ` John W. Linville
2010-09-24 18:50 ` Luciano Coelho
2010-09-24 19:11 ` Tony Lindgren
2010-09-24 19:21 ` Gadiyar, Anand
2010-09-24 21:08 ` Tony Lindgren
[not found] ` <20100924191139.GI4211-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2010-09-24 19:42 ` Koen Kooi
[not found] ` <8004B807-6120-41C0-979B-214BFB75E5CA-QLwJDigV5abLmq1fohREcCpxlwaOVQ5f@public.gmane.org>
2010-09-24 20:00 ` Luciano Coelho
2010-09-24 20:21 ` Koen Kooi
2010-09-24 19:55 ` Luciano Coelho
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).