On 07.03.2022 15:04, Miaoqian Lin wrote: > @@ -136,18 +138,21 @@ static int snd_proto_probe(struct platform_device *pdev) > dai_fmt |= snd_soc_daifmt_parse_clock_provider_as_flag(np, NULL); > } > > - of_node_put(bitclkmaster); > - of_node_put(framemaster); > - dai->dai_fmt = dai_fmt; > > - of_node_put(codec_np); > - of_node_put(cpu_np); > + dai->dai_fmt = dai_fmt; looks like there is an extra line that remains here With this addressed, you can add: Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@xxxxxxxxxxxxx> Thanks!