Re: [PATCH 2/8] x86: use exit_lazy_tlb rather than membarrier_mm_sync_core_before_usermode

2020-12-05 Thread Nicholas Piggin
Excerpts from Andy Lutomirski's message of December 3, 2020 3:09 pm: > On Tue, Dec 1, 2020 at 6:50 PM Nicholas Piggin wrote: >> >> Excerpts from Andy Lutomirski's message of November 29, 2020 3:55 am: >> > On Sat, Nov 28, 2020 at 8:02 AM Nicholas Piggin wrote: >> >> >> >> And get rid of the gener

Re: [PATCH v2 2/2] drivers: gpio: add virtio-gpio guest driver

2020-12-05 Thread Enrico Weigelt, metux IT consult
On 04.12.20 04:35, Jason Wang wrote: >> --- a/drivers/gpio/Kconfig >> +++ b/drivers/gpio/Kconfig >> @@ -1615,6 +1615,15 @@ config GPIO_MOCKUP >>     tools/testing/selftests/gpio/gpio-mockup.sh. Reference the >> usage in >>     it. >>   +config GPIO_VIRTIO >> +    tristate "VirtIO GPIO supp

RE: [PATCH v4 0/8] Refine error history and introduce event_notify vop

2020-12-05 Thread Avri Altman
Hi Stanley, Will you split this series to 3 separate series: Phy initialization cleanup, Error history, and event notification? As those 3 aren't really connected? Please maintain Can's reviewed-by tag for the error history patches, And add mine for the phy initialization, so Martin can pick those

Re: [PATCH 1/2] dt-bindings: phy: add Amlogic G12A Analog MIPI D-PHY bindings

2020-12-05 Thread Vinod Koul
On 23-11-20, 15:51, Neil Armstrong wrote: > The Amlogic G12A SoCs embeds an Analog MIPI D-PHY to communicate with DSI > panels, this adds the bindings. > > This Analog D-PHY works with a separate Digital MIPI D-PHY. Pls cc Rob on dt binding patches (hint get_maintainer.pl would tell you do so) >

RE: [PATCH v4 0/8] Refine error history and introduce event_notify vop

2020-12-05 Thread Avri Altman
> > Hi Stanley, > Will you split this series to 3 separate series: > Phy initialization cleanup, Error history, and event notification? > As those 3 aren't really connected? > > Please maintain Can's reviewed-by tag for the error history patches, > And add mine for the phy initialization, so Mart

RE: [PATCH] ufshcd: fix Wsometimes-uninitialized warning

2020-12-05 Thread Avri Altman
> > > From: Arnd Bergmann > > clang complains about a possible code path in which a variable is > used without an initialization: > > drivers/scsi/ufs/ufshcd.c:7690:3: error: variable 'sdp' is used uninitialized > whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized] >

RE: [PATCH] ufshcd: fix Wsometimes-uninitialized warning

2020-12-05 Thread Avri Altman
> > > From: Arnd Bergmann > > clang complains about a possible code path in which a variable is > used without an initialization: > > drivers/scsi/ufs/ufshcd.c:7690:3: error: variable 'sdp' is used uninitialized > whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized] >

Re: [PATCH v2] phy: rockchip: set pulldown for strobe line in dts

2020-12-05 Thread Vinod Koul
On 29-11-20, 13:44, Chris Ruehl wrote: > This patchset add support to set the strobe line pulldown via dt property Applied, thanks -- ~Vinod

Re: [PATCH v3 9/9] drm/dp: Revert "drm/dp: Introduce EDID-based quirks"

2020-12-05 Thread kernel test robot
submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Lyude-Paul/drm-i915-Add-support-for-Intel-s-eDP-backlight-controls/20201205-063902 base: git://anongit.freedesktop.org/drm-intel for-

Re: [PATCH v2] dt-bindings: phy: Convert Broadcom SATA PHY to YAML

2020-12-05 Thread Vinod Koul
On 04-12-20, 11:35, Florian Fainelli wrote: > Update the Broadcom SATA PHY Device Tree binding to a YAML format. > Applied, thanks -- ~Vinod

Re: [PATCH v9 0/3] Use the generic PHY framework for Ingenic USB PHY.

2020-12-05 Thread Vinod Koul
On 16-11-20, 22:19, 周琰杰 (Zhou Yanjie) wrote: > v3->v4: > Only add new generic-PHY driver, without removing the old one. Because the > jz4740-musb driver is not ready to use the generic PHY framework. When the > jz4740-musb driver is modified to use the generic PHY framework, the old > jz4770-phy dr

[rcu:dev.2020.12.03a] BUILD SUCCESS 8c884f433ec3fb5b009bd14be9cd2a16e960cd40

2020-12-05 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git dev.2020.12.03a branch HEAD: 8c884f433ec3fb5b009bd14be9cd2a16e960cd40 squash! percpu_ref: Add kmem_cache_last_alloc() argument for stack trace elapsed time: 721m configs tested: 132 configs skipped: 3 The fol

Re: [PATCH 0/2] phy: rockchip: set otapdlysec for in dts

2020-12-05 Thread Vinod Koul
On 02-12-20, 16:25, Chris Ruehl wrote: > This patchset add support to set output-tapdelay-selec via dt property Applied, thanks -- ~Vinod

Re: [PATCH v3 3/6] mm: support THP migration to device private memory

2020-12-05 Thread Roger Pau Monné
On Wed, Dec 02, 2020 at 11:08:54AM +0100, Christoph Hellwig wrote: > On Fri, Nov 20, 2020 at 04:01:33PM -0400, Jason Gunthorpe wrote: > > On Wed, Nov 11, 2020 at 03:38:42PM -0800, Ralph Campbell wrote: > > > > > MEMORY_DEVICE_GENERIC: > > > Struct pages are created in dev_dax_probe() and represent

[PATCH] iommu: Up front sanity check in the arm_lpae_map

2020-12-05 Thread Keqian Zhu
... then we have more chance to detect wrong code logic. Signed-off-by: Keqian Zhu --- drivers/iommu/io-pgtable-arm.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/iommu/io-pgtable-arm.c b/drivers/iommu/io-pgtable-arm.c index a7a9bc08dcd1..8ade72adab31 10064

Re: [PATCH 03/22] keembay-ipc: Add Keem Bay IPC module

2020-12-05 Thread Greg KH
On Fri, Dec 04, 2020 at 07:35:17PM -0800, mark gross wrote: > On Wed, Dec 02, 2020 at 08:01:18PM +0100, Greg KH wrote: > > On Wed, Dec 02, 2020 at 09:42:00AM -0800, mark gross wrote: > > > On Wed, Dec 02, 2020 at 07:16:20AM +0100, Greg KH wrote: > > > > On Tue, Dec 01, 2020 at 02:34:52PM -0800, mgr

[PATCH] powerpc/mm: Fix KUAP warning by providing copy_from_kernel_nofault_allowed()

2020-12-05 Thread Christophe Leroy
Since commit c33165253492 ("powerpc: use non-set_fs based maccess routines"), userspace access is not granted anymore when using copy_from_kernel_nofault() However, kthread_probe_data() uses copy_from_kernel_nofault() to check validity of pointers. When the pointer is NULL, it points to userspace,

Re: [PATCH] powerpc/mm: Fix KUAP warning by providing copy_from_kernel_nofault_allowed()

2020-12-05 Thread Christoph Hellwig
On Sat, Dec 05, 2020 at 08:43:06AM +, Christophe Leroy wrote: > Since commit c33165253492 ("powerpc: use non-set_fs based maccess > routines"), userspace access is not granted anymore when using > copy_from_kernel_nofault() > > However, kthread_probe_data() uses copy_from_kernel_nofault() > to

Re: [PATCHv2] clocksource: dw_apb_timer_of: add error handling if no clock available

2020-12-05 Thread Daniel Lezcano
On 04/12/2020 23:39, Dinh Nguyen wrote: > > > On 12/4/20 2:00 PM, Daniel Lezcano wrote: >> On 04/12/2020 16:36, Dinh Nguyen wrote: >>> commit ("b0fc70ce1f02 arm64: berlin: Select DW_APB_TIMER_OF") added the >>> support for the dw_apb_timer into the arm64 defconfig. However, for some >>> platforms

Re: [PATCH v3] bridge: Fix a deadlock when enabling multicast snooping

2020-12-05 Thread Nikolay Aleksandrov
On 05/12/2020 01:56, Joseph Huang wrote: > When enabling multicast snooping, bridge module deadlocks on multicast_lock > if 1) IPv6 is enabled, and 2) there is an existing querier on the same L2 > network. > > The deadlock was caused by the following sequence: While holding the lock, > br_multicas

Re: [PATCH v3] PM: domains: create debugfs nodes when adding power domains

2020-12-05 Thread Greg Kroah-Hartman
On Fri, Dec 04, 2020 at 04:57:16PM -0800, Thierry Strudel wrote: > debugfs nodes were created in genpd_debug_init alled in late_initcall > preventing power domains registered though loadable modules to have > a debugfs entry. > > Create/remove debugfs nodes when the power domain is added/removed >

Re: [PATCH v1] HID: make arrays usage and value to be the same

2020-12-05 Thread Greg KH
On Sat, Dec 05, 2020 at 12:48:48AM +, Will McVicker wrote: > The HID subsystem allows an "HID report field" to have a different > number of "values" and "usages" when it is allocated. When a field > struct is created, the size of the usage array is guaranteed to be at > least as large as the va

Re: [resend/standalone PATCH v4] Add auxiliary bus support

2020-12-05 Thread Greg KH
On Fri, Dec 04, 2020 at 09:10:34AM -0800, Ranjani Sridharan wrote: > On Fri, 2020-12-04 at 13:59 +0100, Greg KH wrote: > > On Fri, Dec 04, 2020 at 02:32:07PM +0200, Leon Romanovsky wrote: > > > On Fri, Dec 04, 2020 at 12:42:46PM +0100, Greg KH wrote: > > > > On Wed, Dec 02, 2020 at 04:54:24PM -0800

[rcu:rcu/next] BUILD SUCCESS 2c05cc5920504514a39df422145c68306f030a60

2020-12-05 Thread kernel test robot
-20201204 i386 randconfig-a002-20201204 i386 randconfig-a006-20201204 i386 randconfig-a003-20201204 i386 randconfig-a005-20201205 i386 randconfig-a004-20201205 i386 randconfig-a001-20201205 i386

mips-linux-ld: decompress.c:(.text.decompress_kernel+0x21c): undefined reference to `__ubsan_handle_out_of_bounds'

2020-12-05 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: b3298500b23f0b53a8d81e0d5ad98a29db71f4f0 commit: 8d58f222e85f01da0c0e1fc1e77986c86de889e2 ubsan: disable UBSAN_ALIGNMENT under COMPILE_TEST date: 7 months ago config: mips-randconfig-r033-20201205

Re: [PATCH] pcmcia: Remove NEC VRC4173 CARDU

2020-12-05 Thread Dominik Brodowski
Am Wed, Sep 16, 2020 at 10:16:29AM +0200 schrieb Sebastian Andrzej Siewior: > This driver is the very definition of bitrotting: > - Introduced in commit > 79a140932c776 ("[PATCH] mips: vR41xx updates") > which is 2.6.11-rc3. > > - Provides ->register_callback which was removed in commit > 7f

[PATCH] phy: mediatek: statify mtk_hdmi_phy_driver

2020-12-05 Thread Vinod Koul
mtk_hdmi_phy_driver is not declared as static, so statify it. drivers/phy/mediatek/phy-mtk-hdmi.c:204:24: warning: symbol 'mtk_hdmi_phy_driver' was not declared. Should it be static? Signed-off-by: Vinod Koul --- drivers/phy/mediatek/phy-mtk-hdmi.c | 2 +- 1 file changed, 1 insertion(+), 1 del

Re: [PATCH] pcmcia: omap: Fix error return code in omap_cf_probe()

2020-12-05 Thread Dominik Brodowski
Am Wed, Nov 25, 2020 at 08:50:57PM +0800 schrieb Wang ShaoBo: > Fix to return proper error code instaed of 0 in omap_cf_probe(), as done > elsewhere in this function. > > Signed-off-by: Wang ShaoBo Applied to pcmcia-next. Thanks, Dominik

Re: [PATCH v2] clk: renesas: r9a06g032: Drop __packed for portability

2020-12-05 Thread Geert Uytterhoeven
Hi Stephen, On Sat, Dec 5, 2020 at 7:24 AM Stephen Boyd wrote: > Quoting Geert Uytterhoeven (2020-11-30 00:57:43) > > The R9A06G032 clock driver uses an array of packed structures to reduce > > kernel size. However, this array contains pointers, which are no longer > > aligned naturally, and can

Re: recursion handling: Re: [PATCH next v2 3/3] printk: remove logbuf_lock, add syslog_lock

2020-12-05 Thread Sergey Senozhatsky
On (20/12/04 17:10), Petr Mladek wrote: > > One reason is the use of per-cpu variables. Alternative solution would > be to store printk_context into task_struct. We can keep per-CPU, disable preemption and have counters for every context (task, soft/hard irq, NMI). Shouldn't be a problem

Re: linux-next: build warning after merge of the akpm tree

2020-12-05 Thread Stephen Rothwell
Hi Andrew, On Fri, 4 Dec 2020 21:19:23 -0800 Andrew Morton wrote: > > Odd. clang wants that signature, according to > https://clang.llvm.org/docs/SanitizerCoverage.html. But gcc seems to > want a different signature. Beats me - best I can do is to cc various > likely culprits ;) > > Which gc

Re: [PATCH] powerpc/mm: Fix KUAP warning by providing copy_from_kernel_nofault_allowed()

2020-12-05 Thread Christophe Leroy
Le 05/12/2020 à 09:48, Christoph Hellwig a écrit : On Sat, Dec 05, 2020 at 08:43:06AM +, Christophe Leroy wrote: Since commit c33165253492 ("powerpc: use non-set_fs based maccess routines"), userspace access is not granted anymore when using copy_from_kernel_nofault() However, kthread_pr

Re: [PATCH] drivers/pcmcia: Fix error return code in electra_cf_probe()

2020-12-05 Thread Dominik Brodowski
Am Tue, Nov 24, 2020 at 03:00:40PM +0800 schrieb Wei Li: > When it fails to call of_get_property(), it just jumps to 'fail1', > while the 'status' which will be returned is not updated. > > Fixes: 2b571a066a2f ("pcmcia: CompactFlash driver for PA Semi Electra boards") > Signed-off-by: Wei Li Thn

Re: [PATCH 1/2] pcmcia: at91_cf: move definitions locally

2020-12-05 Thread Dominik Brodowski
Am Tue, Nov 24, 2020 at 12:07:30PM +0100 schrieb Alexandre Belloni: > On Wed, 30 Sep 2020 20:48:02 +0200, Alexandre Belloni wrote: > > struct at91_cf_data is only used in the driver since all the platforms moved > > to device tree, move its definition locally. > > I've now applied those patches on

[PATCH] tick/nohz: Make the idle_exittime update correctly

2020-12-05 Thread Yunfeng Ye
The idle_exittime field of tick_sched is used to record the time when the idle state was left. but currently the idle_exittime is updated in the function tick_nohz_restart_sched_tick(), which is not always in idle state when nohz_full is configured. tick_irq_exit tick_nohz_irq_exit tic

Re: KMSAN: uninit-value in dvb_usb_adapter_dvb_init (2)

2020-12-05 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:73d62e81 kmsan: random: prevent boot-time reports in _mix_.. git tree: https://github.com/google/kmsan.git master console output: https://syzkaller.appspot.com/x/log.txt?x=170babf750 kernel config: https://syzkall

Re: [PATCH] pcmcia: db1xxx_ss: remove unneeded semicolon

2020-12-05 Thread Dominik Brodowski
Am Thu, Sep 10, 2020 at 10:05:24PM +0800 schrieb Jason Yan: > Eliminate the following coccicheck warning: > > drivers/pcmcia/db1xxx_ss.c:455:2-3: Unneeded semicolon > > Reported-by: Hulk Robot > Signed-off-by: Jason Yan Applied to pcmcia-next. Thanks, Dominik

Linux man-pages maintainership adjustments

2020-12-05 Thread Michael Kerrisk (man-pages)
Gidday, Anyone following linux-man@ in the last few months will have noticed that Alejandro (Alex) Colomar has become rather active in the project. Alex has kindly volunteered to take up some of the work of maintaining the project. In practice, that means he will be reviewing and merging some of t

Re: [PATCH v3 1/3] x86/uprobes: Fix not using prefixes.nbytes for loop over prefixes.bytes

2020-12-05 Thread Borislav Petkov
On Sat, Dec 05, 2020 at 09:10:32AM +0900, Masami Hiramatsu wrote: > In the future, if x86 ISA is expanded and add a legacy prefix > groups, Very unlikely. > then we have to add new insn_prefix_field data structure, which > size will not depend on NUM_INSN_FIELD_BYTES, but still depend on > MAX_LE

Re: [PATCH REPOST] pcmcia: Remove NEC VRC4173 CARDU

2020-12-05 Thread Dominik Brodowski
Am Fri, Dec 04, 2020 at 08:20:09PM +0100 schrieb Sebastian Andrzej Siewior: > On 2020-11-19 18:06:24 [+0100], To linux-kernel@vger.kernel.org wrote: > > On 2020-11-13 22:34:08 [+0100], To linux-kernel@vger.kernel.org wrote: > > > This driver is the very definition of bitrotting: > > > - Introduced

Re: [tip: x86/urgent] x86/uprobes: Do not use prefixes.nbytes when looping over prefixes.bytes

2020-12-05 Thread Borislav Petkov
On Sat, Dec 05, 2020 at 09:12:56AM +0900, Masami Hiramatsu wrote: > This may break tools/objtool build. Please keep "inat.h". How? Please elaborate. Build tests are fine here. Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH v6] checkpatch: add fix for non-standard signature - co-authored-by

2020-12-05 Thread Aditya
On 4/12/20 8:10 pm, Aditya Srivastava wrote: > Currently, checkpatch.pl warns us for BAD_SIGN_OFF on the usage of > non-standard signatures. > > An evaluation on v4.13..v5.8 showed that out of 539 warnings due to > non-standard signatures, 43 are due to the use of 'Co-authored-by' > tag, which may

[PATCH] staging: greybus: audio: Fix possible leak free widgets in gbaudio_dapm_free_controls

2020-12-05 Thread Wang Hai
In gbaudio_dapm_free_controls(), if one of the widgets is not found, an error will be returned directly, which will cause the rest to be unable to be freed, resulting in leak. This patch fixes the bug. If if one of them is not found, just skip and free the others. Fixes: 510e340efe0c ("staging:

Re: [PATCH v3 1/4] irq: export kstat_irqs

2020-12-05 Thread Jarkko Sakkinen
On Fri, Dec 04, 2020 at 06:43:37PM -0700, Jerry Snitselaar wrote: > To try and detect potential interrupt storms that > have been occurring with tpm_tis devices it was suggested > to use kstat_irqs() to get the number of interrupts. > Since tpm_tis can be built as a module it needs kstat_irqs > exp

[tip:core/mm] BUILD SUCCESS 68061c02bb295da4955f0d309b9459f0a7ba83dd

2020-12-05 Thread kernel test robot
randconfig-a006-20201204 i386 randconfig-a003-20201204 i386 randconfig-a005-20201205 i386 randconfig-a004-20201205 i386 randconfig-a001-20201205 i386 randconfig-a002-20201205 i386 randconfig-a006

Re: [PATCH] staging: greybus: audio: Add missing unlock in gbaudio_dapm_free_controls()

2020-12-05 Thread wanghai (M)
在 2020/12/5 2:02, Vaibhav Agarwal 写道: On Fri, Dec 4, 2020 at 2:10 PM Johan Hovold wrote: On Fri, Dec 04, 2020 at 10:13:50AM +0800, Wang Hai wrote: Add the missing unlock before return from function gbaudio_dapm_free_controls() in the error handling case. Fixes: 510e340efe0c ("staging: greyb

Re: [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage

2020-12-05 Thread Joe Perches
On Fri, 2020-12-04 at 16:09 -0800, Jakub Kicinski wrote: > On Wed, 2 Dec 2020 13:49:53 +0100 Enrico Weigelt, metux IT consult > wrote: > > Remove MODULE_VERSION(), as it isn't needed at all: the only version > > making sense is the kernel version. > > > > Signed-off-by: Enrico Weigelt, metux IT c

Re: [PATCHv2] clocksource: dw_apb_timer_of: add error handling if no clock available

2020-12-05 Thread Dinh Nguyen
Hi Daniel, On 12/5/20 2:50 AM, Daniel Lezcano wrote: On 04/12/2020 23:39, Dinh Nguyen wrote: On 12/4/20 2:00 PM, Daniel Lezcano wrote: On 04/12/2020 16:36, Dinh Nguyen wrote: commit ("b0fc70ce1f02 arm64: berlin: Select DW_APB_TIMER_OF") added the support for the dw_apb_timer into the arm64

Re: [PATCH] powerpc: Stop exporting __clear_user which is now inlined.

2020-12-05 Thread Michael Ellerman
Michal Suchanek writes: > Stable commit 452e2a83ea23 ("powerpc: Fix __clear_user() with KUAP > enabled") redefines __clear_user as inline function but does not remove > the export. > > Fixes: 452e2a83ea23 ("powerpc: Fix __clear_user() with KUAP enabled") > > Signed-off-by: Michal Suchanek > --- >

Re: [PATCH RFC] mm: Let readahead submit larger batches of pages in case of ra->ra_pages == 0

2020-12-05 Thread Bean Huo
On Fri, 2020-09-11 at 10:47 +0100, Christoph Hellwig wrote: > > Hi Andrew > > Sorry, I am still not quite understanding your above three > > questions. > > > > Based on my shallow understanding, ra_pages is associated with > > read_ahead_kb. Seems ra_pages controls the maximum read-ahead > > wind

[PATCH v5 0/2] usb: typec: tps6598x: Export some power supply properties

2020-12-05 Thread Guido Günther
This allows downstream supplies and userspace to detect whether external power is supplied. The Librem 5 has the tp65982 in front of bq25980 charge controller. Since that is capable of sinking and sourcing power the online property helps to decide what to do. It also makes upower happy. There wi

[PATCH v5 1/2] usb: typec: tps6598x: Select USB_ROLE_SWITCH and REGMAP_I2C

2020-12-05 Thread Guido Günther
This is more in line with what tcpm does and will be needed to avoid recursive dependency like > drivers/power/supply/Kconfig:2:error: recursive dependency detected! drivers/power/supply/Kconfig:2: symbol POWER_SUPPLY is selected by TYPEC_TPS6598X drivers/usb/typec/Kconfig:64: symbol TYPEC

Re:

2020-12-05 Thread Rasmus Villemoes
On 03/12/2020 19.46, Yury Norov wrote: > I would prefer to avoid changing the find*bit() semantics. As for now, > if any of find_*_bit() > finds nothing, it returns the size of the bitmap it was passed. Yeah, we should actually try to fix that, it causes bad code generation. It's hard, because ca

[PATCH v5 2/2] usb: typec: tps6598x: Export some power supply properties

2020-12-05 Thread Guido Günther
This allows downstream supplies and userspace to detect whether external power is supplied. Signed-off-by: Guido Günther Reviewed-by: Heikki Krogerus Reviewed-by: Andy Shevchenko --- drivers/usb/typec/Kconfig| 1 + drivers/usb/typec/tps6598x.c | 103 +++ 2

Re: [PATCH v4 00/10] Introduced new Cadence USBSSP DRD Driver.

2020-12-05 Thread Peter Chen
On 20-12-05 11:12:44, Aswath Govindraju wrote: > Hi, > On 04/12/20 6:49 am, Peter Chen wrote: > > > >> This patch introduce new Cadence USBSS DRD driver to linux kernel. > >> > >> The Cadence USBSS DRD Controller is a highly configurable IP Core which can > >> be instantiated as Dual-Role Device

Re: [PATCH v3 2/2] clocksource: arm_arch_timer: Correct fault programming of CNTKCTL_EL1.EVNTI

2020-12-05 Thread Daniel Lezcano
Hi Marc, are you fine with this patch ? On 04/12/2020 08:31, Keqian Zhu wrote: > ARM virtual counter supports event stream, it can only trigger an event > when the trigger bit (the value of CNTKCTL_EL1.EVNTI) of CNTVCT_EL0 changes, > so the actual period of event stream is 2^(cntkctl_evnti +

Re: [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage

2020-12-05 Thread Vladimir Oltean
On Wed, Dec 02, 2020 at 01:49:53PM +0100, Enrico Weigelt, metux IT consult wrote: > Remove MODULE_VERSION(), as it isn't needed at all: the only version > making sense is the kernel version. > > Signed-off-by: Enrico Weigelt, metux IT consult > --- > net/8021q/vlan.c | 8 +--- > 1 file chang

Re: [PATCH 2/3] scsi: ufs: Keep device power on only fWriteBoosterBufferFlushDuringHibernate == 1

2020-12-05 Thread Bean Huo
On Thu, 2020-12-03 at 07:27 +, Avri Altman wrote: > > > > From: Bean Huo > > > > Keep device power mode as active power mode and VCC supply only if > > fWriteBoosterBufferFlushDuringHibernate setting 1 is successful. > Hi Avri Thanks so much taking time reiew. > Why would it fail? During

Re: [PATCH] drivers: thermal: Add NULL pointer check before using cooling device stats

2020-12-05 Thread Daniel Lezcano
On 04/12/2020 20:09, Manaf Meethalavalappu Pallikunhi wrote: > There is a possible chance that some cooling device stats buffer > allocation fails due to very high cooling device max state value. > Later cooling device update or cooling stats sysfs will try to > access stats data for the same cooli

Re: [PATCHv2] clocksource: dw_apb_timer_of: add error handling if no clock available

2020-12-05 Thread Daniel Lezcano
On 05/12/2020 11:55, Dinh Nguyen wrote: > Hi Daniel, > > On 12/5/20 2:50 AM, Daniel Lezcano wrote: >> On 04/12/2020 23:39, Dinh Nguyen wrote: >>> >>> >>> On 12/4/20 2:00 PM, Daniel Lezcano wrote: On 04/12/2020 16:36, Dinh Nguyen wrote: > commit ("b0fc70ce1f02 arm64: berlin: Select DW_APB_

[PATCHv3] clocksource: dw_apb_timer_of: add error handling if no clock available

2020-12-05 Thread Dinh Nguyen
commit ("b0fc70ce1f02 arm64: berlin: Select DW_APB_TIMER_OF") added the support for the dw_apb_timer into the arm64 defconfig. However, for some platforms like the Intel Stratix10 and Agilex, the clock manager doesn't get loaded until after the timer driver get loaded. Thus, the driver hits the pan

Re: [PATCH] MAINTAINERS: Update 68k Mac entry

2020-12-05 Thread Michael Ellerman
Finn Thain writes: > Two files under drivers/macintosh are actually m68k-only. I think that > patches for these files should be reviewed in the appropriate forum and > merged via the appropriate tree, rather than falling to the powerpc > maintainers to deal with. Update the "M68K ON APPLE MACINTOS

[PATCH v5 2/4] scsi: ufs: Refine error history functions

2020-12-05 Thread Stanley Chu
Nowadays UFS error history does not only have "history of errors" but also have history of some other events which are not defined as errors. This patch fixes the confused naming of related functions, and change the way for updating and printing history as preparation of next patch. This patch sh

RE: [PATCH v4 0/8] Refine error history and introduce event_notify vop

2020-12-05 Thread Stanley Chu
Hi Avri, On Sat, 2020-12-05 at 16:05 +0800, Avri Altman wrote: > > > > Hi Stanley, > > Will you split this series to 3 separate series: > > Phy initialization cleanup, Error history, and event notification? > > As those 3 aren't really connected? > > > > Please maintain Can's reviewed-by tag for

Re: [PATCH v6] checkpatch: add fix for non-standard signature - co-authored-by

2020-12-05 Thread Aditya
On 5/12/20 5:33 pm, Joe Perches wrote: > On Sat, 2020-12-05 at 15:52 +0530, Aditya wrote: >> On 4/12/20 8:10 pm, Aditya Srivastava wrote: >>> Currently, checkpatch.pl warns us for BAD_SIGN_OFF on the usage of >>> non-standard signatures. > [] >>> The standard signature equivalent for 'Co-authored-b

[PATCH v1 1/4] scsi: ufs: Remove unused setup_regulators variant function

2020-12-05 Thread Stanley Chu
Since setup_regulators variant function is not used by any vendors, simply remove it. Reviewed-by: Avri Altman Signed-off-by: Stanley Chu --- drivers/scsi/ufs/ufshcd.c | 10 +- drivers/scsi/ufs/ufshcd.h | 10 -- 2 files changed, 1 insertion(+), 19 deletions(-) diff --git a/driv

[PATCH v5 1/4] scsi: ufs: Add error history for abort event in UFS Device W-LUN

2020-12-05 Thread Stanley Chu
Add error history for abort event in UFS Device W-LUN. Besides, use specified value as parameter of ufshcd_update_reg_hist() to identify the aborted tag or LUNs. Reviewed-by: Asutosh Das Reviewed-by: Can Guo Signed-off-by: Stanley Chu --- drivers/scsi/ufs/ufshcd.c | 6 -- 1 file changed, 4

[PATCH v1 0/4] scsi: ufs: Cleanup phy_initialization vop

2020-12-05 Thread Stanley Chu
Hi, This series simply cleans up UFS vops and shall not change any functionality. Stanley Chu (4): scsi: ufs: Remove unused setup_regulators variant function scsi: ufs: Introduce phy_initialization helper scsi: ufs-cdns: Use phy_initialization helper scsi: ufs-dwc: Use phy_initialization h

Re: WARN_ON_ONCE

2020-12-05 Thread Michael Ellerman
Alexey Kardashevskiy writes: > On 04/12/2020 12:25, Michael Ellerman wrote: >> Dmitry Vyukov writes: >>> On Thu, Dec 3, 2020 at 10:19 AM Dmitry Vyukov wrote: On Thu, Dec 3, 2020 at 10:10 AM Alexey Kardashevskiy wrote: > > Hi! > > Syzkaller triggered WARN_ON_ONCE at >>

Re: [PATCH v6] checkpatch: add fix for non-standard signature - co-authored-by

2020-12-05 Thread Joe Perches
On Sat, 2020-12-05 at 15:52 +0530, Aditya wrote: > On 4/12/20 8:10 pm, Aditya Srivastava wrote: > > Currently, checkpatch.pl warns us for BAD_SIGN_OFF on the usage of > > non-standard signatures. [] > > The standard signature equivalent for 'Co-authored-by' is > > 'Co-developed-by'. > > > > Provid

[PATCH v5 4/4] scsi: ufs-mediatek: Introduce event_notify implementation

2020-12-05 Thread Stanley Chu
Introduce event_notify implementation on MediaTek UFS platform. A vendor-specific tracepoint is added that could be used for debugging purpose. Reviewed-by: Avri Altman Signed-off-by: Stanley Chu --- drivers/scsi/ufs/ufs-mediatek-trace.h | 37 +++ drivers/scsi/ufs/ufs-m

[PATCH v1 3/4] scsi: ufs-cdns: Use phy_initialization helper

2020-12-05 Thread Stanley Chu
Use phy_initialization helper instead of direct function invoking. Reviewed-by: Avri Altman Signed-off-by: Stanley Chu --- drivers/scsi/ufs/cdns-pltfrm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/scsi/ufs/cdns-pltfrm.c b/drivers/scsi/ufs/cdns-pltfrm.c index d

Re: [PATCH] USB: dummy-hcd: Fix uninitialized array use in init()

2020-12-05 Thread Minh Bùi Quang
Vào Th 6, 4 thg 12, 2020 vào lúc 23:12 Alan Stern đã viết: > Does this initialization end up using less memory than an explicit > memset() call? You mean speed? In my opinion, there is no difference in speed between 2 ways. When I compile this array initialization using gcc 5.4.0, this initial

[PATCH v1 2/4] scsi: ufs: Introduce phy_initialization helper

2020-12-05 Thread Stanley Chu
Introduce phy_initialization helper since this is the only one variant function without helper. Reviewed-by: Avri Altman Signed-off-by: Stanley Chu --- drivers/scsi/ufs/ufshcd.h | 8 1 file changed, 8 insertions(+) diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h ind

[PATCH v1 4/4] scsi: ufs-dwc: Use phy_initialization helper

2020-12-05 Thread Stanley Chu
Use phy_initialization helper instead of direct invoking. Reviewed-by: Avri Altman Signed-off-by: Stanley Chu --- drivers/scsi/ufs/ufshcd-dwc.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/scsi/ufs/ufshcd-dwc.c b/drivers/scsi/ufs/ufshcd-dwc.c index 6a9

[PATCH v5 3/4] scsi: ufs: Introduce event_notify variant function

2020-12-05 Thread Stanley Chu
Introduce event_notify variant function to allow vendor to get notification of important events and connect to any proprietary debugging facilities. Reviewed-by: Asutosh Das Reviewed-by: Can Guo Signed-off-by: Stanley Chu --- drivers/scsi/ufs/ufshcd.c | 2 ++ drivers/scsi/ufs/ufshcd.h | 11 ++

[PATCH v5 0/4] scsi: ufs: Refine error history and introduce event_notify vop

2020-12-05 Thread Stanley Chu
Hi, This series refines error history functions and introduces a new event_notify vop to allow vendor to get notification of important events. Changes since v4: - Seperate patch sets according to Avri's suggestion Changes since v3: - Fix build warning in patch [8/8] Changes since v2: - Ad

Re: [PATCH 2/2] arm64: dts: allwinner: a100: Add CPU Operating Performance Points table

2020-12-05 Thread Frank Lee
HI shuosheng, On Fri, Dec 4, 2020 at 3:11 PM Shuosheng Huang wrote: > > Add an Operating Performance Points table for the CPU cores to > enable Dynamic Voltage & Frequency Scaling on the A100. > > Signed-off-by: Shuosheng Huang > --- > .../allwinner/sun50i-a100-allwinner-perf1.dts | 5 ++ > ..

Re: [PATCH v11 00/10] Introduce memory interconnect for NVIDIA Tegra SoCs

2020-12-05 Thread Krzysztof Kozlowski
On Thu, 3 Dec 2020 22:24:29 +0300, Dmitry Osipenko wrote: > This series brings initial support for memory interconnect to Tegra20, > Tegra30 and Tegra124 SoCs. > > For the starter only display controllers and devfreq devices are getting > interconnect API support, others could be supported later o

Re: [PATCH] i2c: mlxbf: select CONFIG_I2C_SLAVE

2020-12-05 Thread Wolfram Sang
On Thu, Dec 03, 2020 at 11:32:50PM +0100, Arnd Bergmann wrote: > From: Arnd Bergmann > > If this is not enabled, the interfaces used in this driver do not work: > > drivers/i2c/busses/i2c-mlxbf.c:1888:3: error: implicit declaration of > function 'i2c_slave_event' [-Werror,-Wimplicit-function-de

Re: [PATCH] memory: ti-emif-sram: only build for ARMv7

2020-12-05 Thread Krzysztof Kozlowski
On Fri, 4 Dec 2020 00:08:14 +0100, Arnd Bergmann wrote: > The driver can be compile-tested on all ARM machines, but > causes a failure when built for ARMv7-M: > > arm-linux-gnueabi-ld: error: drivers/memory/ti-emif-sram-pm.o: conflicting > architecture profiles A/M > > Limit the target machines

Re: [PATCH] pcmcia/electra_cf: Fix some return values in 'electra_cf_probe()' in case of error

2020-12-05 Thread Dominik Brodowski
Am Wed, Jun 17, 2020 at 09:53:26PM +0200 schrieb Christophe JAILLET: > 'status' is known to be 0 at this point. It must be set to a meaningful > value in order to return an error code if one of the 'of_get_property()' > call fails. > > Return -EINVAL in such a case. > > Fixes: 2b571a066a2f("pcmci

Re: [PATCH] [v2] clk: samsung: mark PM functions as __maybe_unused

2020-12-05 Thread Krzysztof Kozlowski
On Fri, 4 Dec 2020 10:16:11 +0100, Arnd Bergmann wrote: > The use of SIMPLE_DEV_PM_OPS() means that the suspend/resume > functions are now unused when CONFIG_PM is disabled: > > drivers/clk/samsung/clk-exynos-clkout.c:219:12: error: 'exynos_clkout_resume' > defined but not used [-Werror=unused-fu

Re: [PATCH] memory: jz4780_nemc: Fix potential NULL dereference in jz4780_nemc_probe()

2020-12-05 Thread Krzysztof Kozlowski
On Fri, 4 Dec 2020 16:31:57 +0800, Zhang Changzhong wrote: > platform_get_resource() may fail and return NULL, so we should > better check it's return value to avoid a NULL pointer dereference > a bit later in the code. > > This is detected by Coccinelle semantic patch. > > @@ > expression pdev,

Re: [PATCH 3/4] clk: Ingenic: Add missing clocks for Ingenic SoCs.

2020-12-05 Thread Zhou Yanjie
Hi Paul, On 2020/12/3 上午6:18, Paul Cercueil wrote: Hi Zhou, Le jeu. 26 nov. 2020 à 1:26, 周琰杰 (Zhou Yanjie) a écrit : Add CIM, AIC, DMIC clocks for the X1000 SoC, and CIM, AIC, DMIC, I2S clocks for the X1830 SoC from Ingenic. Signed-off-by: 周琰杰 (Zhou Yanjie) ---  drivers/clk/ingenic/x1000-c

Re: [PATCH] staging: greybus: audio: Fix possible leak free widgets in gbaudio_dapm_free_controls

2020-12-05 Thread Vaibhav Agarwal
On Sat, Dec 5, 2020 at 4:02 PM Wang Hai wrote: > > In gbaudio_dapm_free_controls(), if one of the widgets is not found, an error > will be returned directly, which will cause the rest to be unable to be freed, > resulting in leak. > > This patch fixes the bug. If if one of them is not found, just

Re: [PATCH 5/7] regmap: sdw: use no_pm routines for SoundWire 1.2 MBQ

2020-12-05 Thread Pierre-Louis Bossart
MODULE_DESCRIPTION("Regmap SoundWire MBQ Module"); -MODULE_LICENSE("GPL v2"); +MODULE_LICENSE("GPL"); Why do you want to change this ? We only use MODULE_LICENSE("GPL") for new contributions since 'GPL v2' does not bring any information on the license, is equivalent to 'GPL' and only ex

Re: [PATCH 4/7] soundwire/regmap: use _no_pm functions in regmap_read/write

2020-12-05 Thread Pierre-Louis Bossart
-static int sdw_write_no_pm(struct sdw_slave *slave, u32 addr, u8 value) +int sdw_write_no_pm(struct sdw_slave *slave, u32 addr, u8 value) { return sdw_nwrite_no_pm(slave, addr, 1, &value); } +EXPORT_SYMBOL(sdw_write_no_pm); Why not export this is patch 1..? yes, good point. I g

[PATCH 7/9] mm: memcontrol: convert NR_SHMEM_PMDMAPPED account to pages

2020-12-05 Thread Muchun Song
Convert NR_SHMEM_PMDMAPPED account to pages. Signed-off-by: Muchun Song --- drivers/base/node.c | 3 +-- fs/proc/meminfo.c | 2 +- mm/page_alloc.c | 3 +-- mm/rmap.c | 6 -- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/base/node.c b/drivers/base/node

[GIT PULL] Please pull powerpc/linux.git powerpc-5.10-5 tag

2020-12-05 Thread Michael Ellerman
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi Linus, Please pull some more powerpc fixes for 5.10. The change to kernel/cpu.c is just adding an optional arch override, and has Peter's ack. cheers The following changes since commit b6b79dd53082db11070b4368d85dd6699ff0b063: powerpc/64s:

[PATCH 2/9] mm: memcontrol: fix NR_ANON_THPS account

2020-12-05 Thread Muchun Song
The unit of NR_ANON_THPS is HPAGE_PMD_NR already. So it should inc/dec by one rather than nr_pages. Fixes: 468c398233da ("mm: memcontrol: switch to native NR_ANON_THPS counter") Signed-off-by: Muchun Song --- mm/memcontrol.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

Re: [PATCH 4/4] clk: Ingenic: Fill unused bits in parents and reformat code.

2020-12-05 Thread Zhou Yanjie
Hi Paul, On 2020/12/3 上午6:09, Paul Cercueil wrote: Hi Zhou, Le jeu. 26 nov. 2020 à 1:26, 周琰杰 (Zhou Yanjie) a écrit : 1.Fill unused bits in parents in jz4780-cgu.c, x1000-cgu.c,   and x1830-cgu.c, these bits should be filled with -1. 2.Reformat code, add missing blank lines, remove unnecessar

Re: [PATCH 1/2] soc: samsung: exynos-chipid: order list of SoCs by name

2020-12-05 Thread Krzysztof Kozlowski
On Wed, 2 Dec 2020 21:59:54 +0200, Krzysztof Kozlowski wrote: > Bring some order to the list of SoCs. No functional change. Applied, thanks! [1/2] soc: samsung: exynos-chipid: order list of SoCs by name [2/2] soc: samsung: exynos-chipid: initialize later - with arch_initcall commit: 3b4c36

Re: [PATCH] powerpc/mm: Fix KUAP warning by providing copy_from_kernel_nofault_allowed()

2020-12-05 Thread Christoph Hellwig
Yes, I think at this point in the release cycle the specific powerpc fix is safer anyway. But this screams for an eventual general solution.

Re: [PATCH net v1] net: dsa: ksz8795: use correct number of physical ports

2020-12-05 Thread Sven Van Asbroeck
Jakub, Andrew, On Fri, Dec 4, 2020 at 6:24 PM Jakub Kicinski wrote: > > All the port counts here are -1 compared to datasheets, so I'm assuming > the are not supposed to include the host facing port or something? > > Can you describe the exact problem you're trying to solve? > The ksz8795 driver

Re: [PATCH 1/2] cpufreq: sun50i: add a100 cpufreq support

2020-12-05 Thread Frank Lee
HI Maxime, Any comment? BR / Yangtao

Re: [PATCH v3 2/3] iio: hid-sensors: Add hinge sensor driver

2020-12-05 Thread Jonathan Cameron
On Thu, 3 Dec 2020 11:53:51 +0800 Ye Xiang wrote: > The Hinge sensor is a common custom sensor on laptops. It calculates > the angle between the lid (screen) and the base (keyboard). In addition, > it also exposes screen and the keyboard angles with respect to the > ground. Applications can easi

Re: [linux-sunxi] Re: [PATCH 5/8] clk: sunxi-ng: Add support for the Allwinner H616 CCU

2020-12-05 Thread Icenowy Zheng
在 2020-12-02星期三的 23:06 +,André Przywara写道: > On 02/12/2020 21:03, Jernej Škrabec wrote: > > Dne sreda, 02. december 2020 ob 14:54:06 CET je Andre Przywara > > napisal(a): > > > While the clocks are fairly similar to the H6, many differ in > > > tiny > > > details, so a separate clock driver see

[PATCH 0/9] Convert all THP vmstat counters to pages

2020-12-05 Thread Muchun Song
Hi, This patch series is aimed to convert all THP vmstat counters to pages and the kernel stack vmstat counter to bytes. The unit of some vmstat counters are pages, the unit of some vmstat counters are bytes, the unit of some vmstat counters are HPAGE_PMD_NR, and the unit of some vmstat counters

RE: [PATCH 03/22] keembay-ipc: Add Keem Bay IPC module

2020-12-05 Thread Gross, Mark
> -Original Message- > From: Greg KH > Sent: Saturday, December 5, 2020 12:40 AM > To: mark gross > Cc: markgr...@kernel.org; a...@arndb.de; b...@suse.de; > damien.lem...@wdc.com; dragan.cve...@xilinx.com; cor...@lwn.net; > leonard.cres...@nxp.com; palmerdabb...@google.com; > paul.walm

  1   2   3   4   5   >