Hi, Stefan
I agree that redundancy of If else may helps people to understand the code.
However, CONFIG_VDI_WRITE only contributes:
#if defined(CONFIG_VDI_WRITE)
.bdrv_co_pwritev = vdi_co_pwritev,
#endif
I think we don't need CONFIG_VDI_WRITE to document the code.
As its name implies, vdi_co_pwr
Am 30.07.2018 um 04:46 schrieb yuchen...@synology.com:
> From: yuchenlin
>
> This series refine some code in vdi.c, includes:
>
> * Remvoe CONFIG_VDI_WRITE because there is no reason to leave an always on
> and cannot configure option in the code-side.
> * decouple if else if chain to get more
From: yuchenlin
This series refine some code in vdi.c, includes:
* Remvoe CONFIG_VDI_WRITE because there is no reason to leave an always on
and cannot configure option in the code-side.
* decouple if else if chain to get more readability.
Thanks,
yuchenlin
yuchenlin (2):
vdi: remove CONFIG
