Re: [PATCH] ASoC: codecs: aw88399: Correct error handling in aw_dev_get_dsp_status function

From: Markus Elfring
Date: Tue Oct 08 2024 - 03:39:06 EST


* Please check the subsystem specification once more.

* How do you think about to replace the word “function” (in the summary phrase)
by parentheses?


> Added proper error handling for register value check that

Add?


> return -EPERM when register value does not meet expected condition

* It is usually preferred to end a sentence with a dot, isn't it?

* Would you like to add any tags (like “Fixes” and “Cc”) accordingly?


Regards,
Markus