Re: [PATCH v4 2/2] PCI/PM: Fix pci_pm_suspend_noirq() to disable PTM

2022-04-26 Thread David E. Box
On Tue, 2022-04-26 at 11:50 -0500, Bjorn Helgaas wrote: > On Mon, Apr 25, 2022 at 11:32:54AM -0700, David E. Box wrote: > > On Sat, 2022-04-23 at 10:01 -0500, Bjorn Helgaas wrote: > > > On Sat, Apr 23, 2022 at 12:43:14AM +, Jingar, Rajvi wrote: > > > > > -Original Message- > > > > >

[PATCH AUTOSEL 5.10 8/9] powerpc/perf: Fix 32bit compile

2022-04-26 Thread Sasha Levin
From: Alexey Kardashevskiy [ Upstream commit bb82c574691daf8f7fa9a160264d15c5804cb769 ] The "read_bhrb" global symbol is only called under CONFIG_PPC64 of arch/powerpc/perf/core-book3s.c but it is compiled for both 32 and 64 bit anyway (and LLVM fails to link this on 32bit). This fixes it by

[PATCH AUTOSEL 5.15 12/15] powerpc/perf: Fix 32bit compile

2022-04-26 Thread Sasha Levin
From: Alexey Kardashevskiy [ Upstream commit bb82c574691daf8f7fa9a160264d15c5804cb769 ] The "read_bhrb" global symbol is only called under CONFIG_PPC64 of arch/powerpc/perf/core-book3s.c but it is compiled for both 32 and 64 bit anyway (and LLVM fails to link this on 32bit). This fixes it by

[PATCH AUTOSEL 5.17 18/22] powerpc/perf: Fix 32bit compile

2022-04-26 Thread Sasha Levin
From: Alexey Kardashevskiy [ Upstream commit bb82c574691daf8f7fa9a160264d15c5804cb769 ] The "read_bhrb" global symbol is only called under CONFIG_PPC64 of arch/powerpc/perf/core-book3s.c but it is compiled for both 32 and 64 bit anyway (and LLVM fails to link this on 32bit). This fixes it by

[PATCH v6] PCI hotplug: rpaphp: Error out on busy status from get-sensor-state

2022-04-26 Thread Mahesh Salgaonkar
When certain PHB HW failure causes phyp to recover PHB, it marks the PE state as temporarily unavailable until recovery is complete. This also triggers an EEH handler in Linux which needs to notify drivers, and perform recovery. But before notifying the driver about the PCI error it uses

Re: [PATCH v4 2/2] PCI/PM: Fix pci_pm_suspend_noirq() to disable PTM

2022-04-26 Thread Bjorn Helgaas
On Mon, Apr 25, 2022 at 11:32:54AM -0700, David E. Box wrote: > On Sat, 2022-04-23 at 10:01 -0500, Bjorn Helgaas wrote: > > On Sat, Apr 23, 2022 at 12:43:14AM +, Jingar, Rajvi wrote: > > > > -Original Message- > > > > From: Bjorn Helgaas > > > > On Thu, Apr 14, 2022 at 07:54:02PM

Re: [PATCH v5 05/10] serial: termbits: ADDRB to indicate 9th bit addressing mode

2022-04-26 Thread Ilpo Järvinen
On Tue, 26 Apr 2022, Greg KH wrote: > On Tue, Apr 26, 2022 at 05:01:31PM +0300, Ilpo Järvinen wrote: > > > > ADDRB value is the same for all archs (it's just this octal vs hex > > notation I've followed as per the nearby defines within the same file > > which makes them look different). > > >

Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: fsl, ls-extirq: convert to YAML

2022-04-26 Thread Krzysztof Kozlowski
On 26/04/2022 09:28, Michael Walle wrote: > Am 2022-04-26 08:53, schrieb Krzysztof Kozlowski: >> On 25/04/2022 23:58, Michael Walle wrote: > + reg: > +maxItems: 1 > +description: > + Specifies the Interrupt Polarity Control Register (INTPCR) in > the > +

Re: [PATCH v5 05/10] serial: termbits: ADDRB to indicate 9th bit addressing mode

2022-04-26 Thread Greg KH
On Tue, Apr 26, 2022 at 05:01:31PM +0300, Ilpo Järvinen wrote: > One additional thing below I missed on the first read. > > On Tue, 26 Apr 2022, Ilpo Järvinen wrote: > > On Tue, 26 Apr 2022, Greg KH wrote: > > > > > On Tue, Apr 26, 2022 at 03:24:43PM +0300, Ilpo Järvinen wrote: > > > > Add ADDRB

Re: [PATCH v5 05/10] serial: termbits: ADDRB to indicate 9th bit addressing mode

2022-04-26 Thread Ilpo Järvinen
One additional thing below I missed on the first read. On Tue, 26 Apr 2022, Ilpo Järvinen wrote: > On Tue, 26 Apr 2022, Greg KH wrote: > > > On Tue, Apr 26, 2022 at 03:24:43PM +0300, Ilpo Järvinen wrote: > > > Add ADDRB to termbits to indicate 9th bit addressing mode. This change > > > is

Re: [PATCH v5 06/10] serial: General support for multipoint addresses

2022-04-26 Thread Greg KH
On Tue, Apr 26, 2022 at 04:36:49PM +0300, Ilpo Järvinen wrote: > On Tue, 26 Apr 2022, Greg KH wrote: > > > On Tue, Apr 26, 2022 at 03:24:44PM +0300, Ilpo Järvinen wrote: > > > Add generic support for serial multipoint addressing. Two new ioctls > > > are added. TIOCSADDR is used to indicate the

Re: [PATCH v5 06/10] serial: General support for multipoint addresses

2022-04-26 Thread Ilpo Järvinen
On Tue, 26 Apr 2022, Greg KH wrote: > On Tue, Apr 26, 2022 at 03:24:44PM +0300, Ilpo Järvinen wrote: > > Add generic support for serial multipoint addressing. Two new ioctls > > are added. TIOCSADDR is used to indicate the destination/receive > > address. TIOCGADDR returns the current address in

Re: [PATCH v5 05/10] serial: termbits: ADDRB to indicate 9th bit addressing mode

2022-04-26 Thread Ilpo Järvinen
On Tue, 26 Apr 2022, Greg KH wrote: > On Tue, Apr 26, 2022 at 03:24:43PM +0300, Ilpo Järvinen wrote: > > Add ADDRB to termbits to indicate 9th bit addressing mode. This change > > is necessary for supporting devices with RS485 multipoint addressing > > [*]. A later patch in the patch series adds

Re: [PATCH v5 06/10] serial: General support for multipoint addresses

2022-04-26 Thread Greg KH
On Tue, Apr 26, 2022 at 03:24:44PM +0300, Ilpo Järvinen wrote: > Add generic support for serial multipoint addressing. Two new ioctls > are added. TIOCSADDR is used to indicate the destination/receive > address. TIOCGADDR returns the current address in use. The driver > should implement set_addr

Re: [PATCH v5 05/10] serial: termbits: ADDRB to indicate 9th bit addressing mode

2022-04-26 Thread Greg KH
On Tue, Apr 26, 2022 at 03:24:43PM +0300, Ilpo Järvinen wrote: > Add ADDRB to termbits to indicate 9th bit addressing mode. This change > is necessary for supporting devices with RS485 multipoint addressing > [*]. A later patch in the patch series adds support for Synopsys > Designware UART

Re: [PATCH] KVM: powerpc: remove extraneous asterisk from rm_host_ipi_action comment

2022-04-26 Thread Bagas Sanjaya
On 4/26/22 14:47, Bagas Sanjaya wrote: > Link: https://lore.kernel.org/linux-doc/202204252334.cd2isiii-...@intel.com/ > Reported-by: kernel test robot > Cc: Suresh Warrier > Cc: Paul Mackerras > Cc: Anders Roxell > Cc: Greg Kroah-Hartman > Cc: Arnd Bergmann > Cc: Segher Boessenkool > Cc:

[PATCH v5 06/10] serial: General support for multipoint addresses

2022-04-26 Thread Ilpo Järvinen
Add generic support for serial multipoint addressing. Two new ioctls are added. TIOCSADDR is used to indicate the destination/receive address. TIOCGADDR returns the current address in use. The driver should implement set_addr and get_addr to support addressing mode. Adjust ADDRB clearing to

[PATCH v5 05/10] serial: termbits: ADDRB to indicate 9th bit addressing mode

2022-04-26 Thread Ilpo Järvinen
Add ADDRB to termbits to indicate 9th bit addressing mode. This change is necessary for supporting devices with RS485 multipoint addressing [*]. A later patch in the patch series adds support for Synopsys Designware UART capable for 9th bit addressing mode. In this mode, 9th bit is used to

Re: [PATCH] KVM: powerpc: remove extraneous asterisk from rm_host_ipi_action comment

2022-04-26 Thread Segher Boessenkool
On Tue, Apr 26, 2022 at 02:47:51PM +0700, Bagas Sanjaya wrote: > kernel test robot reported kernel-doc warning for rm_host_ipi_action(): > > arch/powerpc/kvm/book3s_hv_rm_xics.c:887: warning: This comment > starts with '/**', but isn't a kernel-doc comment. Refer >

serial hang in qemu-system-ppc64 -M pseries

2022-04-26 Thread Rob Landley
When I cut and paste 80-ish characters of text into the Linux serial console, it reads 16 characters and stops. When I hit space, it reads another 16 characters, and if I keep at it will eventually catch up without losing data. If I type, every character shows up immediately. (On other qemu

[PATCH] KVM: powerpc: remove extraneous asterisk from rm_host_ipi_action comment

2022-04-26 Thread Bagas Sanjaya
kernel test robot reported kernel-doc warning for rm_host_ipi_action(): arch/powerpc/kvm/book3s_hv_rm_xics.c:887: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst Since the function is static, remove the extraneous (second)

Re: [PATCH v2] tty/hvc_opal: simplify if-if to if-else

2022-04-26 Thread Jiri Slaby
On 26. 04. 22, 9:10, Wan Jiabing wrote: Use if and else instead of if(A) and if (!A). Reviewed-by: Jiri Slaby Signed-off-by: Wan Jiabing --- Change log: v2: - add braces to the if block. --- drivers/tty/hvc/hvc_opal.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: fsl, ls-extirq: convert to YAML

2022-04-26 Thread Michael Walle
Am 2022-04-26 08:53, schrieb Krzysztof Kozlowski: On 25/04/2022 23:58, Michael Walle wrote: + reg: +maxItems: 1 +description: + Specifies the Interrupt Polarity Control Register (INTPCR) in the + SCFG or the External Interrupt Control Register (IRQCR) in the ISC. + +

[PATCH v2] tty/hvc_opal: simplify if-if to if-else

2022-04-26 Thread Wan Jiabing
Use if and else instead of if(A) and if (!A). Signed-off-by: Wan Jiabing --- Change log: v2: - add braces to the if block. --- drivers/tty/hvc/hvc_opal.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/hvc/hvc_opal.c b/drivers/tty/hvc/hvc_opal.c index

Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: fsl, ls-extirq: convert to YAML

2022-04-26 Thread Krzysztof Kozlowski
On 25/04/2022 23:58, Michael Walle wrote: >>> + reg: >>> +maxItems: 1 >>> +description: >>> + Specifies the Interrupt Polarity Control Register (INTPCR) in >>> the >>> + SCFG or the External Interrupt Control Register (IRQCR) in the >>> ISC. >>> + >>> + interrupt-map: > >

[PATCH V2] selftests/powerpc/pmu: Fix unsigned function returning negative constant

2022-04-26 Thread Haowen Bai
The function __perf_reg_mask has an unsigned return type, but returns a negative constant to indicate an error condition. So we change unsigned to int. Fixes: 5f6c3061af7c ("selftests/powerpc/pmu: Add utility functions to post process the mmap buffer") Signed-off-by: Haowen Bai --- V1->V2: add

Re: [PATCH] selftests/powerpc/pmu: Fix unsigned function returning negative constant

2022-04-26 Thread kajoljain
On 4/24/22 13:56, Haowen Bai wrote: > The function __perf_reg_mask has an unsigned return type, but returns a > negative constant to indicate an error condition. So we change unsigned > to int. > > Signed-off-by: Haowen Bai > --- > tools/testing/selftests/powerpc/pmu/sampling_tests/misc.c |