[U-Boot] [PATCH 2/3] BeagleBoard: Added userbutton command

Wolfgang Denk wd at denx.de
Thu Sep 8 10:05:17 CEST 2011


Dear Joel A Fernandes,

In message <1313462214-3716-2-git-send-email-agnel.joel at gmail.com> you wrote:
> From: Jason Kridner <jkridner at beagleboard.org>
> 
> Based on commit f1099c7c43caf5bac3bf6a65aa266fade4747072
>     Author: Greg Turner <gregturner at ti.com>
>     Date:   Tue May 25 09:19:06 2010 -0500
> 
>     New u-boot command for status of USER button on BeagleBoard-xM
> 
>          Modified bootcmd to check the staus at boot time and set
> 	 filename of the boot script.
> 
> * Moved to a BeagleBoard specific file.
> * Removed changes to default boot command from adding userbutton
>   command.
> * Made to handle pre-xM boards.
> * Flipped polarity of the return value to avoid confusion.  Success (0)
>   is when the button is pressed.  Failure (1) is when the button is NOT
>   pressed.
> * Used latest revision getting function.
> * Used latest macros for board revision.
> * Added xM-C revision definition (optional, since it was default)
> * updated default configuration with UserButton functionality
>   * Added a separate bootenv variable to load a user defined .txt file
>   * Added an example, showing how a different environment file can be loaded with
>     the user button pressed

Your patch has a large number of cding style issues; please always
run checkpatch before submitting patches.


Also, I agree with Albert: there should be no need for a separate
userbutton command.

Please fix and resubmit.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The average woman would rather have beauty than brains,  because  the
average man can see better than he can think.


More information about the U-Boot mailing list