<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 22, 2022, 4:46 PM Artem Chernyshev <<a href="mailto:artem.chernyshev@red-soft.ru">artem.chernyshev@red-soft.ru</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Error check after mxl111sf_read_reg<br>
<br>
Found by Linux Verification Center (<a href="http://linuxtesting.org" rel="noreferrer noreferrer" target="_blank">linuxtesting.org</a>) with SVACE.<br>
<br>
Fixes: 4c66c9205c07 ("[media] dvb-usb: add ATSC support for the Hauppauge WinTV-Aero-M")<br>
Signed-off-by: Artem Chernyshev <<a href="mailto:artem.chernyshev@red-soft.ru" target="_blank" rel="noreferrer">artem.chernyshev@red-soft.ru</a>><br>
---<br>
 drivers/media/usb/dvb-usb-v2/mxl111sf-phy.c | 3 ++-<br>
 1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/media/usb/dvb-usb-v2/mxl111sf-phy.c b/drivers/media/usb/dvb-usb-v2/mxl111sf-phy.c<br>
index 40b26712ba4c..ad1888514bd0 100644<br>
--- a/drivers/media/usb/dvb-usb-v2/mxl111sf-phy.c<br>
+++ b/drivers/media/usb/dvb-usb-v2/mxl111sf-phy.c<br>
@@ -121,7 +121,8 @@ int mxl111sf_config_mpeg_in(struct mxl111sf_state *state,<br>
        mxl_fail(ret);<br>
<br>
        /* Configure MPEG Clock phase */<br>
-       mxl111sf_read_reg(state, V6_MPEG_IN_CLK_INV_REG, &mode);<br>
+       ret = mxl111sf_read_reg(state, V6_MPEG_IN_CLK_INV_REG, &mode);<br>
+       mxl_fail(ret);<br>
<br>
        if (clock_phase == TSIF_NORMAL)<br>
                mode &= ~V6_INVERTED_CLK_PHASE;<br>
-- <br>
2.30.3<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Artem,</div><div dir="auto"><br></div><div dir="auto">Thank you for this.  If I recall correctly, the read from that particular register often returns a failure despite the correct value being read into memory.  This needs to be tested before being merged, and if my memory is correct about this we should add a comment about it. Unfortunately, I'm out of town at the moment, I won't be able to test this for some time. Can we hold off on this one for a bit? If you don't hear from me after a few weeks, please feel free to ping me again. Thanks.</div><div dir="auto"><br></div><div dir="auto">Best,</div><div dir="auto">Michael Krufky </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div></div>