Replace numbers by named constants. The complete semantic patch used is as follows (http://coccinelle.lip6.fr/): // <smpl> @@ expression pdev; @@ pci_set_power_state(pdev, ( - 0 + PCI_D0 | - 1 + PCI_D1 | - 2 + PCI_D2 | - 3 + PCI_D3hot | - 4 + PCI_D3cold | - 5 + PCI_UNKNOWN | - -1 + PCI_POWER_ERROR ) ) @@ expression e1,e2; @@ pci_enable_wake(e1, ( - 0 + PCI_D0 | - 1 + PCI_D1 | - 2 + PCI_D2 | - 3 + PCI_D3hot | - 4 + PCI_D3cold | - 5 + PCI_UNKNOWN | - -1 + PCI_POWER_ERROR ) ,e2) // </smpl> -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html