<div dir="ltr"><div dir="ltr">Hi Mark,<div><br></div><div>Thanks for the prompt reply.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 8 Aug 2019 at 16:27, Mark Brown <<a href="mailto:broonie@kernel.org">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 04:14:48PM +0300, Tomer Maimon wrote:<br>
<br>
> +     ctrl->mode_bits = SPI_RX_DUAL | SPI_RX_QUAD<br>
> +             | SPI_TX_DUAL | SPI_TX_QUAD;<br>
> +     ctrl->setup = npcm_fiu_setup;<br>
<br>
I'm not seeing where we implement dual or quad modes in the driver?<br>
There's some <br>
<br></blockquote><div> </div><div>Do you mean you do not see where it is implemented in the NPCM FIU driver?</div><div><br></div><div>for example in our driver we modify the access type (singe, dual or quad) according the <span style="color:rgb(0,0,0);font-family:"Courier New";font-size:11pt">op->addr.buswidth</span></div><div>for example in the <span style="font-weight:bolder;color:rgb(0,0,0);font-family:"Courier New";font-size:11pt">npcm_fiu_set_drd function.</span></div><div><div style="font-family:"Courier New";font-size:11pt;color:rgb(0,0,0)">
<pre>        <span style="font-weight:bolder">regmap_update_bits</span>(fiu->regmap, NPCM_FIU_DRD_CFG,
                           NPCM_FIU_DRD_CFG_ACCTYPE,
                           <span style="font-weight:bolder">ilog2</span>(op->addr.buswidth) <<
                           NPCM_FIU_DRD_ACCTYPE_SHIFT);
</pre></div></div><div>we also modify it in the UMA R/W functions.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +     dev_info(dev, "NPCM %s probe succeed\n", fiu->info->name);<br>
<br>
Just remove this, it makes the log more verbose but doesn't really add<br>
any information.<br></blockquote><div><br></div><div>I will remove it.</div><div><br></div><div>Thanks,</div><div><br></div><div>Tomer </div></div></div>