On Thu, Mar 12, 2020 at 2:55 PM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > Hi Sergey, > > thanks for your patch! > > On Fri, Mar 6, 2020 at 2:25 PM <Sergey.Semin@xxxxxxxxxxxxxxxxxxxx> wrote: > > > + err = devm_clk_bulk_get_optional(&pdev->dev, DWAPB_NR_CLOCKS, > > + gpio->clks); > > + if (err) { > > + dev_info(&pdev->dev, "Cannot get APB/Debounce clocks\n"); > > dev_err() again. > > > + err = clk_bulk_prepare_enable(DWAPB_NR_CLOCKS, gpio->clks); > > if (err) { > > - dev_info(&pdev->dev, "Cannot enable APB clock\n"); > > + dev_info(&pdev->dev, "Cannot enable APB/Debounce clocks\n"); > > dev_err() again. With that: Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Yours, Linus Walleij