>On Monday, 8 of September 2008, shaohua.li@xxxxxxxxx wrote: >> This implements PCIe native PME detection. > >Could you please describe how this is done? > >Generally, please add kerneldoc comments to all of the functions and make >it >clear what they are for. Sorry, I'll do it later. Currently the main goal is to discuss the device core changes. We need generic implementation not just for x86. >It looks like 'dev' will always be the root complex. Is that correct? Right, and the port driver is just for root complex. >> + struct work_struct work; >> + u16 bdf; /* device which invokes PME */ >> + int exit; >> +}; >> + > >This one is analogous to pci_pme_active(), isn't it? > >Why don't you want it to clear the status bit as well? No, they are some pcie root complex's native pme bits, I'll document them later. Also, I'll address your other comments later. Thanks, Shaohua -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html