- Original Message -
> From: "Takashi Iwai"
> To: "Jerome Marchand"
> Cc: "Jaroslav Kysela" , alsa-de...@alsa-project.org,
> linux-kernel@vger.kernel.org
> Sent: Tuesday, February 9, 2016 10:56:39 PM
> Subject: Re: Should snd_card_free() check for null pointer?
>
> On Tue, 09 Feb 2016
On Wed, Feb 10, 2016 at 08:41:17AM +0200, Andy Shevchenko wrote:
> On Tue, Feb 9, 2016 at 10:11 PM, Rasmus Villemoes
> wrote:
> > The current code fails to call i2c_del_adapter on
> > dev_prev->gmbus[0].adapter, and if the for loop above failed already
> > at i==0, all hell breaks loose when we
On 02/09/2016 10:04 PM, Bjorn Helgaas wrote:
> On Wed, Jan 13, 2016 at 12:25:34PM +0100, Hannes Reinecke wrote:
>> PCI-2.2 VPD entries have a maximum size of 32k, but might actually
>> be smaller than that. To figure out the actual size one has to read
>> the VPD area until the 'end marker' is
On Tue, Feb 09, 2016 at 06:10:40PM +0100, Maxime Ripard wrote:
>>It seems that the other compatible strings are there for historical
>>reasons. Why do you need a new one with such a specific name?
>>
>>It would have been more sensible to add a generic compatible string as
On 02/09/2016 09:53 PM, Bjorn Helgaas wrote:
> Hi Hannes,
>
> On Wed, Jan 13, 2016 at 12:25:33PM +0100, Hannes Reinecke wrote:
>> It is not always possible to determine the actual size of the VPD
>> data, so allow access to them if the size is set to '0'.
>>
>> Signed-off-by: Hannes Reinecke
>>
PAGE_EXEC is required for inittext, otherwise CONFIG_DEBUG_PAGEALLOC
ends up with an Oops
[0.00] Inode-cache hash table entries: 8192 (order: 1, 32768 bytes)
[0.00] Sorting __ex_table...
[0.00] bootmem::free_all_bootmem_core nid=0 start=0 end=2000
[0.00] Unable to
The cpufreq core now guarantees that policy->rwsem won't be dropped
while running the ->governor callback for the CPUFREQ_GOV_POLICY_EXIT
event and will be held acquired until the complete sequence of governor
state changes has finished.
This allows governor state machine checks to be dropped
Ondemand governor already updates sample_delay_ns immediately on updates
to sampling rate, but conservative isn't doing that.
It was left out earlier as the code has been really complex to get that
done easily. But now things are sorted out very well, and we can follow
the same for conservative
'delay' is updated properly in all paths of the routine od_dbs_timer(),
leaving just one. And can be 0 only in that case.
Move the update to 'delay' as an else part of the if block.
Signed-off-by: Viresh Kumar
---
drivers/cpufreq/cpufreq_ondemand.c | 9 -
1 file changed, 4
Please check. It looks quite possible that something else was intended.
julia
-
Hi Andrew,
[auto build test WARNING on input/next]
[also build test WARNING on v4.5-rc3 next-20160209]
[if your patch is applied to the wrong git tree, please drop us a note
From: Geert Uytterhoeven
The HiSilicon Hi6220 USB PHY is available in HiSilicon Hi6220 SoCs only.
Restrict it to HiSilicon arm64, unless compile-testing.
Signed-off-by: Geert Uytterhoeven
Signed-off-by: Kishon Vijay Abraham I
---
drivers/phy/Kconfig |1 +
1 file changed, 1 insertion(+)
Hi Greg,
Please find the pull request for this -rc cycle below. It contains a fix in
phy core, twl4030-usb phy driver and a fix in Kconfig.
Consider merging it to this -rc cycle. Let me know if you want me to change
anything.
Cheers
Kishon
The following changes since commit
From: Tony Lindgren
If we reload phy-twl4030-usb, we get a warning about unbalanced
pm_runtime_enable. Let's fix the issue and also fix idling of the
device on unload before we attempt to shut it down.
If we don't properly idle the PHY before shutting it down on removal,
the twl4030 ends up
From: Tony Lindgren
Otherwise rmmod omap2430; rmmod phy-twl4030-usb; modprobe omap2430
will try to use a non-existing phy and oops:
Unable to handle kernel paging request at virtual address b6f7c1f0
...
[] (devm_usb_get_phy_by_node) from []
(omap2430_musb_init+0x44/0x2b4 [omap2430])
[]
From: Shawn Lin
If phy_pm_runtime_get_sync failed but we already
enable regulator, current code return directly without
doing regulator_disable. This patch fix this problem
and cleanup err handle of phy_power_on to be more readable.
Fixes: 3be88125d85d ("phy: core: Support regulator ...")
Cc:
On Tue, Feb 9, 2016 at 10:11 PM, Rasmus Villemoes
wrote:
> The current code fails to call i2c_del_adapter on
> dev_prev->gmbus[0].adapter, and if the for loop above failed already
> at i==0, all hell breaks loose when we do the loop body for
> i = -1,-2,...
>
Reviewed-by: Andy Shevchenko
>
On Tue, Feb 9, 2016 at 2:10 PM, Mathieu OTHACEHE wrote:
>> I'm sorry, but it looks like 8250 based driver if I'm not mistaken. In
>> which case why not to use 8250_core.c / 8250_port.c and entire 8250/
>> infrastructure?
>
> Well, the vendor is providing two drivers for his serial pci cards :
^
>
> Caused by commit
>
> 2413306c2566 ("apple-gmux: Add helper for presence detect")
>
> I have used the drm-misc tree from next-20160209 for today.
Can you pls attach your .config? The function is there, I suspect we're
just missing some d
On 02/09/2016 08:44 PM, Maciej W. Rozycki wrote:
> On Tue, 9 Feb 2016, Daniel Wagner wrote:
>
>>> FWIW I think all the MIPS ABI flags stuff also needs to go outside the
>>> conditional, because it's ABI agnostic. I'll make the right change myself
>>> on top of your fixes. It'll remove a
We select COMMON_CLK_IPROC, PINCTRL, and GPIOLIB in arm64 Kconfig
for ARCH_BCM_IPROC so that we can use COMMON_CLK, PINCTRL and GPIOLIB
with iProc SoC drivers.
Signed-off-by: Anup Patel
Signed-off-by: Ray Jui
Signed-off-by: Yendapally Reddy Dhananjaya Reddy
---
arch/arm64/Kconfig.platforms |
Hi all,
Changes since 20160209:
The xfs tree lost its build failure.
The drm-misc tree gained a build failure so I used the version from
next-20160209.
The aio tree still had a build failure so I used the version from
next-20160111.
The akpm-current tree gained a conflict against the xfs tree
Hi Aneesh,
[auto build test ERROR on v4.5-rc3]
[also build test ERROR on next-20160209]
[cannot apply to arc/for-next]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Aneesh-Kumar-K-V/mm-thp
We have one ARM SP805 watchdog instance on NS2 for non-secure software
hence this patch adds appropriate watchdog DT node in NS2 DT.
Signed-off-by: Anup Patel
Reviewed-by: Ray Jui
Reviewed-by: Pramod KUMAR
Reviewed-by: Scott Branden
---
arch/arm64/boot/dts/broadcom/ns2.dtsi | 8
1
The IPROC SDHCI driver works fine for SDIO 3.0 on NS2 so let's enable
it for NS2 SoC in NS2 DT.
Signed-off-by: Anup Patel
Reviewed-by: Vikram Prakash
Reviewed-by: Ray Jui
---
arch/arm64/boot/dts/broadcom/ns2-svk.dts | 4
arch/arm64/boot/dts/broadcom/ns2.dtsi| 18 ++
This patchset primarily adds more DT nodes for NS2 SVK. It also does
minor update to arch/arm64/Kconfig.platforms and adds missing DT
bindings document for sp805 driver.
The patchset is based on v4.5-rc3 tag and is available in ns2_dt2_v1
branch of https://github.com/Broadcom/arm64-linux.git
All
From: Ray Jui
This patch enables PCIe0 and PCIe4 for NS2 by adding
appropriate DT nodes in NS2 DT.
Signed-off-by: Ray Jui
Signed-off-by: Anup Patel
Reviewed-by: Scott Branden
---
arch/arm64/boot/dts/broadcom/ns2-svk.dts | 8
arch/arm64/boot/dts/broadcom/ns2.dtsi| 74
The ARM SP805 DT node is already present in various DTS files.
This patch adds missing DT bindings documentation for ARM SP805.
Signed-off-by: Anup Patel
Reviewed-by: Ray Jui
Reviewed-by: Vikram Prakash
Reviewed-by: Scott Branden
---
.../devicetree/bindings/watchdog/arm,sp805.txt |
We have four ARM SP804 dual-mode timer instances in NS2 SoC
hence this patch adds appropriate DT nodes for NS2.
Signed-off-by: Anup Patel
Reviewed-by: Ray Jui
Reviewed-by: Pramod KUMAR
Reviewed-by: Scott Branden
---
arch/arm64/boot/dts/broadcom/ns2.dtsi | 40
> On Tue, Feb 09, 2016 at 04:11:56PM +0530, Bharat Kumar Gogada wrote:
> > Modifying Xilinx AXI PCIe Host Bridge Soft IP driver to work on both
> > Zynq and Microblaze Architectures.
> > With these modifications drivers/pci/host/pcie-xilinx.c, will work on
> > both Zynq and Microblaze
Hi Andrew,
After merging the akpm-current tree, today's linux-next build (x86_64
allmodconfig) failed like this:
In file included from arch/x86/include/asm/bug.h:35:0,
from include/linux/bug.h:4,
from include/linux/thread_info.h:11,
from
We remove one instace of flush_tlb_range here. That was added by
f714f4f20e59ea6eea264a86b9a51fd51b88fc54 ("mm: numa: call MMU notifiers
on THP migration"). But the pmdp_huge_clear_flush_notify should have
done the require flush for us. Hence remove the extra flush.
Signed-off-by: Aneesh Kumar
On Wed, Feb 10, 2016 at 10:45:14AM +0530, Gautham R Shenoy wrote:
> Hello Rafael,
>
> On Fri, Feb 05, 2016 at 03:11:54AM +0100, Rafael J. Wysocki wrote:
> [..snip..]
> > Index: linux-pm/drivers/cpufreq/cpufreq_performance.c
> > ===
>
Hi,
On Wednesday 10 February 2016 07:12 AM, Suman Anna wrote:
> Hi Paul,
>
> On 02/09/2016 01:36 PM, Paul Walmsley wrote:
>> Hi Suman
>>
>> On Tue, 9 Feb 2016, Suman Anna wrote:
>>
>>> On 02/09/2016 02:49 AM, Paul Walmsley wrote:
On Mon, 8 Feb 2016, Suman Anna wrote:
> On 02/07/2016
On Wednesday 10 February 2016 01:06 AM, Paul Walmsley wrote:
> Hi Suman
>
> On Tue, 9 Feb 2016, Suman Anna wrote:
>
>> On 02/09/2016 02:49 AM, Paul Walmsley wrote:
>>> On Mon, 8 Feb 2016, Suman Anna wrote:
On 02/07/2016 08:48 PM, Paul Walmsley wrote:
> On Tue, 2 Feb 2016, Kishon
On 10-02-16, 01:36, Rafael J. Wysocki wrote:
> > static int cpufreq_governor_stop(struct cpufreq_policy *policy)
> > {
> > - struct policy_dbs_info *policy_dbs = policy->governor_data;
> > -
> > - /* State should be equivalent to START */
> > - if (!policy_dbs->policy)
> > -
The dbs_data_mutex lock is currently used in two places. First,
cpufreq_governor_dbs() uses it to guarantee mutual exclusion between
invocations of governor operations from the core. Second, it is used by
ondemand governor's update_sampling_rate() to ensure the stability of
data structures
On Mon, Feb 08, 2016 at 05:31:23PM -0800, Shaun Ren wrote:
> This patch removes all unnecessary parentheses found by checkpatch.pl.
>
> Signed-off-by: Shaun Ren
This will not apply anymore because of some other changes done by:
9a66d05d82db ("Staging: rts5208: fix check for dma mapping error")
Add "snps,quirk-frame-length-adjustment" property to USB3 node for erratum
A009116.
This property provides value of GFLADJ_30MHZ for post silicon frame length
adjustment.
Signed-off-by: Rajesh Bhagat
---
arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 3 +++
1 file changed, 3 insertions(+)
Hi Andrew,
Today's linux-next merge of the akpm-current tree got a conflict in:
fs/ocfs2/aops.c
between commit:
187372a3b9fa ("direct-io: always call ->end_io if non-NULL")
from the xfs tree and commit:
8e1fbb488a18 ("ocfs2: fix sparse file & data ordering issue in direct io")
from
On Mon, Feb 08, 2016 at 05:31:18PM -0800, Shaun Ren wrote:
> This patch fixes the alignment issue reported by checkpatch.pl:
>
> CHECK: Alignment should match open parenthesis
>
> Signed-off-by: Shaun Ren
> ---
> drivers/staging/rts5208/rtsx_transport.c | 61
> ++--
Hello Rafael,
On Fri, Feb 05, 2016 at 03:11:54AM +0100, Rafael J. Wysocki wrote:
[..snip..]
> Index: linux-pm/drivers/cpufreq/cpufreq_performance.c
> ===
> --- linux-pm.orig/drivers/cpufreq/cpufreq_performance.c
> +++
On 09-02-16, 21:23, Rafael J. Wysocki wrote:
> > + count = dbs_data->usage_count--;
>
> This appears to be planting a bug.
>
> The way you wrote it the decrementation will take place after the assignment,
> so
> count will contain the old value.
Oh crap, what was I smoking :)
--
viresh
On Mon, Feb 08, 2016 at 10:46:57PM +0530, Sudip Mukherjee wrote:
> On Sun, Feb 07, 2016 at 02:04:21PM -0800, Greg Kroah-Hartman wrote:
> > On Tue, Dec 01, 2015 at 04:22:33PM +0530, Sudip Mukherjee wrote:
> > > The success path and the error path both are first doing
> > > spin_unlock_irqrestore()
Kernel modules are partially linked object files with some undefined
symbols that are expected to be matched with EXPORT_SYMBOL() entries
from elsewhere.
Each .tmp_versions/*.mod file currently contains two line of text
separated by a newline character. The first line has the actual module
file
Given the list of exported symbols needed by all modules, we can create
a header file containing preprocessor defines for each of those symbols.
Also, when some symbols are added and/or removed from the list, we can
update the time on the corresponding files used as build dependencies for
those
Like for kconfig options, we now have the ability to compile in and
out individual EXPORT_SYMBOL() declarations based on the content of
include/generated/autoksyms.h. However we don't want the entire
world to be rebuilt whenever that file is touched.
Let's apply the same build dependency trick
Similar to include/generated/autoconf.h, include/generated/autoksyms.h
will contain a list of defines for each EXPORT_SYMBOL() that we want
active. The format is:
#define __KSYM_ 1
This list will be auto-generated with another patch. For now we only
include the preprocessor magic to
Simple code reorganization to make coming changes more obvious.
Signed-off-by: Nicolas Pitre
Acked-by: Rusty Russell
---
scripts/basic/fixdep.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/scripts/basic/fixdep.c b/scripts/basic/fixdep.c
index
This patch series provides the option to omit exported symbols from
the kernel and modules that are never referenced by any of the selected
modules in the current kernel configuration.
The original cover letter with lots of test results can be found here:
The config option to enable it all.
Signed-off-by: Nicolas Pitre
Acked-by: Rusty Russell
---
init/Kconfig | 16
1 file changed, 16 insertions(+)
diff --git a/init/Kconfig b/init/Kconfig
index 22320804fb..e6f666331b 100644
--- a/init/Kconfig
+++ b/init/Kconfig
@@ -1990,6
From: Arve Hjønnevåg
Make sure offsets don't point to overlapping flat_binder_object
structs.
Cc: Colin Cross
Cc: Arve Hjønnevåg
Cc: Dmitry Shmidt
Cc: Rom Lemarchand
Cc: Serban Constantinescu
Cc: Greg Kroah-Hartman
Cc: Android Kernel Team
Signed-off-by: Dmitry Shmidt
Signed-off-by: John
Vineet Gupta writes:
> This showed up on ARC when running LMBench bw_mem tests as
> Overlapping TLB Machine Check Exception triggered due to STLB entry
> (2M pages) overlapping some NTLB entry (regular 8K page).
>
> bw_mem 2m touches a large chunk of vaddr creating NTLB entries.
> In the interim
I noticed there were a few changes from the AOSP side to binder,
so I wanted to send them out to resync w/ upstream.
These have been tested on the HiKey board with the rest of the
android-4.4 kernel changes.
Let me know if there are any concerns or objections.
thanks
-john
Cc: Colin Cross
Cc:
From: Riley Andrews
If a /d/binder/proc/[pid] entry is kept open after linux has
torn down the associated process, binder_proc_show can deference
an invalid binder_proc that has been stashed in the debugfs
inode. Validate that the binder_proc ptr passed into binder_proc_show
has not been freed
> -Original Message-
> From: Dmitry Torokhov [mailto:dmitry.torok...@gmail.com]
> Sent: 09 February 2016 23:14
> To: Raveendra Padasalagi
> Cc: Arnd Bergmann; Russell King; devicet...@vger.kernel.org; linux-arm-
> ker...@lists.infradead.org; linux-in...@vger.kernel.org; Rob Herring;
Pawel
On Tue, 2016-02-09 at 17:41 -0800, Stephen Hemminger wrote:
> On Tue, 9 Feb 2016 18:51:35 -0500
> Jarod Wilson wrote:
>
> > On Tue, Feb 09, 2016 at 11:17:57AM -0800, Stephen Hemminger wrote:
> > > Support for the new rx_nohandler statistic.
> > > This code is designed to handle the case where
I checked locally in BDC,
Seems we don't have these boards available with us (in working state).
Regards
Shashank
-Original Message-
From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
Sent: Tuesday, February 09, 2016 11:53 PM
To: Sharma, Shashank
Cc: Nick Bowler; Daniel Vetter;
32 bit systems using 'struct timeval' will break in the year 2038, so
we replace the code appropriately. However, this driver is not broken
in 2038 since we are only using microseconds portion of the time.
This patch replaces 'struct timeval' with 'struct timespec64'. We only
need to find elapsed
This patch replaces timeval with timespec64 as 32 bit 'struct timeval'
will not give current time beyond year 2038.
The patch changes the code to use ktime_get_real_ts64() which returns
a 'struct timespec64' instead of do_gettimeofday() which returns a
'struct timeval'
This patch also alters the
Hi Guo,
I have an x86 computer with this network card:
02:00.0 Ethernet controller: JMicron Technology Corp. JMC260 PCI
Express Fast Ethernet Controller (rev 03)
Every time I initiate a suspend (systemctl suspend) the machine hangs
at resume unless I unload the jme driver.
Here is a Call Trace
Hi Duc,
[auto build test ERROR on pci/next]
[also build test ERROR on v4.5-rc3 next-20160209]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Duc-Dang/pci-xgene-Add-ECAM-fixups/20160210-095702
On Tue, Feb 9, 2016 at 8:40 AM, Mathieu Poirier
wrote:
> On 8 February 2016 at 15:04, Duc Dang wrote:
>> X-Gene mailbox controller provides 8 mailbox channels, with
>> each channel has a dedicated interrupt line.
>>
>> Signed-off-by: Feng Kan
>> Signed-off-by: Duc Dang
>> ---
>> Changes since
Hi James,
As we discussed I'm sending now a revised pull request with a couple
of new fixes to the one that I sent with the tag:
tags/tpmdd-next-20160120
/Jarkko
The following changes since commit 388f7b1d6e8ca06762e2454d28d6c3c55ad0fe95:
Linux 4.5-rc3 (2016-02-07 15:38:30 -0800)
are
On Wednesday 10 February 2016 03:23 AM, Eduardo Valentin wrote:
On Tue, Feb 09, 2016 at 09:07:27AM +0530, Keerthy wrote:
On Tuesday 09 February 2016 01:27 AM, Tony Lindgren wrote:
* Keerthy [160208 01:19]:
OMAP5 has 3 thermal zones cpu, core and multimedia.
On the other hand DRA7 has 5
On Tue, Feb 2, 2016 at 7:15 PM, Maxime Ripard
wrote:
> Hi Chen-Yu
>
> On Mon, Feb 01, 2016 at 10:42:48PM +0800, Chen-Yu Tsai wrote:
>> A80's APBS clock is not the same as the APB0 clock on A23. The A80's
>> is a zero-based divider, while the A23's is a power-of-two divider.
>>
>> Signed-off-by:
>From f1afe08fbc9797ff63adf03efe564a807a37cfe6 Mon Sep 17 00:00:00 2001
From: Sukadev Bhattiprolu
Date: Tue, 9 Feb 2016 02:47:45 -0500
Subject: [PATCH V2 1/1] powerpc/perf/hv-gpci: Increase request buffer size
The GPCI hcall allows for a 4K buffer but we limit the buffer to 1K.
The problem with
Hi Shaun.
These comments now reflect common kernel style.
Thanks for addressing my comments.
Just one minor note below. It seems the word SCSI
SCSI got duplicated in one comment. (Noted below).
The rest look great to me.
On Tuesday, February 09, 2016 06:45:20 PM Shaun Ren wrote:
> This patch
Michael Ellerman [m...@ellerman.id.au] wrote:
> Here you read from bytes[i] where i can be > 1 (AFAICS).
Yes, buffer is large enough and I thought this construct of
array was used in a several places. Maybe they are being
changed out now (struct pid has one such usage).
>
> That's fishy at
Hi,
El 09/02/16 a las 19:48, Rhyland Klein escribió:
> When clocks are registered, they could be enabled already in
> hardware. As of now, the enable count will start at 0. When this
> happens, it means a clock is enabled and the framework doesn't know
> that, so it will always report it as
On Wed, Feb 10, 2016 at 2:57 AM, Rafael J. Wysocki wrote:
> On Wed, Feb 10, 2016 at 2:02 AM, Steve Muckle wrote:
>> On 02/09/2016 12:05 PM, Rafael J. Wysocki wrote:
> One concern I had was, given that the lone scheduler update hook is in
> CFS, is it possible for governor updates to be
On Friday 05 February 2016 09:40 PM, a...@redhat.com wrote:
> Em Fri, Feb 05, 2016 at 11:18:52AM +, Noam Camus escreveu:
>> Well here for EZchip I also see the:
>> undefined reference to `__sync_add_and_fetch_4'
>> undefined reference to `__sync_sub_and_fetch_4'
>
> Yeah, because there is no:
On Feb 9, 2016 6:24 AM, "Ingo Molnar" wrote:
>
>
> * Matthew Wilcox wrote:
>
> > > I sure hope not. If vm_page_prot was writable, something was already
> > > broken,
> > > because this is the vvar mapping, and the vvar mapping is VM_READ (and not
> > > even VM_MAYREAD).
> >
> > I do beg yor
On Tue, 2016-02-09 at 18:45 -0800, Shaun Ren wrote:
> This patch fixes the alignment issue reported by checkpatch.pl:
>
> CHECK: Alignment should match open parenthesis
>
> Signed-off-by: Shaun Ren
> ---
> drivers/staging/rts5208/rtsx_transport.c | 61
> ++--
> 1
This patch removes all spaces after casts in rtsx_transport.c, as reported
by checkpatch.pl:
CHECK: No space is necessary after a cast
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
This patch fixes the following styling issue in rtsx_transport.c
as reported by checkpatch.pl:
CHECK: spaces preferred around that '-' (ctx:VxV)
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
This patch changes all comparsions to NULL with !..., as reported by
checkpatch.pl.
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx_transport.c
This patch fixes the alignment issue reported by checkpatch.pl:
CHECK: Alignment should match open parenthesis
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 61 ++--
1 file changed, 35 insertions(+), 26 deletions(-)
diff --git
This patch fixes the following naming convention issue in rtsx_transport.c,
as reported by checkpatch.pl:
CHECK: Avoid CamelCase:
Signed-off-by: Shaun Ren
---
Changes since v2
* Update patch to reflect the changes made in PATCH 1/9 (block comments)
drivers/staging/rts5208/rtsx_transport.c |
This patch adds a dma_mapping_error call to debug potential DMA mapping
errors after the dma_map_single call in rtsx_transport.c.
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Hi Andrew,
[auto build test WARNING on input/next]
[also build test WARNING on v4.5-rc3 next-20160209]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Andrew-Duggan/Input-synaptics-rmi4
From: Kuninori Morimoto
Some R-Car sound requests picky register access which needs *force*
register write. To reduce complexity, this patch uses regmap force
function for all register access.
Signed-off-by: Kuninori Morimoto
---
sound/soc/sh/rcar/gen.c | 21 ++---
From: Kuninori Morimoto
Signed-off-by: Kuninori Morimoto
---
drivers/base/regmap/regmap.c | 12
include/linux/regmap.h | 6 --
2 files changed, 4 insertions(+), 14 deletions(-)
diff --git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c
index
drivers/input/rmi4/rmi_i2c.c:373:3-8: No need to set .owner here. The core will
do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Andrew Duggan
Signed-off-by: Fengguang Wu
---
rmi_i2c.c |1 -
1
From: Kuninori Morimoto
We can use _force_ write by regmap_raw_update_bits() option.
Let's remove unused regmap_write_bits()
Signed-off-by: Kuninori Morimoto
---
drivers/base/regmap/regmap.c | 23 ---
include/linux/regmap.h | 9 -
2 files changed, 32
This patch fixes the following issues in rtsx_transport.c as reported by
checkpatch.pl:
CHECK: Blank lines aren't necessary after an open brace '{'
CHECK: Please don't use multiple blank lines
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 3 ---
1 file changed, 3
This patch removes all unnecessary parentheses found by checkpatch.pl.
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx_transport.c
This patch fixes all multiline comments to conform to the coding style,
which states that multiline comments should start with "/*" and end
with "*/" on a separate line.
Also cleans up some comments to make them more clear and/or reflect what
the code is doing.
Signed-off-by: Shaun Ren
---
This set of patches fixes the coding style issues in rtsx_transport.c, and
dds a missing call dma_mapping_error() after dma_map_single().
Changes since v2
* Incorporated Joshua Clayton's suggestions regarding the block comments
From: Kuninori Morimoto
It can be easy to add _check, _async, _check_async functions
Signed-off-by: Kuninori Morimoto
---
drivers/base/regmap/regmap.c | 32 ++--
include/linux/regmap.h | 15 +++
2 files changed, 29 insertions(+), 18 deletions(-)
From: Kuninori Morimoto
Signed-off-by: Kuninori Morimoto
---
include/linux/regmap.h | 4
1 file changed, 4 insertions(+)
diff --git a/include/linux/regmap.h b/include/linux/regmap.h
index 687223d..befa781 100644
--- a/include/linux/regmap.h
+++ b/include/linux/regmap.h
@@ -76,8 +76,12
> From: Greg KH [mailto:gre...@linuxfoundation.org]
> Sent: Monday, February 8, 2016 13:34
> To: KY Srinivasan
> Cc: linux-kernel@vger.kernel.org; de...@linuxdriverproject.org;
> o...@aepfle.de;
> a...@canonical.com; vkuzn...@redhat.com; jasow...@redhat.com; Dexuan
> Cui
> Subject: Re: [PATCH
From: Kuninori Morimoto
Current regmap has many similar update functions, but the difference is
very few. This patch adds new regmap_raw_update_bits() and merge all
update functions into it by macro.
regmap_update_bits()
regmap_update_bits_async()
From: Kuninori Morimoto
It can be easy to add _check, _async, _check_async functions
Signed-off-by: Kuninori Morimoto
---
drivers/base/regmap/regmap.c | 23 ++-
include/linux/regmap.h | 14 ++
2 files changed, 24 insertions(+), 13 deletions(-)
diff
Hi Mark
Current regmap has many similar functions
regmap_update_bits()
regmap_update_bits_async()
regmap_update_bits_check()
regmap_update_bits_check_async()
But difference is very few.
And I would like to have _force_ feature on it.
So, these patches add new
Andrew Morton writes:
> On Tue, 9 Feb 2016 21:41:44 +0530 "Aneesh Kumar K.V"
> wrote:
>
>> With next generation power processor, we are having a new mmu model
>> [1] that require us to maintain a different linux page table format.
>>
>> Inorder to support both current and future ppc64
On 02/09/2016 07:26 AM, Arnd Bergmann wrote:
On Tuesday 09 February 2016 07:08:59 Guenter Roeck wrote:
IS_ERR_VALUE() assumes that its parameter is an unsigned long.
It can not be used to check if an unsigned int reflects an error.
Doing so can result in the following build warning.
"Kirill A. Shutemov" writes:
> On Tue, Feb 09, 2016 at 09:41:44PM +0530, Aneesh Kumar K.V wrote:
>> With next generation power processor, we are having a new mmu model
>> [1] that require us to maintain a different linux page table format.
>>
>> Inorder to support both current and future ppc64
Hi Andrew,
[auto build test WARNING on input/next]
[also build test WARNING on v4.5-rc3 next-20160209]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Andrew-Duggan/Input-synaptics-rmi4
drivers/input/rmi4/rmi_bus.c:295:20-21: WARNING: Use ARRAY_SIZE
Use ARRAY_SIZE instead of dividing sizeof array with sizeof an element
Semantic patch information:
This makes an effort to find cases where ARRAY_SIZE can be used such as
where there is a division of sizeof the array by the
On 2/3/2016 10:24 AM, Doug Anderson wrote:
> John,
>
> On Tue, Feb 2, 2016 at 3:57 PM, John Youn wrote:
>> On 1/28/2016 6:20 PM, Douglas Anderson wrote:
>>> This is a bit of catchall series for all the bug fix and performance
>>> patches I've been working on over the last few months. Note that
1 - 100 of 2040 matches
Mail list logo
