lkml.org 
[lkml]   [2010]   [Nov]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 2/3] regulator: Ensure enough delay time for enabling regulator
    On Fri, Nov 05, 2010 at 03:26:20PM +0800, Axel Lin wrote:

    > - if (delay >= 1000)
    > + if (delay >= 1000) {
    > mdelay(delay / 1000);
    > - else if (delay)
    > + udelay(delay % 1000);
    > + } else if (delay)
    > udelay(delay);

    This looks good but please change this to use { } in the else case also;
    I find this very helpful for legibility.


    \
     
     \ /
      Last update: 2010-11-05 14:23    [W:0.141 / U:0.028 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site