On 23.02.2021 19:29, Krzysztof Kozlowski wrote:
On Tue, 23 Feb 2021 at 19:20, Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com wrote:
On 2/23/21 5:25 AM, Sylwester Nawrocki wrote:
On 22.02.2021 22:33, Pierre-Louis Bossart wrote:
Move the top-level variable to the lower scope where it's needed.
Actually I like your original patch better as there is really no need for multiple lower scope declarations in that fairly small function.
I have no opinion, just let me know what the consensus is.
I proposed to have both variables local scope, to reduce the size of function-scope variables. Their number tends to grow in probe() a lot, so when a variable can be localized more, it makes the code easier to understand. No need to figure out who/where/when uses the variable. Local scope makes it much easier.
I don't have strong opinion, let's keep it local then as in current patch.