[PATCH 4/8] phy: lantiq: rcu-usb2: Constify ltq_rcu_usb2_phy_ops

2020-08-23 Thread Rikard Falkeborn
The only usage is to pass its address to devm_phy_create() which takes a const pointer. Make it const to allow the compiler to put it in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/lantiq/phy-lantiq-rcu-usb2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --gi

[PATCH 0/8] drivers: phy: Constify static phy_ops structs

2020-08-23 Thread Rikard Falkeborn
This series constifies all static phy_ops structs in drivers/phy. Typically the only usage is to pass the address of it to devm_phy_create() which takes a const pointer. The lone exception is in drivers/phy/qualcomm/phy-qcom-ipq4019-usb.c where the address of the structs is assigned to the data-fie

[PATCH 3/8] phy: hisilicon; Constify hi3660_phy_ops

2020-08-23 Thread Rikard Falkeborn
The only usage is to pass its address to devm_phy_create() which takes a const pointer. Make it const to allow the compiler to put it in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/hisilicon/phy-hi3660-usb3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/8] phy: fsl-imx8mq-usb: Constify imx8mq_usb_phy_ops

2020-08-23 Thread Rikard Falkeborn
The only usage is to pass its address to devm_phy_create() which takes a const pointer. Make it const to allow the compiler to put it in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/freescale/phy-fsl-imx8mq-usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --

Re: [GIT pull] x86/urgent for v5.9-rc2

2020-08-23 Thread Thomas Gleixner
On Sun, Aug 23 2020 at 11:29, Linus Torvalds wrote: > On Sun, Aug 23, 2020 at 1:26 AM Thomas Gleixner wrote: >> >> Remove the RDPID optimization, which is not even >> backed by numbers from the paranoid entry path instead. > > Ugh, that's sad. I'd expect the LSL to be quite a bit slower than the >

Re: [PATCH 0/8] drivers: phy: Constify static phy_ops structs

2020-08-23 Thread Joe Perches
On Mon, 2020-08-24 at 00:00 +0200, Rikard Falkeborn wrote: > This series constifies all static phy_ops structs in drivers/phy. > Typically the only usage is to pass the address of it to devm_phy_create() > which takes a const pointer. The lone exception is in > drivers/phy/qualcomm/phy-qcom-ipq4019

arch/powerpc/kexec/core.c:246:29: sparse: sparse: incorrect type in assignment (different base types)

2020-08-23 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d012a7190fc1fd72ed48911e77ca97ba4521bccd commit: 793b08e2efff3ec020c5c5861d00ed394fcdd488 powerpc/kexec: Move kexec files into a dedicated subdir. date: 9 months ago config: powerpc64-randconfig-s032-20200

Re: [PATCH v2] drm/rcar-du: DRM_RCAR_WRITEBACK depends on DRM

2020-08-23 Thread Laurent Pinchart
Hi Qian, On Fri, Aug 21, 2020 at 09:20:37PM -0400, Qian Cai wrote: > On Mon, Jun 08, 2020 at 06:16:22AM +0300, Laurent Pinchart wrote: > > Hi Qian, > > > > I forgot to mention, I think the subject line should be > > > > drm/rcar-du: Make DRM_RCAR_WRITEBACK depend on DRM_RCAR_DU > > > > Could yo

Re: [GIT pull] x86/urgent for v5.9-rc2

2020-08-23 Thread Andy Lutomirski
On Sun, Aug 23, 2020 at 11:29 AM Linus Torvalds wrote: > > On Sun, Aug 23, 2020 at 1:26 AM Thomas Gleixner wrote: > > > > Remove the RDPID optimization, which is not even > > backed by numbers from the paranoid entry path instead. > > Ugh, that's sad. I'd expect the LSL to be quite a bit slower t

Re: [PATCH v2 2/2] net: socket: implement SO_DESCRIPTION

2020-08-23 Thread Pascal Bouchareine
On Sat, Aug 22, 2020 at 2:01 PM David Miller wrote: > > About the pid part - > > On top of multiple pids to scan for a given socket, there's also the > > security provided by /proc - I'm not sure what inet_diag does for that > > So maybe users calling it will need to scan /proc for a long time any

Re: [GIT pull] x86/urgent for v5.9-rc2

2020-08-23 Thread Linus Torvalds
On Sun, Aug 23, 2020 at 3:01 PM Thomas Gleixner wrote: > > > And I don't think we need the GDT for anything else normally, so it's > > not even going to be cached. > > Who cares, really? > > It's pretty irrelevant because the main source of horrors are in having > to run through _ALL_ registered N

Re: [PATCH v1 3/6] leds: Add driver for Acer Iconia Tab A500

2020-08-23 Thread Pavel Machek
Hi! > Acer Iconia Tab A500 is an Android tablet device which has two LEDs > embedded into the Power Button. Orange LED indicates "battery charging" > status and white LED indicates "wake-up/charge-done" status. The new LED > driver provides control over both LEDs to userspace. > @@ -0,0 +1,121 @@

Re: [PATCH v1 3/6] leds: Add driver for Acer Iconia Tab A500

2020-08-23 Thread Pavel Machek
On Sun 2020-08-23 17:08:43, Dmitry Osipenko wrote: > Acer Iconia Tab A500 is an Android tablet device which has two LEDs > embedded into the Power Button. Orange LED indicates "battery charging" > status and white LED indicates "wake-up/charge-done" status. The new LED > driver provides control ove

Re: [GIT pull] x86/urgent for v5.9-rc2

2020-08-23 Thread Linus Torvalds
On Sun, Aug 23, 2020 at 3:27 PM Andy Lutomirski wrote: > > Every interrupt is going to load the CS and SS descriptor cache lines. Yeah, but this isn't even sharing the same GDT cache line. Those two are at least in the same cacheline, and hey, that is forced upon us by the architecture, so we don

[PATCH -next] amdgpu: fix Documentation builds for pm/ file movement

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix Documentation errors for amdgpu.rst due to file rename (moved to another subdirectory). Error: Cannot open file ../drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c WARNING: kernel-doc '../scripts/kernel-doc -rst -enable-lineno -function hwmon ../drivers/gpu/drm/amd/amdgpu/amdgpu_pm

Re: [RFC PATCH 05/10] dt-bindings: pwm: Add bindings for PWM function in Netronix EC

2020-08-23 Thread Jonathan Neuschäfer
On Sun, Jun 21, 2020 at 08:41:23PM +0200, Andreas Kemnade wrote: > On Sun, 21 Jun 2020 00:42:16 +0200 > Jonathan Neuschäfer wrote: > > > The Netronix embedded controller as found in Kobo Aura and Tolino Shine > > supports one PWM channel, which is used to control the frontlight > > brightness on

[PATCH] Documentation: laptops: thinkpad-acpi: fix underline length build warning

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix underline length build warning in thinkpad-acpi.rst documentation: Documentation/admin-guide/laptops/thinkpad-acpi.rst:1437: WARNING: Title underline too short. DYTC Lapmode sensor -- Fixes: acf7f4a59114 ("platform/x86: thinkpad_acpi: lap or desk mode int

Re: [PATCH v2 1/2] arm64: defconfig: Enable USB gadget with configfs

2020-08-23 Thread Olof Johansson
Hi, On Sun, Aug 23, 2020 at 1:42 AM Krzysztof Kozlowski wrote: > > On Sun, Aug 23, 2020 at 09:46:01AM +0800, Shawn Guo wrote: > > On Mon, Aug 17, 2020 at 08:56:53AM +0200, Krzysztof Kozlowski wrote: > > > USB OTG connections are pretty common for embedded and development > > > boards, for example

Re: [PATCH v2 3/8] x86/debug: Move kprobe_debug_handler() into exc_debug_kernel()

2020-08-23 Thread Andy Lutomirski
On Fri, Aug 21, 2020 at 3:21 AM Peter Zijlstra wrote: > > Kprobes are on kernel text, and thus only matter for #DB-from-kernel. > Kprobes are ordered before the generic notifier, preserve that order. > Acked-by: Andy Lutomirski

[PATCH] Documentation: fix pm/intel_pstate build warning and wording

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix documentation build warning and sentence wording: Documentation/admin-guide/pm/intel_pstate.rst:568: WARNING: Unexpected indentation. Fixes: f473bf398bf1 ("cpufreq: intel_pstate: Allow raw energy performance preference value") Signed-off-by: Randy Dunlap Cc: Srinivas P

Re: [PATCH v2 2/8] x86/debug: Sync BTF earlier

2020-08-23 Thread Andy Lutomirski
On Fri, Aug 21, 2020 at 3:21 AM Peter Zijlstra wrote: > > Move the BTF sync near the DR6 load, as this will be the only common > code guaranteed to run on every #DB. I don't think this code is strictly correct, but I think your change is still an improvement. Acked-by: Andy Lutomirski

Re: [PATCH v2 4/8] x86/debug: Remove handle_debug(.user) argument

2020-08-23 Thread Andy Lutomirski
On Fri, Aug 21, 2020 at 3:21 AM Peter Zijlstra wrote: > > The handle_debug(.user) argument is used to terminate the #DB handler > early for the INT1-from-kernel case, since the kernel doesn't use > INT1. > > Remove the argument and handle this explicitly in #DB-from-kernel. Acked-by: Andy Lutomir

[PATCH v8 2/3] arm64: mm: reserve per-numa CMA to localize coherent dma buffers

2020-08-23 Thread Barry Song
Right now, smmu is using dma_alloc_coherent() to get memory to save queues and tables. Typically, on ARM64 server, there is a default CMA located at node0, which could be far away from node2, node3 etc. with this patch, smmu will get memory from local numa node to save command queues and page table

Re: [PATCH v12 09/13] drm/msm: Add support to create a local pagetable

2020-08-23 Thread Guenter Roeck
On Mon, Aug 10, 2020 at 04:26:53PM -0600, Jordan Crouse wrote: > Add support to create a io-pgtable for use by targets that support > per-instance pagetables. In order to support per-instance pagetables the > GPU SMMU device needs to have the qcom,adreno-smmu compatible string and > split pagetable

[PATCH v8 1/3] dma-contiguous: provide the ability to reserve per-numa CMA

2020-08-23 Thread Barry Song
Right now, drivers like ARM SMMU are using dma_alloc_coherent() to get coherent DMA buffers to save their command queues and page tables. As there is only one default CMA in the whole system, SMMUs on nodes other than node0 will get remote memory. This leads to significant latency. This patch prov

[PATCH v8 0/3] make dma_alloc_coherent NUMA-aware by per-NUMA CMA

2020-08-23 Thread Barry Song
Ganapatrao Kulkarni has put some effort on making arm-smmu-v3 use local memory to save command queues[1]. I also did similar job in patch "iommu/arm-smmu-v3: allocate the memory of queues in local numa node" [2] while not realizing Ganapatrao has done that before. But it seems it is much better to

[PATCH v8 3/3] mm: cma: use CMA_MAX_NAME to define the length of cma name array

2020-08-23 Thread Barry Song
CMA_MAX_NAME should be visible to CMA's users as they might need it to set the name of CMA areas and avoid hardcoding the size locally. So this patch moves CMA_MAX_NAME from local header file to include/linux header file and removes the hardcode in both hugetlb.c and contiguous.c. Cc: Mike Kravetz

Re: [PATCH] mmc: mmc_spi: fix timeout calculation

2020-08-23 Thread Tobias Schramm
Hi Uffe, > This looks good to me, but before applying just wanted to check that > you tested this on some HW, to make sure it doesn't break anything? yes, I should have mentioned that. I tested the change on a custom STM32H743 board with a microSD card connected to one of its SPIs. It does still

Re: [PATCH v2 5/8] x86/debug: Simplify #DB signal code

2020-08-23 Thread Andy Lutomirski
On Fri, Aug 21, 2020 at 3:21 AM Peter Zijlstra wrote: > > Get rid of the two variables, avoid computing si_code when not needed > and be consistent about which dr6 value is used. > > - if (tsk->thread.debugreg6 & (DR_STEP | DR_TRAP_BITS) || user_icebp) > - send_sigtrap(regs, 0

Re: [PATCH v2 8/8] x86/debug: Remove the historical junk

2020-08-23 Thread Andy Lutomirski
On Fri, Aug 21, 2020 at 3:21 AM Peter Zijlstra wrote: > > Remove the historical junk and replace it with a WARN and a comment. > > The problem is that even though the kernel only uses TF single-step in > kprobes and KGDB, both of which consume the event before this, > QEMU/KVM has bugs in this are

Re: [GIT pull] x86/urgent for v5.9-rc2

2020-08-23 Thread Andy Lutomirski
On Sun, Aug 23, 2020 at 3:35 PM Linus Torvalds wrote: > > On Sun, Aug 23, 2020 at 3:27 PM Andy Lutomirski wrote: > > > > Every interrupt is going to load the CS and SS descriptor cache lines. > > Yeah, but this isn't even sharing the same GDT cache line. Those two > are at least in the same cache

Re: dt-bindings: display: xlnx: mixer: Inconsistent pixel format terminology in dt docs

2020-08-23 Thread Laurent Pinchart
Hello, On Thu, Aug 20, 2020 at 04:38:18PM -0700, Hyun Kwon wrote: > On Thursday, August 20, 2020 2:18 PM, Kenneth Sloat write: > > Hello, > > > > The Xilinx Video mixer IP uses the DRM fourcc string as a device tree > > binding in > > order to describe the format for a specific DRM layer/plane.

arch/powerpc/platforms/4xx/gpio.c:63:28: sparse: sparse: incorrect type in argument 1 (different base types)

2020-08-23 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d012a7190fc1fd72ed48911e77ca97ba4521bccd commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces date: 9 weeks ago config: powerpc64-randconfig-s032-20200824 (atta

[PATCH] Documentation: fix dma-buf.rst underline length warning

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap /home/rdunlap/lnx/lnx-59-rc2/Documentation/driver-api/dma-buf.rst:182: WARNING: Title underline too short. Indefinite DMA Fences Fixes: 72b6ede73623 ("dma-buf.rst: Document why indefinite fences are a bad idea") Signed-off-by: Randy Dunlap Cc: Daniel Ve

Re: [PATCH] scsi: scsi_debug: Remove superfluous close zone in resp_open_zone()

2020-08-23 Thread Damien Le Moal
On 2020/08/21 22:00, Niklas Cassel wrote: > resp_open_zone() always calls zbc_open_zone() with parameter explicit > set to true. > > If zbc_open_zone() is called with parameter explicit set to true, and > the current zone state is implicit open, it will call zbc_close_zone() > on the zone before p

[PATGCH -next] LSM: SafeSetID: fix underline length warning

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix documentation build warning: Documentation/admin-guide/LSM/SafeSetID.rst:110: WARNING: Title underline too short. Note on GID policies and setgroups() == Fixes: 02e316b088df ("LSM: SafeSetID: Add GID security policy handling") Signed-off-by: Randy Dunlap

Re: [PATCH] ARM: dts: imx6sx-sdb: Add headphone detection for sound card

2020-08-23 Thread Shawn Guo
On Sun, Aug 23, 2020 at 11:12:08AM -0300, Fabio Estevam wrote: > Hi Wang Shengjiu and Shawn, > > On Sun, Aug 23, 2020 at 9:31 AM S.j. Wang wrote: > > > I would like to know your opinion, should I move headphone detect GPIO > > To audmux group? > > What about adding a dedicated pinctrl_hp for th

Re: [PATCH 0/4] mm: Simplfy cow handling

2020-08-23 Thread Peter Xu
On Sat, Aug 22, 2020 at 09:05:37AM -0700, Linus Torvalds wrote: > On Fri, Aug 21, 2020 at 4:50 PM Peter Xu wrote: > > > > - Run a busy loop dirty program [1] that uses 6G of memory, restrict to 1G > > RAM + 5G swap (cgroup). A few hours later, all things still look good. > > Make sure t

Re: kernel BUG at kernel/fork.c:LINE!

2020-08-23 Thread Shakeel Butt
On Sun, Aug 23, 2020 at 10:34 AM Randy Dunlap wrote: > > On 8/7/20 2:16 PM, syzbot wrote: > > Hello, > > > > syzbot found the following issue on: > > > > HEAD commit:fffe3ae0 Merge tag 'for-linus-hmm' of git://git.kernel.org.. > > git tree: upstream > > console output: https://syzkaller.

[PATCH] Documentation: sound/cards: fix heading underline lengths for https: changes

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix documentation build warnings for underline length too short, caused by s/http/https/ and not changing the accompanying underlines. Documentation/sound/cards/audigy-mixer.rst:335: WARNING: Title underline too short. US Patents (https://www.uspto.gov/) -

Re: [PATCH 0/8] Convert the intel iommu driver to the dma-iommu api

2020-08-23 Thread Tom Murphy
Hi Logan/All, I have added a check for the sg_dma_len == 0 : """ } __sgt_iter(struct scatterlist *sgl, bool dma) { struct sgt_iter s = { .sgp = sgl }; + if (sgl && sg_dma_len(sgl) == 0) + s.sgp = NULL; if (s.sgp) { . """ at location [1]. but it do

Re: [PATCH] KVM: LAPIC: Don't kick vCPU which is injecting already-expired timer

2020-08-23 Thread Wanpeng Li
On Sat, 22 Aug 2020 at 12:01, Sean Christopherson wrote: > > On Fri, Aug 21, 2020 at 05:17:34PM +0800, Wanpeng Li wrote: > > From: Wanpeng Li > > > > The kick after setting KVM_REQ_PENDING_TIMER is used to handle the timer > > fires on a different pCPU which vCPU is running on, we don't need this

Re: [PATCH 4/4] mm: Add PGREUSE counter

2020-08-23 Thread Peter Xu
On Sat, Aug 22, 2020 at 09:14:53AM -0700, Linus Torvalds wrote: > On Fri, Aug 21, 2020 at 4:50 PM Peter Xu wrote: > > > > This accounts for wp_page_reuse() case, where we reused a page for COW. > > If we do this, wouldn't it make more sense to also count the COW case > to see how they match up?

linux-next: build failure after merge of the hwmon-staging tree

2020-08-23 Thread Stephen Rothwell
Hi all, After merging the hwmon-staging tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/hwmon/pmbus/adm1266.c: In function 'adm1266_probe': drivers/hwmon/pmbus/adm1266.c:478:31: error: passing argument 2 of 'pmbus_do_probe' from incompatible pointer type [-Werror=

[PATCH] Documentation/powerpc: fix malformed table in syscall64-abi

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Fix malformed table warning in powerpc/syscall64-abi.rst by making two tables and moving the headings. Documentation/powerpc/syscall64-abi.rst:53: WARNING: Malformed table. Text in column margin in table line 2. === = ==

[PATCH] Documentation: submit-checklist: add Documentation clean builds

2020-08-23 Thread Randy Dunlap
From: Randy Dunlap Add to Documentation/process/submit-checklist.rst that patch submitters should run "make htmldocs" and verify that any Documentation/ changes (patches) are clean (no new warnings/errors). Signed-off-by: Randy Dunlap --- Documentation/process/submit-checklist.rst |4

Re: [PATCH v1 05/10] powerpc/pseries/iommu: Add iommu_pseries_alloc_table() helper

2020-08-23 Thread Alexey Kardashevskiy
On 18/08/2020 09:40, Leonardo Bras wrote: > Creates a helper to allow allocating a new iommu_table without the need > to reallocate the iommu_group. > > This will be helpful for replacing the iommu_table for the new DMA window, > after we remove the old one with iommu_tce_table_put(). > > Sign

Re: ..and it built & boot without fuss!!Linux 5.9-rc2

2020-08-23 Thread Bhaskar Chowdhury
On 14:23 Sun 23 Aug 2020, Linus Torvalds wrote: It's Sunday afternoon, which means it's time for another release candidate. Nothing in particular stands out, there's a random collection of fixes and updates in here. It is perhaps a bit filesystem-heavy, because the ext4 updates came in late, so

Re: [PATCH v2] ARM: dts: r8a7742-iwg21d-q7: Add LCD support

2020-08-23 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Thu, Aug 13, 2020 at 03:00:41PM +0100, Lad Prabhakar wrote: > The iwg21d comes with a 7" capacitive touch screen, therefore > add support for it. > > Signed-off-by: Lad Prabhakar > Reviewed-by: Marian-Cristian Rotariu > Everything seems to match the

Re: [PATCH v8 1/4] power: supply: core: add quick charge type property

2020-08-23 Thread Qiwu Huang
On Tue, Aug 18, 2020 at 2:07 PM Greg KH wrote: > > On Tue, Aug 18, 2020 at 09:56:28AM +0800, Qiwu Huang wrote: > > On Fri, Aug 14, 2020 at 2:09 PM Greg KH wrote: > > > > > > On Fri, Aug 14, 2020 at 11:46:54AM +0800, Qiwu Huang wrote: > > > > From: Qiwu Huang > > > > > > > > Reports the kind of q

linux-next: manual merge of the bpf-next tree with Linus' tree

2020-08-23 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the bpf-next tree got a conflict in: tools/lib/bpf/libbpf.c between commit: 1e891e513e16 ("libbpf: Fix map index used in error message") from Linus' tree and commit: 88a82120282b ("libbpf: Factor out common ELF operations and improve logging") from t

[tip:timers/core] BUILD SUCCESS e2d977c9f1abd1d199b412f8f83c1727808b794d

2020-08-23 Thread kernel test robot
powerpc allmodconfig powerpc allnoconfig x86_64 randconfig-a002-20200823 x86_64 randconfig-a003-20200823 x86_64 randconfig-a005-20200823 x86_64 randconfig-a001-20200823 x86_64

Re: [PATCH v3 2/3] media: dt-bindings: media: xilinx: Add Xilinx UHD-SDI Receiver Subsystem

2020-08-23 Thread Laurent Pinchart
Hi Vishal, On Wed, Aug 19, 2020 at 07:40:16PM +0300, Laurent Pinchart wrote: > On Wed, Aug 19, 2020 at 01:45:34PM +, Vishal Sagar wrote: > > On Wednesday, July 15, 2020 9:59 PM, Laurent Pinchart wrote: > >> On Mon, Jul 13, 2020 at 12:54:47PM -0600, Rob Herring wrote: > >>> On Thu, Jun 18, 2020

[PATCH v2] KVM: LAPIC: Narrow down the kick target vCPU

2020-08-23 Thread Wanpeng Li
From: Wanpeng Li The kick after setting KVM_REQ_PENDING_TIMER is used to handle the timer fires on a different pCPU which vCPU is running on, this kick is expensive since memory barrier, rcu, preemption disable/enable operations. We don't need this kick when injecting already-expired timer, we

Re: [PATCH v2 2/2] KVM: LAPIC: Guarantee the timer is in tsc-deadline mode when setting

2020-08-23 Thread Wanpeng Li
ping :) On Wed, 12 Aug 2020 at 14:30, Wanpeng Li wrote: > > From: Wanpeng Li > > Check apic_lvtt_tscdeadline() mode directly instead of apic_lvtt_oneshot() > and apic_lvtt_period() to guarantee the timer is in tsc-deadline mode when > wrmsr MSR_IA32_TSCDEADLINE. > > Signed-off-by: Wanpeng Li > -

答复: [PATCH] mmc: core: Initial support for SD express card/host

2020-08-23 Thread 冯锐
Hi Hansson: If this patch will not be changed, I will post a patch for rtsx driver according your patch. > > Rui, > > On Thu, 16 Jul 2020 at 16:16, Ulf Hansson wrote: > > > > In the SD specification v7.10 the SD express card has been added. This > > new type of removable SD card, can be manag

[PATCH] mmc: sd: Use kobj_to_dev() instead of container_of()

2020-08-23 Thread Tian Tao
Use kobj_to_dev() instead of container_of() Signed-off-by: Tian Tao --- drivers/mmc/core/sd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c index 5a2210c..a0d2c34 100644 --- a/drivers/mmc/core/sd.c +++ b/drivers/mmc/core/sd.c @@

Re: [PATCH v2] ARM: dts: imx7d-zii-rmu2: fix rgmii phy-mode for ksz9031 phy

2020-08-23 Thread Fabio Estevam
Hi Chris, On Sat, Aug 22, 2020 at 11:25 PM Chris Healy wrote: > > From: Chris Healy > > Since commit bcf3440c6dd7 ("net: phy: micrel: add phy-mode support for the > KSZ9031 PHY") the networking is broken on the imx7d-zii-rmu2 board. > > The end result is that network receive behaviour is margina

Re: [PATCH] Add missing bound checks for software 842 decompressor

2020-08-23 Thread Joan Bruguera
On 19.08.20 22:19, Kees Cook wrote: On Sun, Aug 16, 2020 at 02:33:41AM +0200, Joan Bruguera wrote: Any feedback? Hi! I just happened to see this email. I think this should likely be directed to the crypto (which also handled compress/decompress APIs) list and the original author... Additi

Re: [PATCH 1/2] KVM: LAPIC: Fix updating DFR missing apic map recalculation

2020-08-23 Thread Wanpeng Li
ping, :) On Wed, 19 Aug 2020 at 16:55, Wanpeng Li wrote: > > From: Wanpeng Li > > There is missing apic map recalculation after updating DFR, if it is > INIT RESET, in x2apic mode, local apic is software enabled before. > This patch fix it by introducing the function kvm_apic_set_dfr() to > be ca

Re: [PATCH] ntfs: add check for mft record size in superblock

2020-08-23 Thread Rustam Kovhaev
On Mon, Aug 24, 2020 at 01:44:06AM +, Anton Altaparmakov wrote: > Hi Rustam, > > Thank you for the patch but it introduces an endianness bug - you have to us > le32_to_cpu(m->bytes_allocated) both when doing the comparison and then > printing the message. > > Also, please drop the square br

Re: [PATCH] ntfs: add check for mft record size in superblock

2020-08-23 Thread Anton Altaparmakov
Hi Rustam, Thank you for the patch but it introduces an endianness bug - you have to us le32_to_cpu(m->bytes_allocated) both when doing the comparison and then printing the message. Also, please drop the square brackets. Wherever the driver prints such things it never uses brackets around the

[PATCH] regulator: rt4801: Fix W=1 build warning when CONFIG_OF=n

2020-08-23 Thread cy_huang
From: ChiYuan Huang Fix below warning when CONFIG_OF=n: drivers/regulator/rt4801-regulator.c:206:34: warning: unused variable 'rt4801_of_id' [-Wunused-const-variable] 206 | static const struct of_device_id rt4801_of_id[] = { | ^~~~ Signed-off-by

Re: [RFC PATCH 1/5] printk: implement pr_cont_t

2020-08-23 Thread Sergey Senozhatsky
On (20/08/20 12:16), Petr Mladek wrote: > Now that I think about it. This is the biggest problem with any temporary > buffer > for pr_cont() lines. I am more and more convinced that we should just > _keep the current behavior_. It is not ideal. But sometimes mixed > messages are always better than

Re: Re: [PATCH] watchdog: Fix double-free in watchdog_cdev_register

2020-08-23 Thread dinghao . liu
> On 8/23/20 12:13 AM, Dinghao Liu wrote: > > When misc_register() fails, wd_data will be released by the > > release callback function watchdog_core_data_release(), so > > we don't need to free it again. But when watchdog_kworker is > > NULL, we should free wd_data to prevent memleak. > > > > Fix

Re: [PATCH v8 2/3] drm: bridge: Add support for Cadence MHDP DPI/DP bridge

2020-08-23 Thread Laurent Pinchart
Hi Tomi, On Fri, Aug 14, 2020 at 11:22:09AM +0300, Tomi Valkeinen wrote: > On 11/08/2020 05:36, Laurent Pinchart wrote: > > >> +static int cdns_mhdp_connector_init(struct cdns_mhdp_device *mhdp) > >> +{ > >> + u32 bus_format = MEDIA_BUS_FMT_RGB121212_1X36; > >> + struct drm_connector *conn = &m

Re: [PATCH v8 2/3] drm: bridge: Add support for Cadence MHDP DPI/DP bridge

2020-08-23 Thread Laurent Pinchart
Hi Tomi, On Fri, Aug 14, 2020 at 12:29:35PM +0300, Tomi Valkeinen wrote: > On 11/08/2020 05:36, Laurent Pinchart wrote: > > >> +static int cdns_mhdp_mailbox_write(struct cdns_mhdp_device *mhdp, u8 val) > >> +{ > >> + int ret, full; > >> + > >> + WARN_ON(!mutex_is_locked(&mhdp->mbox_mutex)); > >

[PATCH] ntfs: add check for mft record size in superblock

2020-08-23 Thread Rustam Kovhaev
number of bytes allocated for mft record should be equal to the mft record size stored in ntfs superblock as reported by syzbot, userspace might trigger out-of-bounds read by dereferencing ctx->attr in ntfs_attr_find() Reported-and-tested-by: syzbot+aed06913f36eff9b5...@syzkaller.appspotmail.com L

{standard input}:5973: Error: operand out of range (512 is not between -512 and 511)

2020-08-23 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d012a7190fc1fd72ed48911e77ca97ba4521bccd commit: de8f5e4f2dc1f032b46afda0a78cab5456974f89 lockdep: Introduce wait-type checks date: 5 months ago config: arc-randconfig-r004-20200824 (attached as .config) c

Re: [PATCH v2 3/9] iommu/ioasid: Introduce ioasid_set APIs

2020-08-23 Thread Lu Baolu
Hi Jacob, On 8/22/20 12:35 PM, Jacob Pan wrote: ioasid_set was introduced as an arbitrary token that are shared by a group of IOASIDs. For example, if IOASID #1 and #2 are allocated via the same ioasid_set*, they are viewed as to belong to the same set. For guest SVA usages, system-wide IOASID

Re: [PATCH v2 4/9] iommu/ioasid: Add reference couting functions

2020-08-23 Thread Lu Baolu
Hi Jacob, On 8/22/20 12:35 PM, Jacob Pan wrote: There can be multiple users of an IOASID, each user could have hardware contexts associated with the IOASID. In order to align lifecycles, reference counting is introduced in this patch. It is expected that when an IOASID is being freed, each user

[PATCH] regulator: mp886x: use "mps,switch-frequency-hz"

2020-08-23 Thread Jisheng Zhang
As Rob suggested, use the "mps,switch-frequency-hz" instead of the "mps,switch-frequency" for switch frequency. Fortunately, the switch frequency support isn't released, so we can modify it now without any concern. Signed-off-by: Jisheng Zhang --- Documentation/devicetree/bindings/regulator/mp88

Re: [PATCH v2 1/2] irqchip: irq-mst: Add MStar interrupt controller support

2020-08-23 Thread Mark-PK Tsai
From: Daniel Palmer >Hi Mark-PK, > >On Thu, 20 Aug 2020 at 00:38, Mark-PK Tsai wrote: >> >> Add MStar interrupt controller support using hierarchy irq >> domain. >> >> Signed-off-by: Mark-PK Tsai > >I've integrated this version into my MStar/SigmaStar tree and tested >on an MStar MSC313E >based

[PATCH] watchdog: Fix memleak in watchdog_cdev_register

2020-08-23 Thread Dinghao Liu
When watchdog_kworker is NULL, we should free wd_data before the function returns to prevent memleak. Fixes: 664a39236e718 ("watchdog: Introduce hardware maximum heartbeat in watchdog core") Signed-off-by: Dinghao Liu --- drivers/watchdog/watchdog_dev.c | 4 +++- 1 file changed, 3 insertions(+)

Re: [PATCH] usb: hcd: fix the error check

2020-08-23 Thread Peter Chen
On 20-08-20 20:20:38, Tang Bin wrote: > In the function usb_add_hcd(), usb_phy_roothub_alloc() > can return NULL in some cases, so IS_ERR() doesn't meet > the requirements. Thus fix it. > Signed-off-by: Zhang Shengju > Signed-off-by: Tang Bin > --- > drivers/usb/core/hcd.c | 5 +++-- > 1 file ch

Re: [PATCH v5 2/2] Add Intel LGM soc DMA support.

2020-08-23 Thread Reddy, MallikarjunaX
Hi Peter, Thanks for the review comments. Please see my comments inline.. On 8/18/2020 6:16 PM, Peter Ujfalusi wrote: Hi, On 14/08/2020 8.26, Amireddy Mallikarjuna reddy wrote: Add DMA controller driver for Lightning Mountain(LGM) family of SoCs. The main function of the DMA controller is the

Re: [PATCH] ntfs: add check for mft record size in superblock

2020-08-23 Thread Anton Altaparmakov
Hi Andrew, Can you please merge this patch? Thanks a lot in advance! Rustam, thank you for the updated patch! Best regards, Anton > On 24 Aug 2020, at 03:28, Rustam Kovhaev wrote: > > number of bytes allocated for mft record should be equal to the mft > record size stored in ntfs su

Re: [mm] c566586818: BUG:kernel_hang_in_early-boot_stage,last_printk:Probing_EDD(edd=off_to_disable)...ok

2020-08-23 Thread Rong Chen
On 8/21/20 9:01 AM, Qian Cai wrote: On Tue, Aug 18, 2020 at 08:23:51AM +0800, kernel test robot wrote: Greeting, FYI, we noticed the following commit (built with gcc-9): commit: c5665868183fec689dbab9fb8505188b2c4f0757 ("mm: kmemleak: use the memory pool for early allocations") I might se

Re: [RESEND PATCH 4/5] iommu: intel: Drop kerneldoc marker from regular comment

2020-08-23 Thread Lu Baolu
Hi, On 8/20/20 1:53 AM, Krzysztof Kozlowski wrote: Fix W=1 compile warnings (invalid kerneldoc): drivers/iommu/intel/dmar.c:389: warning: Function parameter or member 'header' not described in 'dmar_parse_one_drhd' Signed-off-by: Krzysztof Kozlowski Adjust the commit title to "iommu/v

[PATCH] MAINTAINERS: Add maintenance information for MStar Interrupt Controller

2020-08-23 Thread Mark-PK Tsai
Add entry for MStar Interrupt Controller. Signed-off-by: Mark-PK Tsai --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index deaafb617361..8ab08fccd915 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11754,6 +11754,13 @@ Q: http://patchwork

[PATCH] staging: greybus: fix warnings detected by sparse

2020-08-23 Thread Coiby Xu
This patch fix the following warnings from sparse, $ make C=2 drivers/staging/greybus/ drivers/staging/greybus/audio_codec.c:691:36: warning: incorrect type in initializer (different base types) drivers/staging/greybus/audio_codec.c:691:36:expected unsigned long long [usertype] formats drive

[PATCH] mtd: maps: physmap: Retain mtd-name property from dts

2020-08-23 Thread Chris Packham
In physmap_flash_of_init() the maps[].name can be populated based on the optional 'linux,mtd-name' property in the dts. Make sure this is retained when filling in the rest of the map[] data. Signed-off-by: Chris Packham --- drivers/mtd/maps/physmap-core.c | 3 ++- 1 file changed, 2 insertions(+)

[PATCH v5] kernel/trace: Add DISALLOW_TRACE_PRINTK make option

2020-08-23 Thread Nicolas Boichat
trace_printk is meant as a debugging tool, and should not be compiled into production code without specific debug Kconfig options enabled, or source code changes, as indicated by the warning that shows up on boot if any trace_printk is called: ** NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE

[PATCH] misc: rtsx: do not setting OC_POWER_DOWN reg in rtsx_pci_init_ocp()

2020-08-23 Thread ricky_wu
From: Ricky Wu this power saving action in rtsx_pci_init_ocp() cause INTEL-NUC6 platform missing card reader Signed-off-by: Ricky Wu --- drivers/misc/cardreader/rtsx_pcr.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/misc/cardreader/rtsx_pcr.c b/drivers/misc/cardreader/rtsx_

RE: [PATCH -next] amdgpu: fix Documentation builds for pm/ file movement

2020-08-23 Thread Quan, Evan
[AMD Official Use Only - Internal Distribution Only] Thanks for fixing this. The patch is reviewed-by: Evan Quan BR Evan -Original Message- From: Randy Dunlap Sent: Monday, August 24, 2020 6:36 AM To: dri-devel ; LKML ; amd-...@lists.freedesktop.org; Deucher, Alexander Cc: Quan, Evan

[PATCH] ceph: add column 'mds' to show caps in more user friendly

2020-08-23 Thread Yanhu Cao
In multi-mds, the 'caps' debugfs file will have duplicate ino, add the 'mds' column to indicate which mds session the cap belongs to. Signed-off-by: Yanhu Cao --- fs/ceph/debugfs.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/fs/ceph/debugfs.c b/fs/ceph/debugfs.c in

[PATCH] [v2] watchdog: Use put_device on error

2020-08-23 Thread Dinghao Liu
We should use put_device() instead of freeing device directly after device_initialize(). Fixes: cb36e29bb0e4b ("watchdog: initialize device before misc_register") Signed-off-by: Dinghao Liu --- Changelog: v2: - Use put_device() instead of just removing kfree. Move the memleak part to a se

[PATCH RESEND 00/10] crypto: hisilicon/zip - misc clean up

2020-08-23 Thread Yang Shen
This patchset make some clean up: patch 1:remove useless parameters patch 4:replace 'sprintf' with 'scnprintf' patch 7:fix static check warning and the rest patch fix some coding style Resend this patch series because it depends on https://patchwork.kernel.org/cover/11715927/ (crypto: hisilicon/qm

[PATCH RESEND 01/10] crypto: hisilicon/zip - remove some useless parameters

2020-08-23 Thread Yang Shen
1.Remove the macro 'HZIP_VF_NUM'. 2.Remove 'list' of the struct 'hisi_zip' Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip.h | 1 - drivers/crypto/hisilicon/zip/zip_main.c | 1 - 2 files changed, 2 deletions(-) diff --git a/drivers/crypto/hisilicon/zip

[PATCH RESEND 05/10] crypto: hisilicon/zip - use a enum parameter instead of some macros

2020-08-23 Thread Yang Shen
Macros 'QPC_COMP', 'QPC_DECOMP' and 'HZIP_CTX_Q_NUM' are relative and incremental. So, use an enum instead. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_crypto.c | 23 +-- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a

[PATCH RESEND 09/10] crypto: hisilicon/zip - supplement some comments

2020-08-23 Thread Yang Shen
Supplement some comments. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/hisilicon/zip/zip_main.c b/drivers/crypto/hisilicon/zip/zip_main.c index 7697fa5..49fad18

[PATCH RESEND 08/10] crypto: hisilicon/zip - move some private macros from 'zip.h' to 'zip_crypto.c'

2020-08-23 Thread Yang Shen
Some macros which are defined in 'zip.h' are related to the struct 'hisi_zip_sqe' and are only used in 'zip_crypto.c'. So move them from 'zip.h' to 'zip_crypto.c'. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip.h| 14 -- drivers/crypto/h

[PATCH RESEND 03/10] crypto: hisilicon/zip - modify debugfs interface parameters

2020-08-23 Thread Yang Shen
From: Shukun Tan Update debugfs interface parameters Signed-off-by: Shukun Tan Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_main.c | 55 ++--- 1 file changed, 24 insertions(+), 31 deletions(-) diff --git a/drivers/crypto/hi

[PATCH RESEND 04/10] crypto: hisilicon/zip - replace 'sprintf' with 'scnprintf'

2020-08-23 Thread Yang Shen
Replace 'sprintf' with 'scnprintf' to avoid overrun. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_main.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/crypto/hisilicon/zip/zip_main.c b/drivers/crypto/hisilicon/zi

[PATCH RESEND 07/10] crypto: hisilicon/zip - fix static check warning

2020-08-23 Thread Yang Shen
Fix some code for PClint warning: Warning - Suspicious Cast Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_crypto.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/crypto/hisilicon/zip/zip_crypto.c b/drivers/cry

[PATCH RESEND 10/10] crypto: hisilicon/zip - fix some coding styles

2020-08-23 Thread Yang Shen
1.Unified alignment styles 2.Remove unnecessary goto branch 3.Remove address printf Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_crypto.c | 13 ++--- drivers/crypto/hisilicon/zip/zip_main.c | 16 2 files changed, 10 insertions(+

[PATCH RESEND 06/10] crypto: hisilicon/zip - add print for error branch

2020-08-23 Thread Yang Shen
Add print for some error branches. Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_crypto.c | 68 +++ drivers/crypto/hisilicon/zip/zip_main.c | 8 ++-- 2 files changed, 56 insertions(+), 20 deletions(-) diff --git a/drivers/cr

[PATCH RESEND 02/10] crypto: hisilicon/zip - unify naming style for functions and macros

2020-08-23 Thread Yang Shen
1.Add prefix 'HZIP' for some macros 2.Add prefix 'hisi_zip' for some functions Signed-off-by: Yang Shen Reviewed-by: Zhou Wang --- drivers/crypto/hisilicon/zip/zip_main.c | 35 + 1 file changed, 18 insertions(+), 17 deletions(-) diff --git a/drivers/crypto/hisil

Re: [PATCH] media: staging/intel-ipu3: css: Correctly reset some memory

2020-08-23 Thread Bingbu Cao
Thanks for the patch. On 8/22/20 9:11 PM, Christophe JAILLET wrote: > The intent here is to reset the whole 'scaler_coeffs_luma' array, not just > the first element. > > Fixes:e0a5b744 ("media: staging/intel-ipu3: css: Compute and > program ccs") > Signed-off-by: Christophe JAILLET

[PATCH v10 0/2] pwm: intel: Add PWM driver for a new SoC

2020-08-23 Thread Rahul Tanwar
Patch 1 adds dt binding document in YAML format. Patch 2 add PWM fan controller driver for LGM SoC. v10: - Removed unused of_device.h and added platform_device.h & mod_devicetable.h v9: - Address code quality related review concerns (Andy Shevchenko) - Use devm_add_action_or_reset() instead of

<    1   2   3   4   >