From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Figa Subject: Re: [PATCH 1/2] s5p-mfc: Update mfc-v6 firmware Date: Fri, 16 May 2014 12:22:19 +0200 Message-ID: <5375E6DB.70303@samsung.com> References: <1400233490-5677-1-git-send-email-arun.kk@samsung.com> <04be01cf70ec$d5f37a10$81da6e30$%debski@samsung.com> <04bf01cf70ee$f2184d40$d648e7c0$%debski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout3.w1.samsung.com ([210.118.77.13]:24086 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756815AbaEPKWe (ORCPT ); Fri, 16 May 2014 06:22:34 -0400 Received: from eucpsbgm2.samsung.com (unknown [203.254.199.245]) by mailout3.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0N5N00FLWWTJ7F00@mailout3.w1.samsung.com> for linux-samsung-soc@vger.kernel.org; Fri, 16 May 2014 11:22:31 +0100 (BST) In-reply-to: <04bf01cf70ee$f2184d40$d648e7c0$%debski@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Kamil Debski , 'Arun Kumar K' Cc: 'dwmw2' , 'ben' , 'linux-samsung-soc' Hi Kamil, Arun, On 16.05.2014 12:09, Kamil Debski wrote: > Hi, > > ----Original Message----- >> From: arunkk.samsung@gmail.com [mailto:arunkk.samsung@gmail.com] On >> Behalf Of Arun Kumar K >> Sent: Friday, May 16, 2014 12:00 PM >> >> Hi Kamil, >> >> On Fri, May 16, 2014 at 3:24 PM, Kamil Debski >> wrote: >>> Hi Arun, >>> >>> I asked you to put old and new v6 firmware in separate files. >> >> But wont that require a different filename other than s5p-mfc-v6.fw? > > Yes. > >> But the driver still expects the same file name. >> Can I put the new filename as s5p-mfc-v6-v2.fw and mention in the >> README that the file has to be renamed to s5p-mfc-v6.fw to be used? > > I think this is the way to go, because the new driver (with the patch) > will work with old firmware, while old driver (without the patch) will > not work with the new firmware. > >> >>> You should also mention in the commit message that this new firmware >>> will not work with the s5p-mfc driver without the patch you recently >>> submitted to linux-media mailing list. Please also add a link to the >>> thread with the necessary patch. >> >> Ok will do that. >> >>> >>> In addition to the above I think a readme/description file should be >>> added to the s5p-mfc folder. The issue of firmware for v6 should be >>> explained, so that no one is surprised that something does not work. >>> May I suggest a little different solution? Since to not break compatibility, support for both firmware versions is needed in the driver anyway, why not simply make the new, incompatible firmware always use a different filename. Then let the driver try to load the new one first and if it fails then load the old one? This would also let you get rid of that version check by hardcoded date, as you would know which firmware type was found. Best regards, Tomasz