> +static void sparx5_hw_lock(struct sparx5 *sparx5)
> +{
> +     mutex_lock(&sparx5->lock);
> +}
> +
> +static void sparx5_hw_unlock(struct sparx5 *sparx5)
> +{
> +     mutex_unlock(&sparx5->lock);
> +}

Why is this mutex special and gets a wrapper where as the other two
don't? If there is no reason for the wrapper, please remove it.

       Andrew

Reply via email to