Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH v5 00/15] tc358767 driver improvements
To:     Andrey Smirnov <andrew.smirnov@gmail.com>,
        dri-devel@lists.freedesktop.org
Cc:     Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
        Tomi Valkeinen <tomi.valkeinen@ti.com>,
        Andrey Gusakov <andrey.gusakov@cogentembedded.com>,
        Philipp Zabel <p.zabel@pengutronix.de>,
        Cory Tusar <cory.tusar@zii.aero>,
        Chris Healy <cphealy@gmail.com>,
        Lucas Stach <l.stach@pengutronix.de>,
        linux-kernel@vger.kernel.org
From:   Andrzej Hajda <a.hajda@samsung.com>
Message-ID: <2017e112-2b58-8ba4-1176-9e9e6b212478@samsung.com>
Date:   Wed, 12 Jun 2019 12:31:57 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
        Thunderbird/60.7.0
MIME-Version: 1.0
In-Reply-To: <20190612083252.15321-1-andrew.smirnov@gmail.com>
Content-Transfer-Encoding: 7bit
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 201P
References: <CGME20190612083307epcas1p457198b83aff94f588163cbef724ca0da@epcas1p4.samsung.com>
        <20190612083252.15321-1-andrew.smirnov@gmail.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 12.06.2019 10:32, Andrey Smirnov wrote:
> Everyone:
>
> This series contains various improvements (at least in my mind) and
> fixes that I made to tc358767 while working with the code of the
> driver. Hopefuly each patch is self explanatory.
>
> Feedback is welcome!
>
> Thanks,
> Andrey Smirnov


Tomi, you can queue it to drm-misc-next.


Regards

Andrzej


>
> Changes since [v4]:
>
>     - tc_pllupdate_pllen() renamed to tc_pllupdate()
>
>     - Collected Reviewed-bys from Andrzej for the rest of the series
>
> Changes since [v3]:
>
>     - Collected Reviewed-bys from Andrzej
>     
>     - Dropped explicit check for -ETIMEDOUT in "drm/bridge: tc358767:
>       Simplify polling in tc_main_link_setup()" for consistency
>
>     - AUX transfer code converted to user regmap_raw_read(),
>       regmap_raw_write()
>
> Changes since [v2]:
>
>     - Patchset rebased on top of v4 of Tomi's series that recently
>       went in (https://patchwork.freedesktop.org/series/58176/#rev5)
>       
>     - AUX transfer code converted to user regmap_bulk_read(),
>       regmap_bulk_write()
>
> Changes since [v1]:
>
>     - Patchset rebased on top of
>       https://patchwork.freedesktop.org/series/58176/
>       
>     - Patches to remove both tc_write() and tc_read() helpers added
>
>     - Patches to rework AUX transfer code added
>
>     - Both "drm/bridge: tc358767: Simplify polling in
>       tc_main_link_setup()" and "drm/bridge: tc358767: Simplify
>       polling in tc_link_training()" changed to use tc_poll_timeout()
>       instead of regmap_read_poll_timeout()
>
> [v4] lkml.kernel.org/r/20190607044550.13361-1-andrew.smirnov@gmail.com
> [v3] lkml.kernel.org/r/20190605070507.11417-1-andrew.smirnov@gmail.com
> [v2] lkml.kernel.org/r/20190322032901.12045-1-andrew.smirnov@gmail.com
> [v1] lkml.kernel.org/r/20190226193609.9862-1-andrew.smirnov@gmail.com
>
> Andrey Smirnov (15):
>   drm/bridge: tc358767: Simplify tc_poll_timeout()
>   drm/bridge: tc358767: Simplify polling in tc_main_link_setup()
>   drm/bridge: tc358767: Simplify polling in tc_link_training()
>   drm/bridge: tc358767: Simplify tc_set_video_mode()
>   drm/bridge: tc358767: Drop custom tc_write()/tc_read() accessors
>   drm/bridge: tc358767: Simplify AUX data read
>   drm/bridge: tc358767: Simplify AUX data write
>   drm/bridge: tc358767: Increase AUX transfer length limit
>   drm/bridge: tc358767: Use reported AUX transfer size
>   drm/bridge: tc358767: Add support for address-only I2C transfers
>   drm/bridge: tc358767: Introduce tc_set_syspllparam()
>   drm/bridge: tc358767: Introduce tc_pllupdate()
>   drm/bridge: tc358767: Simplify tc_aux_wait_busy()
>   drm/bridge: tc358767: Drop unnecessary 8 byte buffer
>   drm/bridge: tc358767: Replace magic number in tc_main_link_enable()
>
>  drivers/gpu/drm/bridge/tc358767.c | 648 +++++++++++++++++-------------
>  1 file changed, 372 insertions(+), 276 deletions(-)
>