On 2015年05月21日 23:53, Bjorn Helgaas wrote:
On Thu, May 21, 2015 at 10:21 AM, Bjorn Helgaas <bhelgaas@xxxxxxxxxx> wrote:
On Thu, May 21, 2015 at 10:17 AM, Hanjun Guo <hanjun.guo@xxxxxxxxxx> wrote:
On 2015年05月21日 23:10, Bjorn Helgaas wrote:
On Thu, May 21, 2015 at 8:52 AM, Hanjun Guo <hanjun.guo@xxxxxxxxxx> wrote:
list_head "list" in struct acpi_prt_entry was used to connect
_PRT entries for PCI irq, but after commit 181380b7 (PCI/ACPI:
Don't cache _PRT, and don't associate them with bus numbers),
the list head for _PRT entries was removed, but left "list" in
struct acpi_prt_entry which is useless and stale, remove it now.
Please use the canonical commit reference format, i.e.,
181380b702ee ("PCI/ACPI: Don't cache _PRT, and don't associate them
with bus numbers")
OK, now I know what is a canonical commit reference format :)
Yeah, it'd be nice if that were better documented. checkpatch
complains about it (d311cd44545f ("checkpatch: add test for commit id
formatting style in commit log")), but unfortunately I don't think
it's mentioned anywhere in Documentation/
I guess it actually *is* mentioned:
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches#n171,
but it's a little bit wishy-washy. I'll propose a tweak.
Thanks for the guidance :)
Hanjun
--
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