Hi Subhasish, On 2/11/2011 9:51 AM, Subhasish Ghosh wrote:
> This patch adds the pruss SUART pin mux and registers the device > with the pruss mfd driver. > > Signed-off-by: Subhasish Ghosh <subhas...@mistralsolutions.com> > --- > arch/arm/mach-davinci/board-da850-evm.c | 36 > +++++++++++++++++++++++++++++++ > 1 files changed, 36 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/mach-davinci/board-da850-evm.c > b/arch/arm/mach-davinci/board-da850-evm.c > index f9c38f8..3858516 100644 > --- a/arch/arm/mach-davinci/board-da850-evm.c > +++ b/arch/arm/mach-davinci/board-da850-evm.c > @@ -1060,6 +1060,25 @@ const short da850_evm_pruss_can_pins[] = { > -1 > }; > > +const short da850_evm_pruss_suart_pins[] = { > + DA850_AHCLKX, DA850_ACLKX, DA850_AFSX, > + DA850_AHCLKR, DA850_ACLKR, DA850_AFSR, > + DA850_AXR_13, DA850_AXR_9, DA850_AXR_7, > + DA850_AXR_14, DA850_AXR_10, DA850_AXR_8, > + -1 > +}; > + Shouldn't this pins select PRU[0,1]_XXX type functions and not McASP functions? E.G.: PRU0_R31[17] instead of AHCLKX, PRU0_R31[18] instead of AHCLKR, etc. > +static int __init da850_evm_setup_pruss_suart(void) > +{ > + int ret; > + > + ret = davinci_cfg_reg_list(da850_evm_pruss_suart_pins); > + if (ret) > + pr_warning("%s: da850_evm_pruss_suart_pins " > + "mux setup failed: %d\n", __func__, ret); > + return ret; > +} > + > static int __init da850_evm_setup_pruss_can(void) > { > int ret, val = 0; > @@ -1085,6 +1104,17 @@ static int __init da850_evm_setup_pruss_can(void) > return ret; > } > > +static struct da850_evm_pruss_suart_data suart_data = { > + .version = 1, > + .resource = { > + .name = "da8xx_mcasp0_iomem", > + .start = DAVINCI_DA8XX_MCASP0_REG_BASE, > + .end = DAVINCI_DA8XX_MCASP0_REG_BASE + > + (SZ_1K * 12) - 1, > + .flags = IORESOURCE_MEM, > + }, > +}; > + > static struct da8xx_pruss_can_data can_data = { > .version = 1, > }; > @@ -1094,6 +1124,12 @@ static struct da8xx_pruss_devices pruss_devices[] = { > .dev_name = "da8xx_pruss_can", > .pdata = &can_data, > .pdata_size = sizeof(can_data), > + .setup = da850_evm_setup_pruss_suart, Should this be da850_evm_setup_pruss_can instead? > + }, > + { > + .dev_name = "da8xx_pruss_uart", > + .pdata = &suart_data, > + .pdata_size = sizeof(suart_data), > .setup = da850_evm_setup_pruss_can, Should this be da850_evm_setup_pruss_suart instead? > }, > { -Mike _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source