[PATCH 8/8] mtd: rawnand: qcom: Fix address parsing within ->exec_op()

Sricharan Ramabadhran quic_srichara at quicinc.com
Thu Jul 27 21:14:37 PDT 2023


Hi Miquel,

On 7/28/2023 8:03 AM, Sricharan Ramabadhran wrote:
> Hi Miquel,
> 
> On 7/27/2023 8:29 PM, Miquel Raynal wrote:
>> Hi Sadre,
>>
>> mani at kernel.org wrote on Mon, 17 Jul 2023 12:08:43 +0530:
>>
>>> On Sun, Jul 16, 2023 at 04:46:12PM +0200, Miquel Raynal wrote:
>>>> The naddrs variable is initialized but not used. Fixing this could have
>>>> been a matter of dropping the variable, but the right way to do it 
>>>> looks
>>>> a bit more complex: we can avoid useless writes to the q_op 
>>>> structure by
>>>> using it. In practice we could even have possible out-of-bound bugs 
>>>> with
>>>> the existing implementation. Let's fix all that by just performing the
>>>> right number of assignments in the addr{1,2}_reg fields.
>>>>
>>>> Fixes: 89550beb098e ("mtd: rawnand: qcom: Implement exec_op()")
>>>> Reported-by: kernel test robot <lkp at intel.com>
>>>> Closes: 
>>>> https://lore.kernel.org/oe-kbuild-all/202307131959.PdPSC86K-lkp@intel.com/ 
>>>>
>>>> Closes: 
>>>> https://lore.kernel.org/oe-kbuild-all/202307131730.NOYbcjBr-lkp@intel.com/ 
>>>>
>>>> Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
>>>
>>> LGTM! But I'm relying on Sadre to test it.
>>>
>>> Acked-by: Manivannan Sadhasivam <mani at kernel.org>
>>
>> I don't think I've received feedback from Sadre, I would like to close
>> these issues, can you please test and give us feedback?
>>
> 
>   Sorry for the delay in response. Testing this, will respond back
>   shortly today.

  Tested this series on ipq8074-hk01 board.
  Tested bootcheck, both /dev/mtd and /dev/mtdblock for data integrity.
  All looks fine. So,

  Tested-by: Sricharan Ramabadhran <quic_srichara at quicinc.com>

Regards,
  Sricharan



More information about the linux-arm-kernel mailing list