<div dir="ltr">Hi Mark,<div><br></div><div>Thanks for your prompt reply.</div><div><br></div><div>I will like to send another patch with support for the spi_mem_op->addr field.</div><div><br></div><div>I am going to vacation until 25/08 I will send the new patch (V3) as soon I will be back.</div><div><br></div><div>Thanks for your support,</div><div><br></div><div>Tomer</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 8 Aug 2019 at 21:55, Mark Brown <<a href="mailto:broonie@kernel.org" target="_blank">broonie@kernel.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Aug 08, 2019 at 06:37:06PM +0300, Tomer Maimon wrote:<br>
<br>
> for example in our driver we modify the access type (singe, dual or quad)<br>
> according the op->addr.buswidth<br>
> for example in the npcm_fiu_set_drd function.<br>
<br>
>         regmap_update_bits(fiu->regmap, NPCM_FIU_DRD_CFG,<br>
>                            NPCM_FIU_DRD_CFG_ACCTYPE,<br>
>                            ilog2(op->addr.buswidth) <<<br>
>                            NPCM_FIU_DRD_ACCTYPE_SHIFT);<br>
<br>
> we also modify it in the UMA R/W functions.<br>
<br>
Ah, it's only for the flash functions - that's fine.<br>
</blockquote></div>