On 26/03/2025 15:16, Mukesh Kumar Savaliya wrote:sorry, i ran and i got below error now:
+
+static int i3c_geni_resources_init(struct geni_i3c_dev *gi3c, struct platform_device *pdev)
+{
+ int ret;
+
+ gi3c->se.base = devm_platform_ioremap_resource(pdev, 0);
+ if (IS_ERR(gi3c->se.base))
+ return PTR_ERR(gi3c->se.base);
+
+ gi3c->se.clk = devm_clk_get(&pdev->dev, "se-clk");
Never tested.
Sorry for miss. I did test for this but what happened is while i worked for multiple other changes, i did miss merging of removal of se-clock-frequency from the driver code. I had hard-coded it instead of removing because we don't give this frequency selection to user right now.+ if (IS_ERR(gi3c->se.clk))
+ return dev_err_probe(&pdev->dev, PTR_ERR(gi3c->se.clk),
+ "Unable to get serial engine core clock: %pe\n",
+ gi3c->se.clk);
+
+ ret = device_property_read_u32(&pdev->dev, "se-clock-frequency", &gi3c->clk_src_freq);
I don't see previous comments implemented. Comment was: "Drop".
You did not test the DTS - again - even though I asked for that.
You claim you did internal review, but I have doubts because internalSure, i agree there are misses. will have internal review with dt-bindings and related changes in driver, then will post V3.
review would tell you how to test it (there is comprehensive internal
guide - see go/upstream). Then the testing would point out this issue.
Such trivial things should not be in v1 even. But they happened so you
got the review. Now you sent v2 ignoring that public review.
Sorry guys, please improve internal processes instead of wasting
reviewers time.
NAK.
Best regards,
Krzysztof