[U-Boot] [PATCH v4 2/7] gpio: Add Multi-Function-Pin configuration driver for Marvell SoCs

Albert ARIBAUD albert.aribaud at free.fr
Tue Dec 7 18:39:06 CET 2010


Le 07/12/2010 18:10, Prafulla Wadaskar a écrit :

>>> +                       val&= ~MFP_AF_MASK;
>> Do we need to do this&  here? For val is only 0 here...
>
> This can be removed.

OTOH, with the &, this line makes no assumption about val, and thus will 
work regardless of it. If the & is removed, and if later val is set to 
non-zero before reaching this instruction, it will cause a bug.

IOW, the & makes the statement more resilient.

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list