Currently call to request_irq() contains a line break between the function call and the assignment variable. This gives false positives when attempting to audit (grep) kernel code for correct handling of return value for the function. We can save future developers some time by removing the line break. This arguably makes the code a little cleaner also. Remove line break from between function call and return value assignment. Signed-off-by: Tobin C. Harding <me@xxxxxxxx> --- drivers/mmc/host/via-sdmmc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/mmc/host/via-sdmmc.c b/drivers/mmc/host/via-sdmmc.c index a838bf5480d8..9b9a447f26fe 100644 --- a/drivers/mmc/host/via-sdmmc.c +++ b/drivers/mmc/host/via-sdmmc.c @@ -1144,9 +1144,8 @@ static int via_sd_probe(struct pci_dev *pcidev, via_init_mmc_host(sdhost); - ret = - request_irq(pcidev->irq, via_sdc_isr, IRQF_SHARED, DRV_NAME, - sdhost); + ret = request_irq(pcidev->irq, via_sdc_isr, IRQF_SHARED, + DRV_NAME, sdhost); if (ret) goto unmap; -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html