Hi Jagan, On Thu, 14 May 2020 at 12:09, Jagan Teki <ja...@amarulasolutions.com> wrote: > > Handling probing code for a particular uclass between > dm vs nodm always confusing and requires additional > ifdefs to handle them properly. > > But, having separate low-level code bases for dm and > nodm can make it easy for the command level to use same > function name to probe the devices. This would indeed > avoid extra ifdef call in source code. > > So, this patch probes the spi flash in common legacy > call spi_flash_probe for both dm and nodm devices and > give a chance to handle on respective code bases based > on the build files. > > Cc: Simon Glass <s...@chromium.org> > Cc: Vignesh R <vigne...@ti.com> > Cc: Daniel Schwierzeck <daniel.schwierz...@gmail.com> > Signed-off-by: Jagan Teki <ja...@amarulasolutions.com> > --- > cmd/sf.c | 22 --------------------- > drivers/mtd/spi/sf-uclass.c | 38 +++++++++++++------------------------ > drivers/net/fm/fm.c | 20 ------------------- > env/sf.c | 17 +---------------- > include/spi_flash.h | 20 +++++-------------- > 5 files changed, 19 insertions(+), 98 deletions(-)
+Tom Rini This is really going the wrong way. You would cement the code in limbo forever and no one would be able to migrate property. Instead, you should add a patch to disable SPI flash on boards which have not migrated. Then we can actually clean up the mess properly. The deadline has passed and people have had more than 5 years to migrate. It is time to make the cut. Regards, Simon