Re: [PATCH v2 3/3] ARM: imx_v6_v7_defconfig: Select MXS_DMA

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

On Thu, Jul 26, 2012 at 07:32:54AM -0400, Huang Shijie wrote:
> On Thu, Jul 26, 2012 at 7:08 AM, Fabio Estevam <festevam@xxxxxxxxx> wrote:
> > On Thu, Jul 26, 2012 at 7:56 AM, Huang Shijie <shijie8@xxxxxxxxx> wrote:
> >> On Thu, Jul 26, 2012 at 6:53 AM, Fabio Estevam <festevam@xxxxxxxxx> wrote:
> >>> On Wed, Jul 25, 2012 at 11:24 PM, Huang Shijie <shijie8@xxxxxxxxx> wrote:
> >>>
> >>>> But you have added "depend on MXS_DMA". So If you do not select the
> >>>> mxs-dma, you can not see the
> >>>> gpmi-nand.
> >>>> that's why i think this patch is not needed.
> >>>
> >>> So how CONFIG_MXS_DMA would be selected if not via defconfig?
> >> Use the make menuconfig ARCH=arm:
> >>
> >> "device drivers"-->"DMA engine support" -->"mxs dma support"
> >
> > I prefer to have selected in the defconfig by default instead.
> >
> > It is not obvious for the end user that they need to manually select
> > this option.
> >
> > mxs_defconfig also selects CONFIG_MXS_DMA.
> In mx23/mx28, the mxs-dma has many users, so it's proper to make it default.
> 
> >
> > Can't we do the same here and let imx_v6_v7_defconfig be built again?
> Shawn, do you have any opinion about this?
Most drivers should be kernel modules. mxs_dma needs changes to support
building as module.

Thanks
Richard
> 
> Huang Shijie
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel


[Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [PDAs]     [Linux]     [Linux MIPS]     [Yosemite Campsites]     [Photos]

Add to Google Follow linuxarm on Twitter