Re: [PATCH 1/2] ARM: OMAP2+: Fix MUSB ifdefs for platform init code

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

 



On Wed, Jun 13, 2012 at 03:15:06AM -0700, Tony Lindgren wrote:
> Commit 62285963 (usb: musb: drop a gigantic amount of ifdeferry)
> got rid of a bunch of ifdefs in the MUSB code. Looks like the
> platform init code is still using these dropped defines though,
> which in many cases results the board defaulting always to host
> mode.
> 
> Currently the situation is that USB_MUSB_HDRC is the main
> Kconfig option with additional USB_GADGET_MUSB_HDRC so only
> these two should be used to select between host and OTG mode.
> 
> Fix the situation for omaps. The following users should fix the
> platform init code in a similar way:
> 
> Dropped Kconfig option          Current users
> 
> USB_MUSB_OTG                    blackfin, davinci, not in Kconfigs
> USB_MUSB_PERIPHERAL             davinci, not in Kconfigs
> USB_MUSB_HOST                   davinci, not in Kconfigs
> USB_MUSB_HDRC_HCD               blackfin, not in Kconfigs
> USB_MUSB_OTG                    blackfin, not in Kconfigs
> 
> Cc: Mike Frysinger <vapier@xxxxxxxxxx>
> Cc: Sekhar Nori <nsekhar@xxxxxx>
> Cc: linux-usb@xxxxxxxxxxxxxxx
> Cc: Felipe Balbi <balbi@xxxxxx>
> Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>

Looks ok, though I would stick to things boards are wired to do. e.g.:

all development boards are wired for OTG (with a few exceptions),
n800 is peripheral only
n810 is OTG

and so on. That way you remove all the clutter.

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux