lkml.org 
[lkml]   [2012]   [Mar]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH RESEND v3] USB: Support for LPC32xx SoC
On 08/03/12 22:11, Greg Kroah-Hartman wrote:
>> - pnx4008_unset_usb_bits();
>> + if (machine_is_pnx4008())
>> + pnx4008_unset_usb_bits();
>
> How come some of these pnx4008_* functions you have wrapped in
> machine_is_pnx4008(), but not all of them?
>
> Actually, why don't those functions do that kind of checking themselves,
> without having to do this before you call them each time?

Good idea, I moved the machine dependent execution of the bits()
functions into the functions.

The dependency is only in the bits functions because those are done via
interfaces not available on LPC. The other pnx4008 functions seem to be
mostly compatible.

Thanks for your notes!

Roland


\
 
 \ /
  Last update: 2012-03-08 23:03    [W:0.037 / U:0.972 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site