From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933679AbcAYTAB (ORCPT ); Mon, 25 Jan 2016 14:00:01 -0500 Received: from arrakis.dune.hu ([78.24.191.176]:50011 "EHLO arrakis.dune.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932765AbcAYS77 (ORCPT ); Mon, 25 Jan 2016 13:59:59 -0500 Subject: Re: [PATCH V2 4/4] mfd: mediatek: add MT6323 support to MT6397 driver To: Matthias Brugger References: <1453716887-38442-1-git-send-email-blogic@openwrt.org> <20160125124112.GG3368@x1> <56A64108.7020807@openwrt.org> <3476571.0yU9yvPsKF@linux-gy6r.site> Cc: Steven Liu , Sascha Hauer , linux-kernel@vger.kernel.org, Henry Chen , Flora Fu , linux-mediatek@lists.infradead.org, Lee Jones , linux-arm-kernel@lists.infradead.org From: John Crispin Message-ID: <56A670A3.5030100@openwrt.org> Date: Mon, 25 Jan 2016 19:59:47 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <3476571.0yU9yvPsKF@linux-gy6r.site> 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 On 25/01/2016 19:44, Matthias Brugger wrote: > On Monday 25 Jan 2016 16:36:40 John Crispin wrote: >> Hi, >> >> On 25/01/2016 13:41, Lee Jones wrote: >>> Please honour the subject format of the subsystem you are contributing >>> to. >>> >>> `git log --oneline -- $subsystem` gives you this. >>> >>> On Mon, 25 Jan 2016, John Crispin wrote: >>>> Signed-off-by: John Crispin >>>> --- >> >> [...] >> >>>> @@ -261,6 +271,15 @@ static int mt6397_probe(struct platform_device >>>> *pdev) >>>> >>>> } >>>> >>>> switch (id & 0xff) { >>>> >>>> + case MT6323_CID_CODE: >>>> + mt6397->int_con[0] = MT6323_INT_CON0; >>> >>> This is confusing. You're still using memory allocated for a mt6397 >>> device. >> >> the variable is currently defined as struct mt6397_chip *mt6397; >> shall i only change the name or also create a patch to rename the struct ? >> > > I think we should rename the struct and the file as well. > > Cheers, > Matthias Hi, that would have been my next question. renaming the struct would imply renaming the driver and the whole namespace contained within. We would then also need to change the Kconfig and Makefile. I am happy to do this but want to be sure that is is actually wanted. John