Re: [PATCH 02/10] staging: wilc1000: remove typedef from tenuWIDtype

On 2015년 10월 07일 00:14, Mike Rapoport wrote: On Tue, Oct 06, 2015 at 06:42:13PM +0900, Tony Cho wrote: From: Leo Kim This patch remove typedef from the enum tenuWIDtype and rename it to WID_TYPE. Signed-off-by: Leo Kim Signed-off-by: Tony Cho

[PATCH v2 1/2] fb.h: Provide alternate screen_base pointer

Some drivers use member screen_base of struct fb_info to store non- __iomem pointers, creating the need for ugly __force typecasts to avoid sparse warnings. This adds an alternate pointer without the __iomem qualifyer for this use. Signed-off-by: Lars Svensson ---

[PATCH v2 2/2] staging: fbtft: use alternate screen pointer

Member screen_base in struct fb_info is declared with __iomem qualifier causing sparse warnings when used as a regular ponter. To avoid the warnings, instead use alternate non-__iomem pointer, screen_buffer, troughout the driver. Signed-off-by: Lars Svensson ---

[PATCH 3/3] staging: sm750fb: remove DVI controller related files

The files that implement intefacing with sii164 DVI controller are not used and therefore they can be removed. Signed-off-by: Mike Rapoport --- drivers/staging/sm750fb/ddk750_dvi.c| 96 drivers/staging/sm750fb/ddk750_dvi.h| 62 -

[PATCH 0/3] staging: sm750fb: remove sii164 DVI controller support

Currently the sii164 DVI controller support is unused anywhere in the rest of the driver, and, anyway it does not belong to framebuffer device driver. These patches remove the sii164 DVI controller support. Mike Rapoport (3): staging: sm750fb: don't build DVI controller related objects

[PATCH 2/3] staging: sm750fb: ddk750_display: don't include ddk750_dvi.h

The ddk750_dvi.h provides declarations for unused functions and including it is not needed. Signed-off-by: Mike Rapoport --- drivers/staging/sm750fb/ddk750_display.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/sm750fb/ddk750_display.c

Re: [PATCH 02/10] staging: wilc1000: remove typedef from tenuWIDtype

On Tue, Oct 06, 2015 at 06:42:13PM +0900, Tony Cho wrote: > From: Leo Kim > > This patch remove typedef from the enum tenuWIDtype > and rename it to WID_TYPE. > > Signed-off-by: Leo Kim > Signed-off-by: Tony Cho > --- >

[PATCH 1/3] staging: sm750fb: don't build DVI controller related objects

The ddk750_sii164.o ddk750_dvi.o are required solely for interfacing with sii164 DVI controller and currently there no functions from these modules used in other parts of the driver. That said, both objects can be removed from the driver. Signed-off-by: Mike Rapoport ---

[PATCH next] staging: wilc1000: avoid NULL pointer dereference on error

The host_int_init() function can dereference the pstrWFIDrv pointer while it is NULL on its error path. Jump directly to the _fail_ error label in the end of the error handling path to avoid that. By doing that we also skip stopping our kthread and destroying our message queue, but they

[PATCH 01/13] staging: comedi: mf6x4: prefer using the BIT macro

Fix the checkpatch.pl issues. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/mf6x4.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH 00/13] staging: comedi: mf6x4: cleanup driver

Fix some minor checkpatch.pl issues and tidy up this driver a bit. H Hartley Sweeten (13): staging: comedi: mf6x4: prefer using the BIT macro staging: comedi: mf6x4: remove unnecessary defines staging: comedi: mf6x4: remove unnecessary whitespace staging: comedi: mf6x4: sort the BAR1

[PATCH 02/13] staging: comedi: mf6x4: remove unnecessary defines

The MF6X4_DAC_R macro defines the offsets for all the DAC registers. Remove the unnecessary defines for each register. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

[PATCH 04/13] staging: comedi: mf6x4: sort the BAR1 register defines

For aesthetics, sort the defines in register order. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/mf6x4.c | 8 +++- 1 file changed, 3 insertions(+), 5

[PATCH 03/13] staging: comedi: mf6x4: remove unnecessary whitespace

For aesthetics, remove the extra whitespace and align all the register defines. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/mf6x4.c | 22

[PATCH 06/13] staging: comedi: mf6x4: A/D converter uses 2's complement coding

According to the user's manual, the A/D converter uses 2's complement coding. Use the comedi_offset_munge() helper to convert the data to the offset binary format used by comedi. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg

[PATCH 05/13] staging: comedi: mf6x4: introduce a macro to select the AI channel

For aesthetics, use a macro to set the bit in the ADCTRL register that selects an analog input channel. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

[PATCH 08/13] staging: comedi: mf6x4: rename the digital input register defines

For aesthetics, rename these defines to clarify them a bit. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/mf6x4.c | 6 +++--- 1 file changed, 3

[PATCH 07/13] staging: comedi: mf6x4: remove unnecassary masking of digital outputs

THe 's->state' will always be in range for the 8 digital outputs. Remove the unnecessary masking of the value. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

[PATCH 12/13] staging: comedi: mf6x4: change type of local variable

For aesthetics, change the type of this local variable to unsigned int. This fixes the checkpatch.pl issue about: CHECK: Prefer kernel type 'u32' over 'uint32_t' Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman

[PATCH 09/13] staging: comedi: mf6x4: rename remaining BAR1 register defines

For consistency, rename the ADSTART and DAC register defines to add a bit of clarity. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/mf6x4.c | 8 1

Re: [PATCH 2/2] staging: fbtft: fix sparse warning in call to vfree()

Could you update everything in that directory all at once instead? grep screen_base drivers/staging/fbtft/*c regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org

[PATCH 1/2] fb.h: Provide alternate screen_base pointer

Some drivers use member screen_base of struct fb_info to store non- __iomem pointers, creating the need for ugly __force typecasts to avoid sparse warnings. This adds an alternate pointer without the __iomem qualifyer for this use. Signed-off-by: Lars Svensson ---

[PATCH 2/2] staging: fbtft: fix sparse warning in call to vfree()

Member screen_base in struct fb_info is declared __iomem causing a sparse warning as below when passed to vfree(). Instead use non- __iomem pointer screen_buffer to avoid the warning. fbtft-core.c:922:39: warning: incorrect type in argument 1 \ (different address spaces) fbtft-core.c:922:39:

[RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

From: Laura Abbott This adds a base set of devicetree bindings for the Ion memory manager. This supports setting up the generic set of heaps and their properties. Signed-off-by: Laura Abbott Signed-off-by: Andrew Andrianov ---

[RFC][PATCH 0/2] Devicetree bindings for Ion

From: Laura Abbott Hi, At the last Plumbers and Linaro Connect, there was some discussion related to Ion. One of the items that came up towards eventually getting Ion out of staging was some concept of stable devicetree bindings. This is a proof of concept for bindings.

[RFC][PATCH 2/2] staging: ion: Add files for parsing the devicetree (WIP)

From: Laura Abbott Devicetree is the preferred mechanism for platform definition these days. Add a set of files for supporting Ion with devicetree. This includes a set of bindings for heaps common across all devices and parsing methods. Clients may use the standard bindings

Re: [RFC][PATCH 2/2] staging: ion: Add files for parsing the devicetree (WIP)

Hi Laura, [auto build test WARNING on v4.3-rc4 -- if it's inappropriate base, please ignore] reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >>

[RFC PATCH] staging: ion: ion_parse_dt_heap_common() can be static

Signed-off-by: Fengguang Wu --- ion_of.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/android/ion/ion_of.c index 3c9b1e5..daded0c 100644 --- a/drivers/staging/android/ion/ion_of.c +++

Re: [RFC][PATCH 2/2] staging: ion: Add files for parsing the devicetree (WIP)

Thanks a lot for starting again with dt bindings discussion. I got carried away by work and never had a chance for a respin of my ion-physmem. I'll try to test these on actual hardware next week and provide more detailed feedback. For now just a small pick: - +obj-$(CONFIG_ION_DUMMY) +=

Re: [PATCH] Staging: fbtft: Use BIT macro

> Replace (1 << x) by BIT(x) > > Signed-off-by: Ronit halder Minor nit: you stopped capitalizing your surname in your SoBs sometime between Jul 10 and Aug 19 (judging from a quick peek at git log). > diff --git a/drivers/staging/fbtft/fbtft.h

[PATCH 0/3] staging: sm750fb: refactor MXCLK frequency detection

Hi, The pacthes in this serie refactor detection of MXCLK frequency. Mike Rapoport (3): staging: sm750fb: merge calcPLL and getPllValue into getChipClock staging: sm750fb: rename getChipClock to get_mxclk_freq staging: sm750fb: remove some unnecessary castings

[PATCH 3/3] staging: sm750fb: remove some unnecessary castings

The clock divisor calculations in setMasterClock and setMemoryClock unnecessaryly cast unsigned int to unsigned int. Removing the casting. Signed-off-by: Mike Rapoport --- drivers/staging/sm750fb/ddk750_chip.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH 1/3] staging: sm750fb: merge calcPLL and getPllValue into getChipClock

The getChipClock function is used only to get MXCLK frequency, which makes most of getPllValue function unused and thus. The detection of MXCLK frequency may be implemented directly in getChipClock rendering getPllValue and calcPLL unused. Signed-off-by: Mike Rapoport

[PATCH 2/3] staging: sm750fb: rename getChipClock to get_mxclk_freq

The getChipClock is used to detect MXCLK frequency. Make it's name reflect what the function is actually doing. Signed-off-by: Mike Rapoport --- drivers/staging/sm750fb/ddk750_chip.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [PATCH 2/5] staging: fsl-mc: abstract test for whether a dprc is a root dprc

On Sun, Oct 04, 2015 at 10:09:51AM +0300, itai.k...@freescale.com wrote: > From: Itai Katz > > Instead of relying on assumptions about fields in data > structures, abstract the test for whether a dprc is a root > dprc into a function. > > Signed-off-by: Itai Katz

[PATCH 10/13] staging: comedi: mf6x4: rename private data 'gpio_R'

Rename this CamelCase member of the private data. For consistency, also rename the register defines associated with this member. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

[PATCH 13/13] staging: comedi: mf6x4: tidy up subdevice init

For aesthetics, add some whitespace to the subdevice initialization. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/mf6x4.c | 60

[PATCH 11/13] staging: comedi: mf6x4: refactor block comment

Refactor this block comment to fix the checkpatch.pl issues: WARNING: Block comments use * on subsequent lines WARNING: Block comments use a trailing */ on a separate line Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman

Re: [PATCH v2] staging: rtl8723au: Fix sparse errors in rtl8723a_cmd.c

On Wed, Oct 07, 2015 at 12:11:34AM +0100, Al Viro wrote: > On Tue, Oct 06, 2015 at 12:32:28AM -0400, Jacob Kiefer wrote: > > > int rtl8723a_set_rssi_cmd(struct rtw_adapter *padapter, u8 *param) > > { > > - *((u32 *)param) = cpu_to_le32(*((u32 *)param)); > > + __le32 leparam; > > > > -

[PATCH 01/14] staging: comedi: me_daq: tidy up private data

For aesthetics, rename the local variable used to access dev->private. In comedi drivers this variable is typically named 'devpriv'. For aesthetics, rename the private data members used to mirror the write-only register to, slightly, shorter names. Signed-off-by: H Hartley Sweeten

[PATCH 02/14] staging: comedi: me_daq: tidy up control 1 register defines

Rename the bits of this register so they have association with the register. Use the BIT macro to define the bits. Add a macro to select the ADC mode and remove the magic value used to stop conversion. Reading the control 1 register starts an analog input conversion. For aesthetics, remove the

[PATCH 04/14] staging: comedi: me_daq: tidy up status register defines

Rename the bits of this register so they have association with the register. Use the BIT macro to define the bits. Writing to the status register clears any pending interrupts. For aesthetics, remove the ME_RESET_INTERRUPT define and just use the ME_STATUS_REG define to write the register.

[PATCH 03/14] staging: comedi: me_daq: tidy up control 2 register defines

Rename the bits of this register so they have association with the register. Use the BIT macro to define the bits. Reading the control 2 register updates the DAC registers. For aesthetics, remove the MC_DAC_UPDATE define and just use the ME_CTRL2_REG define to read the register. Signed-off-by: H

[PATCH 07/14] staging: comedi: me_daq: tidy up ai fifo/chanlist register defines

Tidy up this register usage by defining some macros to set the channel and gain bits. Add a define for the bit that enables differential mode. Writing to this offset puts data in the ai chanlist fifo, reading from it gets data from the ai data fifo. For aesthetics, use the same define to read and

[PATCH 05/14] staging: comedi: me_daq: tidy up digital i/o port register defines

For aesthetics, add a _REG suffix to these defines. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/me_daq.c | 8 1 file changed, 4 insertions(+), 4

[PATCH 06/14] staging: comedi: me_daq: tidy up timer data register defines

For aesthetics, convert the register defines into a macro that takes the timer channel and returns the correct register offset. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

[PATCH 00/14] staging: comedi: me_daq: cleanup driver

Cleanup the checkpatch.pl issues and fix some minor issues in this driver. H Hartley Sweeten (14): staging: comedi: me_daq: tidy up private data staging: comedi: me_daq: tidy up control 1 register defines staging: comedi: me_daq: tidy up control 2 register defines staging: comedi: me_daq:

[PATCH 13/14] staging: comedi: me_daq: allow differential analog inputs

The ai (*insn_read) allows differential analog inputs but the subdevice initialization is missing the SDF_DIFF subdev_flag to allow the user to select this mode. Fix the subdevice init. According to the manual, differential operation only works with the first 8 channels and bipolar ranges. Add a

[PATCH 08/14] staging: comedi: me_daq: tidy up dac control register defines

Redefine the dac control register bits as macros that take the channel number and return the correct bit values. This register needs to be read after the new values are written in order to actually update the dac. For aesthetics, use the same define to read and write the registers.

[PATCH 10/14] staging: comedi: me_daq: use comedi_range_is_bipolar() helper

Use the helper function to determine if the dac bipolar bit needs to be set. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/me_daq.c | 6 ++ 1 file

[PATCH 12/14] staging: comedi: me_daq: tidy up counter registers

For aesthetics, convert the counter registers into macros that take the counter channel and return the correct register offset. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

[PATCH 09/14] staging: comedi: me_daq: remove useless for() loop

This for() loop isn't needed. Nothing in the loop uses the 'i' variable. Remove it. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/me_daq.c | 15

[PATCH 14/14] staging: comedi: me_daq: fix me_ai_insn_read()

The comedi core expects (*insn_read) functions to return insn->n data values. Refactor the function to work like the core expects. For aesthetics, use the comedi_offset_munge() helper to munge the 2's complement data. This also fixes a minor issue when comedi_timeout() times out. Currently this

[PATCH] staging: comedi: ke_counter: prefer using the BIT macro

Use a macro to define the clock source options. This fixes the checkpatch.pl issue about: CHECK: Prefer using the BIT macro Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

[PATCH 03/10] staging: wilc1000: rename the member variable, enuWIDtype of wid

From: Leo Kim This patch renames enuWIDtype of struct wid to type. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.h | 2 +- drivers/staging/wilc1000/host_interface.c | 186

[PATCH 05/10] staging: wilc1000: rename the member variable, s32ValueSize of wid

From: Leo Kim This patch renames s32ValueSize of struct wid to size. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 6 +- drivers/staging/wilc1000/coreconfigurator.h | 2 +-

[PATCH 01/10] staging: wilc1000: remove typedef from tstrWID

From: Leo Kim This patch removes typedef from the struct tstrWID and rename it to wid. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 2 +- drivers/staging/wilc1000/coreconfigurator.h

[PATCH 04/10] staging: wilc1000: rename the member variable, u16WIDid of wid

From: Leo Kim This patch renames u16WIDid of struct wid to id. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 8 +- drivers/staging/wilc1000/coreconfigurator.h | 2 +-

[PATCH 06/10] staging: wilc1000: rename the member variable, ps8WidVal of wid

From: Leo Kim This patch renames ps8WidVal of struct wid to val. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.c | 4 +- drivers/staging/wilc1000/coreconfigurator.h | 2 +-

[PATCH 07/10] staging: wilc1000: remove unused struct tstrWidJoinReqExt

From: Leo Kim This patch removes struct tstrWidJoinReqExt because it's not used. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

Re: [PATCH v3 00/24] staging: comedi: quatech_daqp_cs: cleanup driver

On 05/10/15 22:22, H Hartley Sweeten wrote: Following is the cleanup series for the comedi quatech_daqp_cs PCMCIA driver. v2: fix patch 08/17 as pointed out by the kbuild test robot daqp_ai_get_sample() should be static v3: drop the patch that changed the ai cmd timing (patch 03/17)

[PATCH 02/10] staging: wilc1000: remove typedef from tenuWIDtype

From: Leo Kim This patch remove typedef from the enum tenuWIDtype and rename it to WID_TYPE. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/coreconfigurator.h | 2 +-

[PATCH 10/10] staging: wilc1000: remove _tstrJoinParam from join_bss_param

From: Leo Kim This patch removes struct _tstrJoinParam from the struct join_bss_param because it is not used inside the struct. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 1 - 1 file

[PATCH 08/10] staging: wilc1000: remove unused struct tstrBssTable

From: Leo Kim This patch removes struct tstrBssTable with related comment. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 -- 1 file changed, 6 deletions(-) diff --git

Re: [RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

On 10/6/15 3:35 PM, Rob Herring wrote: On Tue, Oct 6, 2015 at 3:47 PM, Laura Abbott wrote: From: Laura Abbott This adds a base set of devicetree bindings for the Ion memory manager. This supports setting up the generic set of heaps and their

[PATCH 1/1] scsi: storvsc: Fix a bug in the handling of SRB status flags

SRB status can have additional information. Mask these out before processing SRB status. This patch was sent as part of a collection of patches more than a year ago. While the rest of the patches in the set were comitted, this patch was not. I woulod like to thank Olaf for noticing that this

Re: [PATCH v2] staging: rtl8723au: Fix sparse errors in rtl8723a_cmd.c

On Tue, Oct 06, 2015 at 12:32:28AM -0400, Jacob Kiefer wrote: > int rtl8723a_set_rssi_cmd(struct rtw_adapter *padapter, u8 *param) > { > - *((u32 *)param) = cpu_to_le32(*((u32 *)param)); > + __le32 leparam; > > - FillH2CCmd(padapter, RSSI_SETTING_EID, 3, param); > + leparam =

Re: [RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

On Tue, Oct 6, 2015 at 3:47 PM, Laura Abbott wrote: > From: Laura Abbott > > > This adds a base set of devicetree bindings for the Ion memory > manager. This supports setting up the generic set of heaps and > their properties. > > Signed-off-by:

Re: [PATCH v2 13/14] staging: comedi: multiq3: add 8254 counter/timer subdevice support

On 05/10/15 23:33, H Hartley Sweeten wrote: The board has an 8254 timer/counter. Add support for this subdevice. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

RE: [PATCH 2/5] staging: fsl-mc: abstract test for whether a dprc is a root dprc

> -Original Message- > From: Mike Rapoport [mailto:mike.rapop...@gmail.com] > Sent: Tuesday, October 06, 2015 11:48 AM > To: katz Itai-RM05202 > Cc: gre...@linuxfoundation.org; a...@arndb.de; > de...@driverdev.osuosl.org; linux-ker...@vger.kernel.org; Yoder

Re: [PATCH] staging: comedi: pcl711: prefer using the BIT marco

On 05/10/15 23:49, H Hartley Sweeten wrote: Fix the checkpatch.pl issues. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcl711.c | 19 ++-

Re: [PATCH v2 13/14] staging: comedi: multiq3: add 8254 counter/timer subdevice support

On 06/10/15 12:06, Ian Abbott wrote: On 05/10/15 23:33, H Hartley Sweeten wrote: The board has an 8254 timer/counter. Add support for this subdevice. According to the manual I linked to, sections 3.2, 3.2.5.1 and 3.2.5.2 indicate that CLK_DATA (your MULTIQ3_CLK_REG) is write-only. This

Re: [PATCH 0/3] staging: comedi: pcl816: checkpatch.pl issues

On 05/10/15 23:44, H Hartley Sweeten wrote: Some minor cleanup to fix the checkpatch.pl issues. H Hartley Sweeten (3): staging: comedi: pcl816: tidy up multi-line comments staging: comedi: pcl816: prefer using the BIT macro staging: comedi: pcl816: remove #if 0'ed out code

Re: [PATCH 0/3] staging: comedi: pcl812: fix checkpatch.pl issues

On 05/10/15 23:47, H Hartley Sweeten wrote: Some minor cleanup to fix the checkpatch.pl issues. H Hartley Sweeten (3): staging: comedi: pcl812: convert hardware type defines into an enum staging: comedi: pcl812: prefer using the BIT macro staging: comedi: pcl812: rename private data

Re: [PATCH 0/9] staging: comedi: mpc624: cleanup driver

On 06/10/15 01:33, H Hartley Sweeten wrote: Fix the checkpatch.pl issues and tidy up this driver a bit. H Hartley Sweeten (9): staging: comedi: mpc624: tidy up multi-line comments staging: comedi: mpc624: tidy up the ai conversion speed code staging: comedi: mpc624: update

Re: [PATCH 0/2] staging: comedi: pcl818: fix checkpatch.pl issues

On 05/10/15 23:40, H Hartley Sweeten wrote: Some minor cleanup to fix the checkpatch.pl issues. H Hartley Sweeten (2): staging: comedi: pcl818: prefer using the BIT macro staging: comedi: pcl818: remove unused "boards constants" drivers/staging/comedi/drivers/pcl818.c | 40

Re: [PATCH v2 00/14] staging: comedi: multiq3: cleanup driver

On 05/10/15 23:33, H Hartley Sweeten wrote: Cleanup this driver a bit and add support for the 8254 counter/timer. v2: Fix patch 11/14, Ian Abbott pointed out that the odd munging of the encoder data does make sense. Add a comment about it. H Hartley Sweeten (14): staging: comedi:

RE: [PATCH v2 13/14] staging: comedi: multiq3: add 8254 counter/timer subdevice support

On Tuesday, October 06, 2015 4:06 AM, Ian Abbott wrote: > On 05/10/15 23:33, H Hartley Sweeten wrote: >> The board has an 8254 timer/counter. Add support for this subdevice. >> >> Signed-off-by: H Hartley Sweeten >> Cc: Ian Abbott >> Cc: Greg