Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors
From: Javier Martinez Canillas
Date: Tue Jul 12 2016 - 11:03:47 EST
Hello Shuah,
On 07/11/2016 06:39 PM, Shuah Khan wrote:
> Fix misspelled error message and existing checkpatch errors in the
> error message conditional.
>
> WARNING: suspect code indent for conditional statements (8, 24)
> if (ctx->state != MFCINST_HEAD_PARSED &&
> [...]
> + mfc_err("Can not get crop information\n");
>
> Signed-off-by: Shuah Khan <shuahkh@xxxxxxxxxxxxxxx>
> ---
Patch looks good to me. Maybe is better to split the message and checkpatch
changes in two different patches. But I don't have a strong opinion on this:
Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America