Hi, On Fri, Dec 2, 2022 at 8:08 AM Doug Anderson <dianders@xxxxxxxxxxxx> wrote: > > Hi, > > On Fri, Dec 2, 2022 at 7:57 AM Krzysztof Kozlowski > <krzysztof.kozlowski@xxxxxxxxxx> wrote: > > > > The pin config entry should have a string, not number, for the GPIO used > > as WCD9340 audio codec interrupt. > > > > Reported-by: Doug Anderson <dianders@xxxxxxxxxxxx> > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > > > > --- > > > > Cc: Doug Anderson <dianders@xxxxxxxxxxxx> > > > > Changes since v2: > > 1. New patch. > > --- > > arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > Presumably should have a "Fixes" tag since this is likely a true bug. > > Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx> FWIW, this got copy-pasta-ed to another place, which I just noticed as I'm looking at your v3. So while your change is correct, it'd be better to also fix "sdm845-xiaomi-beryllium-common.dtsi" -Doug