Re: [PATCH 4/8] staging: erofs: cleanup z_erofs_vle_work_{lookup, register}

From: Gao Xiang
Date: Mon Aug 13 2018 - 09:21:05 EST


Hi Dan,

On 2018/8/13 21:05, Dan Carpenter wrote:
> Yeah. You'd have to remove the const.
>
> Anyway, on looking at it more, I guess this patch is fine for now. We
> will probably end up changing z_erofs_vle_work_lookup() and
> z_erofs_vle_work_register() some more in the future.
>

Thanks for review, I personally tend to leave this patch unmodified for now :( .........

The struct is wrote in order to read-only or assign and read at once, and let compilers notice that
(all modifications are local so compilers can handle them safely)...

Thanks,
Gao Xiang

> regards,
> dan carpenter
>