Re: [PATCH 1/4] Input: goodix - Restructure cfg checksum function

From: Bastien Nocera
Date: Thu Oct 27 2016 - 09:57:44 EST


On Thu, 2016-10-20 at 14:59 -0500, Franklin S Cooper Jr wrote:
> Split generation of checksum into is own function. Currently it is used

"its own".

> only to determine if the checksum for the user provided configuration file
> is valid or not. However, support to dynamically alter the configuration
> file will be supported and needs a way to calculate the updated file's
> checksum. Therefore, by splitting calculating checksum into its own
> function the code can be reused.
>

After the fixup above:

Reviewed-by: Bastien Nocera <hadess@xxxxxxxxxx>