[U-Boot] sf: About sandbox flash driver

[U-Boot] sf: About sandbox flash driver

[Jagan Teki]

Hi Simon,

Few question wrt sandbox flash driver - drivers/mtd/spi/sandbox.c

1) Does this driver used for creating flash emulation environment? or
something else?
2) Can't it use spi_flash.c core for all spi flash operations, how
different is this?

Please let me know your inputs.

thanks!
-- 
Jagan.

[U-Boot] sf: About sandbox flash driver

[Simon Glass]

Hi Jagan,

On 30 December 2015 at 10:03, Jagan Teki <jteki@openedev.com> wrote:
> Hi Simon,
>
> Few question wrt sandbox flash driver - drivers/mtd/spi/sandbox.c
>
> 1) Does this driver used for creating flash emulation environment? or
> something else?

It is an emulation driver, meant to emulate a SPI flash device. See
sandbox_sf_emul_ops - the sandbox_sf_xfer() receives a SPI message
from U-Boot and produces a reply.

> 2) Can't it use spi_flash.c core for all spi flash operations, how
> different is this?

That is for talking to SPI flash devices I think, rather than emulating one.

>
> Please let me know your inputs.
>
> thanks!
> --
> Jagan.

Regards,
Simon

[U-Boot] sf: About sandbox flash driver

[Jagan Teki]

Hi Simon,

On 8 January 2016 at 01:19, Simon Glass <sjg@chromium.org> wrote:
> Hi Jagan,
>
> On 30 December 2015 at 10:03, Jagan Teki <jteki@openedev.com> wrote:
>> Hi Simon,
>>
>> Few question wrt sandbox flash driver - drivers/mtd/spi/sandbox.c
>>
>> 1) Does this driver used for creating flash emulation environment? or
>> something else?
>
> It is an emulation driver, meant to emulate a SPI flash device. See
> sandbox_sf_emul_ops - the sandbox_sf_xfer() receives a SPI message
> from U-Boot and produces a reply.
>
>> 2) Can't it use spi_flash.c core for all spi flash operations, how
>> different is this?
>
> That is for talking to SPI flash devices I think, rather than emulating one.

Yes, that I understood. is it possible to modify this to use spi_flash
core? Because we need to maintain the spi_flash_params separately for
this, see my last series for reference[1] I have moved flash_prams
inside spi_flash core and old ones to sandbox

[1] http://comments.gmane.org/gmane.comp.boot-loaders.u-boot/246568

thanks!
-- 
Jagan.

[U-Boot] sf: About sandbox flash driver

[Simon Glass]

Hi Jagan,


On 8 January 2016 at 03:47, Jagan Teki <jteki@openedev.com> wrote:
> Hi Simon,
>
> On 8 January 2016 at 01:19, Simon Glass <sjg@chromium.org> wrote:
>> Hi Jagan,
>>
>> On 30 December 2015 at 10:03, Jagan Teki <jteki@openedev.com> wrote:
>>> Hi Simon,
>>>
>>> Few question wrt sandbox flash driver - drivers/mtd/spi/sandbox.c
>>>
>>> 1) Does this driver used for creating flash emulation environment? or
>>> something else?
>>
>> It is an emulation driver, meant to emulate a SPI flash device. See
>> sandbox_sf_emul_ops - the sandbox_sf_xfer() receives a SPI message
>> from U-Boot and produces a reply.
>>
>>> 2) Can't it use spi_flash.c core for all spi flash operations, how
>>> different is this?
>>
>> That is for talking to SPI flash devices I think, rather than emulating one.
>
> Yes, that I understood. is it possible to modify this to use spi_flash
> core? Because we need to maintain the spi_flash_params separately for
> this, see my last series for reference[1] I have moved flash_prams
> inside spi_flash core and old ones to sandbox

I'm sure it could be changed. At present it uses
spi_flash_params_table but it could be changed to use a different
table.

>
> [1] http://comments.gmane.org/gmane.comp.boot-loaders.u-boot/246568
>
> thanks!
> --
> Jagan.

Regards,
Simon