Path: csiph.com!aioe.org!bofh.it!news.nic.it!robomod
From: york sun <york.sun@nxp.com>
Newsgroups: linux.kernel
Subject: Re: [Patch v4 3/9] driver/edac/fsl-ddr: Separate FSL DDR EDAC driver  from MPC85xx
Date: Fri, 12 Aug 2016 02:50:01 +0200
Message-ID: <s58PD-2US-1@gated-at.bofh.it>
References: <s4KaC-2JP-37@gated-at.bofh.it> <s4Ynf-4pW-17@gated-at.bofh.it> <s4Ynf-4pW-15@gated-at.bofh.it>
Thread-Topic: [Patch v4 3/9] driver/edac/fsl-ddr: Separate FSL DDR EDAC driver from MPC85xx
Thread-Index: AQHR89VgMyiQ5lf0M0WZuL07gU8aag==
Reply-To: york sun <york.sun@nxp.com>
Accept-Language: en-US
Content-Language: en-US
Received-Spf: None (protection.outlook.com: nxp.com does not designate permitted sender hosts)
Spamdiagnosticoutput: 1:99
Spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
X-Ms-Exchange-Crosstenant-Originalarrivaltime: 11 Aug 2016 20:13:16.5138 (UTC)
X-Ms-Exchange-Crosstenant-Fromentityheader: Hosted
X-Ms-Exchange-Crosstenant-ID: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-Ms-Exchange-Transport-Crosstenantheadersstamped: VI1PR0401MB2637
Sender: robomod@news.nic.it
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Approved: robomod@news.nic.it
Lines: 68
Organization: linux.* mail to news gateway
X-Original-Cc: "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>, "morbidrsa@gmail.com" <morbidrsa@gmail.com>, "oss@buserror.net" <oss@buserror.net>, Stuart Yoder <stuart.yoder@nxp.com>, Doug Thompson <dougthompson@xmission.com>, "mchehab@kernel.org" <mchehab@kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
X-Original-Date: Thu, 11 Aug 2016 20:13:16 +0000
X-Original-Message-ID: <AM4PR0401MB1732472363EE45A886CCA6BB9A1E0@AM4PR0401MB1732.eurprd04.prod.outlook.com>
X-Original-References: <1470779760-16483-1-git-send-email-york.sun@nxp.com> <1470779760-16483-4-git-send-email-york.sun@nxp.com> <20160811133612.GA5801@nazgul.tnic>
X-Original-Sender: linux-kernel-owner@vger.kernel.org
Xref: csiph.com linux.kernel:1460880

On 08/11/2016 06:36 AM, Borislav Petkov wrote:
> On Tue, Aug 09, 2016 at 02:55:40PM -0700, York Sun wrote:
>> The mpc85xx compatible DDR controllers are used on ARM-based SoCs.
>> Separate the DDR part from mpc85xx EDAC driver and prepare to support
>> both architecture.
>>
>> Signed-off-by: York Sun <york.sun@nxp.com>
>>
>> ---
>> Change log
>>   v4: Change comment in file header
>>       Use lower case "fsl_ddr_edac" for EDAC_MOD_STR
>>       Drop exporting probe and remove symbols
>>   v3: Fix compiling errors and warnings caused by patch ordering
>>   v2: Reordered patch
>>       Separate FSL DDR commont code as shared object, not another driver
>>       This patch is generated with "git format-patch -M40 -C40" to show
>>       copy-and-delete.
>
> This patch doesn't apply this way:
>
> $ git apply --check -v < /tmp/03-driver-edac-fsl-ddr-separate_fsl_ddr_edac_driver_from_mpc85xx.patch
> ...
>
> error: patch failed: drivers/edac/mpc85xx_edac.c:724
> error: drivers/edac/mpc85xx_edac.c: patch does not apply
> Checking patch drivers/edac/mpc85xx_edac.h...
>
> $ patch -p1 --dry-run -F10 -i /tmp/03-driver-edac-fsl-ddr-separate_fsl_ddr_edac_driver_from_mpc85xx.patch
> checking file drivers/edac/Makefile
> checking file drivers/edac/fsl_ddr_edac.c (copied from drivers/edac/mpc85xx_edac.c)
> Hunk #3 FAILED at 132.
> Hunk #4 succeeded at 1012 (offset -5 lines).
> Hunk #5 succeeded at 1080 (offset -7 lines).
> Hunk #6 FAILED at 1128.
> Hunk #7 succeeded at 1145 (offset -8 lines).
> Hunk #8 FAILED at 1173.
> 3 out of 8 hunks FAILED
> checking file drivers/edac/fsl_ddr_edac.h (copied from drivers/edac/mpc85xx_edac.h)
> checking file drivers/edac/mpc85xx_edac.c
> Hunk #3 FAILED at 627.
> 1 out of 3 hunks FAILED
> checking file drivers/edac/mpc85xx_edac.h
>
> $ git am -i /tmp/york.sun.03
> Applying: driver/edac/fsl-ddr: Separate FSL DDR EDAC driver from MPC85xx
> error: patch failed: drivers/edac/mpc85xx_edac.c:141
> error: drivers/edac/mpc85xx_edac.c: patch does not apply
> error: patch failed: drivers/edac/mpc85xx_edac.c:724
> error: drivers/edac/mpc85xx_edac.c: patch does not apply
> Patch failed at 0001 driver/edac/fsl-ddr: Separate FSL DDR EDAC driver from MPC85xx
> The copy of the patch that failed is found in: .git/rebase-apply/patch
> When you have resolved this problem, run "git am -i --continue".
> If you prefer to skip this patch, run "git am -i --skip" instead.
> To restore the original branch and stop patching, run "git am -i --abort".
>
> Please regenerate it against:
>
> http://git.kernel.org/cgit/linux/kernel/git/bp/bp.git#edac-for-4.9
>

Boris,

I checked your edac-for-4.9 branch. There are two minor changes when you 
apply the "Replace printk with proper pr_* format". I will send out a 
single "Patch v5 3/9".

York