[lkml]   [2001]   [Dec]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] i810_audio fix for version 0.11
    Andris Pavenis wrote:

    > Why returning non zero from __start_dac() and similar procedures when
    > something real has been done there is so bad.

    Personal preference.

    > Using such return code would
    > ensure we never try to wait for results of __start_dac() if nothing is done
    > by this procedure.

    That's part of the point. In this driver, I try to control when things are
    done and keep track of them in a deterministic way. Using a return code to
    tell us a function we called did nothing when we shouldn't have called it in
    the first place if it wasn't going to do anything is backwards from the way
    I prefer to handle things. Namely, find out why the function was called
    when it shouldn't have been and solve the problem. Note: I don't follow
    that philosophy on all functions, only on very simple ones like this, there
    are a lot of complex functions where you want the function to make those
    decisions. So, like I said, personal preference on how to handle these things.

    > I think such way is also more safe against possible future
    > modifications as real conditions are only in a single place. Keeping them in
    > 2 places is possible source of bitrot if driver will be updated in future.

    It's intended to do exactly that. A lot of what makes this driver work
    properly right now is the LVI handling. That was severly busted when I
    first got hold of the driver. I *want* things to break if the LVI handling
    is changed by someone else because that will alert me to the fact that the
    LVI handling is then busted (at least, if they change it incorrectly, if
    they do things right then they will catch problems like this and fix them
    properly and I won't have to do anything).


    Doug Ledford <>
    Please check my web site for aic7xxx updates/answers before
    e-mailing me about problems

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:14    [W:0.023 / U:38.140 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site