Re: [PATCH v2 09/21] ARM: pxa: magician: Add OV9640 camera support

Petr Cvek
2015-08-20T22:36:27Z

Dne 20.8.2015 v 22:26 Arnd Bergmann napsal(a):
> On Thursday 20 August 2015 21:48:20 Robert Jarzmik wrote:
>> Petr Cvek <petr.cvek@tul.cz> writes:
>>
>>> + mdelay(3);
>> Okay, there are several mdelay(3) in this file.
>>
>> Could you please test if removing these mdelays prevents the camera from working
>> ? If not, I'd like to have them removed.
>>
>>
>
> FWIW, it might be possible to replace some of them with msleep() calls,
> potentially after replacing spinlocks with mutexes where necessary.
>
> Arnd
>

Datasheet says:

tS:RESET Setting time after software/hardware reset 1 ms

So at least one ~1 ms should be left there. Are msleep less than 20ms valid?

(checkpatch: msleep < 20ms can sleep for up to 20ms)

Petr