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

Grygorii Strashko
2017-07-08T18:32:11Z

On 07/08/2017 11:08 AM, Simon Glass wrote:
> Hi,
>
> On 7 July 2017 at 18:32, Grygorii Strashko <grygorii.strashko@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@ti.com>
>>> wrote:
>>>>
>>>> From: Mugunthan V N <mugunthanvnm@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@ti.com>
>>>> Signed-off-by: Grygorii Strashko <grygorii.strashko@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 at 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