On 5/3/19 2:36 PM, Geert Uytterhoeven wrote: Hi, [...] >> &rwdt { >> @@ -309,6 +314,8 @@ >> }; >> >> &iic3 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pmic_irq_pins>; > > Given Blanche has a single device connected to irq2, I think it makes > sense to move the pinctrl properties to the pmic node below. It makes blanche somehow inconsistent with the other boards, but if you insist ... ? > With that fixed: > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > Gr{oetje,eeting}s, > > Geert > -- Best regards, Marek Vasut