From mboxrd@z Thu Jan 1 00:00:00 1970 From: Qianyu Gong Date: Wed, 17 Feb 2016 04:36:40 +0000 Subject: [U-Boot] [Patch V3 2/3] fm: fdt: Move fman ucode fixup to Fman driver code In-Reply-To: <1455657750.2463.69.camel@buserror.net> References: <1453721834-7772-1-git-send-email-Qianyu.Gong@nxp.com> <1453721834-7772-2-git-send-email-Qianyu.Gong@nxp.com> <1453742223.27129.10.camel@buserror.net> <1454959116.2486.38.camel@buserror.net> <1454959510.2486.41.camel@buserror.net> <1455657750.2463.69.camel@buserror.net> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de > -----Original Message----- > From: Scott Wood [mailto:oss at buserror.net] > Sent: Wednesday, February 17, 2016 5:23 AM > To: Qianyu Gong ; york sun ; u- > boot at lists.denx.de > Subject: Re: [U-Boot] [Patch V3 2/3] fm: fdt: Move fman ucode fixup to Fman > driver code > > On Mon, 2016-02-15 at 05:44 +0000, Qianyu Gong wrote: > > > -----Original Message----- > > > From: york sun > > > Sent: Friday, February 12, 2016 1:39 AM > > > To: Scott Wood ; Qianyu Gong > > > ; u- boot at lists.denx.de > > > Subject: Re: [U-Boot] [Patch V3 2/3] fm: fdt: Move fman ucode fixup > > > to Fman driver code > > > > > > On 02/08/2016 11:25 AM, Scott Wood wrote: > > > > On Mon, 2016-02-08 at 19:22 +0000, york sun wrote: > > > > > On 02/08/2016 11:18 AM, Scott Wood wrote: > > > > > > On Mon, 2016-02-08 at 19:03 +0000, york sun wrote: > > > > > > > On 02/01/2016 09:06 AM, york sun wrote: > > > > > > > > On 01/25/2016 09:40 PM, Qianyu Gong wrote: > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > From: Scott Wood [mailto:oss at buserror.net] > > > > > > > > > > Sent: Tuesday, January 26, 2016 1:17 AM > > > > > > > > > > To: Qianyu Gong ; > > > > > > > > > > u-boot at lists.denx.de > > > > > > > > > > Cc: B07421 at freescale.com; Shaohui Xie > > > > > > > > > > > > > > > > > > > > Subject: Re: [U-Boot] [Patch V3 2/3] fm: fdt: Move > > > > > > > > > > fman ucode fixup to Fman driver code > > > > > > > > > > > > > > > > > > > > On Mon, 2016-01-25 at 19:37 +0800, Gong Qianyu wrote: > > > > > > > > > > > Both Freescale Layerscape and powerpc/mpc85xx > > > > > > > > > > > platforms are using > > > > > > > > > > > fdt_fixup_fman_firmware() to insert Fman ucode blob > > > > > > > > > > > into the device tree. So move the function to driver > > > > > > > > > > > code. > > > > > > > > > > > > > > > > > > > > > > Signed-off-by: Gong Qianyu > > > > > > > > > > > --- > > > > > > > > > > > V3: > > > > > > > > > > > - Remove file changes about "qe.h". > > > > > > > > > > > (Should be put in the first patch of this > > > > > > > > > > > patchset) > > > > > > > > > > > V2: > > > > > > > > > > > - New patch. > > > > > > > > > > > > > > > > > > > > > > arch/powerpc/cpu/mpc85xx/fdt.c | 125 > > > > > > > > > > > ++------------------------------------ > > > > > > > > > > > - > > > > > > > > > > > drivers/net/fm/Makefile | 1 + > > > > > > > > > > > drivers/net/fm/fdt.c | 129 > > > > > > > > > > > +++++++++++++++++++++++++++++++++++++++++ > > > > > > > > > > > include/fsl_fman.h | 1 + > > > > > > > > > > > 4 files changed, 136 insertions(+), 120 > > > > > > > > > > > deletions(-) > > > > > > > > > > > > > > > > > > > > Again, pass -M -C to git format-patch. > > > > > > > > > > > > > > > > > > > > -Scott > > > > > > > > > > > > > > > > > > I don't understand but I've already used "git > > > > > > > > > format-patch -M -C > > > > > > > > > - -stat ...". > > > > > > > > > > > > > > > > > > > > > > > > > Scott means using -M and -C, git should detect the moving > > > > > > > > instead of adding and deleting the same code. Try to add > > > > > > > > --find-copies-harder to see if it generates a smaller > > > > > > > > patch. > > > > > > > > > > > > > > > > > > > > > > Qianyu, > > > > > > > > > > > > > > Since you are on holiday, I tried it for you. Adjusting "-M -C" > > > > > > > doesn't > > > > > > > work. > > > > > > > Even you are moving the function from one file to another, > > > > > > > "git format -patch" > > > > > > > cannot detect the moving because both files exist before and > > > > > > > after this change, and the change set is not significant > > > > > > > enough to be detected. > > > > > > > > > > > > It looks like the patch is creating drivers/net/fm/fdt.c... > > > > > > > > > > > > Actually you were right. This patch creates a new file. There are > > > also other small changes here and there. Maybe that's the reason git > > > doesn't detect the move. > > > > > > Anyway, let me know if you have further comment. I am testing these > > > patches. > > > > > > York > > > > Hi York, > > > > Thanks! I did make small changes in the function to make it work for > > both powerpc and arm platforms. > > Those changes should have been a separate patch -- otherwise it's a lot of work to > see what those changes are and review them. > > -Scott OK.. Thanks for your reminder. Regards, Qianyu