On 11/30/2014 11:58 AM, Anjana Sasindran wrote:
This patch fix a checkpatch.pl error:
ERROR: space prohibited before ')' parenthesis closing
Signed-off-by: Anjana Sasindran anjanasasindran...@gmail.com
Thanks for the patch, but this driver is removed in kernel 3.19.
So there is no point in
On Fri, Nov 28, 2014 at 7:54 PM, Dexuan Cui de...@microsoft.com wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Friday, November 28, 2014 18:13 PM
To: Dexuan Cui
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
driverdev-
You resend this one later.
Send a reply to the originial thread when you do that so we don't
respond to this email.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
Chase,
On Sun, Nov 30, 2014 at 11:05:56PM -0600, Chase Southwood wrote:
There are a handful of calls to printk in ni_stc.h without specified log
levels, as well as one in ni_mio_common.c. This patch converts these
calls to pr_err() instead, so that they are now explicitly log level
ERR.
On Sun, Nov 30, 2014 at 04:28:02PM +0530, Anjana Sasindran wrote:
This patch fix a checkpatch.pl error:
ERROR: space prohibited before ')' parenthesis closing
Signed-off-by: Anjana Sasindran anjanasasindran...@gmail.com
---
drivers/staging/media/omap24xx/v4l2-int-device.h | 2 +-
1 file
Dexuan Cui de...@microsoft.com writes:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, December 1, 2014 16:23 PM
To: Dexuan Cui
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-
de...@linuxdriverproject.org; o...@aepfle.de;
On 01/12/14 08:47, Jeremiah Mahler wrote:
Chase,
On Sun, Nov 30, 2014 at 11:05:56PM -0600, Chase Southwood wrote:
There are a handful of calls to printk in ni_stc.h without specified log
levels, as well as one in ni_mio_common.c. This patch converts these
calls to pr_err() instead, so that
--
Caro usuário
O seu endereço de email ultrapassou 2 GB criadas pelo webmaster, que estão
actualmente em execução em 2,30 GB , o que não é possível enviar ou receber
uma nova mensagem nas próximas 24 horas , por favor, informe seus dados
abaixo para verificar e atualizar a sua conta :
(1) E-
On 01/12/14 05:05, Chase Southwood wrote:
There are a handful of calls to printk in ni_stc.h without specified log
levels, as well as one in ni_mio_common.c. This patch converts these
calls to pr_err() instead, so that they are now explicitly log level
ERR.
Signed-off-by: Chase Southwood
On Mon, Dec 1, 2014 at 5:47 PM, Dexuan Cui de...@microsoft.com wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, December 1, 2014 16:23 PM
To: Dexuan Cui
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
driverdev-
When an SMP Hyper-V guest is running on top of 2012R2 Server and secondary
cpus are sent offline (with echo 0 /sys/devices/system/cpu/cpu$cpu/online)
the system freeze is observed. This happens due to the fact that on newer
hypervisors (Win8, WS2012R2, ...) vmbus channel handlers are distributed
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, December 1, 2014 18:18 PM
To: Dexuan Cui
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-
de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY
Srinivasan;
-Original Message-
From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
Sent: Monday, December 1, 2014 18:53 PM
To: KY Srinivasan; Haiyang Zhang
Cc: de...@linuxdriverproject.org; Greg Kroah-Hartman; linux-
ker...@vger.kernel.org; Dexuan Cui
Subject: [PATCH v2] Drivers: hv: vmbus:
We found Freescale imx6 and Rockchip rk3288 and Ingenic JZ4780 (Xburst/MIPS)
use the interface compatible Designware HDMI IP, but they also have some
lightly differences, such as phy pll configuration, register width(imx hdmi
register is one byte, but rk3288 is 4 bytes width and can only be
CHECK: Alignment should match open parenthesis
+ if ((hdmi-vic == 10) || (hdmi-vic == 11) ||
+ (hdmi-vic == 12) || (hdmi-vic == 13) ||
CHECK: braces {} should be used on all arms of this statement
+ if (hdmi-hdmi_data.video_mode.mdvi)
[...]
+ else {
[...]
drm driver may probe before the i2c bus, so the driver should
defer probing until it is available
Signed-off-by: Andy Yan andy@rock-chips.com
Reviewed-by: Daniel Kurtz djku...@chromium.org
---
Changes in v14: None
Changes in v13: None
Changes in v12:
- refactor of_node_put(ddc_node)
IMX6 and Rockchip RK3288 and JZ4780 (Ingenic Xburst/MIPS)
use the interface compatible Designware HDMI IP, but they
also have some lightly differences, such as phy pll configuration,
register width, 4K support, clk useage, and the crtc mux configuration
is also platform specific.
To reuse the imx
hdmi phy configuration is platform specific, which can be adusted
according to the board to get the best SI
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v14: None
Changes in v13:
- split phy configuration from patch#4
Changes in v12: None
Changes in v11: None
Changes in v10:
the original imx hdmi driver is under drm/imx/,
which depends on imx-drm, so move the imx hdmi
driver out to drm/bridge and rename it to dw_hdmi
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v14: None
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10:
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v14: None
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8:
- correct some spelling mistake
- modify ddc-i2c-bus and interrupt description
Changes in v7: None
On rockchip rk3288, only word(32-bit) accesses are
permitted for hdmi registers. Byte width accesses (writeb,
readb) generate an imprecise external abort.
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v14: None
Changes in v13: None
Changes in v12: None
Changes in v11: None
some platform may not support all the display mode,
add mode_valid interface check it
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v14:
- remove drm_connector_register, because imx-drm core has registered
connector
Changes in v13: None
Changes in v12: None
Changes in v11:
HDMI_IH_I2CMPHY_STAT0 is a clear on write register, which indicates i2cm
operation status(i2c transfer done or error), every hdmi phy register
configuration must check this register to make sure the configuration
has complete. But the indication bit should be cleared after check, otherwise
the
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v14: None
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8:
- Add documentation for rockchip dw hdmi
Changes in v7: None
Changes in v6: None
Changes in v5: None
Rockchip RK3288 hdmi is compatible with dw_hdmi
this patch is depend on patch by Mark Yao Add drm
driver for Rockchip Socs
see https://lkml.org/lkml/2014/11/19/1153
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v14: None
Changes in v13: None
Changes in v12:
- add comment for
Hi Philipp:
On 2014年12月01日 19:42, Philipp Zabel wrote:
Hi Andy,
Am Montag, den 01.12.2014, 19:24 +0800 schrieb Andy Yan:
[...]
diff --git a/include/drm/bridge/dw_hdmi.h b/include/drm/bridge/dw_hdmi.h
new file mode 100644
index 000..1bbf3ca
--- /dev/null
+++ b/include/drm/bridge/dw_hdmi.h
Am Freitag, den 28.11.2014, 17:43 +0800 schrieb Andy Yan:
Hi Zabel:
On 2014年11月27日 00:34, Philipp Zabel wrote:
Am Mittwoch, den 26.11.2014, 21:32 +0800 schrieb Andy Yan:
On rockchip rk3288, only word(32-bit) accesses are
permitted for hdmi registers. Byte width accesses (writeb,
readb)
On Thu, 27 Nov 2014, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
Add helper function to write u32 to registers, if we want to put u32
value to 4 continuous register, this can help us reduce tedious work.
Signed-off-by: Micky Ching
On Fri, 28 Nov 2014, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
Add helper function to write u32 to registers, if we want to put u32
value to 4 continuous register, this can help us reduce tedious work.
Signed-off-by: Micky Ching
This series of patches made corrections to the camel casing and typedef
problems in parser header and parser.c. There is a camel case in parser header
where visorchipset_main.c was affected as well. Smaller problems like space
after casts, braces, and logical coninuations in parser.c were solved
Left the functions uisutil_add_proc_line_ex in both files alone
because they were declared in the files and not pulled from the
proc header file. However the PROC_READ_BUFFER_SIZE was changed
to READ_BUFFER_SIZE in the files.
PROC_READ_BUFFER_SIZE = READ_BUFFER_SIZE
Signed-off-by: Jeffrey Brown
Removed proc code from visorchipset header and main. I also assume
that code involving process is not the same as code involving
proc
Replaced two struct proc_dir_entrys and replaced them with voids
struct proc_dir_entry *proc_dir = void reserved
struct proc_dir_entry *proc_info = void reserved2
Fixed camel cases for parser_init
isLocal = is_local
tryAgain = try_again
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 4 ++--
drivers/staging/unisys/visorchipset/parser.h | 3 ++-
2 files changed, 4 insertions(+), 3 deletions(-)
diff
Changed the camel cases for the struct parser_init_bytestream
parser_init_byteStream = parser_init_bytestream
isLocal = is_local
tryAgain = try_again
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c| 4 ++--
Removed the typedef of PARSER_WHICH_STRING and replaced all
instance of the typedef with enum parser_which_string. Also
changed the name of it to parser_which_string
PARSER_WHICH_STRING = parser_which_string
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
This one was a little weird. I replaced all code implemented from
the procobjecttree.h with debugfs code, then deleted
procobjecttree.h and procobjecttree.c. After that I removed the
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/include/procobjecttree.h|
Changed the camel case of the static ulong Controlvm_Payload_Bytes_
Buffered
Controlvm_Payload_Bytes_Buffered =controlvm_payload_bytes_buffered
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 8
1 file changed, 4 insertions(+), 4
Fixed camel cases in the struct parser_init_guts and an Away camel
case in parser_param_start
isLocal = is_local
tryAgain = try_again
HasStandardPayloadHeader = has_standard_payload_header
Away = cleanups
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
Removed proc code from the file uislib.c, the main changes that
removed the proc utility, was the defines, and the include
statement in the beginning.
CALLHOME_PROC_ENTRY_FN = CALLHOME_DEBUGFS_ENTRY_FN
CALLHOME_THROTTLED_PROC_ENTRY_FN = CALLHOME_THROTTLED_PROC_ENTRY_FN
ProcReadBufferValid =
Completely removed all trace of the typedef of PARSER_CONTEXT
and replaced all instance of that typedef with parser_context_tag
also changed PARSER_CONTEXT_Tag
PARSER_CONTEXT_Tag = parser_context_tag
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
Removed all spaces after casts in parser.c
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/parser.c
Inserted a necessary brace for an if statement on line 146 of
parser.c
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/parser.c
Fixed a logical continuation on line 391 by placing the '||'
operator on line 390
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Am Sonntag, den 09.11.2014, 16:51 +0100 schrieb Guennadi Liakhovetski:
On Sun, 9 Nov 2014, Philipp Zabel wrote:
Hi Guennadi,
On Fri, Nov 07, 2014 at 11:06:21PM +0100, Guennadi Liakhovetski wrote:
Hi Philipp,
Thanks for the patch and sorry for a late reply. I did look at your
On Fri, Nov 28, 2014 at 02:31:56PM +0800, micky_ch...@realsil.com.cn wrote:
#ifdef DEBUG
-static void sd_print_debug_regs(struct realtek_pci_sdmmc *host)
+static void dump_reg_range(struct realtek_pci_sdmmc *host, u16 start, u16
end)
{
- struct rtsx_pcr *pcr = host-pcr;
- u16 i;
On Mon, Dec 01, 2014 at 12:18:58PM +, Lee Jones wrote:
On Fri, 28 Nov 2014, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
Add helper function to write u32 to registers, if we want to put u32
value to 4 continuous register, this can help us reduce
Fixed the camel case for parser_byteStream_get and
parser_simpleString_get
parser_simpleString_get = parser_simplestring_get
parser_byteStream_get = parser_bytestream_get
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 4 ++--
Completely removed all trace of the typedef of PARSER_CONTEXT
and replaced all instance of that typedef with parser_context_tag
also changed PARSER_CONTEXT_Tag
PARSER_CONTEXT_Tag = parser_context_tag
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
Fixed camel cases in the struct parser_init_guts and an Away camel
case in parser_param_start
isLocal = is_local
tryAgain = try_again
HasStandardPayloadHeader = has_standard_payload_header
Away = cleanups
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
Fixed a logical continuation on line 391 by placing the '||'
operator on line 390
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Fixed camel cases for parser_init
isLocal = is_local
tryAgain = try_again
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 4 ++--
drivers/staging/unisys/visorchipset/parser.h | 3 ++-
2 files changed, 4 insertions(+), 3 deletions(-)
diff
Changed the camel cases for the struct parser_init_bytestream
parser_init_byteStream = parser_init_bytestream
isLocal = is_local
tryAgain = try_again
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c| 4 ++--
Removed all spaces after casts in parser.c
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/parser.c
This series of patches made corrections to the camel casing and typedef
problems in parser header and parser.c. There is a camel case in parser header
where visorchipset_main.c was affected as well. Smaller problems like space
after casts, braces, and logical coninuations in parser.c were solved
Removed the typedef of PARSER_WHICH_STRING and replaced all
instance of the typedef with enum parser_which_string. Also
changed the name of it to parser_which_string
PARSER_WHICH_STRING = parser_which_string
Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com
---
On Fri, 28 Nov 2014, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
Add helper function to write u32 to registers, if we want to put u32
value to 4 continuous register, this can help us reduce tedious work.
Signed-off-by: Micky Ching
On 12/01/2014 02:01 PM, anjana s wrote:
Should I do anything on this further?
No. The driver will disappear in 3.19, so there is no point to this patch.
BTW, always check the Kconfig to see if a staging driver is marked deprecated.
If so, then patches are pointless.
Regards,
Hans
On Mon, 2014-12-01 at 07:26 -0500, Jeffrey Brown wrote:
This series of patches made corrections to the camel casing and typedef
problems in parser header and parser.c. There is a camel case in parser header
where visorchipset_main.c was affected as well. Smaller problems like space
after
On Wed, Nov 26, 2014 at 4:06 PM, Lars-Peter Clausen l...@metafoo.de wrote:
On 11/26/2014 10:45 PM, George McCollister wrote:
Output can be held high or low for a specified period of time.
Support for waveform capture could be added in the future.
That's a PWM device?
The device I'm adding
-Original Message-
From: Dexuan Cui
Sent: Monday, December 1, 2014 3:01 AM
To: Jason Wang
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-
de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY
Srinivasan; vkuzn...@redhat.com; Haiyang Zhang
On 12/01/2014 09:53 AM, Anjana Sasindran wrote:
This patch fix the checkpatch.pl warning:
WARNING: please, no spaces at the start of a line
Signed-off-by: Anjana Sasindran anjanasasindran...@gmail.com
---
drivers/staging/rtl8723au/hal/odm_RegConfig8723A.c | 6 +++---
1 file changed, 3
On Mon, 2014-12-01 at 08:57 -0600, George McCollister wrote:
On Wed, Nov 26, 2014 at 4:06 PM, Lars-Peter Clausen l...@metafoo.de wrote:
On 11/26/2014 10:45 PM, George McCollister wrote:
Output can be held high or low for a specified period of time.
Support for waveform capture could be
On Mon, 2014-12-01 at 21:23 +0530, Anjana Sasindran wrote:
This patch fix the checkpatch.pl warning:
WARNING: please, no spaces at the start of a line
[]
diff --git a/drivers/staging/rtl8723au/hal/odm_RegConfig8723A.c
b/drivers/staging/rtl8723au/hal/odm_RegConfig8723A.c
[]
@@ -20,7 +20,7 @@
On 11/28/2014 5:31 PM, Zeng Tao wrote:
when cma is located in highmem, virt_to_page will not
work the right way, use pfn_to_page instead.
Signed-off-by: Zeng Tao prime.z...@huawei.com
---
drivers/staging/android/ion/ion_cma_heap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
From: Markus Elfring elfr...@users.sourceforge.net
Date: Mon, 1 Dec 2014 19:49:39 +0100
The vfree() function performs also input parameter validation. Thus the test
around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
On 12/01/2014 03:57 PM, George McCollister wrote:
On Wed, Nov 26, 2014 at 4:06 PM, Lars-Peter Clausen l...@metafoo.de wrote:
On 11/26/2014 10:45 PM, George McCollister wrote:
Output can be held high or low for a specified period of time.
Support for waveform capture could be added in the
This patch adds proper handling of the vNIC hot removal event, which includes
a rescind-channel-offer message from the host side that triggers vNIC close and
removal. In this case, the notices to the host during close and removal is not
necessary because the channel is rescinded. This patch blocks
From: Jes Sorensen jes.soren...@redhat.com
Hi,
This removes a pointless NULL pointer check in the USB endpoint
parsing code, allowing for some cleaning up. I have made the same
change to the rtl8723au driver, which I will push out with the next
set of fixes.
Cheers,
Jes
Jes Sorensen (1):
From: Jes Sorensen jes.soren...@redhat.com
struct usb_host_interface points to an array of
struct usb_host_endpoints - it makes no sense to do a NULL pointer
check for each pointer.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8188eu/os_dep/usb_intf.c | 33
On 12/01/2014 02:20 PM, jes.soren...@redhat.com wrote:
From: Jes Sorensen jes.soren...@redhat.com
struct usb_host_interface points to an array of
struct usb_host_endpoints - it makes no sense to do a NULL pointer
check for each pointer.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
Can you try to elaborate more on how exactly this interface is supposed to
be used, what kind of the devices it is used for, the use case of the device
and so on. This patch is adding userspace ABI which we have to maintain
forever so we should try to give this proper review, which is not easy
From: Markus Elfring elfr...@users.sourceforge.net
Date: Mon, 1 Dec 2014 23:16:34 +0100
Another update suggestion was taken into account after a patch was applied
from static source code analysis.
Markus Elfring (2):
Deletion of an unnecessary check before the function call release_firmware
From: Markus Elfring elfr...@users.sourceforge.net
Date: Mon, 1 Dec 2014 22:55:29 +0100
The release_firmware() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
From: Markus Elfring elfr...@users.sourceforge.net
Date: Mon, 1 Dec 2014 23:15:20 +0100
The release_firmware() function was called by the mn88473_init() function even
if a previous function call request_firmware failed.
This implementation detail could be improved by the introduction of another
From: Micky Ching micky_ch...@realsil.com.cn
Add helper function to write u32 to registers, if we want to put u32
value to 4 continuous register, this can help us reduce tedious work.
Signed-off-by: Micky Ching micky_ch...@realsil.com.cn
Acked-by: Lee Jones lee.jo...@linaro.org
---
From: Micky Ching micky_ch...@realsil.com.cn
v3:
rtsx_pci_sdmmc.c:
- dump_reg_range
- remove unused pointer check
- fix start index
v2:
rtsx_pci.h:
- remove unused rtsx_pci_write_le32
- add SD_CMD_START
rtsx_pci_sdmmc.c:
- dump_reg_range
- alloc data on
From: Micky Ching micky_ch...@realsil.com.cn
Add support for sdio card by SD interface. The main change is data
transfer mode, When read data, host wait data transfer while command
start. When write data, host will start data transfer after command get
response. The transfer mode modify can be
-Original Message-
From: KY Srinivasan
Sent: Monday, December 1, 2014 23:55 PM
To: Dexuan Cui; Jason Wang
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-
de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com;
vkuzn...@redhat.com; Haiyang Zhang
On Tue, Dec 2, 2014 at 1:33 PM, Dexuan Cui de...@microsoft.com wrote:
-Original Message-
From: KY Srinivasan
Sent: Monday, December 1, 2014 23:55 PM
To: Dexuan Cui; Jason Wang
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
driverdev-
de...@linuxdriverproject.org;
We found Freescale imx6 and Rockchip rk3288 and Ingenic JZ4780 (Xburst/MIPS)
use the interface compatible Designware HDMI IP, but they also have some
lightly differences, such as phy pll configuration, register width(imx hdmi
register is one byte, but rk3288 is 4 bytes width and can only be
CHECK: Alignment should match open parenthesis
+ if ((hdmi-vic == 10) || (hdmi-vic == 11) ||
+ (hdmi-vic == 12) || (hdmi-vic == 13) ||
CHECK: braces {} should be used on all arms of this statement
+ if (hdmi-hdmi_data.video_mode.mdvi)
[...]
+ else {
[...]
drm driver may probe before the i2c bus, so the driver should
defer probing until it is available
Signed-off-by: Andy Yan andy@rock-chips.com
Reviewed-by: Daniel Kurtz djku...@chromium.org
---
Changes in v15: None
Changes in v14: None
Changes in v13: None
Changes in v12:
- refactor
IMX6 and Rockchip RK3288 and JZ4780 (Ingenic Xburst/MIPS)
use the interface compatible Designware HDMI IP, but they
also have some lightly differences, such as phy pll configuration,
register width, 4K support, clk useage, and the crtc mux configuration
is also platform specific.
To reuse the imx
hdmi phy configuration is platform specific, which can be adusted
according to the board to get the best SI
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v15: None
Changes in v14: None
Changes in v13:
- split phy configuration from patch#4
Changes in v12: None
Changes in v11:
the original imx hdmi driver is under drm/imx/,
which depends on imx-drm, so move the imx hdmi
driver out to drm/bridge and rename it to dw_hdmi
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v15:
- add prefix dw_hdmi/DW_HDMI for public used dw_hdmi structs
adviced by Philipp
On rockchip rk3288, only word(32-bit) accesses are
permitted for hdmi registers. Byte width accesses (writeb,
readb) generate an imprecise external abort.
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v15:
- remove unio of the multi-byte register access, adviced by Philipp
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v15: None
Changes in v14: None
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8:
- correct some spelling mistake
- modify ddc-i2c-bus and interrupt description
HDMI_IH_I2CMPHY_STAT0 is a clear on write register, which indicates i2cm
operation status(i2c transfer done or error), every hdmi phy register
configuration must check this register to make sure the configuration
has complete. But the indication bit should be cleared after check, otherwise
the
some platform may not support all the display mode,
add mode_valid interface check it
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v15: None
Changes in v14:
- remove drm_connector_register, because imx-drm core has registered
connector
Changes in v13: None
Changes in v12:
RK3288 HDMI will not work without the spare bit of
HDMI_PHY_CONF0 enable
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v15: None
Changes in v14: None
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8: None
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v15: None
Changes in v14: None
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8:
- Add documentation for rockchip dw hdmi
Changes in v7: None
Changes in v6: None
Rockchip RK3288 hdmi is compatible with dw_hdmi
this patch is depend on patch by Mark Yao Add drm
driver for Rockchip Socs
see https://lkml.org/lkml/2014/11/19/1153
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v15:
- remove THIS_MODULE in platform driver
Changes in v14:
92 matches
Mail list logo