linux-kernel - Re: [PATCH v5 00/15] tc358767 driver improvements

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 12 Jun 2019 12:31:57 +0200
From:   Andrzej Hajda <a.hajda@...sung.com>
To:     Andrey Smirnov <andrew.smirnov@...il.com>,
        dri-devel@...ts.freedesktop.org
Cc:     Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
        Tomi Valkeinen <tomi.valkeinen@...com>,
        Andrey Gusakov <andrey.gusakov@...entembedded.com>,
        Philipp Zabel <p.zabel@...gutronix.de>,
        Cory Tusar <cory.tusar@....aero>,
        Chris Healy <cphealy@...il.com>,
        Lucas Stach <l.stach@...gutronix.de>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5 00/15] tc358767 driver improvements

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@...il.com
> [v3] lkml.kernel.org/r/20190605070507.11417-1-andrew.smirnov@...il.com
> [v2] lkml.kernel.org/r/20190322032901.12045-1-andrew.smirnov@...il.com
> [v1] lkml.kernel.org/r/20190226193609.9862-1-andrew.smirnov@...il.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(-)
>