Re: [PATCH v2] staging: exfat: remove fs_func struct.

[Greg Kroah-Hartman]

On Thu, Jan 23, 2020 at 05:32:58PM +0900, Tetsuhiro Kohada wrote:
> From: "Tetsuhiro Kohada" <Kohada.Tetsuhiro@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
> 
> Remove 'fs_func struct' and change indirect calls to direct calls.
> 
> The following issues are described in exfat's TODO.
> > Create helper function for exfat_set_entry_time () and
> > exfat_set_entry_type () because it's sort of ugly to be calling the same functionn directly and other code calling through  the fs_func struc ponters ...
> 
> The fs_func struct was used for switching the helper functions of fat16/fat32/exfat.
> Now, it has lost the role of switching, just making the code less readable.
> 
> Signed-off-by: Tetsuhiro Kohada <Kohada.Tetsuhiro@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
> ---
>  drivers/staging/exfat/exfat.h       |  79 +++++-----
>  drivers/staging/exfat/exfat_core.c  | 214 +++++++++++-----------------
>  drivers/staging/exfat/exfat_super.c | 119 ++++++++--------
>  3 files changed, 178 insertions(+), 234 deletions(-)

What changed from v1?

Always put that below the --- line, as the documentation asks you to, so
that we know what is happening here.

Please fix that up and send a v3.

thanks,

greg k-h