tabs are used first, followed by however may spaces (less than 8) needed to lineup at the end.
I'm not sure how this can be done perfectly with tabs only :)+ if (irq >= 0) {Align arguments on the opening brace.
+ ret = devm_request_irq(&pdev->dev, irq,
+ dmc520_isr, IRQF_SHARED,
+ dev_name(&pdev->dev), mci);
All other comments have been addressed in the next patch, many thanks!