[lkml]   [2005]   [Sep]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [linux-usb-devel] RFC drivers/usb/storage/libusual
    Pete Zaitcev wrote:
    > Patch is attached. I would like someone to look it over and challenge it.
    > The thing looks too complex to me, but I see no other way. Anyone?

    OK, so I'm not very familiar with a lot of the code affected here, but
    since it diddles with unusual_devs, I feel I should chime in. But I'll
    chime in with a question. ;)

    A quick look over the patch shows that there are now two kinds of
    unusual_dev entries: unusual_dev() and unusual_dev_fl(), where the
    latter is for entries that don't need to specify SC or PR (i.e., just
    had US_SC_DEVICE, US_PR_DEVICE in them). While I think that's a
    reasonable change, it's not clear to me why that's useful to the rest of
    the patch, or it's just making unusual_devs.h artificially shorter?

    Phil Dibowitz
    Freeware and Technical Pages Insanity Palace of Metallica

    "They that can give up essential liberty to obtain a little temporary
    safety deserve neither liberty nor safety."
    - Benjamin Franklin, 1759

    [unhandled content-type:application/pgp-signature]
     \ /
      Last update: 2005-09-30 09:11    [W:0.020 / U:10.200 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site