* Re: [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH
2017-07-19 11:47 ` [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH Abhishek Sahu
@ 2017-08-02 5:47 ` Archit Taneja
2017-08-03 15:56 ` Boris Brezillon
2017-08-04 7:46 ` Boris Brezillon
2 siblings, 0 replies; 7+ messages in thread
From: Archit Taneja @ 2017-08-02 5:47 UTC (permalink / raw)
To: Abhishek Sahu, dwmw2, computersforpeace, boris.brezillon,
marek.vasut, richard, cyrille.pitchen, robh+dt, mark.rutland
Cc: linux-mtd, devicetree, linux-kernel, linux-arm-msm, andy.gross,
sricharan, stable
On 07/19/2017 05:17 PM, Abhishek Sahu wrote:
> The configuration for BCH is not correct in the current driver.
> The ECC_CFG_ECC_DISABLE bit defines whether to enable or disable the
> BCH ECC in which
>
> 0x1 : BCH_DISABLED
> 0x0 : BCH_ENABLED
>
> But currently host->bch_enabled is being assined to BCH_DISABLED.
s/assined/assigned
Thanks for fixing this up.
Reviewed-by: Archit Taneja <architt@codeaurora.org>
>
> Fixes: c76b78d8ec05a ("mtd: nand: Qualcomm NAND controller driver")
> Cc: stable@vger.kernel.org
> Signed-off-by: Abhishek Sahu <absahu@codeaurora.org>
> ---
> drivers/mtd/nand/qcom_nandc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/nand/qcom_nandc.c b/drivers/mtd/nand/qcom_nandc.c
> index 57d483a..bc0408c 100644
> --- a/drivers/mtd/nand/qcom_nandc.c
> +++ b/drivers/mtd/nand/qcom_nandc.c
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2016, The Linux Foundation. All rights reserved.
> + * Copyright (c) 2016-2017, The Linux Foundation. All rights reserved.
> *
> * This software is licensed under the terms of the GNU General Public
> * License version 2, as published by the Free Software Foundation, and
> @@ -1893,7 +1893,7 @@ static int qcom_nand_host_setup(struct qcom_nand_host *host)
> | wide_bus << WIDE_FLASH
> | 1 << DEV0_CFG1_ECC_DISABLE;
>
> - host->ecc_bch_cfg = host->bch_enabled << ECC_CFG_ECC_DISABLE
> + host->ecc_bch_cfg = !host->bch_enabled << ECC_CFG_ECC_DISABLE
> | 0 << ECC_SW_RESET
> | host->cw_data << ECC_NUM_DATA_BYTES
> | 1 << ECC_FORCE_CLK_OPEN
>
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH
2017-07-19 11:47 ` [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH Abhishek Sahu
2017-08-02 5:47 ` Archit Taneja
@ 2017-08-03 15:56 ` Boris Brezillon
2017-08-03 17:52 ` Abhishek Sahu
2017-08-04 7:46 ` Boris Brezillon
2 siblings, 1 reply; 7+ messages in thread
From: Boris Brezillon @ 2017-08-03 15:56 UTC (permalink / raw)
To: Abhishek Sahu
Cc: dwmw2, computersforpeace, marek.vasut, richard, cyrille.pitchen,
robh+dt, mark.rutland, devicetree, architt, linux-arm-msm,
linux-kernel, stable, linux-mtd, andy.gross, sricharan
On Wed, 19 Jul 2017 17:17:49 +0530
Abhishek Sahu <absahu@codeaurora.org> wrote:
> The configuration for BCH is not correct in the current driver.
> The ECC_CFG_ECC_DISABLE bit defines whether to enable or disable the
> BCH ECC in which
>
> 0x1 : BCH_DISABLED
> 0x0 : BCH_ENABLED
>
> But currently host->bch_enabled is being assined to BCH_DISABLED.
>
> Fixes: c76b78d8ec05a ("mtd: nand: Qualcomm NAND controller driver")
> Cc: stable@vger.kernel.org
> Signed-off-by: Abhishek Sahu <absahu@codeaurora.org>
> ---
> drivers/mtd/nand/qcom_nandc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/nand/qcom_nandc.c b/drivers/mtd/nand/qcom_nandc.c
> index 57d483a..bc0408c 100644
> --- a/drivers/mtd/nand/qcom_nandc.c
> +++ b/drivers/mtd/nand/qcom_nandc.c
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2016, The Linux Foundation. All rights reserved.
> + * Copyright (c) 2016-2017, The Linux Foundation. All rights reserved.
Hm, this copyright update is not really related to the changes you're
describing in the commit message. I'll drop this line from the commit
(no need to resend it) and ask you to send a separate patch updating the
copyright. Is that ok?
> *
> * This software is licensed under the terms of the GNU General Public
> * License version 2, as published by the Free Software Foundation, and
> @@ -1893,7 +1893,7 @@ static int qcom_nand_host_setup(struct qcom_nand_host *host)
> | wide_bus << WIDE_FLASH
> | 1 << DEV0_CFG1_ECC_DISABLE;
>
> - host->ecc_bch_cfg = host->bch_enabled << ECC_CFG_ECC_DISABLE
> + host->ecc_bch_cfg = !host->bch_enabled << ECC_CFG_ECC_DISABLE
> | 0 << ECC_SW_RESET
> | host->cw_data << ECC_NUM_DATA_BYTES
> | 1 << ECC_FORCE_CLK_OPEN
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH
2017-08-03 15:56 ` Boris Brezillon
@ 2017-08-03 17:52 ` Abhishek Sahu
2017-08-03 18:47 ` Boris Brezillon
0 siblings, 1 reply; 7+ messages in thread
From: Abhishek Sahu @ 2017-08-03 17:52 UTC (permalink / raw)
To: Boris Brezillon
Cc: dwmw2, computersforpeace, marek.vasut, richard, cyrille.pitchen,
robh+dt, mark.rutland, devicetree, architt, linux-arm-msm,
linux-kernel, stable, linux-mtd, andy.gross, sricharan
On 2017-08-03 21:26, Boris Brezillon wrote:
> On Wed, 19 Jul 2017 17:17:49 +0530
> Abhishek Sahu <absahu@codeaurora.org> wrote:
>
>> The configuration for BCH is not correct in the current driver.
>> The ECC_CFG_ECC_DISABLE bit defines whether to enable or disable the
>> BCH ECC in which
>>
>> 0x1 : BCH_DISABLED
>> 0x0 : BCH_ENABLED
>>
>> But currently host->bch_enabled is being assined to BCH_DISABLED.
>>
>> Fixes: c76b78d8ec05a ("mtd: nand: Qualcomm NAND controller driver")
>> Cc: stable@vger.kernel.org
>> Signed-off-by: Abhishek Sahu <absahu@codeaurora.org>
>> ---
>> drivers/mtd/nand/qcom_nandc.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/mtd/nand/qcom_nandc.c
>> b/drivers/mtd/nand/qcom_nandc.c
>> index 57d483a..bc0408c 100644
>> --- a/drivers/mtd/nand/qcom_nandc.c
>> +++ b/drivers/mtd/nand/qcom_nandc.c
>> @@ -1,5 +1,5 @@
>> /*
>> - * Copyright (c) 2016, The Linux Foundation. All rights reserved.
>> + * Copyright (c) 2016-2017, The Linux Foundation. All rights
>> reserved.
>
> Hm, this copyright update is not really related to the changes you're
> describing in the commit message. I'll drop this line from the commit
> (no need to resend it) and ask you to send a separate patch updating
> the
> copyright. Is that ok?
>
The idea was to change the copyright year since we are making
the changes in this file. If it requires separate patch, then I
will do the same.
Also I need to change 'assined' -> assigned in commit text. I can
update the patch and fix this. I am planning to send the v3 patches
tomorrow which incorporates all the review comments.
>> *
>> * This software is licensed under the terms of the GNU General
>> Public
>> * License version 2, as published by the Free Software Foundation,
>> and
>> @@ -1893,7 +1893,7 @@ static int qcom_nand_host_setup(struct
>> qcom_nand_host *host)
>> | wide_bus << WIDE_FLASH
>> | 1 << DEV0_CFG1_ECC_DISABLE;
>>
>> - host->ecc_bch_cfg = host->bch_enabled << ECC_CFG_ECC_DISABLE
>> + host->ecc_bch_cfg = !host->bch_enabled << ECC_CFG_ECC_DISABLE
>> | 0 << ECC_SW_RESET
>> | host->cw_data << ECC_NUM_DATA_BYTES
>> | 1 << ECC_FORCE_CLK_OPEN
--
Abhishek Sahu
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH
2017-08-03 17:52 ` Abhishek Sahu
@ 2017-08-03 18:47 ` Boris Brezillon
2017-08-03 19:02 ` Abhishek Sahu
0 siblings, 1 reply; 7+ messages in thread
From: Boris Brezillon @ 2017-08-03 18:47 UTC (permalink / raw)
To: Abhishek Sahu
Cc: dwmw2, computersforpeace, marek.vasut, richard, cyrille.pitchen,
robh+dt, mark.rutland, devicetree, architt, linux-arm-msm,
linux-kernel, stable, linux-mtd, andy.gross, sricharan
On Thu, 03 Aug 2017 23:22:37 +0530
Abhishek Sahu <absahu@codeaurora.org> wrote:
> On 2017-08-03 21:26, Boris Brezillon wrote:
> > On Wed, 19 Jul 2017 17:17:49 +0530
> > Abhishek Sahu <absahu@codeaurora.org> wrote:
> >
> >> The configuration for BCH is not correct in the current driver.
> >> The ECC_CFG_ECC_DISABLE bit defines whether to enable or disable the
> >> BCH ECC in which
> >>
> >> 0x1 : BCH_DISABLED
> >> 0x0 : BCH_ENABLED
> >>
> >> But currently host->bch_enabled is being assined to BCH_DISABLED.
> >>
> >> Fixes: c76b78d8ec05a ("mtd: nand: Qualcomm NAND controller driver")
> >> Cc: stable@vger.kernel.org
> >> Signed-off-by: Abhishek Sahu <absahu@codeaurora.org>
> >> ---
> >> drivers/mtd/nand/qcom_nandc.c | 4 ++--
> >> 1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/drivers/mtd/nand/qcom_nandc.c
> >> b/drivers/mtd/nand/qcom_nandc.c
> >> index 57d483a..bc0408c 100644
> >> --- a/drivers/mtd/nand/qcom_nandc.c
> >> +++ b/drivers/mtd/nand/qcom_nandc.c
> >> @@ -1,5 +1,5 @@
> >> /*
> >> - * Copyright (c) 2016, The Linux Foundation. All rights reserved.
> >> + * Copyright (c) 2016-2017, The Linux Foundation. All rights
> >> reserved.
> >
> > Hm, this copyright update is not really related to the changes you're
> > describing in the commit message. I'll drop this line from the commit
> > (no need to resend it) and ask you to send a separate patch updating
> > the
> > copyright. Is that ok?
> >
>
> The idea was to change the copyright year since we are making
> the changes in this file. If it requires separate patch, then I
> will do the same.
Yes, I'd prefer a separate patch.
>
> Also I need to change 'assined' -> assigned in commit text.
Yep, already fixed that one.
> I can
> update the patch and fix this. I am planning to send the v3 patches
> tomorrow which incorporates all the review comments.
Can you wait a bit. I already applied some of your patches [1] and would
like to finish my review.
Thanks,
Boris
[1]https://github.com/bbrezillon/linux-0day/commits/nand/next
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH
2017-08-03 18:47 ` Boris Brezillon
@ 2017-08-03 19:02 ` Abhishek Sahu
0 siblings, 0 replies; 7+ messages in thread
From: Abhishek Sahu @ 2017-08-03 19:02 UTC (permalink / raw)
To: Boris Brezillon
Cc: dwmw2, computersforpeace, marek.vasut, richard, cyrille.pitchen,
robh+dt, mark.rutland, devicetree, architt, linux-arm-msm,
linux-kernel, stable, linux-mtd, andy.gross, sricharan
On 2017-08-04 00:17, Boris Brezillon wrote:
> On Thu, 03 Aug 2017 23:22:37 +0530
> Abhishek Sahu <absahu@codeaurora.org> wrote:
>
>> On 2017-08-03 21:26, Boris Brezillon wrote:
>> > On Wed, 19 Jul 2017 17:17:49 +0530
>> > Abhishek Sahu <absahu@codeaurora.org> wrote:
>> >
>> >> The configuration for BCH is not correct in the current driver.
>> >> The ECC_CFG_ECC_DISABLE bit defines whether to enable or disable the
>> >> BCH ECC in which
>> >>
>> >> 0x1 : BCH_DISABLED
>> >> 0x0 : BCH_ENABLED
>> >>
>> >> But currently host->bch_enabled is being assined to BCH_DISABLED.
>> >>
>> >> Fixes: c76b78d8ec05a ("mtd: nand: Qualcomm NAND controller driver")
>> >> Cc: stable@vger.kernel.org
>> >> Signed-off-by: Abhishek Sahu <absahu@codeaurora.org>
>> >> ---
>> >> drivers/mtd/nand/qcom_nandc.c | 4 ++--
>> >> 1 file changed, 2 insertions(+), 2 deletions(-)
>> >>
>> >> diff --git a/drivers/mtd/nand/qcom_nandc.c
>> >> b/drivers/mtd/nand/qcom_nandc.c
>> >> index 57d483a..bc0408c 100644
>> >> --- a/drivers/mtd/nand/qcom_nandc.c
>> >> +++ b/drivers/mtd/nand/qcom_nandc.c
>> >> @@ -1,5 +1,5 @@
>> >> /*
>> >> - * Copyright (c) 2016, The Linux Foundation. All rights reserved.
>> >> + * Copyright (c) 2016-2017, The Linux Foundation. All rights
>> >> reserved.
>> >
>> > Hm, this copyright update is not really related to the changes you're
>> > describing in the commit message. I'll drop this line from the commit
>> > (no need to resend it) and ask you to send a separate patch updating
>> > the
>> > copyright. Is that ok?
>> >
>>
>> The idea was to change the copyright year since we are making
>> the changes in this file. If it requires separate patch, then I
>> will do the same.
>
> Yes, I'd prefer a separate patch.
Sure. will raise a separate patch
>
>>
>> Also I need to change 'assined' -> assigned in commit text.
>
> Yep, already fixed that one.
>
>> I can
>> update the patch and fix this. I am planning to send the v3 patches
>> tomorrow which incorporates all the review comments.
>
> Can you wait a bit. I already applied some of your patches [1] and
> would
> like to finish my review.
Thanks Boris. I will wait for it and will update my patch series
accordingly.
>
> Thanks,
>
> Boris
>
> [1]https://github.com/bbrezillon/linux-0day/commits/nand/next
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH
2017-07-19 11:47 ` [PATCH v2 01/25] mtd: nand: qcom: fix config error for BCH Abhishek Sahu
2017-08-02 5:47 ` Archit Taneja
2017-08-03 15:56 ` Boris Brezillon
@ 2017-08-04 7:46 ` Boris Brezillon
2 siblings, 0 replies; 7+ messages in thread
From: Boris Brezillon @ 2017-08-04 7:46 UTC (permalink / raw)
To: Abhishek Sahu
Cc: dwmw2, computersforpeace, marek.vasut, richard, cyrille.pitchen,
robh+dt, mark.rutland, linux-mtd, devicetree, linux-kernel,
linux-arm-msm, andy.gross, architt, sricharan, stable
On Wed, 19 Jul 2017 17:17:49 +0530
Abhishek Sahu <absahu@codeaurora.org> wrote:
> The configuration for BCH is not correct in the current driver.
> The ECC_CFG_ECC_DISABLE bit defines whether to enable or disable the
> BCH ECC in which
>
> 0x1 : BCH_DISABLED
> 0x0 : BCH_ENABLED
>
> But currently host->bch_enabled is being assined to BCH_DISABLED.
>
> Fixes: c76b78d8ec05a ("mtd: nand: Qualcomm NAND controller driver")
> Cc: stable@vger.kernel.org
> Signed-off-by: Abhishek Sahu <absahu@codeaurora.org>
> ---
> drivers/mtd/nand/qcom_nandc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/nand/qcom_nandc.c b/drivers/mtd/nand/qcom_nandc.c
> index 57d483a..bc0408c 100644
> --- a/drivers/mtd/nand/qcom_nandc.c
> +++ b/drivers/mtd/nand/qcom_nandc.c
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2016, The Linux Foundation. All rights reserved.
> + * Copyright (c) 2016-2017, The Linux Foundation. All rights reserved.
Applied to nand/next after removing the above line.
> *
> * This software is licensed under the terms of the GNU General Public
> * License version 2, as published by the Free Software Foundation, and
> @@ -1893,7 +1893,7 @@ static int qcom_nand_host_setup(struct qcom_nand_host *host)
> | wide_bus << WIDE_FLASH
> | 1 << DEV0_CFG1_ECC_DISABLE;
>
> - host->ecc_bch_cfg = host->bch_enabled << ECC_CFG_ECC_DISABLE
> + host->ecc_bch_cfg = !host->bch_enabled << ECC_CFG_ECC_DISABLE
> | 0 << ECC_SW_RESET
> | host->cw_data << ECC_NUM_DATA_BYTES
> | 1 << ECC_FORCE_CLK_OPEN
^ permalink raw reply [flat|nested] 7+ messages in thread