Re: [PATCH v2] of: remove internal arguments from of_property_for_each_u32()
From: Rob Herring <robh@kernel.org>
Date: 2024-07-24 13:53:49
Also in:
linux-arm-msm, linux-clk, linux-devicetree, linux-gpio, linux-iio, linux-omap, linux-pwm, linux-riscv, linux-samsung-soc, linux-serial, linux-sound, linux-sunxi, linux-usb, lkml, llvm
On Thu, Jul 18, 2024 at 1:57 AM Luca Ceresoli [off-list ref] wrote:
Hello Stephen, On Wed, 17 Jul 2024 16:33:34 -0700 Stephen Boyd [off-list ref] wrote:quoted
quoted
@@ -1191,20 +1191,24 @@ static int si5351_dt_parse(struct i2c_client *client, * property silabs,pll-source : <num src>, [<..>] * allow to selectively set pll source */ - of_property_for_each_u32(np, "silabs,pll-source", prop, p, num) { + sz = of_property_read_variable_u32_array(np, "silabs,pll-source", array, 2, 4); + sz = (sz == -EINVAL) ? 0 : sz; /* Missing property is OK */ + if (sz < 0) + return dev_err_probe(&client->dev, sz, "invalid pll-source");Needs a newline on the printk message.Ouch! Fix queued for v3.
I need v3 like today if I'm going to send it for rc1. Rob