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

Grygorii Strashko grygorii.strashko at ti.com
Sat Jul 8 18:32:11 UTC 2017



On 07/08/2017 11:08 AM, Simon Glass wrote:
> 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[] ?
> 

Its usage expected to be limited to non-DM case only - 
after full conversation of all platforms  to DM it can be removed.

Ref:
- last: https://patchwork.ozlabs.org/patch/722282/
- old: https://patchwork.ozlabs.org/patch/604739/

-- 
regards,
-grygorii


More information about the U-Boot mailing list