On 29/08/2023 11:18, Mateusz Majewski wrote: > The workqueue boilerplate is mostly one-to-one what the threaded > interrupts do. > > Signed-off-by: Mateusz Majewski <m.majewski2@xxxxxxxxxxx> > data->clk = devm_clk_get(&pdev->dev, "tmu_apbif"); > if (IS_ERR(data->clk)) { > dev_err(&pdev->dev, "Failed to get clock\n"); > @@ -1094,8 +1080,10 @@ static int exynos_tmu_probe(struct platform_device *pdev) > goto err_sclk; > } > > - ret = devm_request_irq(&pdev->dev, data->irq, exynos_tmu_irq, > - IRQF_TRIGGER_RISING | IRQF_SHARED, dev_name(&pdev->dev), data); > + ret = devm_request_threaded_irq( > + &pdev->dev, data->irq, NULL, exynos_tmu_threaded_irq, This does not look properly aligned. Best regards, Krzysztof