[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAPDyKFpTkqH4qvLAraWxfxm=LOuWZv7P9nrznJ+o81y+9hDjjA@mail.gmail.com>
Date: Thu, 31 Aug 2017 12:46:12 +0200
From: Ulf Hansson <ulf.hansson@...aro.org>
To: Kevin Hilman <khilman@...libre.com>
Cc: Jerome Brunet <jbrunet@...libre.com>,
Carlo Caione <carlo@...one.org>,
"linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
"open list:ARM/Amlogic Meson..." <linux-amlogic@...ts.infradead.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 00/13] mmc: meson-gx: driver fixups and upgrades
On 30 August 2017 at 21:03, Kevin Hilman <khilman@...libre.com> wrote:
> Hi Ulf,
>
> Ulf Hansson <ulf.hansson@...aro.org> writes:
>
>> On 28 August 2017 at 16:29, Jerome Brunet <jbrunet@...libre.com> wrote:
>>> The patchset features several bugfixes, rework and upgrade for the
>>> meson-gx MMC driver.
>>>
>>> The main goal is to improve readability and enable new high speed
>>> modes, such as eMMC DDR52 and sdcard UHS modes up to SDR50 (100Mhz)
>>>
>>> SDR104 is not working with a few cards on the p200 and the
>>> libretech-cc. I suspect that 200Mhz might be a bit too fast for the PCB
>>> of these boards, adding noise to the signal and eventually breaking
>>> the communication with some cards. The same cards are working well on a
>>> laptop or the nanopi-k2 at 200Mhz.
>>>
>>> This series has been tested on gxbb-p200, gxbb-nanopi-k2 and
>>> gxl-s905x-libretech-cc
>>>
>>> Changes since v2 [1]:
>>> * Drop patches 1 to 3: Applied.
>>> * Drop patch 4: Debug stuff which should not have been sent.
>>> * Added fix to previous patch 3:
>>> If the clock register is not initialized before registering the clk
>>> with CCF, the framework will complain about an illegal divider value.
>>> This had gone unnoticed because it was later fixed by the clock init
>>> rework.
>>>
>>> Ulf, I know it is getting late but it would be nice if patch #1 of
>>> this v3 could go with 3 patches you already applied. The rest can
>>> wait for the following cycle.
>>
>> I decided to pick them all, so applied for next!
>>
>
> Just to double-check, so I can plan for the DT patches
> accordingly... you are queuing this for v4.14, right?
Yes, correct!
Kind regards
Uffe
Powered by blists - more mailing lists