12 comments:
File src/soc/intel/cannonlake/pch_h.c:
Patch Set #12, Line 2: /* This file is part of the coreboot project. */
drop this comment, but leave a blank line between the SPDX identifier and the #includes
Done, and sorted the includes
double blank line
Done
Patch Set #12, Line 18: switch (dev->path.pci.devfn)
agreed.
Done
Patch Set #12, Line 20: // UART0
Such useful comments! I don't think we need them, do we? 😄
Done
Patch Set #12, Line 22: struct acpi_irq irq
maybe declare this outside of the switch? that way, no braces are needed.
Done
Another double blank line
Done
spurious blank line
Done
Patch Set #12, Line 92: switch (dev->path.pci.devfn)
Would be good to fix
Done
Patch Set #12, Line 94: // UART0
We can live without these comments
Done
double empty line
Done
Patch Set #12, Line 119: .acpi_fill_ssdt = pch_h_uart_fill_ssdt,
We can do away with one tab less
Done
File src/soc/intel/cannonlake/pch_h.c:
AAAAA
To view, visit change 40405. To unsubscribe, or for help writing mail filters, visit settings.