Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758777AbYFLJD4 (ORCPT ); Thu, 12 Jun 2008 05:03:56 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753640AbYFLJDr (ORCPT ); Thu, 12 Jun 2008 05:03:47 -0400 Received: from vigor.karmaclothing.net ([217.169.26.28]:57352 "EHLO vigor.karmaclothing.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753442AbYFLJDq (ORCPT ); Thu, 12 Jun 2008 05:03:46 -0400 Date: Thu, 12 Jun 2008 10:02:06 +0100 From: Ralf Baechle To: Manuel Lauss Cc: linux-mips@linux-mips.org, sshtylyov@ru.mvista.com, drzeus@drzeus.cx, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/8] Alchemy: register mmc platform device for db1200/pb1200 boards. Message-ID: <20080612090206.GB21601@linux-mips.org> References: <20080609063521.GA8724@roarinelk.homelinux.net> <20080609063702.GC8724@roarinelk.homelinux.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080609063702.GC8724@roarinelk.homelinux.net> User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7636 Lines: 275 On Mon, Jun 09, 2008 at 08:37:02AM +0200, Manuel Lauss wrote: Only a few small nits ... > >From 536f44d2947f3883016b927a621e30782bd3149f Mon Sep 17 00:00:00 2001 > From: Manuel Lauss > Date: Sat, 17 May 2008 17:48:02 +0200 > Subject: [PATCH] Alchemy: register mmc platform device for db1200/pb1200 boards Please only post as part of the body what is supposed to go into the commit message. Three of these lines are junk which the maintainer applying the patch will have to remove ... > Add au1xmmc platform data for PB1200/DB1200 boards, and wire up > the 2 SD controllers for them. > > Signed-off-by: Manuel Lauss > --- > arch/mips/au1000/common/platform.c | 99 +++++++++++++++++++++++++++--------- > arch/mips/au1000/pb1200/platform.c | 81 +++++++++++++++++++++++++++++ > 2 files changed, 156 insertions(+), 24 deletions(-) > > diff --git a/arch/mips/au1000/common/platform.c b/arch/mips/au1000/common/platform.c > index 8cae775..c235434 100644 > --- a/arch/mips/au1000/common/platform.c > +++ b/arch/mips/au1000/common/platform.c > @@ -16,6 +16,8 @@ > #include > > #include > +#include > +#include > > #define PORT(_base, _irq) \ > { \ > @@ -162,24 +164,6 @@ static struct resource au1xxx_usb_gdt_resources[] = { > }, > }; > > -static struct resource au1xxx_mmc_resources[] = { > - [0] = { > - .start = SD0_PHYS_ADDR, > - .end = SD0_PHYS_ADDR + 0x40, > - .flags = IORESOURCE_MEM, > - }, > - [1] = { > - .start = SD1_PHYS_ADDR, > - .end = SD1_PHYS_ADDR + 0x40, > - .flags = IORESOURCE_MEM, > - }, > - [2] = { > - .start = AU1200_SD_INT, > - .end = AU1200_SD_INT, > - .flags = IORESOURCE_IRQ, > - } > -}; > - > static u64 udc_dmamask = ~(u32)0; Okay, you didn't change this line but using DMA_32BIT_MASK would be cleaner ... > static struct platform_device au1xxx_usb_gdt_device = { > @@ -248,16 +232,80 @@ static struct platform_device au1200_lcd_device = { > > static u64 au1xxx_mmc_dmamask = ~(u32)0; Ditto. > -static struct platform_device au1xxx_mmc_device = { > +extern struct au1xmmc_platform_data au1xmmc_platdata[2]; > + > +static struct resource au1200_mmc0_resources[] = { > + [0] = { > + .start = SD0_PHYS_ADDR, > + .end = SD0_PHYS_ADDR + 0x7ffff, > + .flags = IORESOURCE_MEM, > + }, > + [1] = { > + .start = AU1200_SD_INT, > + .end = AU1200_SD_INT, > + .flags = IORESOURCE_IRQ, > + }, > + [2] = { > + .start = DSCR_CMD0_SDMS_TX0, > + .end = DSCR_CMD0_SDMS_TX0, > + .flags = IORESOURCE_DMA, > + }, > + [3] = { > + .start = DSCR_CMD0_SDMS_RX0, > + .end = DSCR_CMD0_SDMS_RX0, > + .flags = IORESOURCE_DMA, > + } > +}; > + > +static struct platform_device au1200_mmc0_device = { > .name = "au1xxx-mmc", > .id = 0, > .dev = { > - .dma_mask = &au1xxx_mmc_dmamask, > - .coherent_dma_mask = 0xffffffff, > + .dma_mask = &au1xxx_mmc_dmamask, > + .coherent_dma_mask = 0xffffffff, DMA_32BIT_MASK. > + .platform_data = &au1xmmc_platdata[0], > }, > - .num_resources = ARRAY_SIZE(au1xxx_mmc_resources), > - .resource = au1xxx_mmc_resources, > + .num_resources = ARRAY_SIZE(au1200_mmc0_resources), > + .resource = au1200_mmc0_resources, > }; > + > +#ifndef CONFIG_MIPS_DB1200 > +static struct resource au1200_mmc1_resources[] = { > + [0] = { > + .start = SD1_PHYS_ADDR, > + .end = SD1_PHYS_ADDR + 0x7ffff, > + .flags = IORESOURCE_MEM, > + }, > + [1] = { > + .start = AU1200_SD_INT, > + .end = AU1200_SD_INT, > + .flags = IORESOURCE_IRQ, > + }, > + [2] = { > + .start = DSCR_CMD0_SDMS_TX1, > + .end = DSCR_CMD0_SDMS_TX1, > + .flags = IORESOURCE_DMA, > + }, > + [3] = { > + .start = DSCR_CMD0_SDMS_RX1, > + .end = DSCR_CMD0_SDMS_RX1, > + .flags = IORESOURCE_DMA, > + } > +}; > + > + > +static struct platform_device au1200_mmc1_device = { > + .name = "au1xxx-mmc", > + .id = 1, > + .dev = { > + .dma_mask = &au1xxx_mmc_dmamask, > + .coherent_dma_mask = 0xffffffff, DMA_32BIT_MASK. > + .platform_data = &au1xmmc_platdata[1], > + }, > + .num_resources = ARRAY_SIZE(au1200_mmc1_resources), > + .resource = au1200_mmc1_resources, > +}; > +#endif /* #ifndef CONFIG_MIPS_DB1200 */ > #endif /* #ifdef CONFIG_SOC_AU1200 */ > > static struct platform_device au1x00_pcmcia_device = { > @@ -295,7 +343,10 @@ static struct platform_device *au1xxx_platform_devices[] __initdata = { > &au1xxx_usb_gdt_device, > &au1xxx_usb_otg_device, > &au1200_lcd_device, > - &au1xxx_mmc_device, > + &au1200_mmc0_device, > +#ifndef CONFIG_MIPS_DB1200 > + &au1200_mmc1_device, > +#endif > #endif > #ifdef SMBUS_PSC_BASE > &pbdb_smbus_device, > diff --git a/arch/mips/au1000/pb1200/platform.c b/arch/mips/au1000/pb1200/platform.c > index 5930110..faf3d92 100644 > --- a/arch/mips/au1000/pb1200/platform.c > +++ b/arch/mips/au1000/pb1200/platform.c > @@ -19,9 +19,90 @@ > */ > > #include > +#include > #include > > #include > +#include > + > +static int mmc_activity = 0; Don't initialize static variables to 0. > + > +static void pb1200mmc0_set_power(void *mmc_host, int state) > +{ > + if (state) > + bcsr->board |= BCSR_BOARD_SD0PWR; > + else > + bcsr->board &= ~BCSR_BOARD_SD0PWR; > + > + au_sync_delay(1); > +} > + > +static int pb1200mmc0_card_readonly(void *mmc_host) > +{ > + return (bcsr->status & BCSR_STATUS_SD0WP) ? 1 : 0; > +} > + > +static int pb1200mmc0_card_inserted(void *mmc_host) > +{ > + return (bcsr->sig_status & BCSR_INT_SD0INSERT) ? 1 : 0; > +} > + > +static void pb1200_mmcled_set(struct led_classdev *led, > + enum led_brightness brightness) > +{ > + if (brightness != LED_OFF) { > + if (++mmc_activity == 1) > + bcsr->disk_leds &= ~(1 << 8); > + } else { > + if (--mmc_activity == 0) > + bcsr->disk_leds |= (1 << 8); > + } > +} > + > +static struct led_classdev pb1200mmc_led = { > + .brightness_set = pb1200_mmcled_set, > +}; > + > +#ifndef CONFIG_MIPS_DB1200 > +static void pb1200mmc1_set_power(void *mmc_host, int state) > +{ > + if (state) > + bcsr->board |= BCSR_BOARD_SD1PWR; > + else > + bcsr->board &= ~BCSR_BOARD_SD1PWR; > + > + au_sync_delay(1); > +} > + > +static int pb1200mmc1_card_readonly(void *mmc_host) > +{ > + return (bcsr->status & BCSR_STATUS_SD1WP) ? 1 : 0; > +} > + > +static int pb1200mmc1_card_inserted(void *mmc_host) > +{ > + return (bcsr->sig_status & BCSR_INT_SD1INSERT) ? 1 : 0; > +} > +#endif > + > +const struct au1xmmc_platform_data au1xmmc_platdata[2] = { > + [0] = { > + .set_power = pb1200mmc0_set_power, > + .card_inserted = pb1200mmc0_card_inserted, > + .card_readonly = pb1200mmc0_card_readonly, > + .cd_setup = NULL, /* use poll-timer in driver */ > + .led = &pb1200mmc_led, > + }, > +#ifndef CONFIG_MIPS_DB1200 > + [1] = { > + .set_power = pb1200mmc1_set_power, > + .card_inserted = pb1200mmc1_card_inserted, > + .card_readonly = pb1200mmc1_card_readonly, > + .cd_setup = NULL, /* use poll-timer in driver */ > + .led = &pb1200mmc_led, > + }, > +#endif > +}; > > static struct resource ide_resources[] = { > [0] = { > -- > 1.5.5.3 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/