From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753825AbcDKRQL (ORCPT ); Mon, 11 Apr 2016 13:16:11 -0400 Received: from lists.s-osg.org ([54.187.51.154]:41948 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750879AbcDKRQK (ORCPT ); Mon, 11 Apr 2016 13:16:10 -0400 Subject: Re: [PATCH] mfd: cros_ec: allow building for ARM64 To: Brian Norris , Lee Jones References: <1460146297-32535-1-git-send-email-briannorris@chromium.org> <20160411084849.GK4592@x1> <20160411164209.GA11034@localhost> Cc: Olof Johansson , linux-kernel@vger.kernel.org, Brian Norris From: Javier Martinez Canillas Message-ID: <570BDBCC.9060502@osg.samsung.com> Date: Mon, 11 Apr 2016 13:15:56 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <20160411164209.GA11034@localhost> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Brian, On 04/11/2016 12:42 PM, Brian Norris wrote: > On Mon, Apr 11, 2016 at 09:48:49AM +0100, Lee Jones wrote: >> When submitting patches, please use the format appropriate for the >> subsystem. >> >> `git log --oneline -- ` helps with this. > > $ git log --oneline --no-merges drivers/mfd/cros_ec*.c | head -10 > 8827a642a463 mfd: cros_ec_spi: Repair comparison ordering issue > 2756db6c6366 mfd: cros_ec_i2c: Fix trivial 'tabs before spaces' > whitespace issue. > 6d6e44a95316 mfd: cros ec: Lock the SPI bus while holding chipselect > 3821a065f567 spi: Drop owner assignment from spi_drivers > 385c0012dfa0 mfd: cros_ec_i2c: Add OF match table > a78ea195f77a mfd: cros_ec: spi: Add OF match table > 0e777366fb0e mfd: Drop owner assignment from i2c_drivers > cf649e00769a mfd: cros_ec: Staticise some newly introduced structures > ff4378f4b813 mfd: cros_ec: spi: Add delay for asserting CS > 57b33ff077be mfd: cros_ec: Support multiple EC in a system > > So...what's the problem with the subject? > I also don't see what's wrong with the subject line... >> Also please provide a commit log containing a short description of the >> change. Are you enabling ARM64 becuase the driver is able to be used >> on a new platform? If not, how did this work before? Etc. > > I can add a short description. But there's really not much more to it > than the subject. We have ARM64 platforms we want to use this on. Some > are already released, and some are in development. > I think at least a short description is always welcomed. Even if the patch is obvious, it should state why the change was introduced. > Brian > >>> Signed-off-by: Brian Norris >>> --- >>> drivers/mfd/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig >>> index eea61e349e26..1d63c5fb3f3c 100644 >>> --- a/drivers/mfd/Kconfig >>> +++ b/drivers/mfd/Kconfig >>> @@ -134,7 +134,7 @@ config MFD_CROS_EC >>> select MFD_CORE >>> select CHROME_PLATFORMS >>> select CROS_EC_PROTO >>> - depends on X86 || ARM || COMPILE_TEST >>> + depends on X86 || ARM || ARM64 || COMPILE_TEST >>> help >>> If you say Y here you get support for the ChromeOS Embedded >>> Controller (EC) providing keyboard, battery and power services. Patch looks good to me, so after resending with a commit message: Reviewed-by: Javier Martinez Canillas Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America