From: ykk@rock-chips•com (Yakir Yang)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v8 1/2] drm/rockchip: hdmi: add Innosilicon HDMI support
Date: Wed, 3 Feb 2016 17:34:40 +0800 [thread overview]
Message-ID: <56B1C9B0.3030806@rock-chips.com> (raw)
In-Reply-To: <20160129082000.b25fd19cea94c4f3ec45e17f@free.fr>
Hi Jean,
Sorry about miss your comments :)
On 01/29/2016 03:20 PM, Jean-Francois Moine wrote:
> On Fri, 29 Jan 2016 14:47:39 +0800
> Yakir Yang <ykk@rock-chips•com> wrote:
>
>> The Innosilicon HDMI is a low power HDMI 1.4 transmitter
>> IP, and it have been integrated on some rockchip CPUs
>> (like RK3036, RK312x).
>>
>> Signed-off-by: Yakir Yang <ykk@rock-chips•com>
>> ---
>> Changes in v8:
>> - Don't check whether encoder output format is RGB colorspace, cause driver
>> default configure the output colorspace to RGB. (ZhengYang)
>> - Correct the check condition in inno_hdmi_config_video_csc() (ZhengYang)
>> - if (data->enc_out_format == data->enc_out_format) {
>> + if (data->enc_in_format == data->enc_out_format) {
>>
>> Changes in v7:
>> - Correct the module licnese statement (Paul)
>> - MODULE_LICENSE("GPL");
>> + MODULE_LICENSE("GPLv2");
>> - Start indentation with tabs and fix the misspell in Kconfig (Paul)
>> - Carry the lost device-binding document (Heiko)
>>
>> Changes in v6:
>> - Rebase the Makefile/Kconfig files which add by Chris's rockchip-mipi driver (Caeser)
>>
>> Changes in v5:
>> - Use hdmi_infoframe helper functions to packed the infoframe (Russell)
>> - Remove the unused double wait_for_completion_timeout for ddc transfer (Russell)
>> - Remove the unused local variable in "inno_hdmi_i2c_write()" function (Russell)
>>
>> Changes in v4:
>> - Modify the commit title "drm/rockchip: hdmi: ..." (Mark)
>> - Correct the "DKMS" to "DPMS" (Mark)
>> - Fix over 80 characters problems (Mark)
>> - Remove encoder .prepare/.commit helper functions, and move the vop mode
>> configure function into encoder .enable helper functions. (Mark)
>>
>> Changes in v3:
>> - Use encoder enable/disable function, and remove the encoder DPMS function
>> - Keep HDMI PLL power on in standby mode
>>
>> Changes in v2:
>> - Using DRM atomic helper functions for connector init (Mark)
>> - Remove "hdmi->connector.encoder = encoder;" (Mark)
>>
>> drivers/gpu/drm/rockchip/Kconfig | 8 +
>> drivers/gpu/drm/rockchip/Makefile | 1 +
>> drivers/gpu/drm/rockchip/inno_hdmi.c | 939 +++++++++++++++++++++++++++++++++++
>> drivers/gpu/drm/rockchip/inno_hdmi.h | 362 ++++++++++++++
>> 4 files changed, 1310 insertions(+)
>> create mode 100644 drivers/gpu/drm/rockchip/inno_hdmi.c
>> create mode 100644 drivers/gpu/drm/rockchip/inno_hdmi.h
>>
>> diff --git a/drivers/gpu/drm/rockchip/Kconfig b/drivers/gpu/drm/rockchip/Kconfig
>> index 8573985..76b3362 100644
>> --- a/drivers/gpu/drm/rockchip/Kconfig
>> +++ b/drivers/gpu/drm/rockchip/Kconfig
>> @@ -35,3 +35,11 @@ config ROCKCHIP_DW_MIPI_DSI
>> for the Synopsys DesignWare HDMI driver. If you want to
>> enable MIPI DSI on RK3288 based SoC, you should selet this
>> option.
>> +
>> +config ROCKCHIP_INNO_HDMI
>> + tristate "Rockchip specific extensions for Innosilicon HDMI"
>> + depends on DRM_ROCKCHIP
>> + help
>> + This selects support for Rockchip SoC specific extensions
>> + for the Innosilicon HDMI driver. If you want to enable
>> + HDMI on RK3036 based SoC, you should select this option.
>> diff --git a/drivers/gpu/drm/rockchip/Makefile b/drivers/gpu/drm/rockchip/Makefile
>> index f6a809a..df8fbef 100644
>> --- a/drivers/gpu/drm/rockchip/Makefile
>> +++ b/drivers/gpu/drm/rockchip/Makefile
>> @@ -8,5 +8,6 @@ rockchipdrm-$(CONFIG_DRM_FBDEV_EMULATION) += rockchip_drm_fbdev.o
>>
>> obj-$(CONFIG_ROCKCHIP_DW_HDMI) += dw_hdmi-rockchip.o
>> obj-$(CONFIG_ROCKCHIP_DW_MIPI_DSI) += dw-mipi-dsi.o
>> +obj-$(CONFIG_ROCKCHIP_INNO_HDMI) += inno_hdmi.o
>>
>> obj-$(CONFIG_DRM_ROCKCHIP) += rockchipdrm.o rockchip_vop_reg.o
>> diff --git a/drivers/gpu/drm/rockchip/inno_hdmi.c b/drivers/gpu/drm/rockchip/inno_hdmi.c
>> new file mode 100644
>> index 0000000..c99d88d
>> --- /dev/null
>> +++ b/drivers/gpu/drm/rockchip/inno_hdmi.c
>> @@ -0,0 +1,939 @@
>> +/*
>> + * Copyright (C) Fuzhou Rockchip Electronics Co.Ltd
>> + * Zheng Yang <zhengyang@rock-chips•com>
>> + * Yakir Yang <ykk@rock-chips•com>
>> + *
>> + * This software is licensed under the terms of the GNU General Public
>> + * License version 2, as published by the Free Software Foundation, and
>> + * may be copied, distributed, and modified under those terms.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + */
>> +
>> +#include <linux/irq.h>
>> +#include <linux/clk.h>
>> +#include <linux/delay.h>
>> +#include <linux/err.h>
>> +#include <linux/hdmi.h>
>> +#include <linux/mfd/syscon.h>
>> +#include <linux/module.h>
>> +#include <linux/mutex.h>
>> +#include <linux/of_device.h>
>> +
>> +#include <drm/drm_of.h>
>> +#include <drm/drmP.h>
>> +#include <drm/drm_atomic_helper.h>
>> +#include <drm/drm_crtc_helper.h>
>> +#include <drm/drm_edid.h>
>> +#include <drm/drm_encoder_slave.h>
> This is not needed.
Do you mean the "drm_encoder_slave.h" file ?
Yep, seems I haven't called ant encoder slave helper functions, thanks.
>
>> +
>> +#include "rockchip_drm_drv.h"
>> +#include "rockchip_drm_vop.h"
>> +
>> +#include "inno_hdmi.h"
> [snip]
>
next prev parent reply other threads:[~2016-02-03 9:34 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-29 6:42 [PATCH v8 0/2] Add Rockchip Inno-HDMI driver Yakir Yang
2016-01-29 6:47 ` [PATCH v8 1/2] drm/rockchip: hdmi: add Innosilicon HDMI support Yakir Yang
2016-01-29 7:20 ` Jean-Francois Moine
2016-02-03 9:34 ` Yakir Yang [this message]
2016-01-29 6:49 ` [PATCH v8 2/2] dt-bindings: add document for Innosilicon HDMI on Rockchip platform Yakir Yang
2016-01-29 7:10 ` [PATCH v8.1 " Yakir Yang
2016-02-02 21:37 ` [PATCH v8 0/2] Add Rockchip Inno-HDMI driver Heiko Stübner
2016-02-03 9:44 ` [PATCH v8.1 1/2] drm/rockchip: hdmi: add Innosilicon HDMI support Yakir Yang
2016-02-17 21:00 ` Heiko Stuebner
2016-02-18 2:54 ` Mark yao
2016-02-16 0:44 ` [PATCH v8 0/2] Add Rockchip Inno-HDMI driver Mark yao
2016-02-16 14:35 ` Heiko Stuebner
2016-02-18 6:12 ` Mark yao
2016-02-18 7:55 ` Yakir Yang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56B1C9B0.3030806@rock-chips.com \
--to=ykk@rock-chips$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox