All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/3] AM335x : Configs to add USB host support.
Date: Fri, 22 Jun 2012 09:01:07 +0200	[thread overview]
Message-ID: <201206220901.07701.sr@denx.de> (raw)
In-Reply-To: <1340345836-23694-3-git-send-email-harman_sohanpal@ti.com>

On Friday 22 June 2012 08:17:15 Harman Sohanpal wrote:
> This patch adds required configs in config file for
> am335x_evm to add support for usb host mode.
> To enable USB device mode, add CONFIG_MUSB_UDC in
> place of CONFIG_MUSB_HCD
> 
> Signed-off-by: Harman Sohanpal <harman_sohanpal@ti.com>
> ---
>  include/configs/am335x_evm.h |   41
> +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 41
> insertions(+), 0 deletions(-)
> 
> diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
> index d0fbc88..d19f914 100644
> --- a/include/configs/am335x_evm.h
> +++ b/include/configs/am335x_evm.h
> @@ -165,6 +165,47 @@
>  #define CONFIG_SKIP_LOWLEVEL_INIT
>  #endif
> 
> +
> +/*
> + * USB configuration
> + * Enable CONFIG_MUSB_HCD for Host functionalities MSC, keyboard
> + * Enable CONFIG_MUSB_UDC for Device functionalities.
> + */
> +#define CONFIG_USB_AM335X                1
> +#define CONFIG_MUSB_HCD                 1

Please don't add the "1" to the defines. Simply define the value, thats 
enough.

> +
> +#ifdef CONFIG_USB_AM335X
> +
> +#ifdef CONFIG_MUSB_HCD
> +#define CONFIG_CMD_USB
> +
> +#define CONFIG_USB_STORAGE
> +#define CONGIG_CMD_STORAGE
> +#define CONFIG_CMD_FAT
> +
> +#ifdef CONFIG_USB_KEYBOARD
> +#define CONFIG_SYS_USB_EVENT_POLL
> +#define CONFIG_PREBOOT "usb start"
> +#endif /* CONFIG_USB_KEYBOARD */
> +
> +#endif /* CONFIG_MUSB_HCD */
> +
> +#ifdef CONFIG_MUSB_UDC
> +/* USB device configuration */
> +#ifndef CONFIG_SPL_BUILD
> +#define CONFIG_USB_DEVICE               1
> +#define CONFIG_USB_TTY                  1
> +#define CONFIG_SYS_CONSOLE_IS_IN_ENV    1

Here again. No "1" please.

Thanks,
Stefan

  reply	other threads:[~2012-06-22  7:01 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-22  6:17 [U-Boot] [PATCH 0/3] AM335x: Add USB support in u-boot Harman Sohanpal
2012-06-22  6:17 ` [U-Boot] [PATCH 1/3] AM335x : Add USB support for AM335x " Harman Sohanpal
2012-06-22  6:17 ` [U-Boot] [PATCH 2/3] AM335x : Configs to add USB host support Harman Sohanpal
2012-06-22  7:01   ` Stefan Roese [this message]
2012-06-22  6:17 ` [U-Boot] [PATCH 3/3] musb_udc : Fix compile warning Harman Sohanpal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201206220901.07701.sr@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.