RE: [PATCH v2 0/8] ADV7511 driver enhancements

From: Biju Das <biju.das.jz@bp.renesas.com>
To: Andrzej Hajda <andrzej.hajda@intel.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Robert Foss <rfoss@kernel.org>, David Airlie <airlied@gmail.com>,
	Daniel Vetter <daniel@ffwll.ch>
Cc: "Laurent Pinchart" <Laurent.pinchart@ideasonboard.com>,
	"Jonas Karlman" <jonas@kwiboo.se>,
	"Jernej Skrabec" <jernej.skrabec@gmail.com>,
	"Abhinav Kumar" <quic_abhinavk@quicinc.com>,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
	"Andy Shevchenko" <andy.shevchenko@gmail.com>,
	"Javier Martinez Canillas" <javierm@redhat.com>,
	"Ahmad Fatoum" <a.fatoum@pengutronix.de>,
	"Rob Herring" <robh@kernel.org>,
	"Bogdan Togorean" <bogdan.togorean@analog.com>,
	"Adam Ford" <aford173@gmail.com>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"Geert Uytterhoeven" <geert+renesas@glider.be>,
	"linux-renesas-soc@vger.kernel.org"
	<linux-renesas-soc@vger.kernel.org>
Subject: RE: [PATCH v2 0/8] ADV7511 driver enhancements
Date: Mon, 2 Oct 2023 07:14:43 +0000	[thread overview]
Message-ID: <OS0PR01MB5922B6B282401FDB42135F1086C5A@OS0PR01MB5922.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <20230830142358.275459-1-biju.das.jz@bp.renesas.com>

Hi All,

Gentle ping. Are we happy with this patch serries?

Cheers,
Biju

> -----Original Message-----
> From: Biju Das <biju.das.jz@bp.renesas.com>
> Sent: Wednesday, August 30, 2023 3:24 PM
> Subject: [PATCH v2 0/8] ADV7511 driver enhancements
> 
> This patch series aims to improve ADV7511 driver by adding feature bits and
> data instead of comparing enum adv7511_type for various hardware
> differences between ADV7511, ADV7533 and ADV7535.
> 
> This patch series tested with[1] on RZ/G2L SMARC EVK which embeds ADV7535.
> 
> [1]
> 
> v1->v2:
>  * Added Rb tag from Adam and Laurent.
>  * Added tested by tag from Adam and Fabio.
>  * Updated commit description with reason *why* the change is needed.
>  * Dropped the local info variable and instead started using
>    adv7511->info->type in probe().
>  * Replaced max_mode_clock->max_mode_clock_khz in struct adv7511_chip_info
>  * Replaced variable type for max_mode_clock_khz from
>    unsigned int->unsigned long.
>  * Replaced max_lane_freq->max_lane_freq_khz in struct adv7511_chip_info.
>  * Replaced max_lane_freq_khz variable type from
>    unsigned long->unsigned int.
>  * Added trailing commas for num_supplies in adv753{3,5}_chip_info.
>  * Added patch#5 for adding the reg_cec_offset variable to struct
>    adv7511_chip_info.
>  * Replaced has_dsi variable type from unsigned->bool.
>  * Restored check using type for low_refresh_rate and
>    regmap_register_patch().
>  * Replaced link_config variable type from unsigned->bool.
>  * Restored enum adv7511_type as there are users.
>  * Replaced hpd_override_enable variable type from unsigned->bool.
> 
> Biju Das (8):
>   drm: adv7511: Add struct adv7511_chip_info and use
>     i2c_get_match_data()
>   drm: adv7511: Add max_mode_clock_khz variable to struct
>     adv7511_chip_info
>   drm: adv7511: Add max_lane_freq_khz variable to struct
>     adv7511_chip_info
>   drm: adv7511: Add supply_names and num_supplies variables to struct
>     adv7511_chip_info
>   drm: adv7511: Add reg_cec_offset variable to struct adv7511_chip_info
>   drm: adv7511: Add has_dsi variable to struct adv7511_chip_info
>   drm: adv7511: Add link_config variable to struct adv7511_chip_info
>   drm: adv7511: Add hpd_override_enable variable to struct
>     adv7511_chip_info
> 
>  drivers/gpu/drm/bridge/adv7511/adv7511.h     |  16 ++-
>  drivers/gpu/drm/bridge/adv7511/adv7511_cec.c |  14 +--
> drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 104 +++++++++++--------
>  drivers/gpu/drm/bridge/adv7511/adv7533.c     |   7 +-
>  4 files changed, 81 insertions(+), 60 deletions(-)
> 
> --
> 2.25.1