Fixed 'no assignment in if condition' coding style issue and removed unnecessary spaces at the start of a line.
Signed-off-by: Himanshu Jha <himanshujha199...@gmail.com> --- drivers/edac/i82860_edac.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/edac/i82860_edac.c b/drivers/edac/i82860_edac.c index 236c813..c8c1c4d 100644 --- a/drivers/edac/i82860_edac.c +++ b/drivers/edac/i82860_edac.c @@ -282,7 +282,9 @@ static void i82860_remove_one(struct pci_dev *pdev) if (i82860_pci) edac_pci_release_generic_ctl(i82860_pci); - if ((mci = edac_mc_del_mc(&pdev->dev)) == NULL) + mci = edac_mc_del_mc(&pdev->dev); + + if (mci == NULL) return; edac_mc_free(mci); @@ -312,10 +314,11 @@ static int __init i82860_init(void) edac_dbg(3, "\n"); - /* Ensure that the OPSTATE is set correctly for POLL or NMI */ - opstate_init(); + /* Ensure that the OPSTATE is set correctly for POLL or NMI */ + opstate_init(); - if ((pci_rc = pci_register_driver(&i82860_driver)) < 0) + pci_rc = pci_register_driver(&i82860_driver); + if (pci_rc < 0) goto fail0; if (!mci_pdev) { -- 2.7.4