On 12/05/2020 07:38, Anand Jain wrote:
[...]
> +/*
> + * Adjust @level according to the limits of the compression algorithm or
> + * fallback to default
> + */
> +static unsigned int btrfs_compress_set_level(int type, unsigned level)
> +{
> + const struct btrfs_compress_op *ops = btrfs_compress_op[type];
> +
> + if (level == 0)
> + level = ops->default_level;
> + else
> + level = min(level, ops->max_level);
> +
> + return level;
> +}
> +
[...]
> -/*
> - * Adjust @level according to the limits of the compression algorithm or
> - * fallback to default
> - */
> -unsigned int btrfs_compress_set_level(int type, unsigned level)
> -{
> - const struct btrfs_compress_op *ops = btrfs_compress_op[type];
> -
> - if (level == 0)
> - level = ops->default_level;
> - else
> - level = min(level, ops->max_level);
> -
> - return level;
> -}
Why did you move the function?
Apart from that,
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@xxxxxxx>
