[U-Boot] [PATCH V2 02/14] sandisk: sfp: correct function name

Stefano Babic sbabic at denx.de
Tue Oct 20 15:09:29 CEST 2015


Hi Peng,

On 20/10/2015 13:39, Peng Fan wrote:
> board_mxsfb_system_setup shoule be named mxsfb_system_setup.
> 
> Signed-off-by: Peng Fan <Peng.Fan at freescale.com>
> Cc: Marek Vasut <marek.vasut at gmail.com>
> Cc: Stefano Babic <sbabic at denx.de>
> ---
> 
> V2:
>  none
> 
>  board/sandisk/sansa_fuze_plus/sfp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/board/sandisk/sansa_fuze_plus/sfp.c b/board/sandisk/sansa_fuze_plus/sfp.c
> index a3865ad..8d1a797 100644
> --- a/board/sandisk/sansa_fuze_plus/sfp.c
> +++ b/board/sandisk/sansa_fuze_plus/sfp.c
> @@ -311,7 +311,7 @@ static const struct {
>  	{ 0x07, 0  , 0x0173 },
>  };
>  
> -void board_mxsfb_system_setup(void)
> +void mxsfb_system_setup(void)
>  {
>  	struct mxs_lcdif_regs *regs = (struct mxs_lcdif_regs *)MXS_LCDIF_BASE;
>  	uint32_t id;
> 

This is really a bug. Wrong name was not seen before. It looks like this
series mixes some new features (LCDIF) with bug fixes that are quite
orthogonal, as in this case. I will propose that I will start merge the
easy fixes (as this one), and you won't need to post them in a new version.

Acked-by: Stefano Babic <sbabic at denx.de>

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list