[U-Boot] [PATCH v4 01/18] cmd: nand: abstract global variable usage for dm conversion

Simon Glass sjg at chromium.org
Sat Jul 8 16:08:54 UTC 2017


Hi,

On 7 July 2017 at 18:32, Grygorii Strashko <grygorii.strashko at ti.com> wrote:
>
>
> On 07/06/2017 10:58 PM, Simon Glass wrote:
>>
>> Hi,
>>
>> On 26 June 2017 at 18:12, Grygorii Strashko <grygorii.strashko at ti.com>
>> wrote:
>>>
>>> From: Mugunthan V N <mugunthanvnm at ti.com>
>>>
>>> nand_info is used all over the file so abstract it with
>>> get_nand_dev_by_index() which will help for DM conversion.
>>>
>>> Signed-off-by: Mugunthan V N <mugunthanvnm at ti.com>
>>> Signed-off-by: Grygorii Strashko <grygorii.strashko at ti.com>
>>> ---
>>>   cmd/nand.c                   | 69
>>> +++++++++++++++++++++++++-------------------
>>>   drivers/mtd/nand/nand.c      | 21 ++++++++++----
>>>   drivers/mtd/nand/omap_gpmc.c |  7 ++---
>>>   include/nand.h               |  9 ++++++
>>>   4 files changed, 65 insertions(+), 41 deletions(-)
>>
>>
>> Is there a cover letter for this series please? I'm not sure what it
>> does overall, or what your approach is.
>>
>
> It should be and you are in cc, here is the link for your reference:
> https://www.mail-archive.com/u-boot@lists.denx.de/msg254409.html

OK, thank you. In a future series do you plan to remove nand_info[] ?

Regards,
Simon


More information about the U-Boot mailing list