Re: [PATCH -next] xtensa/simdisk: fix error handling in proc_read_simdisk()

From: yiyang (D)
Date: Wed May 11 2022 - 05:46:35 EST




在 2022/5/11 17:04, Max Filippov 写道:
Hi Yi,

On Tue, May 10, 2022 at 1:07 AM Yi Yang <yiyang13@xxxxxxxxxx> wrote:

The frist simple_read_from_buffer() will change ppos. if ppos >= 1.
The second simple_read_from_buffer() will does not work and return 0.

Fixes: a69755b18774 ("xtensa simdisk: switch to proc_create_data()")
Signed-off-by: Yi Yang <yiyang13@xxxxxxxxxx>
---
arch/xtensa/platforms/iss/simdisk.c | 18 +++++++++++++-----
1 file changed, 13 insertions(+), 5 deletions(-)

Thank you for the fix. I've rearranged the patch a bit, expanding the commit
message and fixing the build warning that it introduced and have posted v2.
If there's no objections to it I'll add it to my xtensa tree.


The patch is very nice. Thank for you .

--
Yi