On 5/31/23 03:25, Ping-Ke Shih wrote:
Another way is to add a debug message to show them, and then static checker shouldn't warn this. Also, people can diagnose IQK & LOK results if needed.
1. When CONFIG_RTLWIFI_DEBUG is not set, rtl_dbg() becomes a no-op inline function which doesn't use any of its arguments at all. This may (an will) cause the tool to produce even more warnings. 2. I don't like an idea to add some code just to make some tool happy. 3. In general, is it always (or just sometimes) required to read (some subset of?) BB registers even if we don't interested in their values? Is it explicitly required by the hardware design? Thanks, Dmitry
