[U-Boot] [PATCH] mpc85xx/T104xRDB: Remove vbank check redundant code

York Sun yorksun at freescale.com
Wed Jul 29 18:56:25 CEST 2015



On 07/29/2015 01:31 AM, Jain Priyanka-B32167 wrote:
> 
> 
>> -----Original Message-----
>> From: Sun York-R58495
>> Sent: Tuesday, July 28, 2015 12:37 AM
>> To: Jain Priyanka-B32167; u-boot at lists.denx.de
>> Subject: Re: [PATCH] mpc85xx/T104xRDB: Remove vbank check redundant
>> code
>>
>>
>>
>> On 05/17/2015 11:31 PM, Priyanka Jain wrote:
>>> sw variable in checkboard function is storing vbank value which can
>>> only take 4-bit value.
>>> So check of sw value for if greater than 7 is redundant.
>>>
>>> Signed-off-by: Priyanka Jain <Priyanka.Jain at freescale.com>
>>> ---
>>>  board/freescale/t104xrdb/t104xrdb.c |    5 +----
>>>  1 files changed, 1 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/board/freescale/t104xrdb/t104xrdb.c
>>> b/board/freescale/t104xrdb/t104xrdb.c
>>> index 9cd5e15..c34eea6 100644
>>> --- a/board/freescale/t104xrdb/t104xrdb.c
>>> +++ b/board/freescale/t104xrdb/t104xrdb.c
>>> @@ -35,10 +35,7 @@ int checkboard(void)
>>>  	sw = CPLD_READ(flash_ctl_status);
>>>  	sw = ((sw & CPLD_LBMAP_MASK) >> CPLD_LBMAP_SHIFT);
>>>
>>> -	if (sw <= 7)
>>> -		printf("vBank: %d\n", sw);
>>> -	else
>>> -		printf("Unsupported Bank=%x\n", sw);
>>> +	printf("vBank: %d\n", sw);
>>>
>>>  	return 0;
>>>  }
>>>
>>
>> Priyanka,
>>
>> 4-bit variable can still have value bigger than 7. Is there other reason you
>> want to remove it?
> [Jain Priyanka-B32167] Sorry, Its 3-bit variable. So cannot have value greater than 7.
> CPLD_LBMAP_MASK is 0x3f and CPLD_LBMAB_SHIFT is of value 3.
> So it is basically bits sw[5], sw[4], sw[3].
> 

If it is 3-bit value, it makes sense. Can you update the commit message?

York



More information about the U-Boot mailing list