Date: Mon, 20 Mar 2017 10:45:23 +0000
From: Lee Jones <lee.jones@linaro.org>
To: sean.wang@mediatek.com
Cc: rpurdie@rpsys.net, jacek.anaszewski@gmail.com, matthias.bgg@gmail.com,
        pavel@ucw.cz, robh+dt@kernel.org, mark.rutland@arm.com,
        devicetree@vger.kernel.org, linux-leds@vger.kernel.org,
        linux-mediatek@lists.infradead.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        keyhaede@gmail.com
Subject: Re: [PATCH v6 4/4] mfd: mt6397: Align the placement at which the
 mfd_cell of LED is defined
Message-ID: <20170320104523.wt7t2rpqegdfqwfo@dell>
References: <1489992447-13007-1-git-send-email-sean.wang@mediatek.com>
 <1489992447-13007-5-git-send-email-sean.wang@mediatek.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <1489992447-13007-5-git-send-email-sean.wang@mediatek.com>
User-Agent: Mutt/1.6.2-neo (2016-08-21)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 945
Lines: 35

On Mon, 20 Mar 2017, sean.wang@mediatek.com wrote:

> From: Sean Wang <sean.wang@mediatek.com>
> 
> Align the placement as which the mfd_cell of LED is defined as the other
> members done on the structure.
> 
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
> Acked-by: Lee Jones <lee.jones@linaro.org>
> ---
>  drivers/mfd/mt6397-core.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

Applied, thanks.

> diff --git a/drivers/mfd/mt6397-core.c b/drivers/mfd/mt6397-core.c
> index 8e601c8..04a601f 100644
> --- a/drivers/mfd/mt6397-core.c
> +++ b/drivers/mfd/mt6397-core.c
> @@ -47,8 +47,7 @@
>  	{
>  		.name = "mt6323-regulator",
>  		.of_compatible = "mediatek,mt6323-regulator"
> -	},
> -	{
> +	}, {
>  		.name = "mt6323-led",
>  		.of_compatible = "mediatek,mt6323-led"
>  	},

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog