Re: [PATCH v3 2/3] drm: bridge: add it66121 driver
From: Neil Armstrong
Date: Wed Apr 14 2021 - 02:17:28 EST
Hi,
Le 13/04/2021 à 22:56, Paul Cercueil a écrit :
> Hi Neil,
>
> I get build failures locally:
>
> drivers/gpu/drm/bridge/ite-it66121.c: In function ‘it66121_hw_reset’:
> drivers/gpu/drm/bridge/ite-it66121.c:242:2: error: implicit declaration of function ‘gpiod_set_value’ [-Werror=implicit-function-declaration]
> 242 | gpiod_set_value(ctx->gpio_reset, 1);
> | ^~~~~~~~~~~~~~~
> drivers/gpu/drm/bridge/ite-it66121.c: In function ‘it66121_probe’:
> drivers/gpu/drm/bridge/ite-it66121.c:1016:16: error: implicit declaration of function ‘FIELD_GET’; did you mean ‘FOLL_GET’? [-Werror=implicit-function-declaration]
> 1016 | revision_id = FIELD_GET(IT66121_REVISION_MASK, device_ids[1]);
> | ^~~~~~~~~
> | FOLL_GET
>
> Nothing difficult to fix, but the includes should be added nonetheless.
Exact, I got the CI build failures, I'll fix these for v4.
Were you able to test on your setup ?
The v2 always forced DDR mode, with this v3, I also switch to normal 24input mode, but totally untested.
Thanks,
Neil
>
> Cheers,
> -Paul
>
>
> Le lun. 12 avril 2021 à 17:46, Neil Armstrong <narmstrong@xxxxxxxxxxxx> a écrit :
>> From: Phong LE <ple@xxxxxxxxxxxx>
>>
>> This commit is a simple driver for bridge HMDI it66121.
>> The input format is RBG and there is no color conversion.
>> Audio, HDCP and CEC are not supported yet.
>>
>> Signed-off-by: Phong LE <ple@xxxxxxxxxxxx>
>> Signed-off-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx>
>> ---
>> drivers/gpu/drm/bridge/Kconfig | 8 +
>> drivers/gpu/drm/bridge/Makefile | 1 +
>> drivers/gpu/drm/bridge/ite-it66121.c | 1081 ++++++++++++++++++++++++++
>> 3 files changed, 1090 insertions(+)
>> create mode 100644 drivers/gpu/drm/bridge/ite-it66121.c
>>
>> diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig
>> index e4110d6ca7b3..6915c38fa459 100644
>> --- a/drivers/gpu/drm/bridge/Kconfig
>> +++ b/drivers/gpu/drm/bridge/Kconfig
>> @@ -74,6 +74,14 @@ config DRM_LONTIUM_LT9611UXC
>> HDMI signals
>> Please say Y if you have such hardware.
>>
>> +config DRM_ITE_IT66121
>> + tristate "ITE IT66121 HDMI bridge"
>> + depends on OF
>> + select DRM_KMS_HELPER
>> + select REGMAP_I2C
>> + help
>> + Support for ITE IT66121 HDMI bridge.
>> +
>> config DRM_LVDS_CODEC
>> tristate "Transparent LVDS encoders and decoders support"
>> depends on OF
>> diff --git a/drivers/gpu/drm/bridge/Makefile b/drivers/gpu/drm/bridge/Makefile
>> index 86e7acc76f8d..4f725753117c 100644
>> --- a/drivers/gpu/drm/bridge/Makefile
>> +++ b/drivers/gpu/drm/bridge/Makefile
>> @@ -24,6 +24,7 @@ obj-$(CONFIG_DRM_TI_SN65DSI86) += ti-sn65dsi86.o
>> obj-$(CONFIG_DRM_TI_TFP410) += ti-tfp410.o
>> obj-$(CONFIG_DRM_TI_TPD12S015) += ti-tpd12s015.o
>> obj-$(CONFIG_DRM_NWL_MIPI_DSI) += nwl-dsi.o
>> +obj-$(CONFIG_DRM_ITE_IT66121) += ite-it66121.o
>>
>> obj-y += analogix/
>> obj-y += cadence/
>> diff --git a/drivers/gpu/drm/bridge/ite-it66121.c b/drivers/gpu/drm/bridge/ite-it66121.c
>> new file mode 100644
>> index 000000000000..73af49b29dfa
>> --- /dev/null
>> +++ b/drivers/gpu/drm/bridge/ite-it66121.c
>> @@ -0,0 +1,1081 @@
>> +// SPDX-License-Identifier: GPL-2.0-only
>> +/*
>> + * Copyright (C) 2020 BayLibre, SAS
>> + * Author: Phong LE <ple@xxxxxxxxxxxx>
>> + * Copyright (C) 2018-2019, Artem Mygaiev
>> + * Copyright (C) 2017, Fresco Logic, Incorporated.
>> + *
>> + */
>> +
>> +#include <linux/device.h>
>> +#include <linux/i2c.h>
>> +#include <linux/interrupt.h>
>> +#include <linux/module.h>
>> +#include <linux/property.h>
>> +#include <linux/of_graph.h>
>> +#include <linux/pinctrl/consumer.h>
>> +#include <linux/regmap.h>
>> +#include <linux/regulator/consumer.h>
>> +
>> +#include <drm/drm_atomic_helper.h>
>> +#include <drm/drm_bridge.h>
>> +#include <drm/drm_crtc_helper.h>
>> +#include <drm/drm_edid.h>
>> +#include <drm/drm_modes.h>
>> +#include <drm/drm_print.h>
>> +#include <drm/drm_probe_helper.h>
>> +
>> +#define IT66121_VENDOR_ID0_REG 0x00
>> +#define IT66121_VENDOR_ID1_REG 0x01
>> +#define IT66121_DEVICE_ID0_REG 0x02
>> +#define IT66121_DEVICE_ID1_REG 0x03
>> +
>> +#define IT66121_VENDOR_ID0 0x54
>> +#define IT66121_VENDOR_ID1 0x49
>> +#define IT66121_DEVICE_ID0 0x12
>> +#define IT66121_DEVICE_ID1 0x06
>> +#define IT66121_REVISION_MASK GENMASK(7, 4)
>> +#define IT66121_DEVICE_ID1_MASK GENMASK(3, 0)
>> +
>> +#define IT66121_MASTER_SEL_REG 0x10
>> +#define IT66121_MASTER_SEL_HOST BIT(0)
>> +
>> +#define IT66121_AFE_DRV_REG 0x61
>> +#define IT66121_AFE_DRV_RST BIT(4)
>> +#define IT66121_AFE_DRV_PWD BIT(5)
>> +
>> +#define IT66121_INPUT_MODE_REG 0x70
>> +#define IT66121_INPUT_MODE_RGB (0 << 6)
>> +#define IT66121_INPUT_MODE_YUV422 BIT(6)
>> +#define IT66121_INPUT_MODE_YUV444 (2 << 6)
>> +#define IT66121_INPUT_MODE_CCIR656 BIT(4)
>> +#define IT66121_INPUT_MODE_SYNCEMB BIT(3)
>> +#define IT66121_INPUT_MODE_DDR BIT(2)
>> +
>> +#define IT66121_INPUT_CSC_REG 0x72
>> +#define IT66121_INPUT_CSC_ENDITHER BIT(7)
>> +#define IT66121_INPUT_CSC_ENUDFILTER BIT(6)
>> +#define IT66121_INPUT_CSC_DNFREE_GO BIT(5)
>> +#define IT66121_INPUT_CSC_RGB_TO_YUV 0x02
>> +#define IT66121_INPUT_CSC_YUV_TO_RGB 0x03
>> +#define IT66121_INPUT_CSC_NO_CONV 0x00
>> +
>> +#define IT66121_AFE_XP_REG 0x62
>> +#define IT66121_AFE_XP_GAINBIT BIT(7)
>> +#define IT66121_AFE_XP_PWDPLL BIT(6)
>> +#define IT66121_AFE_XP_ENI BIT(5)
>> +#define IT66121_AFE_XP_ENO BIT(4)
>> +#define IT66121_AFE_XP_RESETB BIT(3)
>> +#define IT66121_AFE_XP_PWDI BIT(2)
>> +
>> +#define IT66121_AFE_IP_REG 0x64
>> +#define IT66121_AFE_IP_GAINBIT BIT(7)
>> +#define IT66121_AFE_IP_PWDPLL BIT(6)
>> +#define IT66121_AFE_IP_CKSEL_05 (0 << 4)
>> +#define IT66121_AFE_IP_CKSEL_1 BIT(4)
>> +#define IT66121_AFE_IP_CKSEL_2 (2 << 4)
>> +#define IT66121_AFE_IP_CKSEL_2OR4 (3 << 4)
>> +#define IT66121_AFE_IP_ER0 BIT(3)
>> +#define IT66121_AFE_IP_RESETB BIT(2)
>> +#define IT66121_AFE_IP_ENC BIT(1)
>> +#define IT66121_AFE_IP_EC1 BIT(0)
>> +
>> +#define IT66121_AFE_XP_EC1_REG 0x68
>> +#define IT66121_AFE_XP_EC1_LOWCLK BIT(4)
>> +
>> +#define IT66121_SW_RST_REG 0x04
>> +#define IT66121_SW_RST_REF BIT(5)
>> +#define IT66121_SW_RST_AREF BIT(4)
>> +#define IT66121_SW_RST_VID BIT(3)
>> +#define IT66121_SW_RST_AUD BIT(2)
>> +#define IT66121_SW_RST_HDCP BIT(0)
>> +
>> +#define IT66121_DDC_COMMAND_REG 0x15
>> +#define IT66121_DDC_COMMAND_BURST_READ 0x0
>> +#define IT66121_DDC_COMMAND_EDID_READ 0x3
>> +#define IT66121_DDC_COMMAND_FIFO_CLR 0x9
>> +#define IT66121_DDC_COMMAND_SCL_PULSE 0xA
>> +#define IT66121_DDC_COMMAND_ABORT 0xF
>> +
>> +#define IT66121_HDCP_REG 0x20
>> +#define IT66121_HDCP_CPDESIRED BIT(0)
>> +#define IT66121_HDCP_EN1P1FEAT BIT(1)
>> +
>> +#define IT66121_INT_STATUS1_REG 0x06
>> +#define IT66121_INT_STATUS1_AUD_OVF BIT(7)
>> +#define IT66121_INT_STATUS1_DDC_NOACK BIT(5)
>> +#define IT66121_INT_STATUS1_DDC_FIFOERR BIT(4)
>> +#define IT66121_INT_STATUS1_DDC_BUSHANG BIT(2)
>> +#define IT66121_INT_STATUS1_RX_SENS_STATUS BIT(1)
>> +#define IT66121_INT_STATUS1_HPD_STATUS BIT(0)
>> +
>> +#define IT66121_DDC_HEADER_REG 0x11
>> +#define IT66121_DDC_HEADER_HDCP 0x74
>> +#define IT66121_DDC_HEADER_EDID 0xA0
>> +
>> +#define IT66121_DDC_OFFSET_REG 0x12
>> +#define IT66121_DDC_BYTE_REG 0x13
>> +#define IT66121_DDC_SEGMENT_REG 0x14
>> +#define IT66121_DDC_RD_FIFO_REG 0x17
>> +
>> +#define IT66121_CLK_BANK_REG 0x0F
>> +#define IT66121_CLK_BANK_PWROFF_RCLK BIT(6)
>> +#define IT66121_CLK_BANK_PWROFF_ACLK BIT(5)
>> +#define IT66121_CLK_BANK_PWROFF_TXCLK BIT(4)
>> +#define IT66121_CLK_BANK_PWROFF_CRCLK BIT(3)
>> +#define IT66121_CLK_BANK_0 0
>> +#define IT66121_CLK_BANK_1 1
>> +
>> +#define IT66121_INT_REG 0x05
>> +#define IT66121_INT_ACTIVE_HIGH BIT(7)
>> +#define IT66121_INT_OPEN_DRAIN BIT(6)
>> +#define IT66121_INT_TX_CLK_OFF BIT(0)
>> +
>> +#define IT66121_INT_MASK1_REG 0x09
>> +#define IT66121_INT_MASK1_AUD_OVF BIT(7)
>> +#define IT66121_INT_MASK1_DDC_NOACK BIT(5)
>> +#define IT66121_INT_MASK1_DDC_FIFOERR BIT(4)
>> +#define IT66121_INT_MASK1_DDC_BUSHANG BIT(2)
>> +#define IT66121_INT_MASK1_RX_SENS BIT(1)
>> +#define IT66121_INT_MASK1_HPD BIT(0)
>> +
>> +#define IT66121_INT_CLR1_REG 0x0C
>> +#define IT66121_INT_CLR1_PKTACP BIT(7)
>> +#define IT66121_INT_CLR1_PKTNULL BIT(6)
>> +#define IT66121_INT_CLR1_PKTGEN BIT(5)
>> +#define IT66121_INT_CLR1_KSVLISTCHK BIT(4)
>> +#define IT66121_INT_CLR1_AUTHDONE BIT(3)
>> +#define IT66121_INT_CLR1_AUTHFAIL BIT(2)
>> +#define IT66121_INT_CLR1_RX_SENS BIT(1)
>> +#define IT66121_INT_CLR1_HPD BIT(0)
>> +
>> +#define IT66121_AV_MUTE_REG 0xC1
>> +#define IT66121_AV_MUTE_ON BIT(0)
>> +#define IT66121_AV_MUTE_BLUESCR BIT(1)
>> +
>> +#define IT66121_PKT_GEN_CTRL_REG 0xC6
>> +#define IT66121_PKT_GEN_CTRL_ON BIT(0)
>> +#define IT66121_PKT_GEN_CTRL_RPT BIT(1)
>> +
>> +#define IT66121_AVIINFO_DB1_REG 0x158
>> +#define IT66121_AVIINFO_DB2_REG 0x159
>> +#define IT66121_AVIINFO_DB3_REG 0x15A
>> +#define IT66121_AVIINFO_DB4_REG 0x15B
>> +#define IT66121_AVIINFO_DB5_REG 0x15C
>> +#define IT66121_AVIINFO_CSUM_REG 0x15D
>> +#define IT66121_AVIINFO_DB6_REG 0x15E
>> +#define IT66121_AVIINFO_DB7_REG 0x15F
>> +#define IT66121_AVIINFO_DB8_REG 0x160
>> +#define IT66121_AVIINFO_DB9_REG 0x161
>> +#define IT66121_AVIINFO_DB10_REG 0x162
>> +#define IT66121_AVIINFO_DB11_REG 0x163
>> +#define IT66121_AVIINFO_DB12_REG 0x164
>> +#define IT66121_AVIINFO_DB13_REG 0x165
>> +
>> +#define IT66121_AVI_INFO_PKT_REG 0xCD
>> +#define IT66121_AVI_INFO_PKT_ON BIT(0)
>> +#define IT66121_AVI_INFO_PKT_RPT BIT(1)
>> +
>> +#define IT66121_HDMI_MODE_REG 0xC0
>> +#define IT66121_HDMI_MODE_HDMI BIT(0)
>> +
>> +#define IT66121_SYS_STATUS_REG 0x0E
>> +#define IT66121_SYS_STATUS_ACTIVE_IRQ BIT(7)
>> +#define IT66121_SYS_STATUS_HPDETECT BIT(6)
>> +#define IT66121_SYS_STATUS_SENDECTECT BIT(5)
>> +#define IT66121_SYS_STATUS_VID_STABLE BIT(4)
>> +#define IT66121_SYS_STATUS_AUD_CTS_CLR BIT(1)
>> +#define IT66121_SYS_STATUS_CLEAR_IRQ BIT(0)
>> +
>> +#define IT66121_DDC_STATUS_REG 0x16
>> +#define IT66121_DDC_STATUS_TX_DONE BIT(7)
>> +#define IT66121_DDC_STATUS_ACTIVE BIT(6)
>> +#define IT66121_DDC_STATUS_NOACK BIT(5)
>> +#define IT66121_DDC_STATUS_WAIT_BUS BIT(4)
>> +#define IT66121_DDC_STATUS_ARBI_LOSE BIT(3)
>> +#define IT66121_DDC_STATUS_FIFO_FULL BIT(2)
>> +#define IT66121_DDC_STATUS_FIFO_EMPTY BIT(1)
>> +#define IT66121_DDC_STATUS_FIFO_VALID BIT(0)
>> +
>> +#define IT66121_EDID_SLEEP_US 20000
>> +#define IT66121_EDID_TIMEOUT_US 200000
>> +#define IT66121_EDID_FIFO_SIZE 32
>> +#define IT66121_AFE_CLK_HIGH 80000 /* Khz */
>> +
>> +struct it66121_ctx {
>> + struct regmap *regmap;
>> + struct drm_bridge bridge;
>> + struct drm_connector connector;
>> + struct device *dev;
>> + struct gpio_desc *gpio_reset;
>> + struct i2c_client *client;
>> + struct regulator_bulk_data supplies[3];
>> + u32 bus_width;
>> + struct mutex lock; /* Protects fields below and device registers */
>> + struct edid *edid;
>> + struct hdmi_avi_infoframe hdmi_avi_infoframe;
>> +};
>> +
>> +static const struct regmap_range_cfg it66121_regmap_banks[] = {
>> + {
>> + .name = "it66121",
>> + .range_min = 0x00,
>> + .range_max = 0x1FF,
>> + .selector_reg = IT66121_CLK_BANK_REG,
>> + .selector_mask = 0x1,
>> + .selector_shift = 0,
>> + .window_start = 0x00,
>> + .window_len = 0x130,
>> + },
>> +};
>> +
>> +static const struct regmap_config it66121_regmap_config = {
>> + .val_bits = 8,
>> + .reg_bits = 8,
>> + .max_register = 0x1FF,
>> + .ranges = it66121_regmap_banks,
>> + .num_ranges = ARRAY_SIZE(it66121_regmap_banks),
>> +};
>> +
>> +static void it66121_hw_reset(struct it66121_ctx *ctx)
>> +{
>> + gpiod_set_value(ctx->gpio_reset, 1);
>> + msleep(20);
>> + gpiod_set_value(ctx->gpio_reset, 0);
>> +}
>> +
>> +static int ite66121_power_on(struct it66121_ctx *ctx)
>> +{
>> + return regulator_bulk_enable(ARRAY_SIZE(ctx->supplies), ctx->supplies);
>> +}
>> +
>> +static int ite66121_power_off(struct it66121_ctx *ctx)
>> +{
>> + return regulator_bulk_disable(ARRAY_SIZE(ctx->supplies), ctx->supplies);
>> +}
>> +
>> +static int it66121_preamble_ddc(struct it66121_ctx *ctx)
>> +{
>> + return regmap_write(ctx->regmap, IT66121_MASTER_SEL_REG,
>> + IT66121_MASTER_SEL_HOST);
>> +}
>> +
>> +static int it66121_fire_afe(struct it66121_ctx *ctx)
>> +{
>> + return regmap_write(ctx->regmap, IT66121_AFE_DRV_REG, 0);
>> +}
>> +
>> +/* TOFIX: Handle YCbCr Input & Output */
>> +static int it66121_configure_input(struct it66121_ctx *ctx)
>> +{
>> + int ret;
>> + u8 mode = IT66121_INPUT_MODE_RGB;
>> +
>> + if (ctx->bus_width == 12)
>> + mode |= IT66121_INPUT_MODE_DDR;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_INPUT_MODE_REG, mode);
>> + if (ret)
>> + return ret;
>> +
>> + return regmap_write(ctx->regmap, IT66121_INPUT_CSC_REG, IT66121_INPUT_CSC_NO_CONV);
>> +}
>> +
>> +/**
>> + * it66121_configure_afe() - Configure the analog front end
>> + * @ctx: it66121_ctx object
>> + * @mode: mode to configure
>> + *
>> + * RETURNS:
>> + * zero if success, a negative error code otherwise.
>> + */
>> +static int it66121_configure_afe(struct it66121_ctx *ctx,
>> + const struct drm_display_mode *mode)
>> +{
>> + int ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_AFE_DRV_REG,
>> + IT66121_AFE_DRV_RST);
>> + if (ret)
>> + return ret;
>> +
>> + if (mode->clock > IT66121_AFE_CLK_HIGH) {
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_XP_REG,
>> + IT66121_AFE_XP_GAINBIT |
>> + IT66121_AFE_XP_ENO,
>> + IT66121_AFE_XP_GAINBIT);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_IP_REG,
>> + IT66121_AFE_IP_GAINBIT |
>> + IT66121_AFE_IP_ER0 |
>> + IT66121_AFE_IP_EC1,
>> + IT66121_AFE_IP_GAINBIT);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_XP_EC1_REG,
>> + IT66121_AFE_XP_EC1_LOWCLK, 0x80);
>> + if (ret)
>> + return ret;
>> + } else {
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_XP_REG,
>> + IT66121_AFE_XP_GAINBIT |
>> + IT66121_AFE_XP_ENO,
>> + IT66121_AFE_XP_ENO);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_IP_REG,
>> + IT66121_AFE_IP_GAINBIT |
>> + IT66121_AFE_IP_ER0 |
>> + IT66121_AFE_IP_EC1, IT66121_AFE_IP_ER0 |
>> + IT66121_AFE_IP_EC1);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_XP_EC1_REG,
>> + IT66121_AFE_XP_EC1_LOWCLK,
>> + IT66121_AFE_XP_EC1_LOWCLK);
>> + if (ret)
>> + return ret;
>> + }
>> +
>> + /* Clear reset flags */
>> + ret = regmap_write_bits(ctx->regmap, IT66121_SW_RST_REG,
>> + IT66121_SW_RST_REF | IT66121_SW_RST_VID,
>> + ~(IT66121_SW_RST_REF | IT66121_SW_RST_VID) &
>> + 0xFF);
>> + if (ret)
>> + return ret;
>> +
>> + return it66121_fire_afe(ctx);
>> +}
>> +
>> +static inline int it66121_wait_ddc_ready(struct it66121_ctx *ctx)
>> +{
>> + int ret, val;
>> + u32 busy = IT66121_DDC_STATUS_NOACK | IT66121_DDC_STATUS_WAIT_BUS |
>> + IT66121_DDC_STATUS_ARBI_LOSE;
>> +
>> + ret = regmap_read_poll_timeout(ctx->regmap, IT66121_DDC_STATUS_REG, val, true,
>> + IT66121_EDID_SLEEP_US, IT66121_EDID_TIMEOUT_US);
>> + if (ret)
>> + return ret;
>> +
>> + if (val & busy)
>> + return -EAGAIN;
>> +
>> + return 0;
>> +}
>> +
>> +static int it66121_clear_ddc_fifo(struct it66121_ctx *ctx)
>> +{
>> + int ret;
>> +
>> + ret = it66121_preamble_ddc(ctx);
>> + if (ret)
>> + return ret;
>> +
>> + return regmap_write(ctx->regmap, IT66121_DDC_COMMAND_REG,
>> + IT66121_DDC_COMMAND_FIFO_CLR);
>> +}
>> +
>> +static int it66121_abort_ddc_ops(struct it66121_ctx *ctx)
>> +{
>> + int ret;
>> + unsigned int swreset, cpdesire;
>> +
>> + ret = regmap_read(ctx->regmap, IT66121_SW_RST_REG, &swreset);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_read(ctx->regmap, IT66121_HDCP_REG, &cpdesire);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_HDCP_REG,
>> + cpdesire & (~IT66121_HDCP_CPDESIRED & 0xFF));
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_SW_RST_REG,
>> + (swreset | IT66121_SW_RST_HDCP));
>> + if (ret)
>> + return ret;
>> +
>> + ret = it66121_preamble_ddc(ctx);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_DDC_COMMAND_REG,
>> + IT66121_DDC_COMMAND_ABORT);
>> + if (ret)
>> + return ret;
>> +
>> + return it66121_wait_ddc_ready(ctx);
>> +}
>> +
>> +static int it66121_get_edid_block(void *context, u8 *buf,
>> + unsigned int block, size_t len)
>> +{
>> + struct it66121_ctx *ctx = context;
>> + unsigned int val;
>> + int remain = len;
>> + int offset = 0;
>> + int ret, cnt;
>> +
>> + offset = (block % 2) * len;
>> + block = block / 2;
>> +
>> + ret = regmap_read(ctx->regmap, IT66121_INT_STATUS1_REG, &val);
>> + if (ret)
>> + return ret;
>> +
>> + if (val & IT66121_INT_STATUS1_DDC_BUSHANG) {
>> + ret = it66121_abort_ddc_ops(ctx);
>> + if (ret)
>> + return ret;
>> + }
>> +
>> + ret = it66121_clear_ddc_fifo(ctx);
>> + if (ret)
>> + return ret;
>> +
>> + while (remain > 0) {
>> + cnt = (remain > IT66121_EDID_FIFO_SIZE) ?
>> + IT66121_EDID_FIFO_SIZE : remain;
>> + ret = it66121_preamble_ddc(ctx);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_DDC_COMMAND_REG,
>> + IT66121_DDC_COMMAND_FIFO_CLR);
>> + if (ret)
>> + return ret;
>> +
>> + ret = it66121_wait_ddc_ready(ctx);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_read(ctx->regmap, IT66121_INT_STATUS1_REG, &val);
>> + if (ret)
>> + return ret;
>> +
>> + if (val & IT66121_INT_STATUS1_DDC_BUSHANG) {
>> + ret = it66121_abort_ddc_ops(ctx);
>> + if (ret)
>> + return ret;
>> + }
>> +
>> + ret = it66121_preamble_ddc(ctx);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_DDC_HEADER_REG,
>> + IT66121_DDC_HEADER_EDID);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_DDC_OFFSET_REG, offset);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_DDC_BYTE_REG, cnt);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_DDC_SEGMENT_REG, block);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write(ctx->regmap, IT66121_DDC_COMMAND_REG,
>> + IT66121_DDC_COMMAND_EDID_READ);
>> + if (ret)
>> + return ret;
>> +
>> + offset += cnt;
>> + remain -= cnt;
>> +
>> + /* Per programming manual, sleep here before emptying the FIFO */
>> + msleep(20);
>> +
>> + ret = it66121_wait_ddc_ready(ctx);
>> + if (ret)
>> + return ret;
>> +
>> + do {
>> + ret = regmap_read(ctx->regmap, IT66121_DDC_RD_FIFO_REG, &val);
>> + if (ret)
>> + return ret;
>> + *(buf++) = val;
>> + cnt--;
>> + } while (cnt > 0);
>> + }
>> +
>> + return 0;
>> +}
>> +
>> +static int it66121_connector_get_modes(struct drm_connector *connector)
>> +{
>> + int ret, num_modes = 0;
>> + struct it66121_ctx *ctx = container_of(connector, struct it66121_ctx,
>> + connector);
>> +
>> + if (ctx->edid)
>> + return drm_add_edid_modes(connector, ctx->edid);
>> +
>> + mutex_lock(&ctx->lock);
>> +
>> + ctx->edid = drm_do_get_edid(connector, it66121_get_edid_block, ctx);
>> + if (!ctx->edid) {
>> + DRM_ERROR("Failed to read EDID\n");
>> + goto unlock;
>> + }
>> +
>> + ret = drm_connector_update_edid_property(connector, ctx->edid);
>> + if (ret) {
>> + DRM_ERROR("Failed to update EDID property: %d\n", ret);
>> + goto unlock;
>> + }
>> +
>> + num_modes = drm_add_edid_modes(connector, ctx->edid);
>> +
>> +unlock:
>> + mutex_unlock(&ctx->lock);
>> +
>> + return num_modes;
>> +}
>> +
>> +static bool it66121_is_hpd_detect(struct it66121_ctx *ctx)
>> +{
>> + int val;
>> +
>> + if (regmap_read(ctx->regmap, IT66121_SYS_STATUS_REG, &val))
>> + return false;
>> +
>> + return val & IT66121_SYS_STATUS_HPDETECT;
>> +}
>> +
>> +static int it66121_connector_detect_ctx(struct drm_connector *connector,
>> + struct drm_modeset_acquire_ctx *c,
>> + bool force)
>> +{
>> + struct it66121_ctx *ctx = container_of(connector, struct it66121_ctx,
>> + connector);
>> +
>> + return it66121_is_hpd_detect(ctx) ? connector_status_connected
>> + : connector_status_disconnected;
>> +}
>> +
>> +static enum drm_mode_status it66121_mode_valid(struct it66121_ctx *ctx,
>> + const struct drm_display_mode *mode)
>> +{
>> + unsigned long max_clock;
>> +
>> + max_clock = (ctx->bus_width == 12) ? 74250 : 148500;
>> +
>> + if (mode->clock > max_clock)
>> + return MODE_CLOCK_HIGH;
>> +
>> + if (mode->clock < 25000)
>> + return MODE_CLOCK_LOW;
>> +
>> + return MODE_OK;
>> +}
>> +
>> +static enum drm_mode_status it66121_connector_mode_valid(struct drm_connector *connector,
>> + struct drm_display_mode *mode)
>> +{
>> + struct it66121_ctx *ctx = container_of(connector, struct it66121_ctx,
>> + connector);
>> +
>> + return it66121_mode_valid(ctx, mode);
>> +}
>> +
>> +static struct drm_connector_helper_funcs it66121_connector_helper_funcs = {
>> + .get_modes = it66121_connector_get_modes,
>> + .detect_ctx = it66121_connector_detect_ctx,
>> + .mode_valid = it66121_connector_mode_valid,
>> +};
>> +
>> +static const struct drm_connector_funcs it66121_connector_funcs = {
>> + .reset = drm_atomic_helper_connector_reset,
>> + .fill_modes = drm_helper_probe_single_connector_modes,
>> + .destroy = drm_connector_cleanup,
>> + .atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
>> + .atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
>> +};
>> +
>> +static int it66121_bridge_attach(struct drm_bridge *bridge,
>> + enum drm_bridge_attach_flags flags)
>> +{
>> + int ret;
>> + struct it66121_ctx *ctx = container_of(bridge, struct it66121_ctx,
>> + bridge);
>> +
>> + if (!bridge->encoder) {
>> + DRM_ERROR("Parent encoder object not found");
>> + return -ENODEV;
>> + }
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_CLK_BANK_REG,
>> + IT66121_CLK_BANK_PWROFF_RCLK, 0);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_INT_REG,
>> + IT66121_INT_TX_CLK_OFF, 0);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_DRV_REG,
>> + IT66121_AFE_DRV_PWD, 0);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_XP_REG,
>> + IT66121_AFE_XP_PWDI | IT66121_AFE_XP_PWDPLL, 0);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_IP_REG,
>> + IT66121_AFE_IP_PWDPLL, 0);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_DRV_REG,
>> + IT66121_AFE_DRV_RST, 0);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_XP_REG,
>> + IT66121_AFE_XP_RESETB, IT66121_AFE_XP_RESETB);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AFE_IP_REG,
>> + IT66121_AFE_IP_RESETB, IT66121_AFE_IP_RESETB);
>> + if (ret)
>> + return ret;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_SW_RST_REG,
>> + IT66121_SW_RST_REF,
>> + IT66121_SW_RST_REF);
>> + if (ret)
>> + return ret;
>> +
>> + /* Per programming manual, sleep here for bridge to settle */
>> + msleep(50);
>> +
>> + /* Start interrupts */
>> + ret = regmap_write_bits(ctx->regmap, IT66121_INT_MASK1_REG,
>> + IT66121_INT_MASK1_DDC_NOACK |
>> + IT66121_INT_MASK1_HPD |
>> + IT66121_INT_MASK1_DDC_FIFOERR |
>> + IT66121_INT_MASK1_DDC_BUSHANG,
>> + ~(IT66121_INT_MASK1_DDC_NOACK |
>> + IT66121_INT_MASK1_HPD |
>> + IT66121_INT_MASK1_DDC_FIFOERR |
>> + IT66121_INT_MASK1_DDC_BUSHANG) & 0xFF);
>> + if (ret)
>> + return ret;
>> +
>> + if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)
>> + return 0;
>> +
>> + ret = drm_connector_init(bridge->dev, &ctx->connector,
>> + &it66121_connector_funcs,
>> + DRM_MODE_CONNECTOR_HDMIA);
>> + if (ret)
>> + return ret;
>> +
>> + ctx->connector.polled = DRM_CONNECTOR_POLL_HPD;
>> + drm_connector_helper_add(&ctx->connector,
>> + &it66121_connector_helper_funcs);
>> +
>> + ret = drm_connector_attach_encoder(&ctx->connector, bridge->encoder);
>> + if (ret)
>> + return ret;
>> +
>> + return drm_connector_register(&ctx->connector);
>> +}
>> +
>> +static int it66121_set_mute(struct it66121_ctx *ctx, bool mute)
>> +{
>> + int ret;
>> + unsigned int val = 0;
>> +
>> + if (mute)
>> + val = IT66121_AV_MUTE_ON;
>> +
>> + ret = regmap_write_bits(ctx->regmap, IT66121_AV_MUTE_REG, IT66121_AV_MUTE_ON, val);
>> + if (ret)
>> + return ret;
>> +
>> + return regmap_write(ctx->regmap, IT66121_PKT_GEN_CTRL_REG,
>> + IT66121_PKT_GEN_CTRL_ON | IT66121_PKT_GEN_CTRL_RPT);
>> +}
>> +
>> +#define MAX_OUTPUT_SEL_FORMATS 1
>> +
>> +static u32 *it66121_bridge_atomic_get_output_bus_fmts(struct drm_bridge *bridge,
>> + struct drm_bridge_state *bridge_state,
>> + struct drm_crtc_state *crtc_state,
>> + struct drm_connector_state *conn_state,
>> + unsigned int *num_output_fmts)
>> +{
>> + u32 *output_fmts;
>> +
>> + output_fmts = kcalloc(MAX_OUTPUT_SEL_FORMATS, sizeof(*output_fmts),
>> + GFP_KERNEL);
>> + if (!output_fmts)
>> + return NULL;
>> +
>> + /* TOFIX handle more than MEDIA_BUS_FMT_RGB888_1X24 as output format */
>> + output_fmts[0] = MEDIA_BUS_FMT_RGB888_1X24;
>> + *num_output_fmts = 1;
>> +
>> + return output_fmts;
>> +}
>> +
>> +#define MAX_INPUT_SEL_FORMATS 1
>> +
>> +static u32 *it66121_bridge_atomic_get_input_bus_fmts(struct drm_bridge *bridge,
>> + struct drm_bridge_state *bridge_state,
>> + struct drm_crtc_state *crtc_state,
>> + struct drm_connector_state *conn_state,
>> + u32 output_fmt,
>> + unsigned int *num_input_fmts)
>> +{
>> + struct it66121_ctx *ctx = container_of(bridge, struct it66121_ctx, bridge);
>> + u32 *input_fmts;
>> +
>> + *num_input_fmts = 0;
>> +
>> + input_fmts = kcalloc(MAX_INPUT_SEL_FORMATS, sizeof(*input_fmts),
>> + GFP_KERNEL);
>> + if (!input_fmts)
>> + return NULL;
>> +
>> + if (ctx->bus_width == 12)
>> + /* IT66121FN Datasheet specifies Little-Endian ordering */
>> + input_fmts[0] = MEDIA_BUS_FMT_RGB888_2X12_LE;
>> + else
>> + /* TOFIX support more input bus formats in 24bit width */
>> + input_fmts[0] = MEDIA_BUS_FMT_RGB888_1X24;
>> + *num_input_fmts = 1;
>> +
>> + return input_fmts;
>> +}
>> +
>> +static void it66121_bridge_enable(struct drm_bridge *bridge)
>> +{
>> + struct it66121_ctx *ctx = container_of(bridge, struct it66121_ctx, bridge);
>> +
>> + it66121_set_mute(ctx, false);
>> +}
>> +
>> +static void it66121_bridge_disable(struct drm_bridge *bridge)
>> +{
>> + struct it66121_ctx *ctx = container_of(bridge, struct it66121_ctx, bridge);
>> +
>> + it66121_set_mute(ctx, true);
>> +}
>> +
>> +static
>> +void it66121_bridge_mode_set(struct drm_bridge *bridge,
>> + const struct drm_display_mode *mode,
>> + const struct drm_display_mode *adjusted_mode)
>> +{
>> + int ret, i;
>> + u8 buf[HDMI_INFOFRAME_SIZE(AVI)];
>> + struct it66121_ctx *ctx = container_of(bridge, struct it66121_ctx, bridge);
>> + const u16 aviinfo_reg[HDMI_AVI_INFOFRAME_SIZE] = {
>> + IT66121_AVIINFO_DB1_REG,
>> + IT66121_AVIINFO_DB2_REG,
>> + IT66121_AVIINFO_DB3_REG,
>> + IT66121_AVIINFO_DB4_REG,
>> + IT66121_AVIINFO_DB5_REG,
>> + IT66121_AVIINFO_DB6_REG,
>> + IT66121_AVIINFO_DB7_REG,
>> + IT66121_AVIINFO_DB8_REG,
>> + IT66121_AVIINFO_DB9_REG,
>> + IT66121_AVIINFO_DB10_REG,
>> + IT66121_AVIINFO_DB11_REG,
>> + IT66121_AVIINFO_DB12_REG,
>> + IT66121_AVIINFO_DB13_REG
>> + };
>> +
>> + mutex_lock(&ctx->lock);
>> +
>> + hdmi_avi_infoframe_init(&ctx->hdmi_avi_infoframe);
>> +
>> + ret = drm_hdmi_avi_infoframe_from_display_mode(&ctx->hdmi_avi_infoframe, &ctx->connector,
>> + adjusted_mode);
>> + if (ret) {
>> + DRM_ERROR("Failed to setup AVI infoframe: %d\n", ret);
>> + goto unlock;
>> + }
>> +
>> + ret = hdmi_avi_infoframe_pack(&ctx->hdmi_avi_infoframe, buf, sizeof(buf));
>> + if (ret < 0) {
>> + DRM_ERROR("Failed to pack infoframe: %d\n", ret);
>> + goto unlock;
>> + }
>> +
>> + /* Write new AVI infoframe packet */
>> + for (i = 0; i < HDMI_AVI_INFOFRAME_SIZE; i++) {
>> + if (regmap_write(ctx->regmap, aviinfo_reg[i], buf[i + HDMI_INFOFRAME_HEADER_SIZE]))
>> + goto unlock;
>> + }
>> + if (regmap_write(ctx->regmap, IT66121_AVIINFO_CSUM_REG, buf[3]))
>> + goto unlock;
>> +
>> + /* Enable AVI infoframe */
>> + if (regmap_write(ctx->regmap, IT66121_AVI_INFO_PKT_REG,
>> + IT66121_AVI_INFO_PKT_ON | IT66121_AVI_INFO_PKT_RPT))
>> + goto unlock;
>> +
>> + /* Set TX mode to HDMI */
>> + if (regmap_write(ctx->regmap, IT66121_HDMI_MODE_REG, IT66121_HDMI_MODE_HDMI))
>> + goto unlock;
>> +
>> + if (regmap_write_bits(ctx->regmap, IT66121_CLK_BANK_REG,
>> + IT66121_CLK_BANK_PWROFF_TXCLK, IT66121_CLK_BANK_PWROFF_TXCLK))
>> + goto unlock;
>> +
>> + if (it66121_configure_input(ctx))
>> + goto unlock;
>> +
>> + if (it66121_configure_afe(ctx, adjusted_mode))
>> + goto unlock;
>> +
>> + regmap_write_bits(ctx->regmap, IT66121_CLK_BANK_REG, IT66121_CLK_BANK_PWROFF_TXCLK, 0);
>> +
>> +unlock:
>> + mutex_unlock(&ctx->lock);
>> +}
>> +
>> +static enum drm_mode_status it66121_bridge_mode_valid(struct drm_bridge *bridge,
>> + const struct drm_display_info *info,
>> + const struct drm_display_mode *mode)
>> +{
>> + struct it66121_ctx *ctx = container_of(bridge, struct it66121_ctx, bridge);
>> +
>> + return it66121_mode_valid(ctx, mode);
>> +}
>> +
>> +static enum drm_connector_status it66121_bridge_detect(struct drm_bridge *bridge)
>> +{
>> + struct it66121_ctx *ctx = container_of(bridge, struct it66121_ctx, bridge);
>> +
>> + return it66121_is_hpd_detect(ctx) ? connector_status_connected
>> + : connector_status_disconnected;
>> +}
>> +
>> +static struct edid *it66121_bridge_get_edid(struct drm_bridge *bridge,
>> + struct drm_connector *connector)
>> +{
>> + struct it66121_ctx *ctx = container_of(bridge, struct it66121_ctx, bridge);
>> + struct edid *edid;
>> +
>> + mutex_lock(&ctx->lock);
>> + edid = drm_do_get_edid(connector, it66121_get_edid_block, ctx);
>> + mutex_unlock(&ctx->lock);
>> +
>> + return edid;
>> +}
>> +
>> +static const struct drm_bridge_funcs it66121_bridge_funcs = {
>> + .attach = it66121_bridge_attach,
>> + .enable = it66121_bridge_enable,
>> + .disable = it66121_bridge_disable,
>> + .mode_set = it66121_bridge_mode_set,
>> + .mode_valid = it66121_bridge_mode_valid,
>> + .detect = it66121_bridge_detect,
>> + .get_edid = it66121_bridge_get_edid,
>> + .atomic_get_output_bus_fmts = it66121_bridge_atomic_get_output_bus_fmts,
>> + .atomic_get_input_bus_fmts = it66121_bridge_atomic_get_input_bus_fmts,
>> +};
>> +
>> +static irqreturn_t it66121_irq_threaded_handler(int irq, void *dev_id)
>> +{
>> + int ret;
>> + unsigned int val;
>> + struct it66121_ctx *ctx = dev_id;
>> + struct device *dev = ctx->dev;
>> + bool event = false;
>> +
>> + mutex_lock(&ctx->lock);
>> +
>> + ret = regmap_read(ctx->regmap, IT66121_SYS_STATUS_REG, &val);
>> + if (ret)
>> + goto unlock;
>> +
>> + if (!(val & IT66121_SYS_STATUS_ACTIVE_IRQ))
>> + goto unlock;
>> +
>> + ret = regmap_read(ctx->regmap, IT66121_INT_STATUS1_REG, &val);
>> + if (ret) {
>> + dev_err(dev, "Cannot read STATUS1_REG %d\n", ret);
>> + } else {
>> + if (val & IT66121_INT_STATUS1_DDC_FIFOERR)
>> + it66121_clear_ddc_fifo(ctx);
>> + if (val & (IT66121_INT_STATUS1_DDC_BUSHANG |
>> + IT66121_INT_STATUS1_DDC_NOACK))
>> + it66121_abort_ddc_ops(ctx);
>> + if (val & IT66121_INT_STATUS1_HPD_STATUS) {
>> + regmap_write_bits(ctx->regmap, IT66121_INT_CLR1_REG,
>> + IT66121_INT_CLR1_HPD, IT66121_INT_CLR1_HPD);
>> +
>> + if (!it66121_is_hpd_detect(ctx)) {
>> + kfree(ctx->edid);
>> + ctx->edid = NULL;
>> + }
>> +
>> + event = true;
>> + }
>> + }
>> +
>> + regmap_write_bits(ctx->regmap, IT66121_SYS_STATUS_REG,
>> + IT66121_SYS_STATUS_CLEAR_IRQ,
>> + IT66121_SYS_STATUS_CLEAR_IRQ);
>> +
>> +unlock:
>> + mutex_unlock(&ctx->lock);
>> +
>> + if (event)
>> + drm_helper_hpd_irq_event(ctx->bridge.dev);
>> +
>> + return IRQ_HANDLED;
>> +}
>> +
>> +static int it66121_probe(struct i2c_client *client,
>> + const struct i2c_device_id *id)
>> +{
>> + u32 vendor_ids[2], device_ids[2], revision_id;
>> + struct device_node *ep;
>> + int ret;
>> + struct it66121_ctx *ctx;
>> + struct device *dev = &client->dev;
>> +
>> + if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) {
>> + dev_err(dev, "I2C check functionality failed.\n");
>> + return -ENXIO;
>> + }
>> +
>> + ep = of_graph_get_endpoint_by_regs(dev->of_node, 0, 0);
>> + if (!ep)
>> + return -EINVAL;
>> +
>> + ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
>> + if (!ctx)
>> + return -ENOMEM;
>> +
>> + ctx->dev = dev;
>> + ctx->client = client;
>> +
>> + of_property_read_u32(ep, "bus-width", &ctx->bus_width);
>> + of_node_put(ep);
>> +
>> + if (ctx->bus_width != 12 && ctx->bus_width != 24)
>> + return -EINVAL;
>> +
>> + i2c_set_clientdata(client, ctx);
>> + mutex_init(&ctx->lock);
>> +
>> + ctx->supplies[0].supply = "vcn33";
>> + ctx->supplies[1].supply = "vcn18";
>> + ctx->supplies[2].supply = "vrf12";
>> + ret = devm_regulator_bulk_get(ctx->dev, 3, ctx->supplies);
>> + if (ret) {
>> + dev_err(ctx->dev, "regulator_bulk failed\n");
>> + return ret;
>> + }
>> +
>> + ret = ite66121_power_on(ctx);
>> + if (ret)
>> + return ret;
>> +
>> + it66121_hw_reset(ctx);
>> +
>> + ctx->regmap = devm_regmap_init_i2c(client, &it66121_regmap_config);
>> + if (IS_ERR(ctx->regmap)) {
>> + ite66121_power_off(ctx);
>> + return PTR_ERR(ctx);
>> + }
>> +
>> + regmap_read(ctx->regmap, IT66121_VENDOR_ID0_REG, &vendor_ids[0]);
>> + regmap_read(ctx->regmap, IT66121_VENDOR_ID1_REG, &vendor_ids[1]);
>> + regmap_read(ctx->regmap, IT66121_DEVICE_ID0_REG, &device_ids[0]);
>> + regmap_read(ctx->regmap, IT66121_DEVICE_ID1_REG, &device_ids[1]);
>> +
>> + /* Revision is shared with DEVICE_ID1 */
>> + revision_id = FIELD_GET(IT66121_REVISION_MASK, device_ids[1]);
>> + device_ids[1] &= IT66121_DEVICE_ID1_MASK;
>> +
>> + if (vendor_ids[0] != IT66121_VENDOR_ID0 || vendor_ids[1] != IT66121_VENDOR_ID1 ||
>> + device_ids[0] != IT66121_DEVICE_ID0 || device_ids[1] != IT66121_DEVICE_ID1) {
>> + ite66121_power_off(ctx);
>> + return -ENODEV;
>> + }
>> +
>> + ctx->bridge.funcs = &it66121_bridge_funcs;
>> + ctx->bridge.of_node = dev->of_node;
>> +
>> + ret = devm_request_threaded_irq(dev, client->irq, NULL, it66121_irq_threaded_handler,
>> + IRQF_SHARED | IRQF_ONESHOT, dev_name(dev), ctx);
>> + if (ret < 0) {
>> + dev_err(dev, "Failed to request irq %d:%d\n", client->irq, ret);
>> + ite66121_power_off(ctx);
>> + return ret;
>> + }
>> +
>> + drm_bridge_add(&ctx->bridge);
>> +
>> + dev_info(ctx->dev, "IT66121 revision %d probed\n", revision_id);
>> +
>> + return 0;
>> +}
>> +
>> +static int it66121_remove(struct i2c_client *client)
>> +{
>> + struct it66121_ctx *ctx = i2c_get_clientdata(client);
>> +
>> + ite66121_power_off(ctx);
>> + drm_bridge_remove(&ctx->bridge);
>> + kfree(ctx->edid);
>> + mutex_destroy(&ctx->lock);
>> +
>> + return 0;
>> +}
>> +
>> +static const struct of_device_id it66121_dt_match[] = {
>> + { .compatible = "ite,it66121" },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(of, it66121_dt_match);
>> +
>> +static const struct i2c_device_id it66121_id[] = {
>> + { "it66121", 0 },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(i2c, it66121_id);
>> +
>> +static struct i2c_driver it66121_driver = {
>> + .driver = {
>> + .name = "it66121",
>> + .of_match_table = it66121_dt_match,
>> + },
>> + .probe = it66121_probe,
>> + .remove = it66121_remove,
>> + .id_table = it66121_id,
>> +};
>> +
>> +module_i2c_driver(it66121_driver);
>> +
>> +MODULE_AUTHOR("Phong LE");
>> +MODULE_DESCRIPTION("IT66121 HDMI transmitter driver");
>> +MODULE_LICENSE("GPL v2");
>> --
>> 2.25.1
>>
>
>