Re: [PATCH] [media] staging: media: Fix build for iMON LIRC driver

2013-10-16 Thread Dan Carpenter
On Tue, Oct 15, 2013 at 04:02:43PM +0200, Thierry Reding wrote: > Add a missing , between parameters in a call to dev_info(). > Greg already reverted the original patch. The original patch wasn't sent to the list. Archana, use scripts/get_maintainer.pl to find out who to send patches to. Don't

Re: [PATCH v2 2/3] ARM: dts: imx6qdl-wandboard: Add HDMI support

2013-10-16 Thread Sascha Hauer
On Tue, Oct 15, 2013 at 11:35:00AM +0100, Russell King - ARM Linux wrote: > On Tue, Oct 15, 2013 at 10:18:30AM +0100, Russell King - ARM Linux wrote: > > On Tue, Oct 15, 2013 at 09:46:18AM +0200, Sascha Hauer wrote: > > > This sounds like the wrong clock polarity. Could you try inverting > > > sig_

[PATCH] mmc: rtsx: fix card poweroff bug

2013-10-16 Thread micky_ching
From: Micky Ching If the host driver removed while card in the slot, the host will not power off card power correctly. This bug is produced because host eject flag set before the last mmc_set_ios callback, we should set the eject flag after power off. Signed-off-by: Micky Ching --- drivers/mmc

Re: [PATCH] mmc: rtsx: change phase searching method

2013-10-16 Thread micky
Hi Chris: have you accepted this patch ? micky ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: A review of dm-writeboost

2013-10-16 Thread Akira Hayakawa
Dave > Akira, can you please post the entire set of messages you are > getting when XFS showing problems? That way I can try to confirm > whether it's a regression in XFS or something else. Environment: - The kernel version is 3.12-rc1 - The debuggee is a KVM virtual machine equipped with 8 vcpus

Re: A review of dm-writeboost

2013-10-16 Thread Dave Chinner
On Wed, Oct 16, 2013 at 07:34:38PM +0900, Akira Hayakawa wrote: > Dave > > > Akira, can you please post the entire set of messages you are > > getting when XFS showing problems? That way I can try to confirm > > whether it's a regression in XFS or something else. > > Environment: > - The kernel v

Web de administración

2013-10-16 Thread Web de administración
Web de administración de notificación de e-mail Este mensaje es de nuestro centro de mensajes de administración para toda nuestra cuenta de correo electrónico owners.We está eliminando el acceso a todos nuestros clientes de correo web. Su correo electrónico cuenta se actualizará a una interfaz d

Re: A review of dm-writeboost

2013-10-16 Thread Akira Hayakawa
Dave > XFS shuts down because you've returned EIO to a log IO. That's a > fatal error. If you do the same to an ext4 journal write, it will do > the equivalent of shut down (e.g. complain and turn read-only). You mean block device should not return -EIO anyway if it doesn't want XFS to suddenly sh

[PATCH] Staging: comedi: fix semicolon coding style issue in ni_660x.c

2013-10-16 Thread Dhanunjaya
This is a patch to the ni_660x.c file that fixes up a semicolon warning found by the checkpatch.pl tool --- drivers/staging/comedi/drivers/ni_660x.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_660x.c b/drivers/staging/comedi/drivers/ni

[PATCH 09/35] staging: comedi: amplc_pci224: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the type of the `ao_scan_vals` member of `struct pci224_private` from `short *` to `unsigned short *` for consistency. Also change the type of the `array` variable in `pci224_ao_munge()` from `short *` to `unsigned short

[PATCH 01/35] staging: comedi: use unsigned samples for comedi_buf_put()/get()

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. `comedi_buf_put()` and `comedi_buf_get()` use a `short` to hold the 16-bit data value being transferred to or from the comedi buffer. Change them to use `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/s

[PATCH 10/35] staging: comedi: amplc_pci230: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Replace all uses of `short` to handle sample values in the "amplc_pci230" driver with `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/amplc_pci230.c | 21 +++-- 1

[PATCH 00/35] staging: comedi: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Some of the comedi buffer handling functions and various drivers use signed short values in places to hold or manipulate 16-bit (or less) sample values. Be a bit more consistent in the use of unsigned types to handle sample valu

[PATCH 06/35] staging: comedi: adl_pci9118: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the `ao_data[]` and `dmabuf_virt[]` members of `struct pci9118_private` and various local variables dealing with sample values to use `unsigned short` instead of `short` for consistency. Also remove the `short *ai_data` m

[PATCH 12/35] staging: comedi: cb_pcidas64: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are unsigned. Change the element type of `ao_bounce_buffer[]` in `struct pcidas64_private` from `short` to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/cb_pcidas64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 21/35] staging: comedi: ni_at_a2150: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented by unsigned values. Change the type of the `dma_buffer` member of `struct a2150_private` from `s16 *` to `uint16_t *`, and change the type of the `dpnt` variable in `a2150_interrupt()` to `unsigned short` for consistency. Signed-off-by: Ian Abbott

[PATCH 07/35] staging: comedi: adv_pci1710: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the element type of the `ao_data[]` member of `struct pci1710_private` from `short` to `unsigned short` for consistency. Also remove the `ai_data` member as it is only assigned to. Change various local variables used to

[PATCH 19/35] staging: comedi: icp_multi: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the element type of the `ao_data[]` member of `struct icp_multi_private` from `short` to `unsigned short` for consistency. Also remove the `ai_data` and `di_data` members as they are not used. Signed-off-by: Ian Abbott

[PATCH 02/35] staging: comedi: use unsigned sample in cfc_write_to_buffer()

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. `cfc_write_to_buffer()` in "comedi_fc.h" currently uses `short` to hold a 16-bit sample value to be written to the buffer. Change the type of the parameter to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- dr

[PATCH 22/35] staging: comedi: ni_labpc: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the type of the `data` variable in `labpc_ai_cmd()` from `short` to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/ni_labpc.c | 2 +- 1 file changed, 1 insertion(+), 1 del

[PATCH 23/35] staging: comedi: ni_mio_common: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the element type of the `ai_fifo_buffer[]` element of `struct ni_board_struct` and the types of various local variables from `short` to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/com

[PATCH 17/35] staging: comedi: dt3000: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the type of the `data` variable in `dt3k_ai_empty_fifo()` from `short` to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/dt3000.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 05/35] staging: comedi: adl_pci9111: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the element type of the `ai_bounce_buffer[]` member of `struct pci9111_private_data` from `short` to `unsigned short` for consistency. Also change the type of the `array` variable in `pci9111_ai_munge()` from `short *` to

[PATCH 20/35] staging: comedi: me4000: sample types are unsigned (and not long!)

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the types of various variables in the "me4000" driver dealing with samples to use unsigned types for consistency. Also replace the `long` or `unsigned long` variables used to handle 32-bit register values to `unsigned int

[PATCH 14/35] staging: comedi: das16m1: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the element type of the `ai_buffer[]` member of `struct das16m1_private_struct` and the types used by `munge_sample()` and `munge_sample_array()` from `short` to `unsigned short` for consistency. Signed-off-by: Ian Abbott

[PATCH 11/35] staging: comedi: cb_pcidas: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented by unsigned values. Change the element types of `ai_buffer[]`, `ao_buffer[]` and `ao_value[]` in `struct cb_pcidas_private` to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/cb_pcidas.c | 6 +++---

[PATCH 24/35] staging: comedi: ni_pcidio: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change `nidio_interrupt()` to use unsigned types for sample values (actually bit-vectors of 1-bit sample values) instead of signed types. Also rename the `AuxData` variable to `auxdata` and change it from `long` to `unsigned int

[PATCH 08/35] staging: comedi: adv_pci1723: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the element type of the `ao_data[]` member of `struct pci1723_private` from `short` to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/adv_pci1723.c | 2 +- 1 file changed,

[PATCH 29/35] staging: comedi: pcmmio: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change `interrupt_pcmmio()` and `ai_rinsn()` in the "pcmmio" module to use unsigned sample values for consistency. Also, make the order in which `interrupt_pcmmio()` writes the two sample values (each actually containing up to 1

[PATCH 27/35] staging: comedi: pcl816: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the "pcl816" module to use unsigned types to handle samples for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/pcl816.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff

[PATCH 16/35] staging: comedi: dt282x: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change various members of `struct dt282x_private` and various parameters and variables dealing with samples to use `unsigned short` instead of `short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drive

[PATCH 15/35] staging: comedi: das1800: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented by unsigned values. Change the type of the `ao_update_bits` member of `struct das1800_private` and the types of various local variables used to hold sample values from `short` to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- dri

[PATCH 18/35] staging: comedi: fl512: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the element type of the `ao_readback[]` member of `struct fl512_private` from `short` to `unsigned short` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/fl512.c | 3 +-- 1 file changed, 1

[PATCH 28/35] staging: comedi: pcl818: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the interrupt data transfer functions in the "pcl818" module to use unsigned types for consistency. Also remove the `short *ai_data` member of `struct pcl818_private` as it is only assigned to and otherwise unused. Signe

[PATCH 13/35] staging: comedi: das16: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the type of the `data` variable in `das16_ai_munge()` from `short *` to `unsigned short *` for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/das16.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 25/35] staging: comedi: pcl711: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Don't cast the sample value parameter of `comedi_buf_put()` to `short`, particularly as it has now been changed to `unsigned short`. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/pcl711.c | 2 +- 1 file changed,

[PATCH 26/35] staging: comedi: pcl812: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change `transfer_from_dma_buf()` and `interrupt_pcl812_ai_dma()` in the "pcl812" module to use `unsigned short` sample values for consistency. Also remove the `short *ai_data` member of `struct pcl812_private` as it is only assi

[PATCH 30/35] staging: comedi: pcmuio: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change `pcmuio_handle_intr_subdev()` in the "pcmuio" module to use unsigned sample values for consistency. Also, make the order in which `pcmuio_handle_intr_subdev()` writes the two sample values (each actually containing up to

[PATCH 35/35] staging: comedi: usbduxsigma: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Use unsigned types consistently for handling comedi sample data and also for the USB data buffers. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/usbduxsigma.c | 20 ++-- 1 file changed, 10 insert

[PATCH 31/35] staging: comedi: quatech_daqp_cs: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change `daqp_interrupt()` to use unsigned sample values for consistency. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/quatech_daqp_cs.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/driv

[PATCH 04/35] staging: comedi: addi_apci_3120: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Change the element type of `ul_DmaBufferVirtual[2]` in `struct addi_private` from `short *` to `unsigned short *` for consistency. Note that several ADDI-DATA drivers use this struct from "addi_common.h", but only the "addi_apci_

[PATCH 03/35] staging: comedi: 8255: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. `subdev_8255_interrupt()` calls `comedi_buf_put()` with a `short` data value merely because that's what was previously expected. Since it now expects an `unsigned short`, change it here for consistency. Signed-off-by: Ian Abbot

[PATCH 32/35] staging: comedi: rtd520: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Use unsigned types for sample value manipulations in the "rtd520" driver for consistency. Also replace the hand-coded munging of 2's complement sample values with calls to `comedi_offset_munge()` and AND with `s->maxdata` to str

[PATCH 34/35] staging: comedi: usbdux: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Use unsigned types consistently for handling comedi sample data and also for the USB data buffers. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/usbdux.c | 35 + 1 file changed, 1

[PATCH 33/35] staging: comedi: s626: sample types are unsigned

2013-10-16 Thread Ian Abbott
Sample values in comedi are generally represented as unsigned values. Use unsigned types consistently in the "s626" module when dealing with sample values. Rewrite `s626_reg_to_uint()` as it can be done with a one-liner. Signed-off-by: Ian Abbott --- drivers/staging/comedi/drivers/s626.c | 22 +

Re: [PATCH] Staging: comedi: fix semicolon coding style issue in ni_660x.c

2013-10-16 Thread Ian Abbott
On 2013-10-16 13:38, Dhanunjaya wrote: This is a patch to the ni_660x.c file that fixes up a semicolon warning found by the checkpatch.pl tool --- drivers/staging/comedi/drivers/ni_660x.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_

Re: [PATCH] [media] staging: media: Fix build for iMON LIRC driver

2013-10-16 Thread Greg Kroah-Hartman
On Wed, Oct 16, 2013 at 10:03:40AM +0300, Dan Carpenter wrote: > On Tue, Oct 15, 2013 at 04:02:43PM +0200, Thierry Reding wrote: > > Add a missing , between parameters in a call to dev_info(). > > > > Greg already reverted the original patch. > > The original patch wasn't sent to the list. Arch

[PATCH] drivers: staging: dgap: Remove useless casting in dgap_parse.c

2013-10-16 Thread Geyslan G. Bem
Casting (void *) value returned by kmalloc is useless as mentioned in Documentation/CodingStyle, Chap 14. Signed-off-by: Geyslan G. Bem --- drivers/staging/dgap/dgap_parse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgap/dgap_parse.c b/drivers/staging/d

[PATCH] drivers: staging: dgnc: Remove useless casting in dgnc_driver.c

2013-10-16 Thread Geyslan G. Bem
Casting (void *) value returned by kzalloc is useless as mentioned in Documentation/CodingStyle, Chap 14. Signed-off-by: Geyslan G. Bem --- drivers/staging/dgnc/dgnc_driver.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c b/drivers/s

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Russell King - ARM Linux
On Tue, Oct 15, 2013 at 10:17:07AM -0300, Fabio Estevam wrote: > On Tue, Oct 15, 2013 at 10:10 AM, Russell King - ARM Linux > wrote: > > Another point on patch 1. Sorry, I don't have patch 1 to reply to, it > > seems it was deleted from linux-arm-kernel's moderation queue. > > > > drm_mode_connec

[PATCH 2/2] Staging: xillybus: fix for quoted string split across lines

2013-10-16 Thread Dhanunjaya
This is a patch to the xillybus_of.c file that fixes up a quoted string split across lines warning found by the checkpatch.pl tool. Signed-off-by: Dhanunjaya --- drivers/staging/xillybus/xillybus_of.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/xilly

Re: [PATCH 2/2] Staging: xillybus: fix for quoted string split across lines

2013-10-16 Thread Greg KH
On Wed, Oct 16, 2013 at 11:27:52PM +0530, Dhanunjaya wrote: > This is a patch to the xillybus_of.c file that fixes up a > quoted string split across lines warning found by the > checkpatch.pl tool. > > Signed-off-by: Dhanunjaya I need a full name here please. > --- > drivers/staging/xillybus/x

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Russell King - ARM Linux
On Wed, Oct 16, 2013 at 06:03:42PM +0100, Russell King - ARM Linux wrote: > On Tue, Oct 15, 2013 at 10:17:07AM -0300, Fabio Estevam wrote: > > On Tue, Oct 15, 2013 at 10:10 AM, Russell King - ARM Linux > > wrote: > > > Another point on patch 1. Sorry, I don't have patch 1 to reply to, it > > > se

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Greg Kroah-Hartman
On Wed, Oct 16, 2013 at 07:07:35PM +0100, Russell King - ARM Linux wrote: > Sorry, but I don't think imx-drm is driving the hardware correctly, and > I know that Greg wants it moved out of drivers/staging, but frankly it > seems to be far from ready for that. Certainly the HDMI parts seems to > be

[PATCH 3/4] Staging: xgifb: fix braces {} are not necessary for single statement blocks

2013-10-16 Thread Dhanunjaya
This is a patch to the vb_setmode.c file that fixes braces {} are not necessary for single statement blocks warning found by the checkpatch.pl tool Signed-of-by: Dhanunjaya --- drivers/staging/xgifb/vb_setmode.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/driv

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Russell King - ARM Linux
On Wed, Oct 16, 2013 at 11:31:07AM -0700, Greg Kroah-Hartman wrote: > On Wed, Oct 16, 2013 at 07:07:35PM +0100, Russell King - ARM Linux wrote: > > Sorry, but I don't think imx-drm is driving the hardware correctly, and > > I know that Greg wants it moved out of drivers/staging, but frankly it > >

Re: [PATCH 2/2] Staging: xillybus: fix for quoted string split across lines

2013-10-16 Thread Eli Billauer
Hello Greg, On 16/10/13 21:06, Greg KH wrote: rc = of_address_to_resource(dev->of_node, 0,&endpoint->res); >if (rc) { > - pr_warn("xillybus: Failed to obtain device tree " > - "resource\n"); > + pr_warn("xillybus: Failed to obtain device tre

Re: [PATCH 2/2] Staging: xillybus: fix for quoted string split across lines

2013-10-16 Thread Greg KH
On Wed, Oct 16, 2013 at 09:49:19PM +0300, Eli Billauer wrote: > Hello Greg, > > On 16/10/13 21:06, Greg KH wrote: > >>rc = of_address_to_resource(dev->of_node, 0,&endpoint->res); > >>> if (rc) { > >>> -pr_warn("xillybus: Failed to obtain device tree " > >>> -

Re: [PATCH 3/4] Staging: xgifb: fix braces {} are not necessary for single statement blocks

2013-10-16 Thread Greg KH
Where are patches 1/4 and 2/4 in this series? On Thu, Oct 17, 2013 at 12:21:33AM +0530, Dhanunjaya wrote: > This is a patch to the vb_setmode.c file that fixes braces {} > are not necessary for single statement blocks warning found > by the checkpatch.pl tool > > Signed-of-by: Dhanunjaya Again,

[PATCH 4/4] Staging: xgifb: fix space prohibited before semicolon.

2013-10-16 Thread Dhanunjaya
This is a patch to the vb_table.h file that fixes the space prohibited before semicolon warning found by the checkpatch.pl tool. Signed-off-by: Dhanunjaya --- drivers/staging/xgifb/vb_table.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/xgifb/vb_table.h b/d

[PATCH 4/4] Staging: xgifb: fix space prohibited before semicolon.

2013-10-16 Thread Dhanunjaya Naidu Ravada
From: Dhanunjaya This is a patch to the vb_table.h file that fixes the space prohibited before semicolon warning found by the checkpatch.pl tool. Signed-off-by: Dhanunjaya Naidu Ravada --- drivers/staging/xgifb/vb_table.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/driv

Re: [PATCH 3/4] Staging: xgifb: fix braces {} are not necessary for single statement blocks

2013-10-16 Thread Greg KH
On Thu, Oct 17, 2013 at 12:51:24AM +0530, Dhanunjaya Naidu Ravada wrote: > From: Dhanunjaya This isn't matching the "From:" line in your email :( And where are patches 1/4 and 2/4 in this series? greg k-h ___ devel mailing list de...@linuxdriverprojec

[PATCH 1/4] Staging: comedi: fix semicolon coding style issue in ni_660x.c

2013-10-16 Thread Dhanunjaya Naidu Ravada
From: Dhanunjaya This is a patch to the ni_660x.c file that fixes up a semicolon warning found by the checkpatch.pl tool. Signed-off-by: Dhanunjaya Naidu Ravada --- drivers/staging/comedi/drivers/ni_660x.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/

Re: [PATCH 4/4] Staging: xgifb: fix space prohibited before semicolon.

2013-10-16 Thread Dan Carpenter
On Thu, Oct 17, 2013 at 12:21:34AM +0530, Dhanunjaya wrote: > This is a patch to the vb_table.h file that fixes the > space prohibited before semicolon warning found by the > checkpatch.pl tool. > Someone already fixed this one. Btw, for people who aren't aware, linux-next has moved to git://git

Re: [PATCH 1/4] Staging: comedi: fix semicolon coding style issue in ni_660x.c

2013-10-16 Thread Greg KH
On Thu, Oct 17, 2013 at 12:54:52AM +0530, Dhanunjaya Naidu Ravada wrote: > From: Dhanunjaya Again, doesn't match the signed-off-by: or From: line on your email :( > This is a patch to the ni_660x.c file that fixes up a semicolon warning > found by the checkpatch.pl tool. No it isn't, it also do

Re: [PATCH] drivers: staging: dgnc: Remove useless casting in dgnc_driver.c

2013-10-16 Thread Greg KH
On Wed, Oct 16, 2013 at 12:57:35PM -0300, Geyslan G. Bem wrote: > Casting (void *) value returned by kzalloc is useless > as mentioned in Documentation/CodingStyle, Chap 14. > > Signed-off-by: Geyslan G. Bem > --- > drivers/staging/dgnc/dgnc_driver.c | 6 ++ > 1 file changed, 2 insertions(+)

Re: [PATCH] Staging: comedi: fix semicolon coding style issue in ni_660x.c

2013-10-16 Thread Greg KH
On Wed, Oct 16, 2013 at 06:08:14PM +0530, Dhanunjaya wrote: > This is a patch to the ni_660x.c file that fixes up a semicolon warning > found by the checkpatch.pl tool > --- No signed-off-by: so I can't take this patch, sorry. greg k-h ___ devel mailing

Re: [PATCH 00/35] staging: comedi: sample types are unsigned

2013-10-16 Thread Greg Kroah-Hartman
On Wed, Oct 16, 2013 at 02:40:03PM +0100, Ian Abbott wrote: > Sample values in comedi are generally represented as unsigned values. > Some of the comedi buffer handling functions and various drivers use > signed short values in places to hold or manipulate 16-bit (or less) > sample values. Be a bi

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Troy Kisky
On 10/16/2013 10:03 AM, Russell King - ARM Linux wrote: On Tue, Oct 15, 2013 at 10:17:07AM -0300, Fabio Estevam wrote: On Tue, Oct 15, 2013 at 10:10 AM, Russell King - ARM Linux wrote: Another point on patch 1. Sorry, I don't have patch 1 to reply to, it seems it was deleted from linux-arm-ke

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Russell King - ARM Linux
On Wed, Oct 16, 2013 at 12:37:42PM -0700, Troy Kisky wrote: > On 10/16/2013 10:03 AM, Russell King - ARM Linux wrote: >> On Tue, Oct 15, 2013 at 10:17:07AM -0300, Fabio Estevam wrote: >>> On Tue, Oct 15, 2013 at 10:10 AM, Russell King - ARM Linux >>> wrote: Another point on patch 1. Sorry, I

[PATCH 3/4] Staging: xgifb: fix braces {} are not necessary for single statement blocks

2013-10-16 Thread Dhanunjaya Naidu Ravada
From: Dhanunjaya This is a patch to the vb_setmode.c file that fixes braces {} are not necessary for single statement blocks warning found by the checkpatch.pl tool Signed-of-by: Dhanunjaya Naidu Ravada --- drivers/staging/xgifb/vb_setmode.c | 11 --- 1 file changed, 4 insertions(+), 7

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Troy Kisky
On 10/16/2013 1:27 PM, Russell King - ARM Linux wrote: On Wed, Oct 16, 2013 at 12:37:42PM -0700, Troy Kisky wrote: On 10/16/2013 10:03 AM, Russell King - ARM Linux wrote: On Tue, Oct 15, 2013 at 10:17:07AM -0300, Fabio Estevam wrote: On Tue, Oct 15, 2013 at 10:10 AM, Russell King - ARM Linux

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Sascha Hauer
On Wed, Oct 16, 2013 at 08:02:11PM +0100, Russell King - ARM Linux wrote: > On Wed, Oct 16, 2013 at 11:31:07AM -0700, Greg Kroah-Hartman wrote: > > On Wed, Oct 16, 2013 at 07:07:35PM +0100, Russell King - ARM Linux wrote: > > > Sorry, but I don't think imx-drm is driving the hardware correctly, and

Re: A review of dm-writeboost

2013-10-16 Thread Dave Chinner
On Wed, Oct 16, 2013 at 09:17:40PM +0900, Akira Hayakawa wrote: > Dave > > > XFS shuts down because you've returned EIO to a log IO. That's a > > fatal error. If you do the same to an ext4 journal write, it will do > > the equivalent of shut down (e.g. complain and turn read-only). > You mean bloc

Re: [PATCH v2 1/3] imx-drm: Add mx6 hdmi transmitter support

2013-10-16 Thread Russell King - ARM Linux
On Wed, Oct 16, 2013 at 02:03:17PM -0700, Troy Kisky wrote: > Freescale's kernel(imx_3.0.35_4.1.0) has this code > > video/mxc_hdmi.c-/* Workaround to clear the overflow condition */ > video/mxc_hdmi.c-static void mxc_hdmi_clear_overflow(void) > video/mxc_hdmi.c-{ > video/mxc_hdmi.c- int coun

Re: [PATCH] drivers: staging: dgnc: Remove useless casting in dgnc_driver.c

2013-10-16 Thread Geyslan Gregório Bem
2013/10/16 Greg KH : > On Wed, Oct 16, 2013 at 12:57:35PM -0300, Geyslan G. Bem wrote: >> Casting (void *) value returned by kzalloc is useless >> as mentioned in Documentation/CodingStyle, Chap 14. >> >> Signed-off-by: Geyslan G. Bem >> --- >> drivers/staging/dgnc/dgnc_driver.c | 6 ++ >> 1