[PATCH v2 5/6] mmc: actions: add MMC driver for Actions OWL S700

[Jaehoon Chung]

Hi Amit,

On 12/23/20 2:59 PM, Amit Tomar wrote:
> Hi Jaehoon
> 
> I had already mentioned about making more readable than now.
> 
>>
>> if (rate <= 1000000) {
>>         rdelay = wdelay = OWL_SD_DELAY_LOW_CLK;
>> } else if ( ...) {
>>         rdelay = wdelay = OWL_SD_DELAY_MID_CLK;
>> } else if (....) {
>>         rdelay = OWL_SD_RDELAY_HIGH;
>>         wdelay = OWL_SD_WDELAY_HIGH;
>> }
>>
>> writel(reg | OWL_SD_CTRL_RDELAY(rdelay) | OWL_SD_CTL_WDELAY(wdelay)...);
>>
>> There are many approach to make readable..but Amit mentioned it's using
>> same code in Linux kernel driver.
>>
>> To be honest, this is *not* the reason but if you see controller also
>> supports DDR50 mode(which we may support in future)
> 
>    where we have different values for read and write delays and we may need
> that many variables to write it cleanly.
> 
>    But if this is not the problem , I will implement the changes as
> suggested by you.

Frankly, i don't have any objection about your patch. :)
Just curious about other driver what using same code with kernel, not only this driver.

I will follow Peng's opinion.


Best Regards,
Jaehoon Chung

> 
>>
>>> Thanks
>>> -Amit
>>>
>>
>>
>