Hi Russell
I'm working on Designware hdmi-audio, also add it as a standard ALSA device.
Before I saw this email, I also planed to submit my patchs to upsteam.
I'm very grateful if you can email those patchs to us.
Best Regards.
于 2014年11月04日 22:29, Russell King - ARM Linux 写道:
On Tue, Nov 04,
Replace references to the v4l2_mbus_pixelcode enum with the new
media_bus_format enum in all common headers.
Signed-off-by: Boris Brezillon
Acked-by: Sakari Ailus
---
include/media/v4l2-mediabus.h| 2 +-
include/media/v4l2-subdev.h | 2 +-
include/uapi/linux/v4l2-subdev.h | 6 +++---
3
Hello,
This patch series prepares the use of media bus formats outside of
the V4L2 subsytem (my final goal is to use it in the Atmel HLCDC DRM
driver where I have to configure my DPI/RGB bus according to the
connected display).
The series first defines a new enum with a neutral name (media_bus_fo
Rename mediabus formats and move the enum into a separate header file so
that it can be used by DRM/KMS subsystem without any reference to the V4L2
subsystem.
Old v4l2_mbus_pixelcode now points to media_bus_format.
Signed-off-by: Boris Brezillon
Acked-by: Guennadi Liakhovetski
---
include/uapi
In order to have subsytem agnostic media bus format definitions we've
moved media bus definition to include/uapi/linux/media-bus-format.h and
prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT.
Update the v4l documentation accordingly.
Signed-off-by: Boris Brezillon
---
Documentat
In order to have subsytem agnostic media bus format definitions we've
moved media bus definitions to include/uapi/linux/media-bus-format.h and
prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT.
Replace all references to the old definitions in i2c drivers.
Signed-off-by: Boris Brezi
In order to have subsytem agnostic media bus format definitions we've
moved media bus definition to include/uapi/linux/media-bus-format.h and
prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT.
Reference new definitions in all usb drivers.
Signed-off-by: Boris Brezillon
---
driver
In order to have subsytem agnostic media bus format definitions we've
moved media bus definition to include/uapi/linux/media-bus-format.h and
prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT.
Reference new definitions in the ipu-v3 driver.
Signed-off-by: Boris Brezillon
---
driv
In order to have subsytem agnostic media bus format definitions we've
moved media bus definition to include/uapi/linux/media-bus-format.h and
prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT.
Replace all references to the old definition in pci drivers.
Signed-off-by: Boris Brezill
Place v4l2_mbus_pixelcode in a #ifndef __KERNEL__ section so that kernel
users don't have access to these definitions.
We have to keep this definition for user-space users even though they're
encouraged to move to the new media_bus_format enum.
Signed-off-by: Boris Brezillon
---
include/uapi/li
In order to have subsytem agnostic media bus format definitions we've
moved media bus definition to include/uapi/linux/media-bus-format.h and
prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT.
Reference new definitions in all media drivers residing in staging.
Signed-off-by: Boris
In order to have subsytem agnostic media bus format definitions we've
moved media bus definition to include/uapi/linux/media-bus-format.h and
prefixed enum values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT.
Reference new definitions in all platform drivers.
Signed-off-by: Boris Brezillon
---
D
On Thu, Nov 06, 2014 at 05:35:40PM +0800, Kuankuan.Yang wrote:
> I'm working on Designware hdmi-audio, also add it as a standard ALSA device.
> Before I saw this email, I also planed to submit my patchs to upsteam.
> I'm very grateful if you can email those patchs to us.
I've attached the set of p
tty's flags can be set by calling tty_alloc_driver().
Signed-off-by: Daeseok Youn
---
drivers/staging/dgap/dgap.c | 16
1 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c
index aa393d2..6418cc5 100644
--- a/
The tty_struct of serial_driver and print_driver were getting
allocated twice. One is allocated in tty_alloc_driver(), the other
is in dgap_tty_register(). So remove these in dgap_tty_register().
Signed-off-by: Daeseok Youn
---
drivers/staging/dgap/dgap.c | 16
1 files changed
The board_t has a tty_struct(serial_driver and print_driver)
that has a major number. When major number is compared in
dgap_tty_open(), just use brd->serial_driver{print_driver}->major.
Signed-off-by: Daeseok Youn
---
drivers/staging/dgap/dgap.c |9 ++---
drivers/staging/dgap/dgap.h |
use find_board_by_major function instead of getting a brd from static
arrary. Becasue tty's major number doesn't start zero and we can
find a brd from dgap_board[].
Signed-off-by: Daeseok Youn
---
drivers/staging/dgap/dgap.c | 28 +---
1 files changed, 17 insertions(+),
On 05/11/14 17:31, H Hartley Sweeten wrote:
Introduce a helper function to determine the number of scans left in
the async command.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/comedidev.h | 2 ++
drivers/staging/comedi/drivers.c | 39
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 difference, such as phy pll configuration, register width(imx hdmi
register is one byte, but rk3288 is 4 bytes width and can only access by w
imx6 and rockchip rk3288 and JZ4780 (Ingenic Xburst/MIPS)
use the interface compatible Designware HDMI IP, but they
also have some lightly difference, such as phy pll configuration,
register width, 4K support, clk useage, and the crtc mux configuration
is also platform specific.
To reuse the imx h
the original imx hdmi driver is under staging/imx-drm,
which depends on imx-drm, so move the imx hdmi drvier out
to drm/bridge and rename imx-hdmi to dw-hdmi
Signed-off-by: Andy Yan
---
drivers/gpu/drm/bridge/Kconfig | 5 +
drivers/gpu/drm/bridge/Makefile
On rockchip rk3288, only word(32-bit) accesses are
permitted for hdmi registers. Byte width access (writeb,
readb) generate an imprecise external abort.
Signed-off-by: Andy Yan
---
drivers/gpu/drm/bridge/dw_hdmi.c | 53
1 file changed, 48 insertions(+),
From: ykk
dw-hdmi is under drm/bridge, so it should be the bridge mode.
hange off the encoder to dw_hdmi-imx.c, keep the connector &
birdge in dw_hdmi.c
Signed-off-by: ykk
Signed-off-by: Andy Yan
---
drivers/gpu/drm/bridge/dw_hdmi.c | 228 +++---
drivers/stagi
Hello Vaishali Thakkar,
The patch 2b365fa9d926: "Staging: rtl8188eu: Use put_unaligned_le32"
from Oct 31, 2014, leads to the following static checker warning:
include/linux/unaligned/access_ok.h:44 put_unaligned_le32()
warn: potential memory corrupting cast 4 vs 3 bytes
drivers/s
On Wed, Oct 29, 2014 at 08:03:51PM +0400, Andrey Utkin wrote:
> This fixes warning from drivers/media/v4l2-core/videobuf2-core.c:2144
>
On my kernel that line is:
q->start_streaming_called = 0;
This changelog is useless.
regards,
dan carpenter
_
From: Ken Depro
This patch removes unneeded spaces after casts within the virthba.c
file. The checkpatch script was run after these changes, and no
"remove spaces after casts" checks were generated.
Later patches will fix the other checkpatch warnings/checks.
Conflicts:
drivers/stagin
Please specify the "error" in the subject in some way.
On Thu, Nov 06, 2014 at 11:46:13AM +0530, Sanjeev Sharma wrote:
> This is a patch to the rtw_cmd.c file that fixes following
> error.
>
> ERROR: spaces required around that '>' (ctx:WxV)
> ERROR: that open brace { should be on the previous li
On Thu, Nov 06, 2014 at 12:06:36PM +0530, Sanjeev Sharma wrote:
> This is a patch to the rtw_cmd.c file that fixes following
> Warning by introducing temporary structure.
>
> WARNING: line over 80 characters
>
> Signed-off-by: Sanjeev Sharma
> ---
> drivers/staging/rtl8723au/core/rtw_cmd.c | 12
On Thu, Nov 06, 2014 at 10:33:28AM -0500, Ken Depro wrote:
> - LOGINF("virtpcidev bus_no<<%d>>devNo<<%d>>", virtpcidev->bus_no,
> -virtpcidev->device_no);
> - virthbainfo = (struct virthba_info *) scsihost->hostdata;
> + LOGINF("virtpcidev busNo<<%d>>devNo<<%d>>", virtpcidev
On Thu, Nov 06, 2014 at 07:26:16PM +0800, Andy Yan wrote:
> From: ykk
We need a "real" name here, sorry.
>
> dw-hdmi is under drm/bridge, so it should be the bridge mode.
> hange off the encoder to dw_hdmi-imx.c, keep the connector &
> birdge in dw_hdmi.c
>
> Signed-off-by: ykk
Same here.
t
On Thu, Nov 06, 2014 at 09:18:44PM +0530, Vaishali Thakkar wrote:
> One more thing, as previously for me it was not showing any error could you
> please tell me what can be the reason behind it and how can I check my
> revised code?? [I am newbie to linux kernel so I it would be nice of you if
> yo
On Thu, Nov 06, 2014 at 10:33:28AM -0500, Ken Depro wrote:
> From: Ken Depro
>
> This patch removes unneeded spaces after casts within the virthba.c
> file. The checkpatch script was run after these changes, and no
> "remove spaces after casts" checks were generated.
>
> Later patches will fix
Dan,
The CamelCase version is what is currently present in our source. I have not
addressed/fixed those checks yet. With the members as busNo and deviceNo, the
kernel builds cleanly. I'm not sure why they showed up as non-CamelCase in the
diff.
Ken
-Original Message-
From: Dan Carp
Greg,
When I executed the format-patch command, it added the conflict lines below.
There was indeed a conflict in this file from yesterday (I had originally
pulled/built from staging-next, but there was another pending patch of mine in
staging-testing that was not in -next, which caused the or
A: No.
Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Thu, Nov 06, 2014 at 10:22:30AM -0600, Depro, Kenneth J wrote:
> Greg,
>
> When I executed the format-patch command, it added the conflict lines below.
> There was indeed a conflict in this file
Register with kernel power-off handler instead of setting pm_power_off
directly. Register with default priority since we don't know any better.
Cc: Julian Andres Klode
Cc: Marc Dietrich
Cc: Greg Kroah-Hartman
Acked-by: Greg Kroah-Hartman
Signed-off-by: Guenter Roeck
---
v5:
- Rebase to v3.18-
This series addresses two issues:
- There is no timeout when communicating with userspace VSS daemon.
- In case we fail to send a message to VSS or KVP userspace daemons we
can report the failure to the host right away avoiding the timeout.
Newly introduced 10 second timeout is something worth d
If we fail to send a message to userspace daemon with cn_netlink_send()
there is no need to wait for userspace to reply as it is not going to
happen. This happens when kvp or vss daemon is stopped after a successful
handshake. Report HV_E_FAIL immediately and cancel the timeout job so
host won't re
In contrast with KVP there is no timeout when communicating with
userspace VSS daemon. In case it gets stuck performing freeze/thaw
operation no message will be sent to the host so it will take very
long (around 10 minutes) before backup fails. Introduce 10 second
timeout using schedule_delayed_wor
From: Ken Depro
This patch renames the following SignalInsert_withLock parameters
to fix the CamelCase checks:
pChannel to pchannel
Queue to queue
pSignal to psignal
Signed-off-by: Ken Depro
---
drivers/staging/unisys/channels/chanstub.c |6 +++---
drivers/staging/unisys/channels/chanstub.
A couple weeks ago I found the bug 87ebccf97f54 ('staging: lustre:
validate size in ll_setxattr()') by manual audit but now I have written
a Smatch check to find bugs like that automatically. The results look
to be fairly high quality generally except in lustre. These have a
potential security im
On 11/6/14, Dan Carpenter wrote:
> On Thu, Nov 06, 2014 at 12:50:10PM -0500, Ken Depro wrote:
>> From: Ken Depro
>>
>> This patch renames the following SignalInsert_withLock parameters
>> to fix the CamelCase checks:
>> pChannel to pchannel
>
> If you're going to rename things, you may as well ge
On Thu, Nov 06, 2014 at 12:50:10PM -0500, Ken Depro wrote:
> From: Ken Depro
>
> This patch renames the following SignalInsert_withLock parameters
> to fix the CamelCase checks:
> pChannel to pchannel
If you're going to rename things, you may as well get rid of the "p".
(My guess is the "p" stan
From: Benjamin Romer
This patch adds the visorclientbus driver to the Unisys s-Par driver set. This
driver is responsible for helping manage virtual devices like keyboards, mice,
serial ports, displays, and any customized drivers for special-purpose guests.
Signed-off-by: Benjamin Romer
Signed-
>> If you're going to rename things, you may as well get rid of the "p".
> And fix the function name as well...
Denis,
I had planned on fixing the function name in a separate patch since
I thought that would constitute two separate "work items" and also
generate a larger set of patches.
>From yo
On Thu, Nov 06, 2014 at 12:53:32PM -0600, Ken Cox wrote:
> From: Benjamin Romer
>
> This patch adds the visorclientbus driver to the Unisys s-Par driver set. This
> driver is responsible for helping manage virtual devices like keyboards, mice,
> serial ports, displays, and any customized drivers
Remove the instances of multiple blank lines in the visorchannel files.
Signed-off-by: Bryan Thompson
---
drivers/staging/unisys/visorchannel/globals.h |1 -
.../unisys/visorchannel/visorchannel_funcs.c |1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/staging/unisys
Remove extraneous parentheses around a variable in visorchannel_funcs.c
Signed-off-by: Bryan Thompson
---
.../unisys/visorchannel/visorchannel_funcs.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/visorchannel/visorchannel_funcs.c
b/drivers/
Add braces to the else arm of an if/else block in visorchannel_funcs.c
Signed-off-by: Bryan Thompson
---
.../unisys/visorchannel/visorchannel_funcs.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/visorchannel/visorchannel_funcs.c
b/drivers
Split multiple logging lines to get them closer to 80 characters per line.
Signed-off-by: Bryan Thompson
---
.../unisys/visorchannel/visorchannel_funcs.c | 18 --
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/unisys/visorchannel/visorchann
Use the appropriate whitespace for multiline statements in visorchannel_funcs.c
Signed-off-by: Bryan Thompson
---
.../unisys/visorchannel/visorchannel_funcs.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/visorchannel/visorchannel_
Move the || logical continuation from the start of the second line of an if
statement to the end of the first line.
Signed-off-by: Bryan Thompson
---
.../unisys/visorchannel/visorchannel_funcs.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/uni
This series of patches address a collection of formatting issues reported
by checkpatch.pl on the files in the visorchannel directory.
Bryan Thompson (8):
staging: unisys: visorchannel: Remove multiple blank lines
staging: unisys: visorchannel: Fix alignment issues
staging: unisys: visorcha
Rename the goto label used throughout visorchannel_funcs.c from Away to cleanup.
Signed-off-by: Bryan Thompson
---
.../unisys/visorchannel/visorchannel_funcs.c | 54 ++--
1 file changed, 27 insertions(+), 27 deletions(-)
diff --git a/drivers/staging/unisys/visorchannel/v
Remove the whitespace between a cast and the variable in visorchannel_funcs.c
Signed-off-by: Bryan Thompson
---
.../unisys/visorchannel/visorchannel_funcs.c | 22 ++--
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/unisys/visorchannel/viso
On Mon, Nov 03, 2014 at 07:52:38PM +0100, Konrad Zapalowicz wrote:
> This commit adds the Digi Classic board implementation to the
> staging/jsm driver.
>
> The code here is taken from the staging/dgnc driver and modified to
> match the serial/jsm state. This work is mostly based on the changes
>
On 11/06/2014 01:05 PM, Greg KH wrote:
On Thu, Nov 06, 2014 at 12:53:32PM -0600, Ken Cox wrote:
From: Benjamin Romer
This patch adds the visorclientbus driver to the Unisys s-Par driver set. This
driver is responsible for helping manage virtual devices like keyboards, mice,
serial ports, disp
On Thu, Nov 06, 2014 at 01:17:56PM -0600, Ken Cox wrote:
> >There are also other checkpatch errors in this patch, which makes me not
> >want to take it at all, as you would now just be required to send more
> >fixes for the file. So why not fix things up right the first time?
> The only checkpatch
On 11/6/14, Depro, Kenneth J wrote:
>>> If you're going to rename things, you may as well get rid of the "p".
>
>> And fix the function name as well...
>
> Denis,
> I had planned on fixing the function name in a separate patch since
> I thought that would constitute two separate "work items" and a
This fixes warning from drivers/media/v4l2-core/videobuf2-core.c,
WARN_ON(atomic_read(&q->owned_by_drv_count)).
Signed-off-by: Andrey Utkin
---
drivers/media/pci/solo6x10/solo6x10-v4l2-enc.c | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/drivers/media/pci/solo6x
On Thu, Nov 06, 2014 at 11:56:05PM +0100, k...@konagma.com wrote:
> From: Krzysztof Konopko
>
> Several stylistics problems are reported by scripts/checkpatch.pl run on
> rtw_led.*:
>
> * FSF_MAILING_ADDRESS
>
> Free Software Foundation's mailing address should not be included in
> th
The staging/dgnc and the serial/jsm drivers have the same origin. They
come from the same codebase however the jsm is the community adopted
version where the dgnc is kind of a direct submission from the Digi
company.
The recent commits to the jsm driver moved in the support for some of
the Neo car
From: Krzysztof Konopko
Several stylistics problems are reported by scripts/checkpatch.pl run on
rtw_led.*:
* FSF_MAILING_ADDRESS
Free Software Foundation's mailing address should not be included in
the sample GPL notice.
* LINE_SPACING
Missing a blank line after declarations.
This commit enables support for the Digi Classic adapters line in the
jsm driver. This means changes in:
- device probing code
- device cleanup code
- driver description (Kconfig)
The init/cleanup code is based on the staging/dgnc driver.
Signed-off-by: Konrad Zapalowicz
This commit adds the Digi Classic board implementation to the
staging/jsm driver.
The code here is taken from the staging/dgnc driver and modified to
match the serial/jsm state. This work is mostly based on the changes
that has been done to the code handling the Digi Neo cards with the
inspiration
This commit removes the address of Free Software Foundation from each
of the mentioned file in order to suppress the checkpatch warning.
Signed-off-by: Konrad Zapalowicz
---
drivers/tty/serial/jsm/jsm.h| 5 -
drivers/tty/serial/jsm/jsm_cls.c| 5 -
drivers/tty/serial/jsm/jsm_d
On 11/06, Greg KH wrote:
> On Mon, Nov 03, 2014 at 07:52:38PM +0100, Konrad Zapalowicz wrote:
> > This commit adds the Digi Classic board implementation to the
> > staging/jsm driver.
> >
> > The code here is taken from the staging/dgnc driver and modified to
> > match the serial/jsm state. This w
On Fri, Nov 07, 2014 at 12:05:34AM +0100, Konrad Zapalowicz wrote:
> This commit removes the staging/dgnc driver in favor of the serial/jsm
> driver.
>
> This is because the serial/jsm driver now supports all of the hardware
> that has been supported by the staging/dgnc plus it offers better
> ove
A rather obvious typo in one of the identifier has been found. This patch
fixes the typo and ensures any lines changed do not exceed 80 characters
as indicated by scripts/checkpatch.pl
Signed-off-by: Krzysztof Konopko
---
drivers/staging/rtl8188eu/core/rtw_led.c| 6 --
drivers/staging/r
rtw_led.* files include Free Software Foundation's mailing address in
the sample GPL notice. This is not desired and picked when running
scripts/checkpatch.pl.
Signed-off-by: Krzysztof Konopko
---
drivers/staging/rtl8188eu/core/rtw_led.c| 4
drivers/staging/rtl8188eu/include/rtw_led.h
According to Linux coding convention leading spaces are not allowed. This
patch removes leading spaces in rtw_led.c
Signed-off-by: Krzysztof Konopko
---
drivers/staging/rtl8188eu/core/rtw_led.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/co
On 11/06, Greg KH wrote:
> On Fri, Nov 07, 2014 at 12:05:34AM +0100, Konrad Zapalowicz wrote:
> > This commit removes the staging/dgnc driver in favor of the serial/jsm
> > driver.
> >
> > This is because the serial/jsm driver now supports all of the hardware
> > that has been supported by the sta
On 2014年11月06日 23:54, Greg Kroah-Hartman wrote:
On Thu, Nov 06, 2014 at 07:26:16PM +0800, Andy Yan wrote:
From: ykk
We need a "real" name here, sorry.
dw-hdmi is under drm/bridge, so it should be the bridge mode.
hange off the encoder to dw_hdmi-imx.c, keep the connector &
birdge in dw_hdmi
On 2014年11月05日 21:41, Philipp Zabel wrote:
Hi Andy,
I think separating the core from the SoC specific part is a good step
into the right direction.
Am Mittwoch, den 05.11.2014, 20:55 +0800 schrieb Andy Yan:
imx6 and rockchip rk3288 and JZ4780 (Ingenic Xburst/MIPS)
use the interface compatible
On 2014年11月05日 21:30, Zubair Lutfullah Kakakhel wrote:
This one patch does too much to be reviewed easily.
One patch is supposed to modify/add one thing at a time in the kernel.
Separating platform specific code from imx-drm/imx-hdmi is one thing.
Adding support for multi-byte register access
On Fri, Nov 07, 2014 at 01:06:18AM +0400, Andrey Utkin wrote:
> This fixes warning from drivers/media/v4l2-core/videobuf2-core.c,
> WARN_ON(atomic_read(&q->owned_by_drv_count)).
>
Thanks!
regards,
dan carpenter
___
devel mailing list
de...@linuxdrive
76 matches
Mail list logo
