From: Andrey Smetanin
Before vmbus_connect() synic is setup per vcpu - this means
hypervisor receives writes at synic msr's and probably allocate
hypervisor resources per synic setup.
If vmbus_connect() failed for some reason it's neccessary to cleanup
synic setup by call hv_synic_cleanup() at e
drivers/staging/wilc1000/host_interface.c:2852:15-22: WARNING: kzalloc should
be used for pu8keybuf, instead of kmalloc/memset
drivers/staging/wilc1000/host_interface.c:2904:15-22: WARNING: kzalloc should
be used for pu8keybuf, instead of kmalloc/memset
drivers/staging/wilc1000/host_interface.c:6
On Wed, Oct 07, 2015 at 07:08:25AM +0200, Vincent Stehlé wrote:
> drivers/staging/wilc1000/host_interface.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/wilc1000/host_interface.c
> b/drivers/staging/wilc1000/host_interface.c
> index 2cf82b2..2fe3023 10
The change to fbtft_write_vmem16_bus9() is a bugfix, which is great, but
it's not mentioned in the changelog. Really it should be a separate
commit.
Fixes: 02eb884f4e26 ('drivers: staging: fbtft: fbtft-bus.c: Fix different
address space warning on I/O mem')
So it looks like this:
[patch 1] fb.h
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
---
include/linux/fb.h | 5 -
1
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
---
v2: removed all references to scre
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
---
drivers/staging/wilc1000/coreconfigurato
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 wer
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;
> >
> > - Fil
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
---
drivers/staging/comedi/drivers/ke_counter.c | 7 ---
1 file changed, 4 insertions(+), 3 de
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 fu
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 ++-
1 file changed, 6 insertions(+), 9 deletions(-)
diff --git a/drivers/sta
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 changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/comedi/drive
For aesthetics, use a macro that takes the analog output channel
and returns the correct offset for the data register.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/drivers/me_daq.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
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 c
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.
Signed-off-by:
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
---
drivers/staging/comedi/drivers/me_daq.c | 9 +++--
1 file changed, 3 insertions(+), 6 d
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
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.
Signed
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
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 deletions(-)
diff --git a/drivers/staging/comedi/drivers/me_daq.c
b/drivers/
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
---
drivers/staging/comedi/drivers/me_daq.c | 4 +---
1 file changed, 1 insertion(+), 3 deletio
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:
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 MC
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
Cc: Ian Abbott
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 = cp
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 patc
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 properties.
Signed-off-by: Laura Abbott
Signed-o
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: Laura Abbott
> Signed-off-by: Andrew Andrianov
>
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) += i
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 >>)
>> drivers/staging/android/ion/ion_o
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
+++ b/drivers/staging/android/
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.
Most of this is based o
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
---
drivers/staging/android/ion/devicetree.txt | 53 +
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 or they can call the
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
---
drivers/staging/comedi/drivers/mf6x4.c | 16
1 file changed, 8 insertion
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 +-
1 file changed, 30 insertions(+), 30 deletions(-)
diff --git a/driver
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
---
drivers/staging/comedi/drivers/mf6x4.c | 4 ++--
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
---
drivers/staging/comedi/drivers/mf6x4.c | 2 +-
1 file ch
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 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/comedi/drivers/mf6x4.c
b/driv
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
---
drivers/staging/comedi/drivers/mf6x4.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --gi
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 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/co
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 Kroah-Hartman
---
drivers/staging/comedi/drivers/mf6x4.c |
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
---
drivers/staging/comedi/drivers/mf6x4.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
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 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers
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 deletions(-)
diff --git a/drivers/staging/comedi/drivers/mf6x4.c
b/drivers/st
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
---
drivers/staging/comedi/drivers/mf6x4.c | 9 -
1 file changed, 9 deletions(-)
diff --git a/d
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 regis
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 a/drivers/staging/comedi/drivers/mf6x4.c
b/drivers/staging/comedi/drivers/mf6
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: multiq
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 Kroah-Hartman
>> ---
>> drivers/staging/comedi/driv
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 -
drivers/staging/sm750fb/ddk750_sii164.c | 4
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
b/drivers/staging/sm750fb/ddk750_disp
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
stagin
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
---
drivers/staging/sm750fb/
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
> ---
> drivers/staging/wilc1000/coreconfigurator.h | 2 +-
> drivers/staging/wil
> -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 Stuart-
> B08248 ; Sharma Bhupe
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 MODULE_DESCRI
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 ++-
1 file changed, 10 insertions(+), 9 deletions(-)
diff --git a/drivers/stag
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 'IR
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
drivers/st
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 +
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 means
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
---
drivers/staging/comedi/drivers/multiq3.c | 147 ++-
1 file changed, 146 in
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 changed, 1 deletion(-)
diff --git a/drivers/staging/wilc1
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 +-
drivers/staging/wilc1000/host_interface.c | 278 ++-
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 a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wilc100
From: Leo Kim
This patch removes typedef from the struct tstrJoinBssParam
and renames it to join_bss_param.
Signed-off-by: Leo Kim
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/host_interface.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drive
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 a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/w
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 +-
drivers/staging/wilc1000/host_interface.c | 188 ++---
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 | 7 +--
drivers/staging/wilc1000/host_interface.c | 92
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 +-
drivers/staging/wilc1000/host_interface.c | 210 ++-
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 ++--
2 files changed, 94 insertions(+), 94 del
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 +-
drivers/staging/wilc1000/host_interface.h | 2 +-
drivers/staging/wilc1000/wilc_wlan_if.h |
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)
r
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
> ---
> drivers/staging/fsl-mc
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(-)
diff --git a/drivers/stagin
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
---
drivers/staging/sm750fb
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 a/drivers/staging/sm750fb/ddk750_chip.c
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
drivers/staging/sm750fb/ddk7
> 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 b/drivers/staging/fbtft/fbtft.h
> index 6dd42b2..375
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
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-d
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:ex
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
---
New version of my previous patch
84 matches
Mail list logo
