Re: [PATCH v3 12/15] drm/bridge: Add ITE IT6263 LVDS to HDMI converter
From: Liu Ying
Date: Tue Oct 22 2024 - 03:30:29 EST
On 10/21/2024, Biju Das wrote:
> Hi Liu Ying,
Hi Biju,
>
> Thanks for the patch.
Thanks for your review.
[...]
>> +struct it6263 {
>> + struct device *dev;
>> + struct i2c_client *hdmi_i2c;
>> + struct i2c_client *lvds_i2c;
>> + struct regmap *hdmi_regmap;
>> + struct regmap *lvds_regmap;
>> + struct drm_bridge bridge;
>> + struct drm_bridge *next_bridge;
>> + struct gpio_desc *reset_gpio;
> This can be dropped, since it is used only in probe().
Will do in next version.
>
> With that fixed,
>
> Reviewed-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
>
> Cheers,
> Biju
--
Regards,
Liu Ying