[PATCH] otg/ulpi.c : fix register write

Eric Bénard eric at eukrea.com
Wed Jun 23 11:07:49 EDT 2010


Le 23/06/2010 16:58, Daniel Mack a écrit :
> On Wed, Jun 23, 2010 at 04:50:32PM +0200, Eric Bénard wrote:
>> -static int ulpi_write(struct otg_transceiver *otg, u32 val, u32 reg)
>> +static int ulpi_write(struct otg_transceiver *otg, u32 reg, u32 val)
>
> Urgs. Is this really necessary? It's not that I have a strong opinion
> about the order of arguments in such cases (I kept to the convention
> of __readl() when I wrote it). But _changing_ it like this is really
> confusing. Once in awhile I stumble over such API changes and I always
> wonder about the reason. The problem is that not even the compiler will
> warn you if you got it wrong, when you copied a sniplet from older
> sources etc.
>
> And you really want to break someone knee caps once you find out what
> caused the breakage ;)
>
> So - if we can avoid that, we should do. If anyone speaks up with a real
> reason for changing it, I'd be fine :)
>
No problem if you prefer to keep as it is (this is "cosmetic" but val 
and reg are reversed in include/linux/usb/otg.h which I took as the 
reference in the present case).
Do you have any opinion on the other part of the patch which removes 
ULPI_SET as this prevents to clear bits in the phy register ? If ok for 
you, I send a revised patch containing only this change.

Thanks,
Eric



More information about the linux-arm-kernel mailing list