<div dir="ltr"><div class="gmail_quote"><div dir="ltr"><div dir="ltr"><div>hi all,</div><div><br></div><div>My name is Ragavendra and I have about fifteen years of work experience in the IT industry mostly in web development. However I did diploma in embedded systems in CDAC in 2005 and have been using Linux since then. I submitted my first patch in 2015 and am planning to contribute again now for a while.</div><div><br></div><div>I went over the emails in the mailing list for the last few weeks or months to see how I can contribute. As a result, I signed up in Coverity and found one issue I thought I can help is say with the `1585136 Uninitialized scalar variable`.</div><div><br></div><div>////</div><div><br>diff --git a/drivers/clk/sophgo/clk-cv18xx-pll.c b/drivers/clk/sophgo/clk-cv18xx-pll.c<br>index 29e24098bf5f..a39a14fc39c2 100644<br>--- a/drivers/clk/sophgo/clk-cv18xx-pll.c<br>+++ b/drivers/clk/sophgo/clk-cv18xx-pll.c<br></div><div>@@ -327,7 +327,8 @@ static int fpll_find_rate(struct cv1800_clk_pll *pll,<br> static int fpll_determine_rate(struct clk_hw *hw, struct clk_rate_request *req)<br> {<br>        struct cv1800_clk_pll *pll = hw_to_cv1800_clk_pll(hw);<br>-       u32 val, ssc_syn_set;<br>+       u32 val = 0;<br>+       u32 ssc_syn_set = 0;<br><br>        if (!fpll_is_factional_mode(pll))<br>                return ipll_determine_rate(hw, req);<br><br>     fpll_find_rate(pll, &pll->pll_limit[2], req->best_parent_rate,<br>                     &req->rate, &val, &ssc_syn_set);</div><div><br></div><div>////</div><div><br></div><div>Here, I am assuming that the default value for u32 is 0, if not please let me know what it should be or how it is determined.</div><div><br></div><div>I know if it is of type pointer, its value should be NULL.</div><div><br></div><div>Another example I saw was like below, where the enum was initialized  which I am assuming to be is based on context.<br></div><div><br></div><div>////</div><div><br></div><div><div>diff --git a/drivers/cpufreq/amd-pstate-ut.c b/drivers/cpufreq/amd-pstate-ut.c<br>index 24432bd78a5a39..d52f26d1ffbab1 100644<br>--- a/<a href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/cpufreq/amd-pstate-ut.c?id=3e39e68dfbb6d4fe8c1943003d2f2a6f3255c902" target="_blank">drivers/cpufreq/amd-pstate-ut.c</a><br>+++ b/<a href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/cpufreq/amd-pstate-ut.c?id=93497752dfed196b41d2804503e80b9a04318adb" target="_blank">drivers/cpufreq/amd-pstate-ut.c</a></div><div>@@ -267,7 +267,7 @@ static int amd_pstate_set_mode(enum amd_pstate_mode mode)</div><div> </div><div> static void amd_pstate_ut_check_driver(u32 index)</div><div> {</div><div>-   enum amd_pstate_mode mode1, mode2;</div><div>+      enum amd_pstate_mode mode1, mode2 = AMD_PSTATE_DISABLE;</div><div>  int ret;</div></div></div><div dir="ltr"><br></div><div dir="ltr">////</div><div dir="ltr"><span class="gmail_signature_prefix"><br></span></div><div dir="ltr"><span class="gmail_signature_prefix"><br></span></div><div dir="ltr"><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Thanks & regards,<br>Ragavendra BN</div></div></div></div></div></div>