[lkml]   [2000]   [Nov]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: Persistent module storage [was Linux 2.4 Status / TODO page]
    On Mon, 06 Nov 2000, David Woodhouse wrote:
    > said:
    > > Irrelevant. The current mixer settings don't matter: what matters is
    > > that the driver does not change them.
    > It does matter. The sound driver needs to be able to _read_ the current
    > levels.

    So do so. That's a hardware/driver issue. If the hardware is broken, put
    a workaround in the driver for that hardware (make the driver persistent,
    as Horst suggested, perhaps). Don't kludge the kernel to mask hardware

    > Almost all mixer programs will start by doing this, to set the
    > slider to the correct place.

    Yippee. As we all know, implementing GUI volume controls
    and putting the slider in the right place is a kernel function,
    and nothing to do with userspace...

    If you want your volume control applet to be able to read the
    current volume settings, even on buggy hardware which can't
    really do that, put the kludge in userspace. Or if you really want,
    put it in your driver for buggy hardware, in the way Horst suggested.

    > > > The driver needs to reset the card to the desired levels.
    > > What desired levels? The only desired levels are the current ones,
    > > which the driver does not and (sometimes) cannot know. Leave well
    > > alone.
    > It does not know them. Correct. But with persistent module storage, it
    > _could_ know them.

    No it cannot. The desired levels have not been defined: there are no
    desired levels to determine! Don't tamper with settings you don't need

    > It cannot know them the _first_ time the module is
    > loaded after booting. That's fine. On subsequent loads, it can and
    > should DTRT.

    The right thing in this context is not to screw with hardware settings
    unless and until it is given settings to set. Do not set values arbitrarily:
    set only the values you are explicitly given. Anything else is simply
    a bug in your driver.

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

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