From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f174.google.com ([209.85.128.174]:33988 "EHLO mail-wr0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750786AbeBUUOF (ORCPT ); Wed, 21 Feb 2018 15:14:05 -0500 Received: by mail-wr0-f174.google.com with SMTP id m5so7989139wrg.1 for ; Wed, 21 Feb 2018 12:14:05 -0800 (PST) Subject: Re: [PATCH 1/2] ARM: shmobile: Factor out complex condition To: Simon Horman Cc: linux-renesas-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Marek Vasut , Geert Uytterhoeven , Kuninori Morimoto , Wolfram Sang References: <20180217020642.21375-1-marek.vasut+renesas@gmail.com> <20180219085837.tsza3xakwxodrwko@verge.net.au> From: Marek Vasut Message-ID: Date: Wed, 21 Feb 2018 20:59:59 +0100 MIME-Version: 1.0 In-Reply-To: <20180219085837.tsza3xakwxodrwko@verge.net.au> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: On 02/19/2018 09:58 AM, Simon Horman wrote: > On Sat, Feb 17, 2018 at 03:06:41AM +0100, Marek Vasut wrote: >> Pull the complex condition in regulator_quirk_notify() into >> regulator_quirk_check(). Moreover, do not hard-code the I2C >> address, but rather use the one in da9xxx_msgs[]. >> >> Signed-off-by: Marek Vasut >> Cc: Geert Uytterhoeven >> Cc: Kuninori Morimoto >> Cc: Simon Horman >> Cc: Wolfram Sang > > There appears to be some review of this series that needs addressing, > I have marked it as "Changes Requested". > Jupp -- Best regards, Marek Vasut From mboxrd@z Thu Jan 1 00:00:00 1970 From: marek.vasut@gmail.com (Marek Vasut) Date: Wed, 21 Feb 2018 20:59:59 +0100 Subject: [PATCH 1/2] ARM: shmobile: Factor out complex condition In-Reply-To: <20180219085837.tsza3xakwxodrwko@verge.net.au> References: <20180217020642.21375-1-marek.vasut+renesas@gmail.com> <20180219085837.tsza3xakwxodrwko@verge.net.au> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 02/19/2018 09:58 AM, Simon Horman wrote: > On Sat, Feb 17, 2018 at 03:06:41AM +0100, Marek Vasut wrote: >> Pull the complex condition in regulator_quirk_notify() into >> regulator_quirk_check(). Moreover, do not hard-code the I2C >> address, but rather use the one in da9xxx_msgs[]. >> >> Signed-off-by: Marek Vasut >> Cc: Geert Uytterhoeven >> Cc: Kuninori Morimoto >> Cc: Simon Horman >> Cc: Wolfram Sang > > There appears to be some review of this series that needs addressing, > I have marked it as "Changes Requested". > Jupp -- Best regards, Marek Vasut