lkml.org 
[lkml]   [2012]   [Feb]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[ 38/72] USB: Remove duplicate USB 3.0 hub feature #defines.
    3.2-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Sarah Sharp <sarah.a.sharp@linux.intel.com>

    commit d9f5343e35d9138432657202afa8e3ddb2ade360 upstream.

    Somehow we ended up with duplicate hub feature #defines in ch11.h.
    Tatyana Brokhman first created the USB 3.0 hub feature macros in 2.6.38
    with commit 0eadcc09203349b11ca477ec367079b23d32ab91 "usb: USB3.0 ch11
    definitions". In 2.6.39, I modified a patch from John Youn that added
    similar macros in a different place in the same file, and committed
    dbe79bbe9dcb22cb3651c46f18943477141ca452 "USB 3.0 Hub Changes".

    Some of the #defines used different names for the same values. Others
    used exactly the same names with the same values, like these gems:

    #define USB_PORT_FEAT_BH_PORT_RESET 28
    ...
    #define USB_PORT_FEAT_BH_PORT_RESET 28

    According to my very geeky husband (who looked it up in the C99 spec),
    it is allowed to have object-like macros with duplicate names as long as
    the replacement list is exactly the same. However, he recalled that
    some compilers will give warnings when they find duplicate macros. It's
    probably best to remove the duplicates in the stable tree, so that the
    code compiles for everyone.

    The macros are now fixed to move the feature requests that are specific
    to USB 3.0 hubs into a new section (out of the USB 2.0 hub feature
    section), and use the most common macro name.

    This patch should be backported to 2.6.39.

    Signed-off-by: Sarah Sharp <sarah.a.sharp@linux.intel.com>
    Cc: Tatyana Brokhman <tlinder@codeaurora.org>
    Cc: John Youn <johnyoun@synopsys.com>
    Cc: Jamey Sharp <jamey@minilop.net>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    include/linux/usb/ch11.h | 10 ++--------
    1 file changed, 2 insertions(+), 8 deletions(-)

    --- a/include/linux/usb/ch11.h
    +++ b/include/linux/usb/ch11.h
    @@ -62,12 +62,6 @@
    #define USB_PORT_FEAT_TEST 21
    #define USB_PORT_FEAT_INDICATOR 22
    #define USB_PORT_FEAT_C_PORT_L1 23
    -#define USB_PORT_FEAT_C_PORT_LINK_STATE 25
    -#define USB_PORT_FEAT_C_PORT_CONFIG_ERROR 26
    -#define USB_PORT_FEAT_PORT_REMOTE_WAKE_MASK 27
    -#define USB_PORT_FEAT_BH_PORT_RESET 28
    -#define USB_PORT_FEAT_C_BH_PORT_RESET 29
    -#define USB_PORT_FEAT_FORCE_LINKPM_ACCEPT 30

    /*
    * Port feature selectors added by USB 3.0 spec.
    @@ -76,8 +70,8 @@
    #define USB_PORT_FEAT_LINK_STATE 5
    #define USB_PORT_FEAT_U1_TIMEOUT 23
    #define USB_PORT_FEAT_U2_TIMEOUT 24
    -#define USB_PORT_FEAT_C_LINK_STATE 25
    -#define USB_PORT_FEAT_C_CONFIG_ERR 26
    +#define USB_PORT_FEAT_C_PORT_LINK_STATE 25
    +#define USB_PORT_FEAT_C_PORT_CONFIG_ERROR 26
    #define USB_PORT_FEAT_REMOTE_WAKE_MASK 27
    #define USB_PORT_FEAT_BH_PORT_RESET 28
    #define USB_PORT_FEAT_C_BH_PORT_RESET 29



    \
     
     \ /
      Last update: 2012-02-28 02:23    [W:4.131 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site