lkml.org 
[lkml]   [2013]   [Jul]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [RFC][PATCH 23/30] ACPI / hotplug / PCI: Do not exectute _PS0 and _PS3 directly
    On Fri, Jul 12, 2013 at 02:01:30AM +0200, Rafael J. Wysocki wrote:
    > Index: linux-pm/drivers/pci/hotplug/acpiphp.h
    > ===================================================================
    > --- linux-pm.orig/drivers/pci/hotplug/acpiphp.h
    > +++ linux-pm/drivers/pci/hotplug/acpiphp.h
    > @@ -160,7 +160,6 @@ struct acpiphp_attention_info
    >
    > /* slot flags */
    >
    > -#define SLOT_POWEREDON (0x00000001)
    > #define SLOT_ENABLED (0x00000002)
    > #define SLOT_MULTIFUNCTION (0x00000004)
    >
    > @@ -168,11 +167,7 @@ struct acpiphp_attention_info
    >
    > #define FUNC_HAS_STA (0x00000001)
    > #define FUNC_HAS_EJ0 (0x00000002)
    > -#define FUNC_HAS_PS0 (0x00000010)
    > -#define FUNC_HAS_PS1 (0x00000020)
    > -#define FUNC_HAS_PS2 (0x00000040)
    > -#define FUNC_HAS_PS3 (0x00000080)
    > -#define FUNC_HAS_DCK (0x00000100)
    > +#define FUNC_HAS_DCK (0x00000003)

    These are flags not enum so the above wants to be

    #define FUNC_HAS_DCK (0x00000004)

    otherwise we accidentally match checks like:

    /* install notify handler */
    if (!(newfunc->flags & FUNC_HAS_DCK)) {
    ...


    \
     
     \ /
      Last update: 2013-07-12 15:41    [W:2.108 / U:0.180 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site