[lkml]   [2011]   [Aug]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 6/8] staging: vme: rename *_slot_get to *_get_slot
    On 01/08/11 13:31, Manohar Vanga wrote:
    > Hi Martyn,
    >> If your doing vme_slot_get, then there's also vme_lm_get, vme_master_get and
    >> vme_slave_get.
    >> Doing just this would then lead to more inconsistency in the naming and
    >> wouldn't even get rid of all the functions using the *_get naming convention.
    > I can simply change those as well and resend :)
    >> I'm not sure this change is worth it.
    > I thought it would be worth changing this especially with the addition of bridge
    > refcounting functions (which have the *_get/_set convention).

    The naming of the API is reasonably consistent at the moment. It looks like we
    aren't going to need to export vme_bus_get and vme_bus_put based on the
    conversation on patch 5. In addition to this, I've discovered that the
    functions for USB are actually called usb_get_dev and usb_put_dev, so the
    *_put/get convention isn't very strictly adhered to. I'd prefer to keep the
    API as is.


    > --
    > /manohar

    Martyn Welch (Principal Software Engineer) | Registered in England and
    GE Intelligent Platforms | Wales (3828642) at 100
    T +44(0)127322748 | Barbirolli Square, Manchester,
    E | M2 3AB VAT:GB 927559189

     \ /
      Last update: 2011-08-09 17:21    [W:4.737 / U:0.092 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site