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