[PATCH]ARM/footbridge: fix wrong parenthesis

[Wan ZongShun]

So should I send  a true one again or fix it by your self?

2010/6/7, Russell King - ARM Linux <linux at arm.linux.org.uk>:
> On Fri, Jun 04, 2010 at 05:28:41PM +0800, Wan ZongShun wrote:
>> There is a wrong redundant right parenthesis in this code.
>
> This patch is wrong.
>
>>  unsigned long __pfn_to_bus(unsigned long pfn)
>>  {
>> -	return __pfn_to_phys(pfn) + (fb_bus_sdram_offset() - PHYS_OFFSET));
>> +	return __pfn_to_phys(pfn + (fb_bus_sdram_offset() - PHYS_OFFSET));
>
> It makes no sense to add byte address offsets to a PFN.  The correct
> solution is to delete the right paren at the end of the line instead.
>


-- 
*linux-arm-kernel mailing list
mail addr:linux-arm-kernel at lists.infradead.org
you can subscribe by:
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

* linux-arm-NUC900 mailing list
mail addr:NUC900 at googlegroups.com
main web: https://groups.google.com/group/NUC900
you can subscribe it by sending me mail:
mcuos.com at gmail.com