<div dir="ltr"><div class="gmail_quote"><br><div dir="ltr">Hello, <div>I had submitted following patch:</div><div><br></div><div>```</div><div><font face="arial narrow, sans-serif">---<br> drivers/staging/rtl8712/rtl871x_pwrctrl.h | 12 ++++++------<br> 1 file changed, 6 insertions(+), 6 deletions(-)<br><br>diff --git a/drivers/staging/rtl8712/rtl871x_pwrctrl.h b/drivers/staging/rtl8712/rtl871x_pwrctrl.h<br>index b35b9c792..29f8b6136 100644<br>--- a/drivers/staging/rtl8712/rtl871x_pwrctrl.h<br>+++ b/drivers/staging/rtl8712/rtl871x_pwrctrl.h<br>@@ -74,14 +74,14 @@ struct reportpwrstate_parm {<br><br> struct pwrctrl_priv {<br>        struct mutex mutex_lock;<br>-       /*volatile*/ u8 rpwm; /* requested power state for fw */<br>+       u8 rpwm; /* requested power state for fw */<br>        /* fw current power state. updated when 1. read from HCPWM or<br>         * 2. driver lowers power level<br>         */<br>-       /*volatile*/ u8 cpwm;<br>-       /*volatile*/ u8 tog; /* toggling */<br>-       /*volatile*/ u8 cpwm_tog; /* toggling */<br>-       /*volatile*/ u8 tgt_rpwm; /* wanted power state */<br>+       u8 cpwm;<br>+       u8 tog; /* toggling */<br>+       u8 cpwm_tog; /* toggling */<br>+       u8 tgt_rpwm; /* wanted power state */<br>        uint pwr_mode;<br>        uint smart_ps;<br>        uint alives;<br>@@ -106,7 +106,7 @@ void r8712_unregister_cmd_alive(struct _adapter *padapter);<br> void r8712_cpwm_int_hdl(struct _adapter *padapter,<br>                        struct reportpwrstate_parm *preportpwrstate);<br> void r8712_set_ps_mode(struct _adapter *padapter, uint ps_mode,<br>-                       uint smart_ps);<br>+                      uint smart_ps);<br> void r8712_set_rpwm(struct _adapter *padapter, u8 val8);<br> void r8712_flush_rwctrl_works(struct _adapter *padapter);</font><br></div><div>```</div><div>And I was asked to verify if there is some specific meaning is attached to comment here (which was causing the issue).</div><div><font color="#000000">I would be glad you could explain me how should I approach this issue? One way would<br>be to rewrite that these variables could be defined as volatile (just add a comment) and then compile driver and see that build goes through without any error. </font></div><div><font color="#000000">Other way would be that try to understand what this function is supposed to be doing and then figure out author's intent of putting volatile there. How should I take decision on these (or if they are wrong approaches) ?</font><br></div><div><font color="#000000"><br></font></div><div><font color="#000000">Thanks,</font></div><div><font color="#000000">Ankit</font></div><div><br></div></div>
</div></div>