Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: isgXIEVo9whT9d2IzSQE7ZVA32XXAMQPf2m4toZdeyXUrStHs72k4AjbCkh5jlhDEgGcNrAvD5
 /Gf0kqE8ySwA==
IronPort-SDR: s+Ji13/ZuR5HcMwMZftRZZSKUuLxfL/rVjaU/LHRZ21UChLMi8eHN1VmkJNNj8zXdkageINVO9
 I6jn2Y0qYFYw==
Date:   Wed, 22 Jul 2020 14:07:11 +0300
From:   Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To:     Rayagonda Kokatanur <rayagonda.kokatanur@broadcom.com>
Cc:     Wolfram Sang <wsa@kernel.org>, linux-i2c@vger.kernel.org,
        linux-kernel@vger.kernel.org, Ray Jui <rjui@broadcom.com>,
        Scott Branden <sbranden@broadcom.com>,
        bcm-kernel-feedback-list@broadcom.com,
        Lori Hikichi <lori.hikichi@broadcom.com>,
        Robert Richter <rrichter@marvell.com>,
        Nishka Dasgupta <nishkadg.linux@gmail.com>,
        linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 0/2] add PEC support on slave side
Message-ID: <20200722110711.GY3703480@smile.fi.intel.com>
References: <20200717090155.10383-1-rayagonda.kokatanur@broadcom.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200717090155.10383-1-rayagonda.kokatanur@broadcom.com>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, Jul 17, 2020 at 02:31:53PM +0530, Rayagonda Kokatanur wrote:
> This patch set adds support for PEC on Slave side.

LGTM! FWIW,
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> Changes from v1:
>  -Address review comments from Andy Shevchenko
>   Update commit message,
>   Rewrite bcm_iproc_smbus_check_slave_pec() to remove local
>   variable ret and type casting,
>   Use positive condition.
> 
> Rayagonda Kokatanur (2):
>   i2c: add PEC error event
>   i2c: iproc: add slave pec support
> 
>  drivers/i2c/busses/i2c-bcm-iproc.c | 49 +++++++++++++++++++++++++++---
>  include/linux/i2c.h                |  1 +
>  2 files changed, 46 insertions(+), 4 deletions(-)
> 
> -- 
> 2.17.1
> 

-- 
With Best Regards,
Andy Shevchenko