From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javier Martinez Canillas Subject: Re: [PATCH 1/2] i2c: exynos5: simplify clock frequency handling Date: Thu, 20 Apr 2017 10:54:22 -0400 Message-ID: References: <1487934962-8703-1-git-send-email-a.hajda@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1487934962-8703-1-git-send-email-a.hajda@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org To: Andrzej Hajda , Wolfram Sang , Krzysztof Kozlowski , linux-i2c@vger.kernel.org, linux-samsung-soc@vger.kernel.org Cc: Bartlomiej Zolnierkiewicz , Marek Szyprowski List-Id: linux-i2c@vger.kernel.org Hello Andrzej, On 02/24/2017 08:16 AM, Andrzej Hajda wrote: > There is no need to keep separate settings for high and fast speed clock. > > Signed-off-by: Andrzej Hajda > --- > drivers/i2c/busses/i2c-exynos5.c | 45 +++++++++++----------------------------- > 1 file changed, 12 insertions(+), 33 deletions(-) > Patch looks good to me: Reviewed-by: Javier Martinez Canillas I've also tested it on an Exynos5800 Peach Pi Chromebook and the I2C devices are working properly after this change: Tested-by: Javier Martinez Canillas Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America