Re: [PATCH v4] PM / clk: make PM clock layer compatible with clocks that must sleep

2021-02-10 Thread Geert Uytterhoeven
Hi Stephen, On Thu, Feb 11, 2021 at 4:35 AM Stephen Boyd wrote: > Quoting Nicolas Pitre (2021-01-25 11:29:18) > > The clock API splits its interface into sleepable ant atomic contexts: > > > > - clk_prepare/clk_unprepare for stuff that might sleep > > > > - clk_enable_clk_disable for anything

Re: [PATCH] staging: rtl8723bs: remove obsolete commented out code

2021-02-10 Thread Dan Carpenter
On Thu, Feb 11, 2021 at 12:40:41AM +0530, karthik alapati wrote: > fix some checkpatch "Block comments use * on subsequent lines" > warnings and remove obsolete code > This isn't the correct commit message. It should say something like: "There is a bunch of messy, commented out code. Just

[PATCH net-next v4 2/9] net: phy: icplus: use PHY_ID_MATCH_EXACT() for IP101A/G

2021-02-10 Thread Michael Walle
According to the datasheet of the IP101A/G there is no revision field and MII_PHYSID2 always reads as 0x0c54. Use PHY_ID_MATCH_EXACT() then. Signed-off-by: Michael Walle Reviewed-by: Andrew Lunn --- Changes since v3: - none Changes since v2: - none Changes since v1: - none

Re: [PATCH v4 1/2] pinctrl: use to octal permissions for debugfs files

2021-02-10 Thread Drew Fustini
On Wed, Feb 10, 2021 at 11:36:39PM -0800, Joe Perches wrote: > On Wed, 2021-02-10 at 14:28 -0800, Drew Fustini wrote: > > Switch over pinctrl debugfs files to use octal permissions as they are > > preferred over symbolic permissions. Refer to commit f90774e1fd27 > > ("checkpatch: look for symbolic

[PATCH net-next v4 6/9] net: phy: icplus: don't set APS_EN bit on IP101G

2021-02-10 Thread Michael Walle
This bit is reserved as 'always-write-1'. While this is not a particular error, because we are only setting it, guard it by checking the model to prevent errors in the future. Signed-off-by: Michael Walle Reviewed-by: Andrew Lunn --- Changes since v3: - none Changes since v2: - none Changes

[PATCH net-next v4 7/9] net: phy: icplus: fix paged register access

2021-02-10 Thread Michael Walle
Registers >= 16 are paged. Be sure to set the page. It seems this was working for now, because the default is correct for the registers used in the driver at the moment. But this will also assume, nobody will change the page select register before linux is started. The page select register is

[PATCH net-next v4 9/9] net: phy: icplus: add MDI/MDIX support for IP101A/G

2021-02-10 Thread Michael Walle
Implement the operations to set desired mode and retrieve the current mode. This feature was tested with an IP101G. Signed-off-by: Michael Walle Reviewed-by: Andrew Lunn --- Changes since v3: - added return code check on phy_select_page() Changes since v2: - none Changes since v1: - none,

[PATCH net-next v4 8/9] net: phy: icplus: add PHY counter for IP101G

2021-02-10 Thread Michael Walle
The IP101G provides three counters: RX packets, CRC errors and symbol errors. The error counters can be configured to clear automatically on read. Unfortunately, this isn't true for the RX packet counter. Because of this and because the RX packet counter is more likely to overflow, than the error

[PATCH net-next v4 1/9] net: phy: icplus: use PHY_ID_MATCH_MODEL() macro

2021-02-10 Thread Michael Walle
Simpify the initializations of the structures. There is no functional change. Signed-off-by: Michael Walle Reviewed-by: Andrew Lunn --- Changes since v3: - none Changes since v2: - none Changes since v1: - none drivers/net/phy/icplus.c | 19 ++- 1 file changed, 10

[PATCH net-next v4 0/9] net: phy: icplus: cleanups and new features

2021-02-10 Thread Michael Walle
Cleanup the PHY drivers for IPplus devices and add PHY counters and MDIX support for the IP101A/G. Patch 5 adds a model detection based on the behavior of the PHY. Unfortunately, the IP101A shares the PHY ID with the IP101G. But the latter provides more features. Try to detect the newer model by

[PATCH net-next v4 3/9] net: phy: icplus: drop address operator for functions

2021-02-10 Thread Michael Walle
Don't sometimes use the address operator and sometimes not. Drop it and make the code look uniform. Signed-off-by: Michael Walle Reviewed-by: Andrew Lunn --- Changes since v3: - none Changes since v2: - none Changes since v1: - none drivers/net/phy/icplus.c | 10 +- 1 file

Re: [PATCH v3] driver core: auxiliary bus: Fix calling stage for auxiliary bus init

2021-02-10 Thread Greg KH
On Wed, Feb 10, 2021 at 01:16:11PM -0700, Dave Jiang wrote: > MODULE_LICENSE("GPL v2"); > MODULE_DESCRIPTION("Auxiliary Bus"); > MODULE_AUTHOR("David Ertman "); As this code can not be built as a module, can you remove these lines as well? I don't think they do anything, and were never

Re: [PATCH] staging: rtl8723bs: remove obsolete commented out code

2021-02-10 Thread Dan Carpenter
On Thu, Feb 11, 2021 at 12:49:30AM +0530, karthek wrote: > Hey dan, > check this out too I have no idea what you're talking about. Anyway, I just read my inbox in chronological order. That means I tend to send my replies before reading Greg's comments and before seeing the v2 patches. Just be

[PATCH] powerpc/bug: Remove specific powerpc BUG_ON()

2021-02-10 Thread Christophe Leroy
powerpc BUG_ON() is based on using twnei or tdnei instruction, which obliges gcc to format the condition into a 0 or 1 value in a register. By using a generic implementation, gcc will generate a branch to the unconditional trap generated by BUG(). As modern powerpc implement branch folding,

Re: [PATCH v4 2/2] pinctrl: pinmux: Add pinmux-select debugfs file

2021-02-10 Thread Dan Carpenter
On Wed, Feb 10, 2021 at 11:24:23PM -0800, Joe Perches wrote: > On Thu, 2021-02-11 at 10:11 +0300, Dan Carpenter wrote: > > On Wed, Feb 10, 2021 at 02:28:54PM -0800, Drew Fustini wrote: > > > + ret = strncpy_from_user(buf, user_buf, PINMUX_MAX_NAME * 2); > > > + if (ret < 0) { > > > +

[PATCH v2] powerpc/traps: Declare unrecoverable_exception() as __noreturn

2021-02-10 Thread Christophe Leroy
unrecoverable_exception() is never expected to return, most callers have an infiniteloop in case it returns. Ensure it really never returns by terminating it with a BUG(), and declare it __no_return. It always GCC to really simplify functions calling it. In the exemple below, it avoids the stack

Re: [PATCH 2/3] ARM: dts: exynos: Add charger supply for I9100

2021-02-10 Thread Krzysztof Kozlowski
On Thu, Feb 11, 2021 at 05:59:04AM +, Timon Baetz wrote: > On Sat, 30 Jan 2021 17:29:31 +, Timon Baetz wrote: > > The regulator is used for charging control by max8997_charger driver. > > > > Signed-off-by: Timon Baetz > > --- > > arch/arm/boot/dts/exynos4210-i9100.dts | 2 ++ > > 1

Re: [PATCH][next] soc: xilinx: vcu: remove deadcode on null divider check

2021-02-10 Thread Michael Tretter
On Wed, 10 Feb 2021 19:28:18 -0800, Stephen Boyd wrote: > Quoting Colin King (2021-02-10 10:49:38) > > From: Colin Ian King > > > > The pointer 'divider' has previously been null checked followed by > > a return, hence the subsequent null check is redundant deadcode > > that can be removed.

Re: [PATCH v4 1/2] pinctrl: use to octal permissions for debugfs files

2021-02-10 Thread Joe Perches
On Wed, 2021-02-10 at 14:28 -0800, Drew Fustini wrote: > Switch over pinctrl debugfs files to use octal permissions as they are > preferred over symbolic permissions. Refer to commit f90774e1fd27 > ("checkpatch: look for symbolic permissions and suggest octal instead"). > > Note: S_IFREG flag is

Re: [PATCH v2 2/2] power: supply: bq25980: Move props from battery node

2021-02-10 Thread Krzysztof Kozlowski
On Wed, Feb 10, 2021 at 04:56:46PM -0600, Ricardo Rivera-Matos wrote: > Currently POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT and > > POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE are exposed on the battery node > > and this is incorrect. > > This patch exposes

Re: [PATCH v2 1/2] power: supply: bq25980 Apply datasheet revision changes

2021-02-10 Thread Krzysztof Kozlowski
On Wed, Feb 10, 2021 at 04:56:45PM -0600, Ricardo Rivera-Matos wrote: > The latest datasheet revision for BQ25980, BQ25975, and BQ25960 changed > > various register step sizes and offset values. > > This patch changes the following header file > > values for POWER_SUPPLY_PROP_CURRENT_NOW, >

Re: [PATCH] ARM: kexec: Remove unused kexec_reinit callback

2021-02-10 Thread Arnd Bergmann
On Thu, Feb 11, 2021 at 12:52 AM Joel Stanley wrote: > > The last (only?) user of this was removed in commit ba364fc752da ("ARM: > Kirkwood: Remove mach-kirkwood"), back in v3.17. > > Signed-off-by: Joel Stanley Reviewed-by: Arnd Bergmann

Re: [PATCH v2 3/3] vdpa/mlx5: defer clear_virtqueues to until DRIVER_OK

2021-02-10 Thread Eli Cohen
On Wed, Feb 10, 2021 at 01:48:00PM -0800, Si-Wei Liu wrote: > While virtq is stopped, get_vq_state() is supposed to > be called to get sync'ed with the latest internal > avail_index from device. The saved avail_index is used > to restate the virtq once device is started. Commit >

Re: [PATCH v2 2/3] vdpa/mlx5: fix feature negotiation across device reset

2021-02-10 Thread Eli Cohen
On Wed, Feb 10, 2021 at 01:47:59PM -0800, Si-Wei Liu wrote: > The mlx_features denotes the capability for which > set of virtio features is supported by device. In > principle, this field needs not be cleared during > virtio device reset, as this capability is static > and does not change across

Re: [PATCH][next] soc: xilinx: vcu: remove deadcode on null divider check

2021-02-10 Thread Michael Tretter
On Wed, 10 Feb 2021 18:49:38 +, Colin King wrote: > From: Colin Ian King > > The pointer 'divider' has previously been null checked followed by > a return, hence the subsequent null check is redundant deadcode > that can be removed. Clean up the code and remove it. > > Fixes: 9c789deea206

Re: phy_attach_direct()'s use of device_bind_driver()

2021-02-10 Thread Heiner Kallweit
On 11.02.2021 00:29, Saravana Kannan wrote: > On Wed, Feb 10, 2021 at 2:52 PM Andrew Lunn wrote: >> >> On Wed, Feb 10, 2021 at 02:13:48PM -0800, Saravana Kannan wrote: >>> Hi, >>> >>> This email was triggered by this other email[1]. >>> >>> Why is phy_attach_direct() directly calling

[PATCH] The Patch fixes the missing a blank line warning

2021-02-10 Thread Adithya Chandrakasan
FILE: mm/util.c:930: WARNING: Missing a blank line after declarations Signed-off-by: Adithya Chandrakasan --- mm/util.c | 1 + 1 file changed, 1 insertion(+) diff --git a/mm/util.c b/mm/util.c index 8c9b7d1e7c49..60286876636d 100644 --- a/mm/util.c +++ b/mm/util.c @@ -927,6 +927,7 @@ int

Re: [PATCH v4 2/2] pinctrl: pinmux: Add pinmux-select debugfs file

2021-02-10 Thread Joe Perches
On Thu, 2021-02-11 at 10:11 +0300, Dan Carpenter wrote: > On Wed, Feb 10, 2021 at 02:28:54PM -0800, Drew Fustini wrote: > > + ret = strncpy_from_user(buf, user_buf, PINMUX_MAX_NAME * 2); > > + if (ret < 0) { > > + dev_err(pctldev->dev, "failed to copy buffer from userspace"); > > +

Re: [PATCH v3 2/2] kunit: ubsan integration

2021-02-10 Thread kernel test robot
in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Daniel-Latypov/kunit-support-failure-from-dynamic-analysis-tools/20210210-074913 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git e0756cfc7d7cd08c98a53b6009c091a3f6a50be6 config

Re: [PATCH v7 3/6] mfd: bd9576: Add IRQ support

2021-02-10 Thread Vaittinen, Matti
Hello Again Lee & All, On Wed, 2021-02-10 at 09:01 +0200, Matti Vaittinen wrote: > On Tue, 2021-02-09 at 15:25 +, Lee Jones wrote: > > On Fri, 22 Jan 2021, Matti Vaittinen wrote: > > + /* BD9573 only supports fatal IRQs which we do not > > > handle */ > > > > Why not? > > Because

Re: [PATCH] powerpc/traps: Declare unrecoverable_exception() as __noreturn

2021-02-10 Thread Christophe Leroy
Le 11/02/2021 à 07:34, Christophe Leroy a écrit : unrecoverable_exception() is never expected to return, most callers have an infiniteloop in case it returns. Ensure it really never returns by terminating it with a BUG(), and declare it __no_return. Not so easy, gcc complains about

Re: [PATCH v4] staging: gdm724x: Fix DMA from stack

2021-02-10 Thread Dan Carpenter
On Thu, Feb 11, 2021 at 11:08:19AM +0530, Amey Narkhede wrote: > Stack allocated buffers cannot be used for DMA > on all architectures so allocate hci_packet buffer > using kmalloc. > > Signed-off-by: Amey Narkhede Looks good. Reviewed-by: Dan Carpenter regards, dan carpenter

Re: [PATCH v17 07/10] mm: introduce memfd_secret system call to create "secret" memory areas

2021-02-10 Thread Mike Rapoport
On Tue, Feb 09, 2021 at 02:17:11PM +0100, Michal Hocko wrote: > On Tue 09-02-21 11:09:38, Mike Rapoport wrote: > > On Tue, Feb 09, 2021 at 09:47:08AM +0100, Michal Hocko wrote: > > > > > > OK, so IIUC this means that the model is to hand over memory from host > > > to guest. I thought the guest

Re: [PATCH v4 2/2] pinctrl: pinmux: Add pinmux-select debugfs file

2021-02-10 Thread Dan Carpenter
On Wed, Feb 10, 2021 at 02:28:54PM -0800, Drew Fustini wrote: > + ret = strncpy_from_user(buf, user_buf, PINMUX_MAX_NAME * 2); > + if (ret < 0) { > + dev_err(pctldev->dev, "failed to copy buffer from userspace"); > + goto free_gname; > + } > + buf[len-1] =

Re: [PATCH] scripts: Fix linking extract-cert against libcrypto

2021-02-10 Thread Masahiro Yamada
On Tue, Feb 9, 2021 at 2:01 PM Daniel Díaz wrote: > > When compiling under OpenEmbedded, the following error is seen > as of recently: > > /srv/oe/build/tmp/hosttools/ld: cannot find /lib/libc.so.6 inside / > /srv/oe/build/tmp/hosttools/ld: cannot find /usr/lib/libc_nonshared.a > inside / >

[PATCH] irq: simplify condition in irq_matrix_reserve()

2021-02-10 Thread Juergen Gross
The if condition in irq_matrix_reserve() can be much simpler. While at it fix a typo in the comment. Signed-off-by: Juergen Gross --- kernel/irq/matrix.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/kernel/irq/matrix.c b/kernel/irq/matrix.c index

Re: [PATCH net-next] misc: Add Renesas Synchronization Management Unit (SMU) support

2021-02-10 Thread Greg KH
On Wed, Feb 10, 2021 at 10:03:31PM -0500, min.li...@renesas.com wrote: > From: Min Li > > This driver supports 1588 related functions of ClockMatrix(TM) > and 82P33xxx families of timing and synchronization devices. The > supported functons are: > > - set combomode > - get dpll's state > - get

Re: [PATCH v5 2/4] USB: misc: Add onboard_usb_hub driver

2021-02-10 Thread Greg Kroah-Hartman
On Wed, Feb 10, 2021 at 09:10:37AM -0800, Matthias Kaehlcke wrote: > +static int onboard_hub_add_usbdev(struct onboard_hub *hub, struct usb_device > *udev) > +{ > + struct udev_node *node; > + char link_name[64]; > + int ret = 0; > + > + mutex_lock(>lock); > + > + if

Re: [PATCH v5 3/4] usb: host: xhci-plat: Create platform device for onboard hubs in probe()

2021-02-10 Thread Greg Kroah-Hartman
On Wed, Feb 10, 2021 at 09:10:38AM -0800, Matthias Kaehlcke wrote: > Check during probe() if a hub supported by the onboard_usb_hub > driver is connected to the controller. If such a hub is found > create the corresponding platform device. This requires the > device tree to have a node for the hub

Re: [PATCH] arm/xen: Don't probe xenbus as part of an early initcall

2021-02-10 Thread Jürgen Groß
On 10.02.21 18:06, Julien Grall wrote: From: Julien Grall After Commit 3499ba8198cad ("xen: Fix event channel callback via INTX/GSI"), xenbus_probe() will be called too early on Arm. This will recent to a guest hang during boot. If there hang wasn't there, we would have ended up to call

Re: [PATCH v3] driver core: auxiliary bus: Fix calling stage for auxiliary bus init

2021-02-10 Thread Greg KH
On Wed, Feb 10, 2021 at 01:16:11PM -0700, Dave Jiang wrote: > When the auxiliary device code is built into the kernel, it can be executed > before the auxiliary bus is registered. This causes bus->p to be not > allocated and triggers a NULL pointer dereference when the auxiliary bus > device gets

Re: [PATCH] Macros with complex values should be enclosed in parentheses.

2021-02-10 Thread Aakash Hemadri
On 21/02/11 09:25AM, Fatih YILDIRIM wrote: > Signed-off-by: Fatih YILDIRIM > --- > Hi, > I have a coding style fix. > By the way, I'm following the Eudyptula Challenge Linux kernel tasks > and this is my first patch related to my task no 10. > I hope I'm doing it the right way. > Thanks for your

Re: [PATCH 1/2] uio: Add late_release callback to uio_info

2021-02-10 Thread Greg Kroah-Hartman
On Wed, Feb 10, 2021 at 08:57:11PM +0100, Bodo Stroesser wrote: > On 10.02.21 20:47, Greg Kroah-Hartman wrote: > > On Wed, Feb 10, 2021 at 08:40:30PM +0100, Bodo Stroesser wrote: > > > If uio_unregister_device() is called while userspace daemon > > > still holds the uio device open or mmap'ed, uio

Re: [PATCH 3/5] ath10k: change ath10k_offchan_tx_work() peer present msg to a warn

2021-02-10 Thread Kalle Valo
Shuah Khan wrote: > Based on the comment block in this function and the FIXME for this, peer > being present for the offchannel tx is unlikely. Peer is deleted once tx > is complete. Change peer present msg to a warn to detect this condition. > > Signed-off-by: Shuah Khan > Signed-off-by:

Re: [PATCH] ath9k: fix data bus crash when setting nf_override via debugfs

2021-02-10 Thread Kalle Valo
Linus Lüssing wrote: > When trying to set the noise floor via debugfs, a "data bus error" > crash like the following can happen: > > [ 88.433133] Data bus error, epc == 80221c28, ra == 83314e60 > [ 88.438895] Oops[#1]: > [ 88.441246] CPU: 0 PID: 7263 Comm: sh Not tainted 4.14.195 #0 > [

Re: [PATCH] Macros with complex values should be enclosed in parentheses.

2021-02-10 Thread Greg KH
On Thu, Feb 11, 2021 at 09:25:43AM +0300, Fatih YILDIRIM wrote: > Signed-off-by: Fatih YILDIRIM > --- > Hi, > I have a coding style fix. > By the way, I'm following the Eudyptula Challenge Linux kernel tasks > and this is my first patch related to my task no 10. > I hope I'm doing it the right

Re: brcmsmac: Fix the spelling configation to configuration in the file d11.h

2021-02-10 Thread Kalle Valo
Bhaskar Chowdhury wrote: > s/configation/configuration/ > > Signed-off-by: Bhaskar Chowdhury > Acked-by: Randy Dunlap Patch applied to wireless-drivers-next.git, thanks. 1899e49385fd brcmsmac: Fix the spelling configation to configuration in the file d11.h --

Re: [PATCH 1/1] Macros with multiple statements should be enclosed in a do - while loop.

2021-02-10 Thread Greg KH
On Thu, Feb 11, 2021 at 01:14:39AM +0300, Fatih YILDIRIM wrote: > Signed-off-by: Fatih YILDIRIM > --- > drivers/staging/greybus/loopback.c | 10 ++ > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/greybus/loopback.c >

[PATCH] powerpc/traps: Declare unrecoverable_exception() as __noreturn

2021-02-10 Thread Christophe Leroy
unrecoverable_exception() is never expected to return, most callers have an infiniteloop in case it returns. Ensure it really never returns by terminating it with a BUG(), and declare it __no_return. It always GCC to really simplify functions calling it. In the exemple below, it avoids the stack

[PATCH] Macros with complex values should be enclosed in parentheses.

2021-02-10 Thread Fatih YILDIRIM
Signed-off-by: Fatih YILDIRIM --- Hi, I have a coding style fix. By the way, I'm following the Eudyptula Challenge Linux kernel tasks and this is my first patch related to my task no 10. I hope I'm doing it the right way. Thanks for your understanding and kind comments.

Re: [PATCH v2 3/3] fpga: versal-fpga: Add versal fpga manager driver

2021-02-10 Thread Randy Dunlap
Hi-- On 2/10/21 10:05 PM, Nava kishore Manne wrote: > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig > index bf85b9a65ec2..dcd2ed5a7956 100644 > --- a/drivers/fpga/Kconfig > +++ b/drivers/fpga/Kconfig > @@ -223,4 +223,12 @@ config FPGA_MGR_ZYNQMP_FPGA > to configure the

[PATCH 1/3] mm/page_alloc: Fix pageblock_order when HUGETLB_PAGE_ORDER >= MAX_ORDER

2021-02-10 Thread Anshuman Khandual
With HUGETLB_PAGE_SIZE_VARIABLE enabled, pageblock_order cannot be assigned as HUGETLB_PAGE_ORDER when it is greater than or equal to MAX_ORDER during set_pageblock_order(). Otherwise the following warning is triggered during boot as detected on an arm64 platform. WARNING: CPU: 5 PID: 124 at

[PATCH 2/3] arm64/hugetlb: Enable HUGETLB_PAGE_SIZE_VARIABLE

2021-02-10 Thread Anshuman Khandual
MAX_ORDER which invariably depends on FORCE_MAX_ZONEORDER can be a variable for a given page size, depending on whether TRANSPARENT_HUGEPAGE is enabled or not. In certain page size and THP combinations HUGETLB_PAGE_ORDER can be greater than MAX_ORDER, making it unusable as pageblock_order. This

[PATCH 3/3] dma-contiguous: Type cast MAX_ORDER as unsigned int

2021-02-10 Thread Anshuman Khandual
Type cast MAX_ORDER as unsigned int to fix the following build warning. In file included from ./include/linux/kernel.h:14, from ./include/asm-generic/bug.h:20, from ./arch/arm64/include/asm/bug.h:26, from ./include/linux/bug.h:5,

[PATCH 0/3] mm/page_alloc: Fix pageblock_order with HUGETLB_PAGE_SIZE_VARIABLE

2021-02-10 Thread Anshuman Khandual
The following warning gets triggered while trying to boot a 64K page size without THP config kernel on arm64 platform. WARNING: CPU: 5 PID: 124 at mm/vmstat.c:1080 __fragmentation_index+0xa4/0xc0 Modules linked in: CPU: 5 PID: 124 Comm: kswapd0 Not tainted 5.11.0-rc6-4-ga0ea7d62002 #159

Re: linux-next: build failure after merge of the modules tree

2021-02-10 Thread Masahiro Yamada
On Wed, Feb 10, 2021 at 5:37 PM Masahiro Yamada wrote: > > On Wed, Feb 10, 2021 at 5:06 PM Jessica Yu wrote: > > > > +++ Stephen Rothwell [10/02/21 08:50 +1100]: > > >Hi Jessica, > > > > > >On Tue, 9 Feb 2021 16:16:20 +0100 Jessica Yu wrote: > > >> > > >> Hmm, these errors don't look like it's

Re: [PATCH v4 5/5] x86/sgx: Remove redundant if conditions in sgx_encl_create

2021-02-10 Thread Tianjia Zhang
On 2/3/21 6:04 AM, Jarkko Sakkinen wrote: On Mon, Feb 01, 2021 at 09:26:53PM +0800, Tianjia Zhang wrote: In this scenario, there is no case where va_page is NULL, and the error has been checked. The if condition statement here is if-condition, i.e. dash missing Will do in the next

[PATCH] kbuild: fix CONFIG_TRIM_UNUSED_KSYMS build for ppc64

2021-02-10 Thread Masahiro Yamada
Stephen Rothwell reported a build error on ppc64 when CONFIG_TRIM_UNUSED_KSYMS is enabled. Jessica Yu pointed out the cause of the error with the reference to the ppc64 elf ABI: "Symbol names with a dot (.) prefix are reserved for holding entry point addresses. The value of a symbol named

Re: [PATCH v4 2/5] x86/sgx: Reduce the locking range in sgx_sanitize_section()

2021-02-10 Thread Tianjia Zhang
On 2/3/21 6:00 AM, Jarkko Sakkinen wrote: On Mon, Feb 01, 2021 at 09:26:50PM +0800, Tianjia Zhang wrote: The spin lock of sgx_epc_section only locks the page_list. The EREMOVE operation and init_laundry_list is not necessary in the protection range of the spin lock. This patch reduces the

[PATCH v2 3/3] fpga: versal-fpga: Add versal fpga manager driver

2021-02-10 Thread Nava kishore Manne
Add support for Xilinx Versal FPGA manager. PDI source type can be DDR, OCM, QSPI flash etc.. But driver allocates memory always from DDR, Since driver supports only DDR source type. Signed-off-by: Appana Durga Kedareswara rao Signed-off-by: Nava kishore Manne --- Changes for v2:

[PATCH v2 1/3] drivers: firmware: Add PDI load API support

2021-02-10 Thread Nava kishore Manne
This patch adds load PDI API support to enable PDI/partial loading from linux. Programmable Device Image (PDI) is combination of headers, images and bitstream files to be loaded. Partial PDI is partial set of image/ images to be loaded. Signed-off-by: Nava kishore Manne --- Changes for v2:

[PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager

2021-02-10 Thread Nava kishore Manne
From: Appana Durga Kedareswara rao This patch adds binding doc for versal fpga manager driver. Signed-off-by: Nava kishore Manne Signed-off-by: Appana Durga Kedareswara rao --- Changes for v2: -Fixed file format and syntax issues. .../bindings/fpga/xlnx,versal-fpga.yaml

[PATCH v2 0/3] Add Bitstream configuration support for Versal

2021-02-10 Thread Nava kishore Manne
Appana Durga Kedareswara rao (1): dt-bindings: fpga: Add binding doc for versal fpga manager Nava kishore Manne (2): drivers: firmware: Add PDI load API support fpga: versal-fpga: Add versal fpga manager driver .../bindings/fpga/xlnx,versal-fpga.yaml | 33 +

Re: [PATCH v4 4/5] x86/sgx: Allows ioctl PROVISION to execute before CREATE

2021-02-10 Thread Tianjia Zhang
On 2/3/21 5:57 AM, Jarkko Sakkinen wrote: On Mon, Feb 01, 2021 at 09:26:52PM +0800, Tianjia Zhang wrote: In the function sgx_create_enclave(), the direct assignment operation of attributes_mask determines that the ioctl PROVISION operation must be executed after the ioctl CREATE operation,

Re: [PATCH v4 3/5] x86/sgx: Optimize the free_cnt count in sgx_epc_section

2021-02-10 Thread Tianjia Zhang
On 2/3/21 5:54 AM, Jarkko Sakkinen wrote: On Mon, Feb 01, 2021 at 09:26:51PM +0800, Tianjia Zhang wrote: 'section->free_cnt' represents the free page in sgx_epc_section, which is assigned once after initialization. In fact, just after the initialization is completed, the pages are in the

Re: [PATCH v3 3/5] x86/sgx: Optimize the free_cnt count in sgx_epc_section

2021-02-10 Thread Tianjia Zhang
Hi, Sorry for the late reply. On 1/28/21 1:40 AM, Jarkko Sakkinen wrote: I could bet some money that this does not bring any significant performance gain. Yes, this does not bring performance gains. This is not a change for performance, mainly to make the value of free_cnt look more

Re: [PATCH 06/24] perf daemon: Add config file support

2021-02-10 Thread Namhyung Kim
Hi Jiri, On Tue, Feb 9, 2021 at 5:09 AM Jiri Olsa wrote: > +static int daemon__reconfig(struct daemon *daemon) > +{ > + struct daemon_session *session, *n; > + > + list_for_each_entry_safe(session, n, >sessions, list) { > + /* No change. */ > + if

Re: [PATCH 2/3] ARM: dts: exynos: Add charger supply for I9100

2021-02-10 Thread Timon Baetz
On Sat, 30 Jan 2021 17:29:31 +, Timon Baetz wrote: > The regulator is used for charging control by max8997_charger driver. > > Signed-off-by: Timon Baetz > --- > arch/arm/boot/dts/exynos4210-i9100.dts | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git

Re: KASAN: null-ptr-deref Read in tcf_idrinfo_destroy

2021-02-10 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:291009f6 Merge tag 'pm-5.11-rc8' of git://git.kernel.org/p.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=14470d18d0 kernel config:

linux-next: manual merge of the rcu tree with the block tree

2021-02-10 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the rcu tree got conflicts in: include/linux/rcupdate.h kernel/rcu/tree.c kernel/rcu/tree_plugin.h between commits: 3a7b5c87a0b2 ("rcu/nocb: Perform deferred wake up before last idle's need_resched() check") e4234f21d2ea ("rcu: Pull deferred rcuog

Re: Clínica de Recuperação para Dependentes Químicos e Alcoólatras

2021-02-10 Thread GRUPO VIDA OFICIAL CENTRAL DE INTERNAÇÕES
O Grupo ViDA é uma Clínica de Recuperação e Reabilitação para dependentes químicos e alcoólatras. O Grupo ViDA oferece tratamentos contra a dependência química e o alcoolismo. Internação involuntária, resgate 24 horas para usuários de drogas e o tratamento com ibogaína. Desde 1988, recuperamos

Re: [PATCH net-next] misc: Add Renesas Synchronization Management Unit (SMU) support

2021-02-10 Thread kernel test robot
Hi, Thank you for the patch! Yet something to improve: [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/min-li-xe-renesas-com/misc-Add-Renesas-Synchronization-Management-Unit-SMU-support/20210211-113223 base:

Re: [PATCH v3 1/5] clk: sunxi-ng: mp: fix parent rate change flag check

2021-02-10 Thread Jernej Škrabec
Dne četrtek, 11. februar 2021 ob 03:28:00 CET je Stephen Boyd napisal(a): > Quoting Maxime Ripard (2021-02-10 02:29:04) > > > Hi Mike, Stephen, > > > > On Tue, Feb 09, 2021 at 06:58:56PM +0100, Jernej Skrabec wrote: > > > CLK_SET_RATE_PARENT flag is checked on parent clock instead of current > >

[PATCH v4] staging: gdm724x: Fix DMA from stack

2021-02-10 Thread Amey Narkhede
Stack allocated buffers cannot be used for DMA on all architectures so allocate hci_packet buffer using kmalloc. Signed-off-by: Amey Narkhede --- Changes in v4: - Use struct_size to allocate memory for hci_packet - Fix memory corruption drivers/staging/gdm724x/gdm_usb.c | 10

[PATCH v2] [RFC] mm: fs: Invalidate BH LRU during page migration

2021-02-10 Thread Chris Goldsworthy
Pages containing buffer_heads that are in one of the per-CPU buffer_head LRU caches will be pinned and thus cannot be migrated. This can prevent CMA allocations from succeeding, which are often used on platforms with co-processors (such as a DSP) that can only use physically contiguous memory. It

[PATCH v2] [RFC] Invalidate BH LRU during page migration

2021-02-10 Thread Chris Goldsworthy
A page containing buffer_heads can be pinned if any of its constituent buffer_heads belongs to the BH LRU cache [1], which can prevent that page from being migrated. After going through several iterations of a patch that attempts to solve this by removing BH entries inside of the drop_buffers()

Re: [PATCH 1/2] include: fpga: adi-axi-common.h: add definitions for supported FPGAs

2021-02-10 Thread Alexandru Ardelean
On Thu, Feb 11, 2021 at 6:00 AM Moritz Fischer wrote: > > Alexandru, > > On Wed, Feb 10, 2021 at 12:15:34PM +0200, Alexandru Ardelean wrote: > > From: Mircea Caprioru > > > > All (newer) FPGA IP cores supported by Analog Devices, store information in > > Nit: extra ',' ? > > the synthesized

[PATCH v2 7/7] ARM: mstar: Add mpll to base dtsi

2021-02-10 Thread Daniel Palmer
All of the currently known MStar/SigmaStar ARMv7 SoCs have at least one MPLL and it seems to always be at the same place so add it to the base dtsi. Signed-off-by: Daniel Palmer --- arch/arm/boot/dts/mstar-v7.dtsi | 8 1 file changed, 8 insertions(+) diff --git

Re: [PATCH 0/6] Support second Image Signal Processor on rk3399

2021-02-10 Thread Sebastian Fricke
Hey Heiko, On 10.02.2021 12:15, Heiko Stübner wrote: Hi Sebastian, Am Freitag, 5. Februar 2021, 15:55:56 CET schrieb Heiko Stübner: Hi Sebastian, I did some tests myself today as well and can confirm your hdmi related finding - at least when plugged in on boot. I tried some combinations of

Re: [RESEND PATCH v2 0/9] soundwire/regmap: use _no_pm routines

2021-02-10 Thread Vinod Koul
On 10-02-21, 19:58, Mark Brown wrote: > On Mon, Feb 08, 2021 at 08:34:49PM +0530, Vinod Koul wrote: > > On 08-02-21, 12:50, Mark Brown wrote: > > > > Is there a tag I can pull the new APIs from? > > > Yes, please pull from below: > > >

[PATCH v2 6/7] ARM: mstar: Add the external clocks to the base dsti

2021-02-10 Thread Daniel Palmer
All of the currently known MStar/SigmaStar ARMv7 SoCs have an "xtal" clock input that is usually 24MHz and an "RTC xtal" that is usually 32KHz. The xtal input has to be connected to something so it's enabled by default. The MSC313 and MSC313E do not bring the RTC clock input out to the pins so

[PATCH v2 4/7] clk: mstar: MStar/SigmaStar MPLL driver

2021-02-10 Thread Daniel Palmer
This adds a basic driver for the MPLL block found in MStar/SigmaStar ARMv7 SoCs. Currently this driver is only good for calculating the rates of it's outputs and the actual configuration must be done before the kernel boots. Usually this is done even before u-boot starts. This driver targets the

[PATCH v2 5/7] ARM: mstar: Select MSTAR_MSC313_MPLL

2021-02-10 Thread Daniel Palmer
All of the ARCH_MSTARV7 chips have an MPLL as the source for peripheral clocks so select MSTAR_MSC313_MPLL. Signed-off-by: Daniel Palmer --- arch/arm/mach-mstar/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-mstar/Kconfig b/arch/arm/mach-mstar/Kconfig index

[PATCH v2 2/7] dt-bindings: clk: mstar msc313 mpll binding description

2021-02-10 Thread Daniel Palmer
Add a binding description for the MStar/SigmaStar MPLL clock block. Signed-off-by: Daniel Palmer --- .../bindings/clock/mstar,msc313-mpll.yaml | 46 +++ MAINTAINERS | 1 + 2 files changed, 47 insertions(+) create mode 100644

[PATCH v2 3/7] clk: fixed: add devm helper for clk_hw_register_fixed_factor()

2021-02-10 Thread Daniel Palmer
Add a devm helper for clk_hw_register_fixed_factor() so that drivers that internally register fixed factor clocks for things like dividers don't need to manually unregister them on remove or if probe fails. Signed-off-by: Daniel Palmer --- drivers/clk/clk-fixed-factor.c | 39

[PATCH v2 1/7] dt-bindings: clk: mstar msc313 mpll binding header

2021-02-10 Thread Daniel Palmer
Simple header to document the relationship between the MPLL outputs and which divider they come from. Output 0 is missing because it should not be consumed. Signed-off-by: Daniel Palmer Acked-by: Rob Herring --- MAINTAINERS | 1 +

[PATCH v2 0/7] ARM: mstar: Basic MPLL support

2021-02-10 Thread Daniel Palmer
This series adds support for the MPLL block that is present in MStar/SigmaStar ARMv7 SoCs. This block is intended to be set and forgotten about before Linux is running so all it actually does it read the registers and calculate what the output frequencies should be. We only care about this block

Re: Very slow unlockall()

2021-02-10 Thread Hugh Dickins
On Wed, 10 Feb 2021, Michal Hocko wrote: > On Wed 10-02-21 17:57:29, Michal Hocko wrote: > > On Wed 10-02-21 16:18:50, Vlastimil Babka wrote: > [...] > > > And the munlock (munlock_vma_pages_range()) is slow, because it uses > > > follow_page_mask() in a loop incrementing addresses by PAGE_SIZE,

[PATCH v2 1/2] dt-bindings: fpga: Add compatible value for Xilinx DFX AXI shutdown manager

2021-02-10 Thread Nava kishore Manne
This patch Adds compatible value for Xilinx Dynamic Function eXchnage(DFX) AXI Shutdown manager IP. Signed-off-by: Nava kishore Manne --- Changes for v2: -Modified the doc and added DFX axi shutdown manager node example node as suggested by Tom Rix.

[PATCH v2 2/2] fpga: Add support for Xilinx DFX AXI Shutdown manager

2021-02-10 Thread Nava kishore Manne
This patch adds support for Xilinx Dynamic Function eXchange(DFX) AXI shutdown manager IP. It can be used to safely handling the AXI traffic on a Reconfigurable Partition when it is undergoing dynamic reconfiguration and there by preventing system deadlock that may occur if AXI transactions are

[PATCH v2 0/2] Add DFX AXI Shutdown manager IP support for Xilinx

2021-02-10 Thread Nava kishore Manne
Nava kishore Manne (2): dt-bindings: fpga: Add compatible value for Xilinx DFX AXI shutdown manager fpga: Add support for Xilinx DFX AXI Shutdown manager .../bindings/fpga/xilinx-pr-decoupler.txt | 24 +++- drivers/fpga/Kconfig | 9 -

Re: [PATCH v17 10/10] arm64: Enable passing IMA log to next kernel on kexec

2021-02-10 Thread Thiago Jung Bauermann
Lakshmi Ramasubramanian writes: > Update CONFIG_KEXEC_FILE to select CONFIG_HAVE_IMA_KEXEC, if CONFIG_IMA > is enabled, to indicate that the IMA measurement log information is > present in the device tree for ARM64. > > Co-developed-by: Prakhar Srivastava > Signed-off-by: Prakhar Srivastava

Re: [PATCH v17 09/10] powerpc: Delete unused function delete_fdt_mem_rsv()

2021-02-10 Thread Thiago Jung Bauermann
Lakshmi Ramasubramanian writes: > delete_fdt_mem_rsv() defined in "arch/powerpc/kexec/file_load.c" > has been renamed to fdt_find_and_del_mem_rsv(), and moved to > "drivers/of/kexec.c". > > Remove delete_fdt_mem_rsv() in "arch/powerpc/kexec/file_load.c". > > Co-developed-by: Prakhar Srivastava

Re: [PATCH v17 07/10] powerpc: Move arch independent ima kexec functions to drivers/of/kexec.c

2021-02-10 Thread Thiago Jung Bauermann
Lakshmi Ramasubramanian writes: > The functions defined in "arch/powerpc/kexec/ima.c" handle setting up > and freeing the resources required to carry over the IMA measurement > list from the current kernel to the next kernel across kexec system call. > These functions do not have architecture

[rcu:dev.2021.02.01a] BUILD SUCCESS 52a8ca059c5eca36698a33cb16a373a6a1ed8978

2021-02-10 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git dev.2021.02.01a branch HEAD: 52a8ca059c5eca36698a33cb16a373a6a1ed8978 fixup! torture: Provide bare-metal modprobe-based advice elapsed time: 724m configs tested: 118 configs skipped: 2 The following configs

Re: [PATCH v2 4/8] cxl/mem: Add basic IOCTL interface

2021-02-10 Thread Dan Williams
On Wed, Feb 10, 2021 at 10:47 AM Jonathan Cameron wrote: [..] > > +#define CXL_CMDS \ > > + ___C(INVALID, "Invalid Command"), \ > > + ___C(IDENTIFY, "Identify Command"),

Re: [PATCH] regulator: qcom-rpmh: fix pm8009 ldo7

2021-02-10 Thread Vinod Koul
On 10-02-21, 22:49, Jonathan Marek wrote: > Use the correct name to avoid ldo7 commands being sent to ldo6's address. Thanks for spotting and fixing it. Reviewed-by: Vinod Koul > Fixes: 06369bcc15a1 ("regulator: qcom-rpmh: Add support for SM8150") > Signed-off-by: Jonathan Marek > --- >

Re: linux-next: Tree for Feb 10 (acrn)

2021-02-10 Thread Randy Dunlap
On 2/10/21 7:29 PM, Liu, Shuo A wrote: > > > On 2/11/2021 01:52, Randy Dunlap wrote: >> On 2/10/21 3:42 AM, Stephen Rothwell wrote: >>> Hi all, >>> >>> Changes since 20210209: >>> >> >> ../drivers/virt/acrn/hsm.c: In function ‘remove_cpu_store’: >> ../drivers/virt/acrn/hsm.c:389:3: error:

Re: [PATCH v3 net-next 10/11] net: mscc: ocelot: offload bridge port flags to device

2021-02-10 Thread Florian Fainelli
On 2/10/2021 1:14 AM, Vladimir Oltean wrote: > From: Vladimir Oltean > > We should not be unconditionally enabling address learning, since doing > that is actively detrimential when a port is standalone and not offloading > a bridge. Namely, if a port in the switch is standalone and others

Re: [PATCH v3 net-next 09/11] net: mscc: ocelot: use separate flooding PGID for broadcast

2021-02-10 Thread Florian Fainelli
On 2/10/2021 1:14 AM, Vladimir Oltean wrote: > From: Vladimir Oltean > > In preparation of offloading the bridge port flags which have > independent settings for unknown multicast and for broadcast, we should > also start reserving one destination Port Group ID for the flooding of > broadcast

  1   2   3   4   5   6   7   8   9   10   >