[PATCH v2 1/3] dt-bindings: vendor-prefixes: Add techwell vendor prefix

2020-11-13 Thread Maxime Chevallier
Add prefix for Techwell, Inc. Acked-by: Rob Herring Signed-off-by: Maxime Chevallier --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendo

[PATCH 06/40] drm/msm/adreno/a6xx_gpu: Staticise local function 'a6xx_idle'

2020-11-13 Thread Lee Jones
Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/msm/adreno/a6xx_gpu.c:33:6: warning: no previous prototype for ‘a6xx_idle’ [-Wmissing-prototypes] Cc: Rob Clark Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter Cc: linux-arm-...@vger.kernel.org Cc: dri-de...@lists.freedesktop

[PATCH 03/40] drm/amd/display/dc/core/dc_link_ddc: Move DP_DVI_CONVERTER_ID_{4,5} to where they're used

2020-11-13 Thread Lee Jones
This patch fixes >200 warnings. Fixes the following W=1 kernel build warning(s): from drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:31: drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:126:22: warning: ‘DP_DVI_CONVERTER_ID_5’ defined but not used [-Wunused-const-

[PATCH 07/40] drm/mediatek/mtk_disp_rdma: Fix formatting and supply missing struct member description

2020-11-13 Thread Lee Jones
Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/mediatek/mtk_disp_rdma.c:66: warning: Function parameter or member 'ddp_comp' not described in 'mtk_disp_rdma' drivers/gpu/drm/mediatek/mtk_disp_rdma.c:66: warning: Function parameter or member 'crtc' not described in 'mtk_disp_r

[PATCH v2 3/3] media: i2c: Introduce a driver for the Techwell TW9900 decoder

2020-11-13 Thread Maxime Chevallier
The Techwell video decoder supports PAL, NTSC and SECAM input formats, and outputs a BT.656 signal. This commit adds support for this device, with basic support for NTSC and PAL, along with brightness and contrast controls. Signed-off-by: Maxime Chevallier --- v1 -> v2: Set the media entity type

[PATCH v2 2/3] media: dt-bindings: media: i2c: Add bindings for TW9900

2020-11-13 Thread Maxime Chevallier
The Techwell TW9900 is a video decoder supporting multiple input standards, such as PAL, NTSC and SECAM, and outputs a BT.656 video signal. It's designed to be low-power, posesses some features such as a programmable comb-filter, and automatic input standard detection. Signed-off-by: Maxime Cheva

[PATCH v2 0/3] media: i2c: Introduce driver for the TW9900 decoder

2020-11-13 Thread Maxime Chevallier
Hello everyone, This is the second version of the series adding support for the Techwell TW9900 multi standard decoder. It's a pretty simple decoder compared to the TW9910, since it doesn't have a built-in scaler/crop engine. Since V2, I've fixed a few issues with the binding, and changed the med

Re: Regression: QCA6390 fails with "mm/page_alloc: place pages to tail in __free_pages_core()"

2020-11-13 Thread David Hildenbrand
On 13.11.20 14:36, wi nk wrote: On Fri, Nov 13, 2020 at 1:52 PM Pavel Procopiuc wrote: Op 13.11.2020 om 12:08 schreef Carl Huang: Checked some logs. Looks when the error happens, the physical address are very small. Its' between 20M - 30M. So could you have a try to reserve the memory starti

[PATCH] checkpatch: fix typo and remove duplicate word

2020-11-13 Thread Dwaipayan Ray
Delete repeated word in scripts/checkpatch.pl: "are are" -> "are" Fix typos: "commments" -> "comments" "falsly" -> "falsely" Signed-off-by: Dwaipayan Ray --- scripts/checkpatch.pl | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatc

Re: [PATCH v1 1/2] mmc: Support kmsg dumper based on pstore/blk

2020-11-13 Thread kernel test robot
Hi Bhaskara, Thank you for the patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.10-rc3] [cannot apply to next-20201112] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base

Re: [PATCH v1 1/4] dt-bindings: ata: tegra: Convert binding documentation to YAML

2020-11-13 Thread Rob Herring
On Thu, Nov 12, 2020 at 9:54 PM Sowjanya Komatineni wrote: > > Hi Rob, > > Updated yamllint and dt-schema as well. > > When I do make dt_binding_check, I see it failed as > processed-schema-examples.json is not generated. > Any idea of what I may be missing? > > Also it did not go thru bindings/at

Re: [PATCH v8 4/9] mm: introduce memfd_secret system call to create "secret" memory areas

2020-11-13 Thread Matthew Wilcox
On Tue, Nov 10, 2020 at 05:14:39PM +0200, Mike Rapoport wrote: > +static vm_fault_t secretmem_fault(struct vm_fault *vmf) > +{ > + struct address_space *mapping = vmf->vma->vm_file->f_mapping; > + struct inode *inode = file_inode(vmf->vma->vm_file); > + pgoff_t offset = vmf->pgoff; > +

Re: [v10] i2c: imx: support slave mode for imx I2C driver

2020-11-13 Thread Oleksij Rempel
On Wed, Nov 11, 2020 at 07:32:55PM +0800, Biwen Li wrote: > From: Biwen Li > > The patch supports slave mode for imx I2C driver > > Signed-off-by: Biwen Li Thank you for your work! Acked-by: Oleksij Rempel @Wolfram, Christian Eggers I2SR_IAL patches should go before this one. > --- > Chang

Re: [PATCH] lib: vsprintf: Avoid 32-bit truncation in vsscanf number parsing

2020-11-13 Thread Petr Mladek
On Thu 2020-11-12 12:04:27, Steven Rostedt wrote: > On Thu, 12 Nov 2020 15:46:46 + > Richard Fitzgerald wrote: > > > See this thread from 2014 where the field width problem was raised and > > explained: > > http://lkml.iu.edu/hypermail/linux/kernel/1401.1/03443.html > > > > and the reply fro

Re: [PATCH RFC] checkpatch: add new cases to commit handling

2020-11-13 Thread Lukas Bulwahn
On Fri, Nov 13, 2020 at 2:37 PM Lukas Bulwahn wrote: > > On Fri, Nov 13, 2020 at 1:31 PM Dwaipayan Ray wrote: > > > > Commit extraction in checkpatch fails in some cases. > > One of the most common false positives is a split line > > between "commit" and the git SHA of the commit. > > > > Improve

Re: [PATCH v8 4/9] mm: introduce memfd_secret system call to create "secret" memory areas

2020-11-13 Thread Matthew Wilcox
On Tue, Nov 10, 2020 at 05:14:39PM +0200, Mike Rapoport wrote: > diff --git a/mm/Kconfig b/mm/Kconfig > index c89c5444924b..d8d170fa5210 100644 > --- a/mm/Kconfig > +++ b/mm/Kconfig > @@ -884,4 +884,7 @@ config ARCH_HAS_HUGEPD > config MAPPING_DIRTY_HELPERS > bool > > +config SECRETMEM

[PATCH] mm/page_counter: use page_counter_read in page_counter_set_max

2020-11-13 Thread Hui Su
use page_counter_read() in page_counter_set_max(). Signed-off-by: Hui Su --- mm/page_counter.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mm/page_counter.c b/mm/page_counter.c index b24a60b28bb0..c6860f51b6c6 100644 --- a/mm/page_counter.c +++ b/mm/page_counter.c @@

Re: [PATCH v4 1/3] arm64: wrap and generalise counter read functions

2020-11-13 Thread Sudeep Holla
On Fri, Nov 06, 2020 at 12:53:32PM +, Ionela Voinescu wrote: > In preparation for other uses of Activity Monitors (AMU) cycle counters, > place counter read functionality in generic functions that can reused: > read_corecnt() and read_constcnt(). > > As a result, implement update_freq_counters

Re: [PATCH 0/3] platform/chrome: cros_ec_typec: Add plug and plug altmodes

2020-11-13 Thread Greg KH
On Wed, Nov 11, 2020 at 05:23:25PM -0800, Prashant Malani wrote: > This patch series add plug registration support to the cros-ec-typec > driver. It also adds support for registering alternate modes for the > registered plug. These features utilize the API provided by the Type C > connector class f

Re: [PATCH v4 2/3] arm64: split counter validation function

2020-11-13 Thread Sudeep Holla
On Fri, Nov 06, 2020 at 12:53:33PM +, Ionela Voinescu wrote: > In order for the counter validation function to be reused, split > validate_cpu_freq_invariance_counters() into: > - freq_counters_valid(cpu) - check cpu for valid cycle counters > - freq_inv_set_max_ratio(int cpu, u64 max_rate, u

Re: [PATCH v4 3/3] arm64: implement CPPC FFH support using AMUs

2020-11-13 Thread Sudeep Holla
On Fri, Nov 06, 2020 at 12:53:34PM +, Ionela Voinescu wrote: > If Activity Monitors (AMUs) are present, two of the counters can be used > to implement support for CPPC's (Collaborative Processor Performance > Control) delivered and reference performance monitoring functionality > using FFH (Fun

Re: [PATCH] sysctl: move local variable in proc_do_large_bitmap() to proper scope

2020-11-13 Thread Luis Chamberlain
On Mon, Nov 09, 2020 at 05:26:37PM -0800, Tom Rix wrote: > > On 11/8/20 11:11 PM, Lukas Bulwahn wrote: > > make clang-analyzer caught my attention with: > > > > kernel/sysctl.c:1511:4: warning: Value stored to 'first' is never read \ > > [clang-analyzer-deadcode.DeadStores] > >

Re: [PATCH 2/2] uio: pruss: use devm_clk_get() for clk init

2020-11-13 Thread kernel test robot
Hi Alexandru, I love your patch! Perhaps something to improve: [auto build test WARNING on char-misc/char-misc-testing] [also build test WARNING on linus/master linux/master v5.10-rc3 next-20201113] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch

Re: [PATCH RFC] checkpatch: add new cases to commit handling

2020-11-13 Thread Dwaipayan Ray
On Fri, Nov 13, 2020 at 7:31 PM Lukas Bulwahn wrote: > > On Fri, Nov 13, 2020 at 2:37 PM Lukas Bulwahn wrote: > > > > On Fri, Nov 13, 2020 at 1:31 PM Dwaipayan Ray > > wrote: > > > > > > Commit extraction in checkpatch fails in some cases. > > > One of the most common false positives is a split

Re: [PATCH 0/8] linker-section array fix and clean ups

2020-11-13 Thread Johan Hovold
On Wed, Nov 11, 2020 at 04:47:16PM +0100, Jessica Yu wrote: > Thanks for providing the links and references. Your explanation and > this reply from Jakub [1] clarified things for me. I was not aware of > the distinction gcc made between aligned attributes on types vs. on > variables. So from what

Re: [PATCH v2] firmware: arm_scmi: fix missing destroy_workqueue()

2020-11-13 Thread Sudeep Holla
On Tue, 10 Nov 2020 15:42:21 +0800, Qinglang Miao wrote: > destroy_workqueue seems necessary before return from > scmi_notification_init in the error handling case when > fails to do devm_kcalloc(). Fix this by simply moving > devm_kcalloc to the front. Applied to sudeep.holla/linux (for-next/scm

Re: [RESEND][PATCH 2/2] iommu: Avoid crash if iommu_group is null

2020-11-13 Thread Robin Murphy
On 2020-11-12 22:05, John Stultz wrote: In trying to handle a possible driver probe ordering issue brought up by Robin Murphy, I ran across a separate null pointer crash in the iommu core in iommu_group_remove_device(): [2.732803] dwc3-qcom a6f8800.usb: failed to get usb-ddr path: -517 [2

[patch 01/19] parisc: Remove bogus __IRQ_STAT macro

2020-11-13 Thread Thomas Gleixner
This is a leftover from a historical array based implementation and unused. Signed-off-by: Thomas Gleixner Cc: "James E.J. Bottomley" Cc: Helge Deller Cc: linux-par...@vger.kernel.org --- arch/parisc/include/asm/hardirq.h |1 - 1 file changed, 1 deletion(-) --- a/arch/parisc/include/asm/h

[patch 00/19] softirq: Cleanups and RT awareness

2020-11-13 Thread Thomas Gleixner
RT runs softirq processing always in thread context and it requires that both the softirq execution and the BH disabled sections are preemptible. This is achieved by serialization through per CPU local locks and substituting a few parts of the existing softirq processing code with helper functions

[patch 10/19] preempt: Cleanup the macro maze a bit

2020-11-13 Thread Thomas Gleixner
Make the macro maze consistent and prepare it for adding the RT variant for BH accounting. - Use nmi_count() for the NMI portion of preempt count - Introduce in_hardirq() to make the naming consistent and non-ambiguos - Use the macros to create combined checks (e.g. in_task()) so the softirq

[patch 07/19] asm-generic/irqstat: Add optional __nmi_count member

2020-11-13 Thread Thomas Gleixner
Add an optional __nmi_count member to irq_cpustat_t so more architectures can use the generic version. Signed-off-by: Thomas Gleixner --- include/asm-generic/hardirq.h |3 +++ 1 file changed, 3 insertions(+) --- a/include/asm-generic/hardirq.h +++ b/include/asm-generic/hardirq.h @@ -7,6 +7,

[patch 02/19] sh: Get rid of nmi_count()

2020-11-13 Thread Thomas Gleixner
nmi_count() is a historical leftover and SH is the only user. Replace it with regular per cpu accessors. Signed-off-by: Thomas Gleixner Cc: Yoshinori Sato Cc: Rich Felker Cc: linux...@vger.kernel.org --- arch/sh/kernel/irq.c |2 +- arch/sh/kernel/traps.c |2 +- 2 files changed, 2 ins

[patch 09/19] irqstat: Move declaration into asm-generic/hardirq.h

2020-11-13 Thread Thomas Gleixner
Move the declaration of the irq_cpustat per cpu variable to asm-generic/hardirq.h and remove the now empty linux/irq_cpustat.h header. Signed-off-by: Thomas Gleixner --- include/asm-generic/hardirq.h |3 ++- include/linux/irq_cpustat.h | 24 2 files changed, 2 in

[patch 15/19] tick/sched: Prevent false positive softirq pending warnings on RT

2020-11-13 Thread Thomas Gleixner
On RT a task which has soft interrupts disabled can block on a lock and schedule out to idle while soft interrupts are pending. This triggers the warning in the NOHZ idle code which complains about going idle with pending soft interrupts. But as the task is blocked soft interrupt processing is temp

[patch 05/19] ARM: irqstat: Get rid of duplicated declaration

2020-11-13 Thread Thomas Gleixner
irq_cpustat_t is exactly the same as the asm-generic one. Define ack_bad_irq so the generic header does not emit the generic version of it. Signed-off-by: Thomas Gleixner Cc: Russell King Cc: Marc Zyngier Cc: Valentin Schneider Cc: linux-arm-ker...@lists.infradead.org --- arch/arm/include/asm

[patch 14/19] softirq: Make softirq control and processing RT aware

2020-11-13 Thread Thomas Gleixner
Provide a local lock based serialization for soft interrupts on RT which allows the local_bh_disabled() sections and servicing soft interrupts to be preemptible. Provide the necessary inline helpers which allow to reuse the bulk of the softirq processing code. Signed-off-by: Thomas Gleixner ---

[patch 17/19] softirq: Replace barrier() with cpu_relax() in tasklet_unlock_wait()

2020-11-13 Thread Thomas Gleixner
A barrier() in a tight loop which waits for something to happen on a remote CPU is a pointless exercise. Replace it with cpu_relax() which allows HT siblings to make progress. Signed-off-by: Thomas Gleixner --- include/linux/interrupt.h |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

[patch 18/19] tasklets: Use static inlines for stub implementations

2020-11-13 Thread Thomas Gleixner
Inlines exist for a reason. Signed-off-by: Thomas Gleixner --- include/linux/interrupt.h |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) --- a/include/linux/interrupt.h +++ b/include/linux/interrupt.h @@ -672,9 +672,9 @@ static inline void tasklet_unlock_wait(s cp

[patch 11/19] softirq: Move related code into one section

2020-11-13 Thread Thomas Gleixner
To prepare for adding a RT aware variant of softirq serialization and processing move related code into one section so the necessary #ifdeffery is reduced to one. Signed-off-by: Thomas Gleixner --- kernel/softirq.c | 107 +++ 1 file changed, 5

[patch 19/19] tasklets: Prevent kill/unlock_wait deadlock on RT

2020-11-13 Thread Thomas Gleixner
tasklet_kill() and tasklet_unlock_wait() spin and wait for the TASKLET_STATE_SCHED resp. TASKLET_STATE_RUN bit in the tasklet state to be cleared. This works on !RT nicely because the corresponding execution can only happen on a different CPU. On RT softirq processing is preemptible, therefore a t

[patch 16/19] rcu: Prevent false positive softirq warning on RT

2020-11-13 Thread Thomas Gleixner
Soft interrupt disabled sections can legitimately be preempted or schedule out when blocking on a lock on RT enabled kernels so the RCU preempt check warning has to be disabled for RT kernels. Signed-off-by: Thomas Gleixner --- include/linux/rcupdate.h |3 ++- 1 file changed, 2 insertions(+)

[patch 12/19] softirq: Add RT specific softirq accounting

2020-11-13 Thread Thomas Gleixner
RT requires the softirq to be preemptible and uses a per CPU local lock to protect BH disabled sections and softirq processing. Therefore RT cannot use the preempt counter to keep track of BH disabled/serving. Add a RT only counter to task struct and adjust the relevant macros in preempt.h. Signe

[patch 13/19] softirq: Move various protections into inline helpers

2020-11-13 Thread Thomas Gleixner
To allow reuse of the bulk of softirq processing code for RT and to avoid #ifdeffery all over the place, split protections for various code sections out into inline helpers so the RT variant can just replace them in one go. Signed-off-by: Thomas Gleixner --- kernel/softirq.c | 53 +

[patch 06/19] arm64: irqstat: Get rid of duplicated declaration

2020-11-13 Thread Thomas Gleixner
irq_cpustat_t is exactly the same as the asm-generic one. Define ack_bad_irq so the generic header does not emit the generic version of it. Signed-off-by: Thomas Gleixner Cc: Catalin Marinas Cc: Will Deacon Cc: Marc Zyngier Cc: linux-arm-ker...@lists.infradead.org --- arch/arm64/include/asm/h

[patch 04/19] um/irqstat: Get rid of the duplicated declarations

2020-11-13 Thread Thomas Gleixner
irq_cpustat_t and ack_bad_irq() are exactly the same as the asm-generic ones. Signed-off-by: Thomas Gleixner Cc: Jeff Dike Cc: Richard Weinberger Cc: Anton Ivanov Cc: linux...@lists.infradead.org --- arch/um/include/asm/hardirq.h | 17 + 1 file changed, 1 insertion(+), 16 de

[patch 08/19] sh: irqstat: Use the generic irq_cpustat_t

2020-11-13 Thread Thomas Gleixner
SH can now use the generic irq_cpustat_t. Define ack_bad_irq so the generic header does not emit the generic version of it. Signed-off-by: Thomas Gleixner Cc: Yoshinori Sato Cc: Rich Felker Cc: linux...@vger.kernel.org --- arch/sh/include/asm/hardirq.h | 14 -- 1 file changed, 4

[patch 03/19] irqstat: Get rid of nmi_count() and __IRQ_STAT()

2020-11-13 Thread Thomas Gleixner
Nothing uses this anymore. Signed-off-by: Thomas Gleixner --- include/linux/irq_cpustat.h |4 1 file changed, 4 deletions(-) --- a/include/linux/irq_cpustat.h +++ b/include/linux/irq_cpustat.h @@ -19,10 +19,6 @@ #ifndef __ARCH_IRQ_STAT DECLARE_PER_CPU_ALIGNED(irq_cpustat_t, irq_sta

Re: [PATCH bpf-next v5 01/34] mm: memcontrol: use helpers to read page's memcg data

2020-11-13 Thread Shakeel Butt
On Thu, Nov 12, 2020 at 8:02 PM Roman Gushchin wrote: > > On Thu, Nov 12, 2020 at 07:25:48PM -0800, Alexei Starovoitov wrote: > > On Thu, Nov 12, 2020 at 7:18 PM Andrew Morton > > wrote: > > > > > > On Thu, 12 Nov 2020 19:04:56 -0800 Alexei Starovoitov > > > wrote: > > > > > > > On Thu, Nov 12

Re: [PATCH v2 2/2] firmware: arm_scmi: Augment SMC/HVC to allow optional interrupt

2020-11-13 Thread Jim Quinlan
Hi, these are fast calls. Regards, Jim On Fri, Nov 13, 2020 at 4:47 AM Sudeep Holla wrote: > > On Thu, Nov 12, 2020 at 12:56:27PM -0500, Jim Quinlan wrote: > > The SMC/HVC SCMI transport is modified to allow the completion of an SCMI > > message to be indicated by an interrupt rather than the r

[PATCH 0/2] KVM: arm64: vgic: Fix handling of userspace register accesses

2020-11-13 Thread Zenghui Yu
We had recently seen a kernel panic when accidently programming QEMU in an inappropriate way (in short, accessing RD registers before setting the RD base address. See patch #1 for details). And it looks like we're missing some basic checking when handling userspace register access. I've only teste

[PATCH 1/2] KVM: arm64: vgic: Forbid invalid userspace Redistributor accesses

2020-11-13 Thread Zenghui Yu
It's expected that users will access registers in the redistributor *if* the RD has been initialized properly. Unfortunately userspace can be bogus enough to access registers before setting the RD base address, and KVM implicitly allows it (we handle the access anyway, regardless of whether the bas

[PATCH 2/2] KVM: arm64: vgic: Forbid invalid userspace Distributor accesses

2020-11-13 Thread Zenghui Yu
Accessing registers in the Distributor before setting its base address should be treated as an invalid userspece behaviour. But KVM implicitly allows it as we handle the access anyway, regardless of whether the base address is set or not. Fix this issue by informing userspace what had gone wrong (

Re: [PATCH v1 11/30] drm/tegra: dc: Support OPP and SoC core voltage scaling

2020-11-13 Thread Mark Brown
On Fri, Nov 13, 2020 at 01:37:01AM +0300, Dmitry Osipenko wrote: > 12.11.2020 23:01, Mark Brown пишет: > >> But it's not allowed to change voltage of a dummy regulator, is it > >> intentional? > > Of course not, we can't know if the requested new voltage is valid - the > > driver would have to hav

Re: [PATCH] drivers: staging: small refactor

2020-11-13 Thread Greg KH
On Sat, Nov 07, 2020 at 06:01:06PM +0200, Sotir Danailov wrote: > Small refactor of 3 files in the staging directory. > > Signed-off-by: Sotir Danailov > --- > drivers/staging/mt7621-pci/pci-mt7621.c | 1 - > drivers/staging/rtl8188eu/core/rtw_ap.c | 4 +++- > drivers/staging/rtl8188eu/cor

Re: ERROR: modpost: "mac_pton" undefined!

2020-11-13 Thread Andy Shevchenko
On Fri, Nov 13, 2020 at 3:19 PM Laurent Vivier wrote: > On 13/11/2020 14:12, Andy Shevchenko wrote: > > On Fri, Nov 13, 2020 at 2:42 PM Laurent Vivier wrote: ... > > But what I want to say is that API is under NET_UTILS, I dunno if the > > driver requires full NET to be selected. And IIRC NET_U

Re: [PATCH] tracing: remove the unused value assignment in test_create_synth_event

2020-11-13 Thread Steven Rostedt
On Fri, 13 Nov 2020 15:33:30 +0800 xiakaixu1...@gmail.com wrote: > From: Kaixu Xia > > The value of variable ret is overwritten on the delete branch in the > test_create_synth_event(), so here the value assignment is useless. > Remove it. > No. The correct fix is to remove the useless assignme

Re: [PATCH v3 2/2] usb: typec: Expose Product Type VDOs via sysfs

2020-11-13 Thread Heikki Krogerus
On Thu, Nov 12, 2020 at 08:50:55AM -0800, Prashant Malani wrote: > Hi Heikki, > > On Thu, Nov 12, 2020 at 4:43 AM Heikki Krogerus > wrote: > > > > On Wed, Nov 11, 2020 at 06:40:55PM -0800, Prashant Malani wrote: > > > Hi Heikki, > > > > > > On Tue, Nov 10, 2020 at 01:54:53PM +0200, Heikki Krogeru

Re: [PATCH] highmem: fix highmem for xtensa

2020-11-13 Thread Thomas Gleixner
Max, On Fri, Nov 13 2020 at 05:50, Max Filippov wrote: > On Fri, Nov 13, 2020 at 5:40 AM Thomas Gleixner wrote: >> On Fri, Nov 13 2020 at 04:23, Max Filippov wrote: >> > Fixmap on xtensa grows upwards, i.e. bigger fixmap entry index >> > corresponds to a higher virtual address. This was lost in h

Re: [PATCH v2 2/2] firmware: arm_scmi: Augment SMC/HVC to allow optional interrupt

2020-11-13 Thread Sudeep Holla
On Fri, Nov 13, 2020 at 09:26:43AM -0500, Jim Quinlan wrote: > Hi, these are fast calls. Regards, Jim > > > On Fri, Nov 13, 2020 at 4:47 AM Sudeep Holla wrote: > > > > On Thu, Nov 12, 2020 at 12:56:27PM -0500, Jim Quinlan wrote: > > > The SMC/HVC SCMI transport is modified to allow the completi

Re: [PATCH v1 00/30] Introduce core voltage scaling for NVIDIA Tegra20/30 SoCs

2020-11-13 Thread Ulf Hansson
On Thu, 12 Nov 2020 at 23:14, Dmitry Osipenko wrote: > > 12.11.2020 23:43, Thierry Reding пишет: > >> The difference in comparison to using voltage regulator directly is > >> minimal, basically the core-supply phandle is replaced is replaced with > >> a power-domain phandle in a device tree. > > T

Re: [PATCH v2 03/17] Drivers: hv: vmbus: skip VMBus initialization if Linux is root

2020-11-13 Thread Wei Liu
On Thu, Nov 12, 2020 at 04:24:38PM +0100, Vitaly Kuznetsov wrote: > Wei Liu writes: > > > There is no VMBus and the other infrastructures initialized in > > hv_acpi_init when Linux is running as the root partition. > > > > Signed-off-by: Wei Liu > > --- > > drivers/hv/vmbus_drv.c | 3 +++ > > 1

[PATCH] ASoC: samsung: remove the unused variable value assignment

2020-11-13 Thread xiakaixu1987
From: Kaixu Xia The value of variable ret is overwritten by the following call devm_snd_soc_register_card(), so here the value assignment is useless. Remove it. Reported-by: Tosk Robot Signed-off-by: Kaixu Xia --- sound/soc/samsung/smdk_wm8994.c | 4 +--- 1 file changed, 1 insertion(+), 3 del

[PATCH v10 0/3] Adding support for Microchip/Microsemi serial GPIO controller

2020-11-13 Thread Lars Povlsen
The series add support for the serial GPIO controller used by Microchip Sparx5, as well as (MSCC) ocelot/jaguar2 SoCs. v10 changes - anniversary edition (from Andy): - Fixed "Author" comment - Added missing "break;" in default switch case - Return -EINVAL when requesting pin disabled in bitstre

[PATCH v10 2/3] pinctrl: pinctrl-microchip-sgpio: Add pinctrl driver for Microsemi Serial GPIO

2020-11-13 Thread Lars Povlsen
This adds a pinctrl driver for the Microsemi/Microchip Serial GPIO (SGPIO) device used in various SoC's. The driver is added as a pinctrl driver, albeit only having just GPIO support currently. The hardware supports other functions that will be added following. Signed-off-by: Lars Povlsen --- M

[PATCH v10 3/3] arm64: dts: sparx5: Add SGPIO devices

2020-11-13 Thread Lars Povlsen
This adds SGPIO devices for the Sparx5 SoC and configures it for the applicable reference boards. Signed-off-by: Lars Povlsen --- arch/arm64/boot/dts/microchip/sparx5.dtsi | 91 ++ .../boot/dts/microchip/sparx5_pcb125.dts | 5 + .../dts/microchip/sparx5_pcb134_board.dtsi| 258

[PATCH v10 1/3] dt-bindings: pinctrl: Add bindings for pinctrl-microchip-sgpio driver

2020-11-13 Thread Lars Povlsen
This adds DT bindings for the Microsemi/Microchip SGPIO controller, bindings microchip,sparx5-sgpio, mscc,ocelot-sgpio and mscc,luton-sgpio. Signed-off-by: Lars Povlsen Reviewed-by: Rob Herring --- .../pinctrl/microchip,sparx5-sgpio.yaml | 145 ++ 1 file changed, 145 inser

Re: [PATCH v2 04/17] iommu/hyperv: don't setup IRQ remapping when running as root

2020-11-13 Thread Wei Liu
On Thu, Nov 12, 2020 at 04:27:14PM +0100, Vitaly Kuznetsov wrote: > Wei Liu writes: > > > The IOMMU code needs more work. We're sure for now the IRQ remapping > > hooks are not applicable when Linux is the root. > > Super-nitpick: I would suggest we always say 'root partition' as 'root' > has a

[PATCH] clk: imx6q: demote warning about pre-boot ldb_di_clk reparenting

2020-11-13 Thread Ahmad Fatoum
Since 5d283b083800 ("clk: imx6: Fix procedure to switch the parent of LDB_DI_CLK"), the clock driver warns if ldb_di\d_sel is changed from reset value on system boot. This warning is printed even if the bootloader (or a previous kernel that did kexec) followed the correct procedure for glitch-free

Re: [PATCH] rtl8xxxu: remove the unused variable timeout value assignment

2020-11-13 Thread Jes Sorensen
On 11/13/20 4:50 AM, xiakaixu1...@gmail.com wrote: > From: Kaixu Xia > > The value of variable timeout is overwritten by the following statement in > rtl8xxxu_gen1_init_aggregation(), so here the value assignment is useless. > Remove it. > > Reported-by: Tosk Robot > Signed-off-by: Kaixu Xia >

Re: [PATCH][next] drm/kmb: fix array out-of-bounds writes to kmb->plane_status[]

2020-11-13 Thread Sam Ravnborg
Hi Colin. On Fri, Nov 13, 2020 at 12:01:21PM +, Colin King wrote: > From: Colin Ian King > > Writes to elements in the kmb->plane_status array in function > kmb_plane_atomic_disable are overrunning the array when plane_id is > more than 1 because currently the array is KMB_MAX_PLANES element

Re: [PATCH] iosched: Add i10 I/O Scheduler

2020-11-13 Thread Ming Lei
Hello, On Thu, Nov 12, 2020 at 09:07:52AM -0500, Rachit Agarwal wrote: > From: Rachit Agarwal > > > Hi All, > > I/O batching is beneficial for optimizing IOPS and throughput for various > applications. For instance, several kernel block drivers would benefit from > batching, > including mmc [1]

Re: [PATCH] checkpatch: fix typo and remove duplicate word

2020-11-13 Thread Joe Perches
On Fri, 2020-11-13 at 19:27 +0530, Dwaipayan Ray wrote: > Delete repeated word in scripts/checkpatch.pl: > "are are" -> "are" > > Fix typos: > "commments" -> "comments" > "falsly" -> "falsely" > > Signed-off-by: Dwaipayan Ray > --- style: this should have subject prefix: "[PATCH V2] checkpatch

Re: [PATCH v2 3/3] media: i2c: Introduce a driver for the Techwell TW9900 decoder

2020-11-13 Thread Paul Kocialkowski
Hi, On Fri 13 Nov 20, 14:44, Maxime Chevallier wrote: > The Techwell video decoder supports PAL, NTSC and SECAM input formats, > and outputs a BT.656 signal. > > This commit adds support for this device, with basic support for NTSC > and PAL, along with brightness and contrast controls. > > Sign

Re: [PATCH] hwmon: fix corsair-psu fan rpm calculation

2020-11-13 Thread Guenter Roeck
On Fri, Nov 13, 2020 at 01:19:54PM +0100, Wilken Gottwalt wrote: > The correct fan rpm value is also a LINEAR11 value but without a factor. > Verified by using the fan test button on the psu to let the fan spin up > to maximum for some seconds. > > Fixes: 933222c98445 ("hwmon: (corsair-psu) fix un

ks-sa-rng.c:undefined reference to `devm_platform_ioremap_resource'

2020-11-13 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 585e5b17b92dead8a3aca4e3c9876fbca5f7e0ba commit: 90c4b29eb1e555fee66f8329a18cb8a070090ad6 hwrng: ks-sa - Enable COMPILE_TEST date: 12 months ago config: s390-randconfig-r022-20201113 (attached as .config

Re: [PATCH v2] reset: make shared pulsed reset controls re-triggerable

2020-11-13 Thread Philipp Zabel
On Fri, 2020-11-13 at 00:00 +0100, Amjad Ouled-Ameur wrote: > The current reset framework API does not allow to release what is done by > reset_control_reset(), IOW decrement triggered_count. Add the new > reset_control_rearm() call to do so. > > When reset_control_reset() has been called once, th

Re: [PATCH][next] drm/kmb: fix array out-of-bounds writes to kmb->plane_status[]

2020-11-13 Thread Colin Ian King
On 13/11/2020 14:55, Sam Ravnborg wrote: > Hi Colin. > > On Fri, Nov 13, 2020 at 12:01:21PM +, Colin King wrote: >> From: Colin Ian King >> >> Writes to elements in the kmb->plane_status array in function >> kmb_plane_atomic_disable are overrunning the array when plane_id is >> more than 1 be

Re: [PATCH v2 06/17] x86/hyperv: allocate output arg pages if required

2020-11-13 Thread Wei Liu
On Thu, Nov 12, 2020 at 04:35:48PM +0100, Vitaly Kuznetsov wrote: > Wei Liu writes: [...] > > @@ -209,14 +219,23 @@ static int hv_cpu_die(unsigned int cpu) > > unsigned int new_cpu; > > unsigned long flags; > > void **input_arg; > > - void *input_pg = NULL; > > + void *pg; > > >

Re: [PATCH v4 11/19] sched/core: Make migrate disable and CPU hotplug cooperative

2020-11-13 Thread Qian Cai
000 [#1] SMP KASAN PTI [ 809.423445][ T428] CPU: 72 PID: 428 Comm: migration/72 Tainted: G I 5.10.0-rc3-next-20201113+ #1 [ 809.433678][ T428] Hardware name: HPE ProLiant DL560 Gen10/ProLiant DL560 Gen10, BIOS U34 11/13/2019 [ 809.442951][ T428] Stopper: multi_cpu_stop

[PATCH] optee: add writeback to valid memory type

2020-11-13 Thread Rui Miguel Silva
Only in smp systems the cache policy is setup as write alloc, in single cpu systems the cache policy is set as writeback and it is normal memory, so, it should pass the is_normal_memory check in the share memory registration. Add the right condition to make it work in no smp systems. Fixes: cdbcf

[PATCH] dt-bindings: fsl-imx-drm: fix example compatible string

2020-11-13 Thread Cengiz Can
Example `display-subsystem` has an incorrect compatible string. Required properties section tells that developers should use "fsl,imx-display-subsystem" as "compatible" string but the example misses 'imx-' prefix. Change example to have correct "compatible" string. Signed-off-by: Cengiz Can ---

Re: [PATCH] checkpatch: fix typo and remove duplicate word

2020-11-13 Thread Dwaipayan Ray
On Fri, Nov 13, 2020 at 8:30 PM Joe Perches wrote: > > On Fri, 2020-11-13 at 19:27 +0530, Dwaipayan Ray wrote: > > Delete repeated word in scripts/checkpatch.pl: > > "are are" -> "are" > > > > Fix typos: > > "commments" -> "comments" > > "falsly" -> "falsely" > > > > Signed-off-by: Dwaipayan Ray

[PATCH] x86/boot/compressed/64: Drop the now-unused finalize_identity_maps()

2020-11-13 Thread Vitaly Kuznetsov
Since commit 8570978ea030 ("x86/boot/compressed/64: Don't pre-map memory in KASLR code") finalize_identity_maps() has no users, drop it. Signed-off-by: Vitaly Kuznetsov --- arch/x86/boot/compressed/ident_map_64.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/arch/x86/boot/compre

Re: [PATCH] mm/page_counter: use page_counter_read in page_counter_set_max

2020-11-13 Thread Pankaj Gupta
> use page_counter_read() in page_counter_set_max(). > > Signed-off-by: Hui Su > --- > mm/page_counter.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/mm/page_counter.c b/mm/page_counter.c > index b24a60b28bb0..c6860f51b6c6 100644 > --- a/mm/page_counter.c > +++ b/m

[PATCH 1/2] phy: qualcomm: usb: Fix SuperSpeed PHY OF dependency

2020-11-13 Thread Bryan O'Donoghue
This Kconfig entry should declare a dependency on OF Fixes: 6076967a500c ("phy: qualcomm: usb: Add SuperSpeed PHY driver") Reported-by: kernel test robot Link: https://lkml.org/lkml/2020/11/13/414 Signed-off-by: Bryan O'Donoghue --- drivers/phy/qualcomm/Kconfig | 2 +- 1 file changed, 1 inserti

[PATCH 2/2] phy: qualcomm: Fix 28 nm Hi-Speed USB PHY OF dependency

2020-11-13 Thread Bryan O'Donoghue
This Kconfig entry should declare a dependency on OF Fixes: 67b27dbeac4d ("phy: qualcomm: Add Synopsys 28nm Hi-Speed USB PHY driver") Signed-off-by: Bryan O'Donoghue --- drivers/phy/qualcomm/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/phy/qualcomm/Kconfig

[PATCH 0/2] phy: qualcomm: usb: Fix two COMPILE_TEST errors

2020-11-13 Thread Bryan O'Donoghue
kernel build robot fired this email at me about an hour ago https://lkml.org/lkml/2020/11/13/414 The build robot has flagged the super-speed PHY driver but both Kconfig entries have the same error. Fix those now. verified with "make menuconfig ARCH=s390" Bryan O'Donoghue (2): phy: qualcomm: us

Re: [PATCH v2 2/2] firmware: arm_scmi: Augment SMC/HVC to allow optional interrupt

2020-11-13 Thread Jim Quinlan
On Fri, Nov 13, 2020 at 9:36 AM Sudeep Holla wrote: > > On Fri, Nov 13, 2020 at 09:26:43AM -0500, Jim Quinlan wrote: > > Hi, these are fast calls. Regards, Jim > > > > > > On Fri, Nov 13, 2020 at 4:47 AM Sudeep Holla wrote: > > > > > > On Thu, Nov 12, 2020 at 12:56:27PM -0500, Jim Quinlan wrote:

Re: [PATCH v2] reset: make shared pulsed reset controls re-triggerable

2020-11-13 Thread Jerome Brunet
On Fri 13 Nov 2020 at 16:04, Philipp Zabel wrote: > On Fri, 2020-11-13 at 00:00 +0100, Amjad Ouled-Ameur wrote: >> The current reset framework API does not allow to release what is done by >> reset_control_reset(), IOW decrement triggered_count. Add the new >> reset_control_rearm() call to do s

Re: [PATCH] x86/boot/compressed/64: Drop the now-unused finalize_identity_maps()

2020-11-13 Thread Arvind Sankar
On Fri, Nov 13, 2020 at 04:08:10PM +0100, Vitaly Kuznetsov wrote: > Since commit 8570978ea030 ("x86/boot/compressed/64: Don't pre-map memory in > KASLR code") finalize_identity_maps() has no users, drop it. > > Signed-off-by: Vitaly Kuznetsov > --- > arch/x86/boot/compressed/ident_map_64.c | 10

Re: [EXTERNAL] [tip: x86/apic] x86/io_apic: Cleanup trigger/polarity helpers

2020-11-13 Thread David Woodhouse
On Wed, 2020-11-11 at 14:30 -0600, Tom Lendacky wrote: > I had trouble cloning your tree for some reason, so just took the top > three patches and applied them to the tip tree. This all appears to be > working. I'll let the IOMMU experts take a closer look (adding Suravee). Thanks. I see Thomas ha

Re: [PATCH] mm/shmem.c: make shmem_mapping() inline

2020-11-13 Thread Pankaj Gupta
> inline the shmem_mapping(), and use shmem_mapping() > instead of 'inode->i_mapping->a_ops == &shmem_aops' > in shmem_evict_inode(). > > Signed-off-by: Hui Su > --- > include/linux/shmem_fs.h | 2 +- > mm/shmem.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff

Re: [PATCH 01/40] drm/amd/include/vega10_ip_offset: Mark _BASE structs as __maybe_unused

2020-11-13 Thread Alex Deucher
On Fri, Nov 13, 2020 at 8:49 AM Lee Jones wrote: > > This patch fixes nearly 400 warnings! > > These structures are too widely used in too many varying > configurations to be split-up into different headers or moved into > source files. > > Instead, we'll mark them as __maybe_unused which tells th

Re: ERROR: modpost: "mac_pton" undefined!

2020-11-13 Thread Michael S. Tsirkin
bca5f7e0ba > > commit: 0c86d774883fa17e7c81b0c8838b88d06c2c911e vdpasim: allow to assign a > > MAC address > > date: 2 weeks ago > > config: openrisc-randconfig-r035-20201113 (attached as .config) > > compiler: or1k-linux-gcc (GCC) 9.3.0 > > reproduce (this is a W=1 build): > >

Re: ERROR: modpost: "mac_pton" undefined!

2020-11-13 Thread Michael S. Tsirkin
On Fri, Nov 13, 2020 at 02:19:31PM +0100, Laurent Vivier wrote: > On 13/11/2020 14:12, Andy Shevchenko wrote: > > On Fri, Nov 13, 2020 at 2:42 PM Laurent Vivier wrote: > > > > ... > > > >>> If you fix the issue, kindly add following tag as appropriate > >>> Reported-by: kernel test robot > >>>

[PATCH v3 4/6] regulator: mcp16502: adapt for get/set on other registers

2020-11-13 Thread Claudiu Beznea
MCP16502 have multiple registers for each regulator (as described in enum mcp16502_reg). Adapt the code to be able to get/set all these registers. This is necessary for the following commits. Signed-off-by: Claudiu Beznea --- drivers/regulator/mcp16502.c | 43 +++-

[PATCH v3 5/6] regulator: mcp16502: add support for ramp delay

2020-11-13 Thread Claudiu Beznea
MCP16502 have configurable ramp delay support (via DVSR bits in regulators' CFG register). Signed-off-by: Claudiu Beznea --- drivers/regulator/mcp16502.c | 89 +++- 1 file changed, 87 insertions(+), 2 deletions(-) diff --git a/drivers/regulator/mcp16502.c

[PATCH v3 0/6] regulator: mcp16502: add support for ramp delay

2020-11-13 Thread Claudiu Beznea
Hi, This series adds support for ramp delay on mcp16502. It also adds some cleanup on mcp16502. Apart from that patches 1/6 fixes the selector validation in case the regulator::desc::linear_min_sel is not zero. Thank you, Claudiu Beznea Changes in v3: - fix compilation error in patch 5/6 Repo

[PATCH v3 1/6] regulator: core: validate selector against linear_min_sel

2020-11-13 Thread Claudiu Beznea
There are regulators who's min selector is not zero. Selectors loops (looping b/w zero and regulator::desc::n_voltages) might throw errors because invalid selectors are used (lower than regulator::desc::linear_min_sel). For this situations validate selectors against regulator::desc::linear_min_sel.

[PATCH v2] tracing: remove the useless value assignment in test_create_synth_event

2020-11-13 Thread xiakaixu1987
From: Kaixu Xia The value of variable ret is overwritten on the delete branch in the test_create_synth_event() and we care more about the above error than this delete portion. Remove it. Reported-by: Tosk Robot Signed-off-by: Kaixu Xia --- kernel/trace/synth_event_gen_test.c | 2 +- 1 file ch

<    1   2   3   4   5   6   7   8   9   10   >