Re: [PATCH] Add omap3_defconfig

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

 



On Mon, Nov 30, 2009 at 11:17:47AM +0530, Gadiyar, Anand wrote:
> Olof Johansson wrote:
> 
> > Having one combined defconfig that is the superset of the individual
> > defconfigs for OMAP3 platforms is useful for easily finding build
> > errors. Not to mention convenient as a base if you want to boot several
> > platforms with a single kernel image.
> > 
> > Signed-off-by: Olof Johansson <olof@xxxxxxxxx>
> > 
> > 
> > ---
> > 
> > Tony, I noticed linux-next doesn't build many of the OMAP defconfigs,
> > and a handful of them won't build to date. I suggest adding this config,
> > and I'll as Stephen Rothwell to add it as one of the configs that are
> > tested (and build errors reported) for every release.
> > 
> > He's reluctant to add too many individual configs since it adds time to
> > each cycle, but one more can surely be acceptable.
> > 
> 
> Good idea. Kevin does something similar with the pm branch. There's
> an omap3_pm_defconfig.
> 
> A few minor comments below.
> 
> > 
> > -Olof
> > 
> <snip>
> 
> > +CONFIG_USER_SCHED=y
> > +# CONFIG_CGROUP_SCHED is not set
> > +# CONFIG_CGROUPS is not set
> > +CONFIG_SYSFS_DEPRECATED=y
> > +CONFIG_SYSFS_DEPRECATED_V2=y
> 
> Would you consider getting rid of these two?
> There was a patch recently to remove these from the
> other omap defconfigs, so this would be one inconsistency
> between them.

Sure. I created this by cat:ing the various board configs together to
get the superset, I didn't spend a whole lot of time looking at the
individual options.

> <snip>
> 
> > +# Miscellaneous USB options
> > +#
> > +CONFIG_USB_DEVICEFS=y
> > +CONFIG_USB_DEVICE_CLASS=y
> > +# CONFIG_USB_DYNAMIC_MINORS is not set
> > +CONFIG_USB_SUSPEND=y
> > +CONFIG_USB_OTG=y
> > +CONFIG_USB_OTG_WHITELIST=y
> > +CONFIG_USB_OTG_BLACKLIST_HUB=y
> 
> This removes the ability to connect all devices to MUSB
> in host mode. (Most flash drives would not be on the
> whitelist). Also, blacklisting hubs means you can't use
> an external hub either - which is commonly used with
> beagle and other similar boards.

Sure.

> <snip>
> 
> > +# OMAP 343x high speed USB support
> > +#
> > +# CONFIG_USB_MUSB_HOST is not set
> > +# CONFIG_USB_MUSB_PERIPHERAL is not set
> > +CONFIG_USB_MUSB_OTG=y
> > +CONFIG_USB_GADGET_MUSB_HDRC=y
> > +CONFIG_USB_MUSB_HDRC_HCD=y
> > +CONFIG_MUSB_PIO_ONLY=y
> 
> And this would remove the MUSB DMA support on these boards.
> I think you should make this n. It's always possible to
> disable dma by adding a parameter "musb_hdrc.use_dma=n" to
> the kernel command line

No problem. I'll hold off reposting until tonight to let more comments
come in.


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux