[PATCH v3 01/15] phy: rockchip-emmc: give DLL some extra time to be ready

Guenter Roeck groeck at google.com
Mon Jun 20 12:36:37 PDT 2016


On Mon, Jun 20, 2016 at 12:30 PM, Doug Anderson <dianders at chromium.org> wrote:
> Hi,
>
> On Mon, Jun 20, 2016 at 12:23 PM, Guenter Roeck <groeck at google.com> wrote:
>> On Mon, Jun 20, 2016 at 10:56 AM, Douglas Anderson
>> <dianders at chromium.org> wrote:
>>> From: Shawn Lin <shawn.lin at rock-chips.com>
>>>
>>> According to the databook, 10.2us is the max time for dll to be ready to
>>> work. However in testing, some chips need 20us for dll to be ready. This
>>
>> Nitpick, but description (20uS) and code (more than 25 uS) don't match.
>
> Good catch.  If I spin the series I'll fix this, otherwise I'll assume
> that the person applying this (Ulf?) can fix the description at their
> discretion.
>

Not that it matters much, as the code is replaced in a later patch
anyway. It might make more sense to merge the two patches into one.

Guenter



More information about the linux-arm-kernel mailing list