[RFC PATCH 14/35] i2c/busses: Change PCIBIOS_SUCCESSFUL to 0

2020-07-13 Thread Saheed O. Bolarinwa
In reference to the PCI spec (Chapter 2), PCIBIOS* is an x86 concept. Their scope should be limited within arch/x86. Change all PCIBIOS_SUCCESSFUL to 0 Signed-off-by: "Saheed O. Bolarinwa" --- drivers/i2c/busses/i2c-ali15x3.c | 4 ++-- drivers/i2c/busses/i2c-nforce2.c | 2 +-

[RFC PATCH 16/35] hwmon: (sis5595) Change PCIBIOS_SUCCESSFUL to 0

2020-07-13 Thread Saheed O. Bolarinwa
In reference to the PCI spec (Chapter 2), PCIBIOS* is an x86 concept. Their scope should be limited within arch/x86. Change all PCIBIOS_SUCCESSFUL to 0 Signed-off-by: "Saheed O. Bolarinwa" --- drivers/hwmon/sis5595.c | 8 drivers/hwmon/via686a.c | 8 drivers/hwmon/vt8231.c |

[RFC PATCH 26/35] powerpc: Change PCIBIOS_SUCCESSFUL to 0

2020-07-13 Thread Saheed O. Bolarinwa
In reference to the PCI spec (Chapter 2), PCIBIOS* is an x86 concept. Their scope should be limited within arch/x86. Change all PCIBIOS_SUCCESSFUL to 0 Signed-off-by: "Saheed O. Bolarinwa" --- arch/powerpc/kernel/rtas_pci.c | 4 ++-- arch/powerpc/platforms/4xx/pci.c

[RFC PATCH 27/35] powerpc: Tidy Success/Failure checks

2020-07-13 Thread Saheed O. Bolarinwa
Remove unnecessary check for 0. Signed-off-by: "Saheed O. Bolarinwa" --- This patch depends on PATCH 26/35 arch/powerpc/platforms/powernv/eeh-powernv.c | 4 ++-- arch/powerpc/platforms/pseries/eeh_pseries.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git

[RFC PATCH 11/35] r8169: Change PCIBIOS_SUCCESSFUL to 0

2020-07-13 Thread Saheed O. Bolarinwa
In reference to the PCI spec (Chapter 2), PCIBIOS* is an x86 concept. Their scope should be limited within arch/x86. Change all PCIBIOS_SUCCESSFUL to 0 Signed-off-by: "Saheed O. Bolarinwa" --- drivers/net/ethernet/realtek/r8169_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[RFC PATCH 06/35] PCI: Tidy Success/Failure checks

2020-07-13 Thread Saheed O. Bolarinwa
Remove unnecessary check for 0. Signed-off-by: "Saheed O. Bolarinwa" --- This patch depends on PATCH 05/35 drivers/pci/controller/dwc/pci-meson.c| 2 +- drivers/pci/controller/dwc/pcie-designware-host.c | 2 +- drivers/pci/controller/pci-xgene.c| 3 +--

[RFC PATCH 03/35] scsi: ipr: Change PCIBIOS_SUCCESSFUL to 0

2020-07-13 Thread Saheed O. Bolarinwa
In reference to the PCI spec (Chapter 2), PCIBIOS* is an x86 concept. Their scope should be limited within arch/x86. Change all PCIBIOS_SUCCESSFUL to 0 Signed-off-by: "Saheed O. Bolarinwa" --- drivers/scsi/ipr.c | 16 drivers/scsi/pmcraid.c | 6 +++--- 2 files changed, 11

[RFC PATCH 09/35] nvme-pci: Change PCIBIOS_SUCCESSFUL to 0

2020-07-13 Thread Saheed O. Bolarinwa
In reference to the PCI spec (Chapter 2), PCIBIOS* is an x86 concept. Their scope should be limited within arch/x86. Change all PCIBIOS_SUCCESSFUL to 0 Signed-off-by: "Saheed O. Bolarinwa" --- drivers/nvme/host/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[RFC PATCH 08/35] PCI: Tidy Success/Failure checks

2020-07-13 Thread Saheed O. Bolarinwa
Remove unnecessary check for 0. Signed-off-by: "Saheed O. Bolarinwa" --- This patch depends on PATCH 07/35 drivers/pci/pci-bridge-emul.c | 2 +- drivers/pci/pci.c | 8 drivers/pci/pcie/bw_notification.c | 4 ++-- drivers/pci/probe.c| 4 ++--

Re: [PATCH v3 6/7] sched/topology: Introduce SD metaflag for flags needing > 1 groups

2020-07-13 Thread Valentin Schneider
On 13/07/20 13:39, Peter Zijlstra wrote: > On Wed, Jul 01, 2020 at 08:06:54PM +0100, Valentin Schneider wrote: >> +/* Generate a mask of SD flags with the SDF_NEEDS_GROUPS metaflag */ >> +#define SD_FLAG(name, idx, mflags) (BIT(idx) * (((mflags) & >> SDF_NEEDS_GROUPS) / SDF_NEEDS_GROUPS)) | > >

[PATCHv3 3/4] watchdog: rti-wdt: attach to running watchdog during probe

2020-07-13 Thread Tero Kristo
If the RTI watchdog is running already during probe, the driver must configure itself to match the HW. Window size and timeout is probed from hardware, and the last keepalive ping is adjusted to match it also. Signed-off-by: Tero Kristo --- drivers/watchdog/rti_wdt.c | 111

Re: [PATCH v3 7/7] sched/topology: Use prebuilt SD flag degeneration mask

2020-07-13 Thread Valentin Schneider
On 13/07/20 13:55, Peter Zijlstra wrote: > On Wed, Jul 01, 2020 at 08:06:55PM +0100, Valentin Schneider wrote: >> Leverage SD_DEGENERATE_GROUPS_MASK in sd_degenerate() and >> sd_degenerate_parent(). >> >> Note that this changes sd_degenerate() somewhat: I'm using the negation of >>

[PATCH 1/6] dt bindings: remoteproc: Add bindings for MT8183 APU

2020-07-13 Thread Alexandre Bailon
This adds dt bindings for the APU present in the MT8183. Signed-off-by: Alexandre Bailon --- .../bindings/remoteproc/mtk,apu.yaml | 121 ++ 1 file changed, 121 insertions(+) create mode 100644 Documentation/devicetree/bindings/remoteproc/mtk,apu.yaml diff --git

[PATCH 5/6] remoteproc: mtk_apu: Don't try to use the APU local RAM

2020-07-13 Thread Alexandre Bailon
Currently, this local RAM is not accessible from the CPU. If the CPU tries to access it, then the CPU will hang. Remoteproc may try to use it when it load a firmware that has some sections in the local RAM. This workarounds the issue by skiping this section. Signed-off-by: Alexandre Bailon ---

[PATCH 4/6] remoteproc: mtk_vpu_rproc: Don't try to load empty PT_LOAD segment

2020-07-13 Thread Alexandre Bailon
The firmware generated by our toolchain contains many empty PT_LOAD segments. The elf loader don't manage it and will raise an error: "bad phdr da 0x0 mem 0x0". To workaround it, implement the sanity_check callback to detect the empty PT_LOAD segment and change it to PT_NULL. In that way, the elf

[PATCH 6/6] ARM64: mt8183: Add support of APU to mt8183

2020-07-13 Thread Alexandre Bailon
This adds the support of APU to mt8183. Signed-off-by: Alexandre Bailon --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 42 1 file changed, 42 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi index

[PATCH 3/6] remoteproc: mtk_vpu_rproc: Add support of JTAG

2020-07-13 Thread Alexandre Bailon
The DSP could be debugged using JTAG. The support of JTAG could enabled at build time and it could be enabled using debugfs. Signed-off-by: Alexandre Bailon --- drivers/remoteproc/Kconfig | 9 ++ drivers/remoteproc/mtk_apu_rproc.c | 156 - 2 files changed,

[PATCH 2/6] remoteproc: Add a remoteproc driver for the MT8183's APU

2020-07-13 Thread Alexandre Bailon
This adds a driver to control the APU present in the MT8183. This loads the firmware and start the DSP. Signed-off-by: Alexandre Bailon --- drivers/remoteproc/Kconfig | 10 + drivers/remoteproc/Makefile| 1 + drivers/remoteproc/mtk_apu_rproc.c | 308

[PATCH 0/6] Add support of mt8183 APU

2020-07-13 Thread Alexandre Bailon
Some Mediatek's SoC have an Accelerated Processing Unit. This adds support of the one available in the mt8183 (aswell some derivative SoC). This series depends on two other series: - Mediatek MT8183 scpsys support - arm64: dts: Add m4u and smi-larbs nodes for mt8183 Notes: This series include

Re: [PATCH v6 00/10] MT6779 IOMMU SUPPORT

2020-07-13 Thread Joerg Roedel
On Sat, Jul 11, 2020 at 03:11:33PM +0800, Yong Wu wrote: > The SMI part always go with the IOMMU, Could you also help apply the > mt6779 SMI basical part [1][2]. Both has already got reviewed-by from > Rob and Matthias. and the [3] in that patchset is for performance > improvement, it's not so

[PATCHv3 0/4] watchdog: rti: support attach to running timer

2020-07-13 Thread Tero Kristo
Hi, Changes from previous version: - Documentation changes for patch #2 - Dropped the configurable module parameter for window size - Merged any needed functionality from old patches #3 and #4 to patch #3 now - Added new rti driver internal API for getting remaining milliseconds left on the

Re: [PATCH 4/6] interconnect: qcom: Add SM8250 interconnect provider driver

2020-07-13 Thread Georgi Djakov
Hi Jonathan, Thanks for posting these patches! On 7/1/20 16:42, Jonathan Marek wrote: > Add driver for the Qualcomm interconnect buses found in SM8250 based > platforms. The topology consists of several NoCs that are controlled by > a remote processor that collects the aggregated bandwidth for

Re: [PATCH v2] doc:kmsg: explictly state the return value in case of SEEK_CUR

2020-07-13 Thread Bruno Meneguele
On Mon, Jul 13, 2020 at 03:15:09PM +0200, Petr Mladek wrote: > On Mon 2020-07-13 11:25:58, Sergey Senozhatsky wrote: > > On (20/07/10 14:44), Bruno Meneguele wrote: > > > The commit 625d3449788f ("Revert "kernel/printk: add kmsg SEEK_CUR > > > handling"") reverted a change done to the return value

[PATCH] fault-inject: Replace HTTP links with HTTPS ones

2020-07-13 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

RE: [PATCH v11 1/2] ACPI / APEI: Add a notifier chain for unknown (vendor) CPER records

2020-07-13 Thread Shiju Jose
Hi Rafael, >-Original Message- >From: Rafael J. Wysocki [mailto:raf...@kernel.org] >Sent: 13 July 2020 12:18 >To: Shiju Jose >Cc: linux-a...@vger.kernel.org; linux-...@vger.kernel.org; linux- >ker...@vger.kernel.org; r...@rjwysocki.net; helg...@kernel.org; >b...@alien8.de;

Re: [PATCH v2] iommu/arm-smmu: Mark qcom_smmu_client_of_match as possibly unused

2020-07-13 Thread Will Deacon
On Mon, Jun 08, 2020 at 04:13:08PM +0100, Will Deacon wrote: > On Thu, Jun 04, 2020 at 02:39:04PM -0600, Jordan Crouse wrote: > > When CONFIG_OF=n of_match_device() gets pre-processed out of existence > > leaving qcom-smmu_client_of_match unused. Mark it as possibly unused to > > keep the compiler

Re: [PATCH] Staging: comedi: driver: Remove condition with no effect

2020-07-13 Thread Greg KH
On Sun, Jul 12, 2020 at 12:36:28PM +0530, Saurav Girepunje wrote: > Remove below warning in das1800.c > WARNING: possible condition with no effect (if == else) > > Signed-off-by: Saurav Girepunje > --- > drivers/staging/comedi/drivers/das1800.c | 6 -- > 1 file changed, 6 deletions(-) > >

Re: [PATCH v29 03/16] leds: multicolor: Introduce a multicolor class definition

2020-07-13 Thread Dan Murphy
Pavel Thanks for the review On 7/11/20 10:57 AM, Pavel Machek wrote: Hi! Introduce a multicolor class that groups colored LEDs within a LED node. +What: /sys/class/leds//multi_intensity +Date: March 2020 +KernelVersion: 5.8 +Contact: Dan Murphy +Description:

[PATCHv3 2/4] watchdog: add support for adjusting last known HW keepalive time

2020-07-13 Thread Tero Kristo
Certain watchdogs require the watchdog only to be pinged within a specific time window, pinging too early or too late cause the watchdog to fire. In cases where this sort of watchdog has been started before kernel comes up, we must adjust the watchdog keepalive window to match the actually running

Re: [PATCH] Staging: vc04_services: Fix code indent error

2020-07-13 Thread Greg Kroah-Hartman
On Mon, Jul 13, 2020 at 12:52:24PM +0530, Baidyanath Kundu wrote: > This patch fixes the checkpatch.pl error: > > ERROR: code indent should use tabs where possible > > Signed-off-by: Baidyanath Kundu > --- > .../staging/vc04_services/vchiq-mmal/mmal-vchiq.c | 14 +++--- > 1 file

Re: [PATCH v6 1/2] sched/uclamp: Add a new sysctl to control RT default boost value

2020-07-13 Thread Peter Zijlstra
On Mon, Jul 13, 2020 at 01:12:46PM +0100, Qais Yousef wrote: > On 07/13/20 13:21, Peter Zijlstra wrote: > > It's monday, and I cannot get my brain working.. I cannot decipher the > > comments you have with the smp_[rw]mb(), what actual ordering do they > > enforce? > > It was a bit of a

Re: [PATCH 4/6] staging: qlge: qlge_main: Simplify while statements.

2020-07-13 Thread Greg KH
On Mon, Jul 13, 2020 at 05:50:14PM +0530, Suraj Upadhyay wrote: > Simplify while loops into more readable and simple for loops. > > Signed-off-by: Suraj Upadhyay > --- > drivers/staging/qlge/qlge_main.c | 49 ++-- > 1 file changed, 22 insertions(+), 27 deletions(-)

Re: [PATCH] Staging: vc04_services: Fix code indent error

2020-07-13 Thread Dan Carpenter
On Mon, Jul 13, 2020 at 03:35:27PM +0200, Greg Kroah-Hartman wrote: > On Mon, Jul 13, 2020 at 12:52:24PM +0530, Baidyanath Kundu wrote: > > This patch fixes the checkpatch.pl error: > > > > ERROR: code indent should use tabs where possible > > > > Signed-off-by: Baidyanath Kundu > > --- > >

Re: [PATCH v11 1/2] ACPI / APEI: Add a notifier chain for unknown (vendor) CPER records

2020-07-13 Thread Rafael J. Wysocki
On Mon, Jul 13, 2020 at 3:33 PM Shiju Jose wrote: > > Hi Rafael, > > >-Original Message- > >From: Rafael J. Wysocki [mailto:raf...@kernel.org] > >Sent: 13 July 2020 12:18 > >To: Shiju Jose > >Cc: linux-a...@vger.kernel.org; linux-...@vger.kernel.org; linux- > >ker...@vger.kernel.org;

Re: [PATCH 1/3] lib: Add a generic copy_oldmem_page()

2020-07-13 Thread Arnd Bergmann
On Mon, Jul 13, 2020 at 3:07 PM Christoph Hellwig wrote: > On Fri, Jul 10, 2020 at 08:55:42PM -0700, Palmer Dabbelt wrote: > > +ssize_t copy_oldmem_page(unsigned long pfn, char *buf, > > + size_t csize, unsigned long offset, > > + int userbuf) > > +{ > >

[PATCH] fpga: region: Replace HTTP links with HTTPS ones

2020-07-13 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

[PATCH] leds: add NCT6795D driver

2020-07-13 Thread Alexandre Courbot
Add support for the LED feature of the NCT6795D chip found on some motherboards, notably MSI ones. The LEDs are typically used using a RGB connector so this driver creates one LED device for each color component. Also add self as maintainer. Signed-off-by: Alexandre Courbot --- MAINTAINERS

Re: [PATCH v3 7/7] sched/topology: Use prebuilt SD flag degeneration mask

2020-07-13 Thread Peter Zijlstra
On Mon, Jul 13, 2020 at 02:28:29PM +0100, Valentin Schneider wrote: > > On 13/07/20 13:55, Peter Zijlstra wrote: > > On Wed, Jul 01, 2020 at 08:06:55PM +0100, Valentin Schneider wrote: > >> Leverage SD_DEGENERATE_GROUPS_MASK in sd_degenerate() and > >> sd_degenerate_parent(). > >> > >> Note that

Re: [PATCH v10 5/5] iommu/arm-smmu: Add global/context fault implementation hooks

2020-07-13 Thread Will Deacon
On Tue, Jul 07, 2020 at 10:00:17PM -0700, Krishna Reddy wrote: > Add global/context fault hooks to allow vendor specific implementations > override default fault interrupt handlers. > > Update NVIDIA implementation to override the default global/context fault > interrupt handlers and handle

Re: [RFC PATCH 12/35] r8169: Tidy Success/Failure checks

2020-07-13 Thread Heiner Kallweit
On 13.07.2020 14:22, Saheed O. Bolarinwa wrote: > Remove unnecessary check for 0. > > Signed-off-by: "Saheed O. Bolarinwa" > --- > This patch depends on PATCH 11/35 > > drivers/net/ethernet/realtek/r8169_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH] Staging: comedi: driver: Remove condition with no effect

2020-07-13 Thread Ian Abbott
On 13/07/2020 14:34, Greg KH wrote: On Sun, Jul 12, 2020 at 12:36:28PM +0530, Saurav Girepunje wrote: Remove below warning in das1800.c WARNING: possible condition with no effect (if == else) Signed-off-by: Saurav Girepunje --- drivers/staging/comedi/drivers/das1800.c | 6 -- 1 file

Re: [RFC PATCH 4/7] x86: use exit_lazy_tlb rather than membarrier_mm_sync_core_before_usermode

2020-07-13 Thread Nicholas Piggin
Excerpts from Nicholas Piggin's message of July 13, 2020 2:45 pm: > Excerpts from Andy Lutomirski's message of July 11, 2020 3:04 am: >> Also, as it stands, I can easily see in_irq() ceasing to promise to >> serialize. There are older kernels for which it does not promise to >> serialize. And I

Re: [PATCH 4/6] interconnect: qcom: Add SM8250 interconnect provider driver

2020-07-13 Thread Jonathan Marek
On 7/13/20 9:30 AM, Georgi Djakov wrote: Hi Jonathan, Thanks for posting these patches! On 7/1/20 16:42, Jonathan Marek wrote: Add driver for the Qualcomm interconnect buses found in SM8250 based platforms. The topology consists of several NoCs that are controlled by a remote processor that

Re: [PATCH] staging: comedi: s626: Remove pci-dma-compat wrapper APIs.

2020-07-13 Thread Ian Abbott
On 11/07/2020 14:38, Christophe JAILLET wrote: Le 11/07/2020 à 14:35, Suraj Upadhyay a écrit : The legacy API wrappers in include/linux/pci-dma-compat.h should go away as it creates unnecessary midlayering for include/linux/dma-mapping.h APIs, instead use dma-mapping.h APIs directly. The patch

Re: [PATCH v10 0/5] NVIDIA ARM SMMU Implementation

2020-07-13 Thread Will Deacon
On Tue, Jul 07, 2020 at 10:00:12PM -0700, Krishna Reddy wrote: > Changes in v10: > Perform SMMU base ioremap before calling implementation init. > Check for Global faults across both ARM MMU-500s during global interrupt. > Check for context faults across all contexts of both ARM MMU-500s during >

RE: [PATCH v11 1/2] ACPI / APEI: Add a notifier chain for unknown (vendor) CPER records

2020-07-13 Thread Shiju Jose
Hi Rafael, >-Original Message- >From: linux-pci-ow...@vger.kernel.org [mailto:linux-pci- >ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki >Sent: 13 July 2020 14:38 >To: Shiju Jose >Cc: Rafael J. Wysocki ; linux-a...@vger.kernel.org; linux- >p...@vger.kernel.org;

Re: [PATCH] SCSI RDMA PROTOCOL (SRP) TARGET: Replace HTTP links with HTTPS ones

2020-07-13 Thread Jason Gunthorpe
On Sun, Jul 12, 2020 at 10:15:29PM +0200, Alexander A. Klimov wrote: > > > Am 12.07.20 um 21:52 schrieb Bart Van Assche: > > On 2020-07-10 11:12, Alexander A. Klimov wrote: > > > Am 10.07.20 um 16:22 schrieb Bart Van Assche: > > > > On 2020-07-09 12:48, Alexander A. Klimov wrote: > > > > > diff

[PATCH] gcc-plugins: Replace HTTP links with HTTPS ones

2020-07-13 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

Re: [PATCH] cpufreq: intel_pstate: Fix static checker warning for epp variable

2020-07-13 Thread Rafael J. Wysocki
On Thu, Jul 9, 2020 at 10:05 PM Srinivas Pandruvada wrote: > > Fix warning for: > drivers/cpufreq/intel_pstate.c:731 store_energy_performance_preference() > error: uninitialized symbol 'epp'. > > This warning is for a case, when energy_performance_preference attribute > matches pre defined

Re: [PATCH v3 7/7] sched/topology: Use prebuilt SD flag degeneration mask

2020-07-13 Thread Valentin Schneider
On 13/07/20 14:43, Peter Zijlstra wrote: > On Mon, Jul 13, 2020 at 02:28:29PM +0100, Valentin Schneider wrote: >> >> On 13/07/20 13:55, Peter Zijlstra wrote: >> > On Wed, Jul 01, 2020 at 08:06:55PM +0100, Valentin Schneider wrote: >> >> Leverage SD_DEGENERATE_GROUPS_MASK in sd_degenerate() and

Re: [PATCH v5 1/2] dt-bindings: phy: add bcm63xx-usbh bindings

2020-07-13 Thread Álvaro Fernández Rojas
Hi Rob, If adding "additionalProperties: false” throws the following error, which I don’t understand: /home/noltari/workspace/linux/Documentation/devicetree/bindings/phy/brcm,bcm63xx-usbh-phy.example.dt.yaml: usb-phy@10001700: 'clock-names', 'clocks' do not match any of the regexes:

Re: [PATCH v9 0/2] Adding support for Versal CPM as Root Port driver

2020-07-13 Thread Lorenzo Pieralisi
On Tue, Jun 16, 2020 at 06:26:52PM +0530, Bharat Kumar Gogada wrote: > - Adding support for Versal CPM as Root port. > - The Versal ACAP devices include CCIX-PCIe Module (CPM). The integrated > block for CPM along with the integrated bridge can function > as PCIe Root Port. > - Versal CPM uses

[PATCH] cpufreq: intel_pstate: Fix active mode setting from command line

2020-07-13 Thread Rafael J. Wysocki
From: Rafael J. Wysocki If intel_pstate starts in the passive mode by default (that happens when the processor in the system doesn't support HWP), passing intel_pstate=active in the kernel command line doesn't work, so fix that. Fixes: 33aa46f252c7 ("cpufreq: intel_pstate: Use passive mode by

Re: [PATCH 0/4] drm: core: Convert logging to drm_* functions.

2020-07-13 Thread Suraj Upadhyay
On Sun, Jul 12, 2020 at 12:07:45PM -0700, Joe Perches wrote: > On Mon, 2020-07-13 at 00:24 +0530, Suraj Upadhyay wrote: > > On Sat, Jul 11, 2020 at 11:16:33AM -0700, Joe Perches wrote: > [] > > > Perhaps change the __drm_printk macro to not > > > dereference the drm argument when NULL. > > > > >

Re: [PATCH v5 2/2] phy: bcm63xx-usbh: Add BCM63xx USBH driver

2020-07-13 Thread Álvaro Fernández Rojas
Hi Vinod, > El 13 jul 2020, a las 7:51, Vinod Koul escribió: > > On 19-06-20, 12:00, Álvaro Fernández Rojas wrote: >> Add BCM63xx USBH PHY driver for BMIPS. >> >> Signed-off-by: Álvaro Fernández Rojas >> --- >> v5: use devm_reset_control_get_exclusive. >> v4: several improvements: >> - Use

RE: [RFC PATCH 06/35] PCI: Tidy Success/Failure checks

2020-07-13 Thread Gustavo Pimentel
On Mon, Jul 13, 2020 at 13:22:18, Saheed O. Bolarinwa wrote: > Remove unnecessary check for 0. > > Signed-off-by: "Saheed O. Bolarinwa" > --- > This patch depends on PATCH 05/35 > > drivers/pci/controller/dwc/pci-meson.c| 2 +- > drivers/pci/controller/dwc/pcie-designware-host.c

[PATCH] hwmon: (adm1025) Replace HTTP links with HTTPS ones

2020-07-13 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

[PATCH] timekeeping: correct typo

2020-07-13 Thread Claudiu Beznea
Correct typo in logarithmic_accumulation() description. Signed-off-by: Claudiu Beznea --- kernel/time/timekeeping.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c index d20d489841c8..6b436d7a751f 100644 ---

Re: [PATCH v10 2/5] iommu/arm-smmu: ioremap smmu mmio region before implementation init

2020-07-13 Thread Robin Murphy
On 2020-07-08 06:00, Krishna Reddy wrote: ioremap smmu mmio region before calling into implementation init. This is necessary to allow mapped address available during vendor specific implementation init. Reviewed-by: Robin Murphy Signed-off-by: Krishna Reddy --- drivers/iommu/arm-smmu.c

Re: [PATCH v2 11/12] PM, libnvdimm: Add 'mem-quiet' state and callback for firmware activation

2020-07-13 Thread Rafael J. Wysocki
On Thursday, July 9, 2020 9:04:30 PM CEST Dan Williams wrote: > On Thu, Jul 9, 2020 at 7:57 AM Rafael J. Wysocki wrote: > > > > On Tuesday, July 7, 2020 3:59:32 AM CEST Dan Williams wrote: > > > The runtime firmware activation capability of Intel NVDIMM devices > > > requires memory transactions

[rcu:rcu/test 64/65] arch/x86/kvm/mmu/page_track.c:232:2: error: expected expression

2020-07-13 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git rcu/test head: e4d26431bafa60b212117ec5750833d39ce1b2aa commit: f89bf7a94ffa9446a341fefc18153e8f9bebffbb [64/65] kvm: mmu: page_track: Fix RCU list API usage config: x86_64-randconfig-a011-20200713 (attached

[PATCH] crypto: qat: fix double free in qat_uclo_create_batch_init_list

2020-07-13 Thread trix
From: Tom Rix clang static analysis flags this error qat_uclo.c:297:3: warning: Attempt to free released memory [unix.Malloc] kfree(*init_tab_base); ^ When input *init_tab_base is null, the function allocates memory for the head of the

Re: [PATCH] rwsem: fix commas in initialisation

2020-07-13 Thread Alexey Dobriyan
On Mon, Jul 13, 2020 at 01:51:41PM +0200, Peter Zijlstra wrote: > On Sat, Jul 11, 2020 at 05:59:54PM +0300, Alexey Dobriyan wrote: > > Leading comma prevents arbitrary reordering of initialisation clauses. > > The whole point of C99 initialisation is to allow any such reordering. > > I'm

[TEGRA194_CPUFREQ PATCH v5 0/4] Add cpufreq driver for Tegra194

2020-07-13 Thread Sumit Gupta
Hi Viresh, The patch series adds cpufreq driver for Tegra194 SOC. Incorporated the feedback on previous version of patchset. Please consider this patch series for merging in 5.9. Hi Rob, Can you please review/ack DT patches (1-2). v4[4] -> v5 - Don't call destroy_workqueue() if

[TEGRA194_CPUFREQ PATCH v5 3/4] cpufreq: Add Tegra194 cpufreq driver

2020-07-13 Thread Sumit Gupta
Add support for CPU frequency scaling on Tegra194. The frequency of each core can be adjusted by writing a clock divisor value to a MSR on the core. The range of valid divisors is queried from the BPMP. Signed-off-by: Mikko Perttunen Signed-off-by: Sumit Gupta --- drivers/cpufreq/Kconfig.arm

[TEGRA194_CPUFREQ PATCH v5 2/4] arm64: tegra: Add t194 ccplex compatible and bpmp property

2020-07-13 Thread Sumit Gupta
On Tegra194, data on valid operating points for the CPUs needs to be queried from BPMP. In T194, there is no node representing CPU complex. So, add compatible string to the 'cpus' node instead of using dummy node to bind cpufreq driver. Also, add reference to the BPMP instance for the CPU complex.

[TEGRA194_CPUFREQ PATCH v5 4/4] soc/tegra: cpufreq: select cpufreq for Tegra194

2020-07-13 Thread Sumit Gupta
Select ARM_TEGRA194_CPUFREQ by default to enable CPU frequency scaling support for Tegra194 SOC. Signed-off-by: Sumit Gupta --- drivers/soc/tegra/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/soc/tegra/Kconfig b/drivers/soc/tegra/Kconfig index 2e95809..6955cee 100644 ---

[TEGRA194_CPUFREQ PATCH v5 1/4] dt-bindings: arm: Add t194 ccplex compatible and bpmp property

2020-07-13 Thread Sumit Gupta
To do frequency scaling on all CPUs within T194 CPU Complex, we need to query BPMP for data on valid operating points. Document a compatible string under 'cpus' node to represent the CPU Complex for binding drivers like cpufreq which don't have their node or CPU Complex node to bind to. Also,

Re: [PATCH net-next v3 5/7] devlink: Add devlink health port reporters API

2020-07-13 Thread Ido Schimmel
On Mon, Jul 13, 2020 at 09:18:25AM -0400, Qian Cai wrote: > On Fri, Jul 10, 2020 at 03:25:11PM +0300, Moshe Shemesh wrote: > > From: Vladyslav Tarasiuk > > > > In order to use new devlink port health reporters infrastructure, add > > corresponding constructor and destructor functions. > > > >

Re: [RFC PATCH 12/35] r8169: Tidy Success/Failure checks

2020-07-13 Thread Saheed Bolarinwa
Thank you for the review. On 7/13/20 3:45 PM, Heiner Kallweit wrote: Patches 11 and 12 are both trivial, wouldn't it make sense to merge them? Apart from that: Acked-by: Heiner Kallweit I separated them for easy review, I will merge them in the next version. - Saheed

[PATCH] ia64: Replace HTTP links with HTTPS ones

2020-07-13 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

Re: [PATCH v10 4/5] dt-bindings: arm-smmu: add binding for Tegra194 SMMU

2020-07-13 Thread Robin Murphy
On 2020-07-10 21:29, Krishna Reddy wrote: Thanks Rob. One question on setting "minItems: ". Please see below. +allOf: + - if: + properties: +compatible: + contains: +enum: + - nvidia,tegra194-smmu +then: + properties: +reg: +

[PATCH v12 0/2] ACPI / APEI: Add support to notify the vendor specific HW errors

2020-07-13 Thread Shiju Jose
CPER records describing a firmware-first error are identified by GUID. The ghes driver currently logs, but ignores any unknown CPER records. This prevents describing errors that can't be represented by a standard entry, that would otherwise allow a driver to recover from an error. The UEFI spec

[PATCH v12 1/2] ACPI / APEI: Add a notifier chain for unknown (vendor) CPER records

2020-07-13 Thread Shiju Jose
CPER records describing a firmware-first error are identified by GUID. The ghes driver currently logs, but ignores any unknown CPER records. This prevents describing errors that can't be represented by a standard entry, that would otherwise allow a driver to recover from an error. The UEFI spec

Re: [PATCH 0/9] drm/msm: Avoid possible infinite probe deferral and speed booting

2020-07-13 Thread Rob Herring
On Fri, Jul 10, 2020 at 5:02 PM Douglas Anderson wrote: > > I found that if I ever had a little mistake in my kernel config, > or device tree, or graphics driver that my system would sit in a loop > at bootup trying again and again and again. An example log was: Why do we care about optimizing

[PATCH v12 2/2] PCI: hip: Add handling of HiSilicon HIP PCIe controller errors

2020-07-13 Thread Shiju Jose
From: Yicong Yang The HiSilicon HIP PCIe controller is capable of handling errors on root port and perform port reset separately at each root port. Add error handling driver for HIP PCIe controller to log and report recoverable errors. Perform root port reset and restore link status after the

Re: [PATCH 4/6] staging: qlge: qlge_main: Simplify while statements.

2020-07-13 Thread Dan Carpenter
On Mon, Jul 13, 2020 at 05:50:14PM +0530, Suraj Upadhyay wrote: > Simplify while loops into more readable and simple for loops. > I don't think either is more clear that the other. Walter Harms hates count down loops and he's not entirely wrong... regards, dan carpenter

Re: [RFC PATCH 4/7] x86: use exit_lazy_tlb rather than membarrier_mm_sync_core_before_usermode

2020-07-13 Thread Mathieu Desnoyers
- On Jul 13, 2020, at 9:47 AM, Nicholas Piggin npig...@gmail.com wrote: > Excerpts from Nicholas Piggin's message of July 13, 2020 2:45 pm: >> Excerpts from Andy Lutomirski's message of July 11, 2020 3:04 am: >>> Also, as it stands, I can easily see in_irq() ceasing to promise to >>>

[PATCH -next] ftrace: Make ftrace_profile_pages_init() static

2020-07-13 Thread Wei Yongjun
The sparse tool complains as follows: kernel/trace/ftrace.c:587:5: warning: symbol 'ftrace_profile_pages_init' was not declared. Should it be static? ftrace_profile_pages_init() is not used out of ftrace.c, so marks it static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun ---

Re: [Freedreno] [v1] drm/msm/dpu: enumerate second cursor pipe for external interface

2020-07-13 Thread Rob Clark
On Mon, Jul 13, 2020 at 3:18 AM wrote: > > On 2020-07-10 22:19, Rob Clark wrote: > > On Thu, Jun 25, 2020 at 5:46 AM Kalyan Thota > > wrote: > >> > >> Setup an RGB HW pipe as cursor which can be used on > >> secondary interface. > >> > >> For SC7180 2 HW pipes are enumerated as cursors > >> 1 -

[PATCH -next] iommu: Make some functions static

2020-07-13 Thread Wei Yongjun
The sparse tool complains as follows: drivers/iommu/iommu.c:386:5: warning: symbol 'iommu_insert_resv_region' was not declared. Should it be static? drivers/iommu/iommu.c:2182:5: warning: symbol '__iommu_map' was not declared. Should it be static? Those functions are not used outside of

Re: [PATCH] ARM: dts: kirkwood: Replace HTTP links with HTTPS ones

2020-07-13 Thread Andrew Lunn
On Mon, Jul 13, 2020 at 09:09:08AM +0200, Alexander A. Klimov wrote: > Rationale: > Reduces attack surface on kernel devs opening the links for MITM > as HTTPS traffic is much harder to manipulate. > > Deterministic algorithm: > For each file: > If not .svg: > For each line: > If

Re: [PATCH 6/6] staging: qlge: qlge_ethtool: Remove one byte memset.

2020-07-13 Thread Dan Carpenter
On Mon, Jul 13, 2020 at 05:52:22PM +0530, Suraj Upadhyay wrote: > Use direct assignment instead of using memset with just one byte as an > argument. > Issue found by checkpatch.pl. > > Signed-off-by: Suraj Upadhyay > --- > Hii Maintainers, > Please correct me if I am wrong here. > --- > >

Re: [PATCH] crypto: qat: fix double free in qat_uclo_create_batch_init_list

2020-07-13 Thread Greg KH
On Mon, Jul 13, 2020 at 07:06:34AM -0700, t...@redhat.com wrote: > From: Tom Rix > > clang static analysis flags this error > > qat_uclo.c:297:3: warning: Attempt to free released memory > [unix.Malloc] > kfree(*init_tab_base); > ^ > > When

Re: [PATCH 4/6] interconnect: qcom: Add SM8250 interconnect provider driver

2020-07-13 Thread Georgi Djakov
On 7/13/20 16:47, Jonathan Marek wrote: > On 7/13/20 9:30 AM, Georgi Djakov wrote: >> Hi Jonathan, >> >> Thanks for posting these patches! >> >> On 7/1/20 16:42, Jonathan Marek wrote: >>> Add driver for the Qualcomm interconnect buses found in SM8250 based >>> platforms. The topology consists of

Re: [PATCH] Replace HTTP links with HTTPS ones: kprobes

2020-07-13 Thread Masami Hiramatsu
Hi Naveen and Alexander, On Fri, 10 Jul 2020 19:14:47 +0530 "Naveen N. Rao" wrote: > Masami Hiramatsu wrote: > > On Tue, 7 Jul 2020 21:49:59 +0200 > > "Alexander A. Klimov" wrote: > > > >> Rationale: > >> Reduces attack surface on kernel devs opening the links for MITM > >> as HTTPS traffic

[PATCH] ib_srp: Replace HTTP links with HTTPS ones

2020-07-13 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

Re: [PATCH v2 2/2] soc: mediatek: add mtk-devapc driver

2020-07-13 Thread Chun-Kuang Hu
Hi, Neal: Neal Liu 於 2020年7月13日 週一 下午4:27寫道: > > Hi Chun-Kuang, > > Thanks for your review. > > On Fri, 2020-07-10 at 22:21 +0800, Chun-Kuang Hu wrote: > > Hi, Neal: > > > > Neal Liu 於 2020年7月10日 週五 上午11:23寫道: > > > > > > Hi Chun-Kuang, > > > > > > Thanks for your review. > > > > > > On Thu,

[PATCH net-next v1] hinic: add firmware update support

2020-07-13 Thread Luo bin
add support to update firmware by the devlink flashing API Signed-off-by: Luo bin --- V0~V1: remove the implementation from ethtool to devlink drivers/net/ethernet/huawei/hinic/Makefile| 2 +- drivers/net/ethernet/huawei/hinic/hinic_dev.h | 1 +

Re: [PATCH] fsi: fsi-occ: fix return value check in occ_probe()

2020-07-13 Thread Eddie James
On 7/12/20 10:33 PM, Xu Wang wrote: In case of error, the function platform_device_register_full() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Thanks, Reviewed-by: Eddie James Signed-off-by: Xu Wang ---

Re: [PATCH] staging: r8188eu: remove unused members of struct xmit_buf

2020-07-13 Thread Dan Carpenter
On Mon, Jul 13, 2020 at 04:16:07PM +0300, Dan Carpenter wrote: > On Sun, Jul 12, 2020 at 03:38:21PM +0300, Ivan Safonov wrote: > > Remove unused members of struct xmit_buf: alloc_sz, ff_hwaddr, > > dma_transfer_addr, bpending and last. > > > > Signed-off-by: Ivan Safonov > > --- > >

[PATCH][next] drm/i915/selftest: fix an error return path where err is not being set

2020-07-13 Thread Colin King
From: Colin Ian King There is an error condition where err is not being set and an uninitialized garbage value in err is being returned. Fix this by assigning err to an appropriate error return value before taking the error exit path. Addresses-Coverity: ("Uninitialized scalar value") Fixes:

Re: [PATCH] pinctrl: qcom: Handle broken PDC dual edge case on sc7180

2020-07-13 Thread Maulik Shah
Hi, On 7/10/2020 9:40 PM, Doug Anderson wrote: Hi, On Fri, Jul 10, 2020 at 2:03 AM Marc Zyngier wrote: Hi Doug, On Wed, 08 Jul 2020 22:16:25 +0100, Douglas Anderson wrote: As per Qualcomm, there is a PDC hardware issue (with the specific IP rev that exists on sc7180) that causes the PDC

Re: [PATCH v2 2/2] PM / devfreq: Add governor flags to clarify the features

2020-07-13 Thread Dmitry Osipenko
13.07.2020 15:26, Chanwoo Choi пишет: ... >> BTW, I'm curious what do you think about hiding the unsupported debugfs > > Do you mean that sysfs? Yes, sysfs :) >> attributes per-device instead of returning the -EACCES? > > I considered the hiding of sysfs node too instead of -EACCES. If there

Re: [PATCH v2 2/2] arm64: tlb: Use the TLBI RANGE feature in arm64

2020-07-13 Thread Jon Hunter
On 10/07/2020 10:44, Zhenyu Ye wrote: > Add __TLBI_VADDR_RANGE macro and rewrite __flush_tlb_range(). > > When cpu supports TLBI feature, the minimum range granularity is > decided by 'scale', so we can not flush all pages by one instruction > in some cases. > > For example, when the pages =

Re: [PATCH v6 1/2] sched/uclamp: Add a new sysctl to control RT default boost value

2020-07-13 Thread Qais Yousef
On 07/13/20 15:35, Peter Zijlstra wrote: > On Mon, Jul 13, 2020 at 01:12:46PM +0100, Qais Yousef wrote: > > On 07/13/20 13:21, Peter Zijlstra wrote: > > > > It's monday, and I cannot get my brain working.. I cannot decipher the > > > comments you have with the smp_[rw]mb(), what actual ordering

Re: [PATCH] Revert "ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb"

2020-07-13 Thread Kalle Valo
Viktor Jägersküpper writes: > Kalle Valo wrote: >> Viktor Jägersküpper writes: >> >>> Kalle Valo writes: Roman Mamedov writes: > On Sat, 4 Apr 2020 12:18:38 +0800 > Qiujun Huang wrote: > >> In ath9k_hif_usb_rx_cb interface number is assumed to be 0. >>

Re: [PATCH 1/2] dt-bindings: leds: pca955x: Add IBM implementation compatible string

2020-07-13 Thread Eddie James
On 7/11/20 8:48 AM, Pavel Machek wrote: Hi! IBM created an implementation of the PCA9552 on a PIC16F microcontroller. Document the new compatible string for this device. Is the implementation opensource? Hi, no it is not. Signed-off-by: Eddie James +++

Re: [PATCH 2/2] leds: pca955x: Add an IBM software implementation of the PCA9552 chip

2020-07-13 Thread Eddie James
On 7/9/20 3:50 PM, Andy Shevchenko wrote: On Thu, Jul 9, 2020 at 11:16 PM Eddie James wrote: IBM created an implementation of the PCA9552 on a PIC16F microcontroller. The I2C device addresses are different from the hardware PCA9552, so add a new compatible string and associated platform data

[PATCH] isdn/capi: Replace HTTP links with HTTPS ones

2020-07-13 Thread Alexander A. Klimov
Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain `\bxmlns\b`: For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:

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