On 07/08/2017 11:08 AM, Simon Glass wrote: > Hi, > > On 7 July 2017 at 18:32, Grygorii Strashko <grygorii.stras...@ti.com> wrote: >> >> >> On 07/06/2017 10:58 PM, Simon Glass wrote: >>> >>> Hi, >>> >>> On 26 June 2017 at 18:12, Grygorii Strashko <grygorii.stras...@ti.com> >>> wrote: >>>> >>>> From: Mugunthan V N <mugunthan...@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 <mugunthan...@ti.com> >>>> Signed-off-by: Grygorii Strashko <grygorii.stras...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot