On 12/8/21 2:44 PM, Mark Brown wrote:
On Wed, Dec 08, 2021 at 02:27:34PM -0700, Shuah Khan wrote:
} }
- snd_config_delete(config); } /*
This open comment at the end of the patch looks odd. Does this compile?
Yes, it's the start of the comment describing the next function, more complete context is:
} snd_config_delete(config);
}
/* * Check that we can read the default value and it is valid. Write * tests use the read value to restore the default. */ void test_ctl_get_value(struct ctl_data *ctl) {
Yes my bad :)
Reviewed-by: Shuah Khan skhan@linuxfoundation.org
thanks, -- Shuah