From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Crispin Subject: Re: [PATCH V2 4/4] mfd: mediatek: add MT6323 support to MT6397 driver Date: Mon, 25 Jan 2016 19:59:47 +0100 Message-ID: <56A670A3.5030100@openwrt.org> References: <1453716887-38442-1-git-send-email-blogic@openwrt.org> <20160125124112.GG3368@x1> <56A64108.7020807@openwrt.org> <3476571.0yU9yvPsKF@linux-gy6r.site> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <3476571.0yU9yvPsKF@linux-gy6r.site> Sender: linux-kernel-owner@vger.kernel.org To: Matthias Brugger 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 List-Id: linux-mediatek@lists.infradead.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