[PATCH v5 09/19] dt-bindings: usb: renesas-xhci: Refer to the usb-xhci.yaml file

2020-12-05 Thread Serge Semin
With minor peculiarities (like uploading some vendor-specific firmware) these are just Generic xHCI controllers fully compatible with its properties. Make sure the Renesas USB xHCI DT nodes are also validated against the Generic xHCI DT schema. Signed-off-by: Serge Semin Reviewed-by: Rob Herring

Re: [PATCH v2 1/5] arm64: dts: meson: g12b: odroid-n2: fix PHY deassert timing requirements

2020-12-05 Thread Martin Blumenstingl
On Tue, Dec 1, 2020 at 2:21 PM Stefan Agner wrote: > > According to the datasheet (Rev. 1.9) the RTL8211F requires at least > 72ms "for internal circuits settling time" before accessing the PHY > egisters. This fixes an issue where the Ethernet link doesn't come up > when using ip link set down/up

Re: [PATCH] mm/memblock:use a more appropriate order calculation when free memblock pages

2020-12-05 Thread Anders Roxell
On Fri, 4 Dec 2020 at 18:44, Jon Hunter wrote: > > > On 04/12/2020 16:07, Marek Szyprowski wrote: > > Hi All, > > > > On 04.12.2020 14:42, Qian Cai wrote: > >> On Thu, 2020-12-03 at 23:23 +0800, carver4...@163.com wrote: > >>> From: Hailong Liu > >>> > >>> When system in the booting stage, pages

[PATCH 4/8] tty: serial: jsm: Fixed file by added more spacing in line 613

2020-12-05 Thread Clement Smith
Fixed a coding style issue Signed-off-by: Clement Smith --- drivers/tty/serial/jsm/jsm_tty.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/serial/jsm/jsm_tty.c index 2bc5f32e59f4..45e212be64c4 100644 --- a/drive

[PATCH v2] Revert "ACPI / resources: Use AE_CTRL_TERMINATE to terminate resources walks"

2020-12-05 Thread Daniel Scally
This reverts commit 8a66790b7850a6669129af078768a1d42076a0ef. Switching this function to AE_CTRL_TERMINATE broke the documented behaviour of acpi_dev_get_resources() - AE_CTRL_TERMINATE does not, in fact, terminate the resource walk because acpi_walk_resource_buffer() ignores it (specifically conv

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

2020-12-05 Thread Greg KH
On Sat, Dec 05, 2020 at 01:20:18PM +0200, Vladimir Oltean wrote: > 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 Weig

Re: [PATCH 2/7] net: batman-adv: remove unneeded MODULE_VERSION() usage

2020-12-05 Thread Jakub Kicinski
On Sat, 05 Dec 2020 08:06:40 +0100 Sven Eckelmann wrote: > On Wednesday, 2 December 2020 13:49:54 CET 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. > > Is there some explanation besides

[PATCH v5 04/19] dt-bindings: usb: Add "ulpi/serial/hsic" PHY types

2020-12-05 Thread Serge Semin
Aside from the UTMI+ there are also ULPI, Serial and HSIC PHY types that can be specified in the phy_type HCD property. Add them to the enumeration of the acceptable values. Signed-off-by: Serge Semin Reviewed-by: Rob Herring --- Changelog v2: - Grammar fix: "s/PHY types can be/PHY types that

Re: [Bug 202453] TRACE irq/18-i801_smb Tainted when enabled threadirqs in kernel commandline.

2020-12-05 Thread Oleksandr Natalenko
On Sat, Dec 05, 2020 at 05:19:18PM +0100, Thomas Gleixner wrote: > On Fri, Dec 04 2020 at 21:19, Oleksandr Natalenko wrote: > > On Thu, Dec 03, 2020 at 07:04:00PM +, > > bugzilla-dae...@bugzilla.kernel.org wrote: > >>2) Have a wrapper around handle_generic_irq() which ensures that > >>

Re: [External] Re: [PATCH 5/9] mm: memcontrol: convert NR_FILE_THPS account to pages

2020-12-05 Thread Greg KH
On Sat, Dec 05, 2020 at 11:39:24PM +0800, Muchun Song wrote: > On Sat, Dec 5, 2020 at 11:32 PM Greg KH wrote: > > > > On Sat, Dec 05, 2020 at 11:29:26PM +0800, Muchun Song wrote: > > > On Sat, Dec 5, 2020 at 10:09 PM Greg KH > > > wrote: > > > > > > > > On Sat, Dec 05, 2020 at 09:02:20PM +0800,

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

2020-12-05 Thread Greg KH
On Sat, Dec 05, 2020 at 08:52:50AM -0600, Pierre-Louis Bossart wrote: > > > > 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 s

Re: [PATCH v3 3/3] iio:Documentation: Add documentation for hinge sensor channels

2020-12-05 Thread Jonathan Cameron
On Thu, 3 Dec 2020 11:53:52 +0800 Ye Xiang wrote: > Add channel description for hinge sensor, including channel label > attribute and raw data description. > > Signed-off-by: Ye Xiang > --- > Documentation/ABI/testing/sysfs-bus-iio | 17 + > 1 file changed, 17 insertions(+) >

Re: [PATCH 3/7] net: macb: unprepare clocks in case of failure

2020-12-05 Thread Andrew Lunn
On Fri, Dec 04, 2020 at 02:34:17PM +0200, Claudiu Beznea wrote: > Unprepare clocks in case of any failure in fu540_c000_clk_init(). Hi Claudiu Nice patchset. Simple to understand. > > +err_disable_clocks: > + clk_disable_unprepare(*tx_clk); > + clk_disable_unprepare(*hclk); > + clk

[PATCH] proc: fix lookup in /proc/net subdirectories after setns(2)

2020-12-05 Thread Alexey Dobriyan
commit 1fde6f21d90f8ba5da3cb9c54ca991ed72696c43 proc: fix /proc/net/* after setns(2) only forced revalidation of regular files under /proc/net/ However, /proc/net/ is unusual in the sense of /proc/net/foo handlers take netns pointer from parent directory which is old netns. Steps

Re: [External] Re: [PATCH 5/9] mm: memcontrol: convert NR_FILE_THPS account to pages

2020-12-05 Thread Muchun Song
On Sat, Dec 5, 2020 at 11:32 PM Greg KH wrote: > > On Sat, Dec 05, 2020 at 11:29:26PM +0800, Muchun Song wrote: > > On Sat, Dec 5, 2020 at 10:09 PM Greg KH wrote: > > > > > > On Sat, Dec 05, 2020 at 09:02:20PM +0800, Muchun Song wrote: > > > > Converrt NR_FILE_THPS account to pages. > > > > > > >

Re: [PATCH 5/9] mm: memcontrol: convert NR_FILE_THPS account to pages

2020-12-05 Thread Greg KH
On Sat, Dec 05, 2020 at 09:02:20PM +0800, Muchun Song wrote: > Converrt NR_FILE_THPS account to pages. > > Signed-off-by: Muchun Song > --- > drivers/base/node.c | 3 +-- > fs/proc/meminfo.c | 2 +- > mm/filemap.c| 2 +- > mm/huge_memory.c| 3 ++- > mm/khugepaged.c | 2 +- > mm

Re: crypto: sun4i-ss: error with kmap

2020-12-05 Thread Corentin Labbe
On Fri, Dec 04, 2020 at 09:58:21PM +0100, Thomas Gleixner wrote: > On Fri, Dec 04 2020 at 20:27, Corentin Labbe wrote: > > On Fri, Dec 04, 2020 at 04:08:27PM +0100, Thomas Gleixner wrote: > >> On Fri, Dec 04 2020 at 14:26, Corentin Labbe wrote: > >> > On Fri, Dec 04, 2020 at 12:34:05AM +0100, Thoma

Re: [External] Re: [PATCH 5/9] mm: memcontrol: convert NR_FILE_THPS account to pages

2020-12-05 Thread Greg KH
On Sat, Dec 05, 2020 at 11:29:26PM +0800, Muchun Song wrote: > On Sat, Dec 5, 2020 at 10:09 PM Greg KH wrote: > > > > On Sat, Dec 05, 2020 at 09:02:20PM +0800, Muchun Song wrote: > > > Converrt NR_FILE_THPS account to pages. > > > > > > Signed-off-by: Muchun Song > > > --- > > > drivers/base/nod

Re: [PATCH 4/7] dt-bindings: add documentation for sama7g5 ethernet interface

2020-12-05 Thread Andrew Lunn
On Fri, Dec 04, 2020 at 02:34:18PM +0200, Claudiu Beznea wrote: > Add documentation for SAMA7G5 ethernet interface. > > Signed-off-by: Claudiu Beznea Reviewed-by: Andrew Lunn Andrew

Re: [PATCH 6/7] net: macb: add support for sama7g5 gem interface

2020-12-05 Thread Andrew Lunn
On Fri, Dec 04, 2020 at 02:34:20PM +0200, Claudiu Beznea wrote: > Add support for SAMA7G5 gigabit ethernet interface. > > Signed-off-by: Claudiu Beznea Reviewed-by: Andrew Lunn Andrew

Re: [PATCH] phy: mediatek: statify mtk_hdmi_phy_driver

2020-12-05 Thread Chun-Kuang Hu
Hi, Vinod: Vinod Koul 於 2020年12月5日 週六 下午5:11寫道: > > 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? > Acked-by: Chun-Kuang Hu > Signed-off-by: Vinod Kou

Re: [PATCH v3 3/6] ARM: dts: sun8i: v3s: Add node for system control

2020-12-05 Thread Samuel Holland
On 12/3/20 10:44 AM, Chen-Yu Tsai wrote: > On Fri, Dec 4, 2020 at 12:25 AM Martin Cerveny wrote: >> >> Hello. >> >> On Thu, 3 Dec 2020, Chen-Yu Tsai wrote: >> >>> Hi, >>> >>> On Mon, Nov 16, 2020 at 8:57 PM Martin Cerveny >>> wrote: Allwinner V3s has system control and SRAM C1 region s

Re: [PATCH 1/3] iio: dummy: convert all simple allocation devm_ variants

2020-12-05 Thread Jonathan Cameron
On Thu, 3 Dec 2020 11:50:03 +0200 Alexandru Ardelean wrote: > Since a main requirement for an IIO device is to have a parent device > object, it makes sense to attach more of the IIO device's objects to the > lifetime of the parent object, to clean up the code. > The idea is to also provide a bas

Re: [PATCH 7/7] net: macb: add support for sama7g5 emac interface

2020-12-05 Thread Andrew Lunn
On Fri, Dec 04, 2020 at 02:34:21PM +0200, Claudiu Beznea wrote: > Add support for SAMA7G5 10/100Mbps interface. > > Signed-off-by: Claudiu Beznea Reviewed-by: Andrew Lunn Andrew

Re: [PATCH net v3] bonding: fix feature flag setting at init time

2020-12-05 Thread Jarod Wilson
On Thu, Dec 3, 2020 at 11:45 AM Jakub Kicinski wrote: ... > nit: let's narrow down the ifdef-enery > > no need for the ifdef here, if the helper looks like this: > > +static void bond_set_xfrm_features(struct net_device *bond_dev, u64 mode) > +{ > +#ifdef CONFIG_XFRM_OFFLOAD > + if (mode ==

Re: [PATCH 2/7] net: macb: add capability to not set the clock rate

2020-12-05 Thread Andrew Lunn
On Fri, Dec 04, 2020 at 02:34:16PM +0200, Claudiu Beznea wrote: > SAMA7G5's ethernet IPs TX clock could be provided by its generic clock or > by the external clock provided by the PHY. The internal IP logic divides > properly this clock depending on the link speed. The patch adds a new > capability

Re: [PATCH v4 net-next 2/2] net: dsa: qca: ar9331: export stats64

2020-12-05 Thread Andrew Lunn
> +static void ar9331_stats_update(struct ar9331_sw_port *port, > + struct rtnl_link_stats64 *stats) > +{ > + struct ar9331_sw_stats *s = &port->stats; > + > + stats->rx_packets = s->rxbroad + s->rxmulti + s->rx64byte + > + s->rx128byte + s->rx256byte

Re: [PATCH 5/7] dt-bindings: add documentation for sama7g5 gigabit ethernet interface

2020-12-05 Thread Andrew Lunn
On Fri, Dec 04, 2020 at 02:34:19PM +0200, Claudiu Beznea wrote: > Add documentation for SAMA7G5 gigabit ethernet interface. > > Signed-off-by: Claudiu Beznea Reviewed-by: Andrew Lunn Andrew

Re: [PATCH 1/7] net: macb: add userio bits as platform configuration

2020-12-05 Thread Andrew Lunn
On Fri, Dec 04, 2020 at 02:34:15PM +0200, Claudiu Beznea wrote: > This is necessary for SAMA7G5 as it uses different values for > PHY interface and also introduces hdfctlen bit. > > Signed-off-by: Claudiu Beznea Reviewed-by: Andrew Lunn Andrew

Re: [PATCH v4 net-next 1/2] net: dsa: add optional stats64 support

2020-12-05 Thread Andrew Lunn
On Fri, Dec 04, 2020 at 03:56:23PM +0100, Oleksij Rempel wrote: > Allow DSA drivers to export stats64 > > Signed-off-by: Oleksij Rempel > Reviewed-by: Vladimir Oltean Reviewed-by: Andrew Lunn Andrew

[PATCH] proc: fix lookup in /proc/net subdirectories after setns(2)

2020-12-05 Thread Alexey Dobriyan
commit 1fde6f21d90f8ba5da3cb9c54ca991ed72696c43 proc: fix /proc/net/* after setns(2) only forced revalidation of regular files under /proc/net/ However, /proc/net/ is unusual in the sense of /proc/net/foo handlers take netns pointer from parent directory which is old netns. Steps

[PATCH][next] arm64: fix the mm build error in mm/kfence/core.c

2020-12-05 Thread Hui Su
When I build kernel with ARCH=arm64, the building errors came out like this: CC mm/kfence/core.o In file included from ../mm/kfence/core.c:21: ../arch/arm64/include/asm/kfence.h: In function ‘kfence_protect_page’: ../arch/arm64/include/asm/kfence.h:12:2: error: implicit declaration of functi

Re: [PATCH 3/3] exec: Transform exec_update_mutex into a rw_semaphore

2020-12-05 Thread Eric W. Biederman
Davidlohr Bueso writes: > On Fri, 04 Dec 2020, Linus Torvalds wrote: > >>On Fri, Dec 4, 2020 at 12:30 PM Bernd Edlinger >> wrote: >>> >perf_event_open (exec_update_mutex -> ovl_i_mutex) >> >>Side note: this one looks like it should be easy to fix. >> >>Is there any real reason why exec_

Re: [Bug 202453] TRACE irq/18-i801_smb Tainted when enabled threadirqs in kernel commandline.

2020-12-05 Thread Thomas Gleixner
On Sat, Dec 05 2020 at 17:24, Oleksandr Natalenko wrote: > On Sat, Dec 05, 2020 at 05:19:18PM +0100, Thomas Gleixner wrote: >> +/** >> + * generic_dispatch_irq - Dispatch an interrupt from an interrupt handler >> + * @irq:The irq number to handle >> + * >> + * A wrapper around generic_handle_ir

Re: [RFC PATCH v1 07/12] efi: Replace strstarts() by str_has_prefix().

2020-12-05 Thread Francis Laniel
Le vendredi 4 décembre 2020, 19:02:09 CET James Bottomley a écrit : > On Fri, 2020-12-04 at 18:07 +0100, Ard Biesheuvel wrote: > > On Fri, 4 Dec 2020 at 18:06, > > > > wrote: > > > From: Francis Laniel > > > > > > The two functions indicates if a string begins with a given prefix. > > > The onl

Re: [PATCH v2 2/2] drm: panel: add Khadas TS050 panel driver

2020-12-05 Thread Sam Ravnborg
Hi Neil, > + > +static int khadas_ts050_panel_probe(struct mipi_dsi_device *dsi) > +{ > + struct khadas_ts050_panel *khadas_ts050; > + int err; > + > + dsi->lanes = 4; > + dsi->format = MIPI_DSI_FMT_RGB888; > + dsi->mode_flags = MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_VIDEO_BURST |

[PATCH 00/20] ethernet: ucc_geth: assorted fixes and simplifications

2020-12-05 Thread Rasmus Villemoes
While trying to figure out how to allow bumping the MTU with the ucc_geth driver, I fell into a rabbit hole and stumbled on a whole bunch of issues of varying importance - some are outright bug fixes, while most are a matter of simplifying the code to make it more accessible. At the end of digging

[PATCH 03/20] ethernet: ucc_geth: remove unused read of temoder field

2020-12-05 Thread Rasmus Villemoes
In theory, such a read-after-write might be required by the hardware, but nothing in the data sheet suggests that to be the case. The name test also suggests that it's some debug leftover. Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 3 --- 1 file changed, 3 de

[PATCH 04/20] soc: fsl: qe: make cpm_muram_offset take a const void* argument

2020-12-05 Thread Rasmus Villemoes
Allow passing const-qualified pointers without requiring a cast in the caller. Signed-off-by: Rasmus Villemoes --- drivers/soc/fsl/qe/qe_common.c | 2 +- include/soc/fsl/qe/qe.h| 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/soc/fsl/qe/qe_common.c b/drive

[PATCH 02/20] ethernet: ucc_geth: fix definition and size of ucc_geth_tx_global_pram

2020-12-05 Thread Rasmus Villemoes
Table 8-53 in the QUICC Engine Reference manual shows definitions of fields up to a size of 192 bytes, not just 128. But in table 8-111, one does find the text Base Address of the Global Transmitter Parameter RAM Page. [...] The user needs to allocate 128 bytes for this page. The address must

Re: [PATCH 1/1] dt-bindings: eliminate yamllint warnings

2020-12-05 Thread Sam Ravnborg
On Fri, Dec 04, 2020 at 10:42:26AM +0800, Zhen Lei wrote: > All warnings are related only to "wrong indentation", except one: > Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml:4:1: \ > [error] missing document start "---" (document-start) > > Signed-off-by: Zhen Lei > Cc: Rob Herring >

[PATCH 09/20] ethernet: ucc_geth: replace kmalloc+memset by kzalloc

2020-12-05 Thread Rasmus Villemoes
Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/net/ethernet/freescale/ucc_geth.c b/drivers/net/ethernet/freescale/ucc_geth.c index f854ff90f238..e1574c14b7e5 100644 --- a/drivers/net/et

[PATCH 07/20] ethernet: ucc_geth: use qe_muram_free_addr()

2020-12-05 Thread Rasmus Villemoes
This removes the explicit NULL checks, and allows us to stop storing at least some of the _offset values separately. Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 77 ++- 1 file changed, 33 insertions(+), 44 deletions(-) diff --git a/drivers

Re: [PATCH v3 1/3] HID: hid-sensor-custom: Add custom sensor iio support

2020-12-05 Thread Jonathan Cameron
On Thu, 3 Dec 2020 11:53:50 +0800 Ye Xiang wrote: > Currently custom sensors properties are not decoded and it is up to > user space to interpret. > > Some manufacturers already standardized the meaning of some custom sensors. > They can be presented as a proper IIO sensor. We can identify thes

[PATCH 06/20] soc: fsl: qe: add cpm_muram_free_addr() helper

2020-12-05 Thread Rasmus Villemoes
Add a helper that takes a virtual address rather than the muram offset. This will be used in a couple of places to avoid having to store both the offset and the virtual address, as well as removing NULL checks from the callers. Signed-off-by: Rasmus Villemoes --- drivers/soc/fsl/qe/qe_common.c |

[PATCH 10/20] ethernet: ucc_geth: remove {rx,tx}_glbl_pram_offset from struct ucc_geth_private

2020-12-05 Thread Rasmus Villemoes
These fields are only used within ucc_geth_startup(), so they might as well be local variables in that function rather than being stashed in struct ucc_geth_private. Aside from making that struct a tiny bit smaller, it also shortens some lines (getting rid of pointless casts while here), and fixes

[PATCH 12/20] ethernet: ucc_geth: factor out parsing of {rx,tx}-clock{,-name} properties

2020-12-05 Thread Rasmus Villemoes
Reduce the code duplication a bit by moving the parsing of rx-clock-name and the fallback handling to a helper function. Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 79 ++- 1 file changed, 35 insertions(+), 44 deletions(-) diff --git a/dri

[PATCH 15/20] ethernet: ucc_geth: use UCC_GETH_{RX,TX}_BD_RING_ALIGNMENT macros directly

2020-12-05 Thread Rasmus Villemoes
These macros both have the value 32, there's no point first initializing align to a lower value. If anything, one could throw in a BUILD_BUG_ON(UCC_GETH_TX_BD_RING_ALIGNMENT < 4), but it's not worth it - lots of code depends on named constants having sensible values. Signed-off-by: Rasmus Villemo

[PATCH 11/20] ethernet: ucc_geth: fix use-after-free in ucc_geth_remove()

2020-12-05 Thread Rasmus Villemoes
ugeth is the netdiv_priv() part of the netdevice. Accessing the memory pointed to by ugeth (such as done by ucc_geth_memclean() and the two of_node_puts) after free_netdev() is thus use-after-free. Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 4 ++-- 1 file cha

[PATCH 08/20] ethernet: ucc_geth: remove unnecessary memset_io() calls

2020-12-05 Thread Rasmus Villemoes
These buffers have all just been handed out from qe_muram_alloc(), aka cpm_muram_alloc(), and the helper cpm_muram_alloc_common() already does memset_io(cpm_muram_addr(start), 0, size); Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 19 --

[PATCH 18/20] ethernet: ucc_geth: add helper to replace repeated switch statements

2020-12-05 Thread Rasmus Villemoes
The translation from the ucc_geth_num_of_threads enum value to the actual count can be written somewhat more compactly with a small lookup table, allowing us to replace the four switch statements. Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 100 +--

[PATCH 19/20] ethernet: ucc_geth: inform the compiler that numQueues is always 1

2020-12-05 Thread Rasmus Villemoes
The numQueuesTx and numQueuesRx members of struct ucc_geth_info are never set to anything but 1, and never have been. It's unclear how well the code supporting multiple queues would work. Until somebody wants to play with enabling that, help the compiler eliminate a lot of dead code and loops that

[PATCH 16/20] ethernet: ucc_geth: remove bd_mem_part and all associated code

2020-12-05 Thread Rasmus Villemoes
The bd_mem_part member of ucc_geth_info always has the value MEM_PART_SYSTEM, and AFAICT, there has never been any code setting it to any other value. Moreover, muram is a somewhat precious resource, so there's no point using that when normal memory serves just as well. Apart from removing a lot o

[PATCH 17/20] ethernet: ucc_geth: replace kmalloc_array()+for loop by kcalloc()

2020-12-05 Thread Rasmus Villemoes
Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/net/ethernet/freescale/ucc_geth.c b/drivers/net/ethernet/freescale/ucc_geth.c index c9f619908561..3aebea191b52 100644 --- a/dr

[PATCH 20/20] ethernet: ucc_geth: simplify rx/tx allocations

2020-12-05 Thread Rasmus Villemoes
Since kmalloc() is nowadays [1] guaranteed to return naturally aligned (i.e., aligned to the size itself) memory for power-of-2 sizes, we don't need to over-allocate the align amount, compute an aligned address within the allocation, and (for later freeing) also storing the original pointer [2]. I

[PATCH 14/20] ethernet: ucc_geth: don't statically allocate eight ucc_geth_info

2020-12-05 Thread Rasmus Villemoes
struct ucc_geth_info is somewhat large, and on systems with only one or two UCC instances, that just wastes a few KB of memory. So allocate and populate a chunk of memory at probe time instead of initializing them all during driver init. Note that the existing "ug_info == NULL" check was dead code

[PATCH 13/20] ethernet: ucc_geth: constify ugeth_primary_info

2020-12-05 Thread Rasmus Villemoes
Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/freescale/ucc_geth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/freescale/ucc_geth.c b/drivers/net/ethernet/freescale/ucc_geth.c index 700eafef4921..a06744d8b4af 100644 --- a/drivers/net/ethern

[PATCH 05/20] soc: fsl: qe: store muram_vbase as a void pointer instead of u8

2020-12-05 Thread Rasmus Villemoes
The two functions cpm_muram_offset() and cpm_muram_dma() both need a cast currently, one casts muram_vbase to do the pointer arithmetic on void pointers, the other casts the the passed-in address u8*. It's simpler and more consistent to just always use void* and drop all the casting. Signed-off-b

[PATCH 01/20] ethernet: ucc_geth: set dev->max_mtu to 1518

2020-12-05 Thread Rasmus Villemoes
All the buffers and registers are already set up appropriately for an MTU slightly above 1500, so we just need to expose this to the networking stack. AFAICT, there's no need to implement .ndo_change_mtu when the receive buffers are always set up to support the max_mtu. This fixes several warnings

Re: [GIT PULL] SMB3 Fixes

2020-12-05 Thread pr-tracker-bot
The pull request you sent on Fri, 4 Dec 2020 19:23:07 -0600: > git://git.samba.org/sfrench/cifs-2.6.git tags/5.10-rc6-smb3-fixes-part2 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/d4e904198c5b46c140fdd04492df6ec31f1f03a5 Thank you! -- Deet-doot-dot, I am a bot. ht

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

2020-12-05 Thread pr-tracker-bot
The pull request you sent on Sat, 05 Dec 2020 23:19:45 +1100: > https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git > tags/powerpc-5.10-5 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/32f741b02f1a84dd15cdaf74ea3c8d724f812318 Thank you! -- Deet-doot-d

Re: [GIT PULL] SCSI fixes for 5.10-rc6

2020-12-05 Thread pr-tracker-bot
The pull request you sent on Sat, 05 Dec 2020 09:30:29 -0800: > git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git scsi-fixes has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/312b0bcd402a003053914e13d962e82be906cf41 Thank you! -- Deet-doot-dot, I am a bot. ht

Re: [PATCH] acpi: resource: Use AE_ABORT_METHOD to terminate acpi_dev_get_resources()

2020-12-05 Thread Rafael J. Wysocki
On Friday, December 4, 2020 1:27:40 AM CET Daniel Scally wrote: > Switching this function to AE_CTRL_TERMINATE broke the documented > behaviour of acpi_dev_get_resources() - AE_CTRL_TERMINATE does not, in > fact, terminate the resource walk because acpi_walk_resource_buffer() > ignores it (specific

[PATCH v3 02/10] arm64: dts: amlogic: meson-g12: Set FL-adj property value

2020-12-05 Thread Serge Semin
In accordance with the DWC USB3 bindings the property is supposed to have uint32 type. It's erroneous from the DT schema and driver points of view to declare it as boolean. As Neil suggested set it to 0x20 so not break the platform and to make the dtbs checker happy. Link: https://lore.kernel.org

Re: [PATCH v2 2/6] genirq: Allow an interrupt to be marked as 'raw'

2020-12-05 Thread Valentin Schneider
On 03/12/20 15:52, Valentin Schneider wrote: > On 03/12/20 13:03, Peter Zijlstra wrote: [...] >> The scheduler IPI really doesn't need RCU either ;-) [...] > But as with any other interrupt, we could then go through: > > preempt_schedule_irq() ~> pick_next_task_fair() -> newidle_balance() > > w

[PATCH v3 01/10] arm: dts: keystone: Correct DWC USB3 compatible string

2020-12-05 Thread Serge Semin
Syonpsys IP cores are supposed to be defined with "snps" vendor-prefix. Use it instead of the deprecated "synopsys" one. Signed-off-by: Serge Semin Reviewed-by: Krzysztof Kozlowski --- arch/arm/boot/dts/keystone-k2e.dtsi | 2 +- arch/arm/boot/dts/keystone.dtsi | 2 +- 2 files changed, 2 ins

Re: [PATCH] drm: panel: Fully transition panel_desc kerneldoc to inline style

2020-12-05 Thread Sam Ravnborg
Hi Douglas, On Tue, Dec 01, 2020 at 12:59:12PM -0800, Douglas Anderson wrote: > In commit 131f909ad55f ("drm: panel: simple: Fixup the struct > panel_desc kernel doc") I transitioned the more deeply nested > kerneldoc comments into the inline style. Apparently it is desirable > to continue the jo

Re: [PATCH v2] drm: panel: add flags to BOE NV110WTM-N61

2020-12-05 Thread Sam Ravnborg
Hi Douglas, On Tue, Dec 01, 2020 at 12:56:11PM -0800, Douglas Anderson wrote: > I forgot to add these when posting up the support for BOE > NV110WTM-N61. Add them now. > > Fixes: a96ee0f6b58d ("drm: panel: simple: Add BOE NV110WTM-N61") > Signed-off-by: Douglas Anderson > Cc: Douglas Anderson

Re: [PATCH v2] pwm: bcm2835: Support apply function for atomic configuration

2020-12-05 Thread Uwe Kleine-König
Hello Sean, On Sat, Dec 05, 2020 at 05:34:44PM +, Sean Young wrote: > On Sat, Dec 05, 2020 at 12:28:34AM +0100, Uwe Kleine-König wrote: > > On Fri, Dec 04, 2020 at 11:38:46AM +, Sean Young wrote: > > > On Fri, Dec 04, 2020 at 12:13:26PM +0100, Uwe Kleine-König wrote: > > > > On Fri, Dec 04

[PATCH] ACPI: scan: Add PNP0D80 to the _DEP exceptions list

2020-12-05 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The PNP0D80 ("Windows-compatible System Power Management Controller") device ID is used for identifying the special device object providing the LPI (Low-power S0 Idle) _DSM interface [1]. That device object does not supply any operation regions, but it appears in _DEP lis

Re: [PATCH v6 1/8] dt-bindings: display: simple: fix alphabetical order for EDT compatibles

2020-12-05 Thread Sam Ravnborg
Hi Oleksij On Wed, Dec 02, 2020 at 09:18:19AM +0100, Oleksij Rempel wrote: > Reorder it alphabetically and remove one double entry. > > Signed-off-by: Oleksij Rempel > --- > .../bindings/display/panel/panel-simple.yaml | 16 +++- > 1 file changed, 7 insertions(+), 9 deletions(-)

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

2020-12-05 Thread Michael S. Tsirkin
On Sat, Dec 05, 2020 at 08:59:55AM +0100, Enrico Weigelt, metux IT consult wrote: > 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. Refer

Re: [PATCH v3 3/3] iio:Documentation: Add documentation for hinge sensor channels

2020-12-05 Thread Jonathan Cameron
On Fri, 04 Dec 2020 03:40:15 -0800 Srinivas Pandruvada wrote: > On Thu, 2020-12-03 at 11:53 +0800, Ye Xiang wrote: > > Add channel description for hinge sensor, including channel label > > attribute and raw data description. > > > > Signed-off-by: Ye Xiang > > --- > > Documentation/ABI/testing

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

2020-12-05 Thread Greg KH
On Fri, Dec 04, 2020 at 08:41:09AM -0800, Dan Williams wrote: > On Fri, Dec 4, 2020 at 3:41 AM Greg KH wrote: > > > > On Wed, Dec 02, 2020 at 04:54:24PM -0800, Dan Williams wrote: > > > From: Dave Ertman > > > > > > Add support for the Auxiliary Bus, auxiliary_device and auxiliary_driver. > > > I

Re: [PATCH v6 2/8] dt-bindings: display: simple: add EDT compatibles already supported by the driver

2020-12-05 Thread Sam Ravnborg
Hi Oleksij, On Wed, Dec 02, 2020 at 09:18:20AM +0100, Oleksij Rempel wrote: > Some EDT compatibles are already supported by the driver but will fail > on checkpatch script. Fix it by syncing dt-bindings documentation with the > driver. > > Signed-off-by: Oleksij Rempel > --- > .../devicetree/bi

Re: [PATCH v6 3/8] dt-bindings: display: simple: Add Kyocera tcg070wvlq panel

2020-12-05 Thread Sam Ravnborg
On Wed, Dec 02, 2020 at 09:18:21AM +0100, Oleksij Rempel wrote: > So far, this panel seems to be compatible with "lg,lb070wv8", on other > hand it is better to set this compatible in the devicetree. So, let's > add it for now only to the dt-binding documentation to fix the > checkpatch warnings. >

Re: [RFC PATCH v1 07/12] efi: Replace strstarts() by str_has_prefix().

2020-12-05 Thread Ard Biesheuvel
On Fri, 4 Dec 2020 at 19:02, James Bottomley wrote: > > On Fri, 2020-12-04 at 18:07 +0100, Ard Biesheuvel wrote: > > On Fri, 4 Dec 2020 at 18:06, > > wrote: > > > From: Francis Laniel > > > > > > The two functions indicates if a string begins with a given prefix. > > > The only difference is tha

[PATCH] Revert "mei: virtio: virtualization frontend driver"

2020-12-05 Thread Michael S. Tsirkin
This reverts commit d162219c655c8cf8003128a13840d6c1e183fb80. The device uses a VIRTIO device ID out of a not-for-production range. Releasing Linux using an ID out of this range will make it conflict with development setups. An official request to reserve an ID for an MEI device is yet to be submit

Re: [char-misc-next 13/13] mei: virtio: virtualization frontend driver

2020-12-05 Thread Michael S. Tsirkin
On Thu, Dec 03, 2020 at 11:01:45PM +0100, Greg Kroah-Hartman wrote: > On Thu, Dec 03, 2020 at 04:51:10PM -0500, Michael S. Tsirkin wrote: > > On Wed, Nov 25, 2020 at 09:18:04PM +, Winkler, Tomas wrote: > > > > > > > > On Tue, Aug 18, 2020 at 02:51:47PM +0300, Tomas Winkler wrote: > > > > > +#i

arch/powerpc/crypto/sha256-spe-glue.c:186:16: sparse: sparse: incorrect type in assignment (different base types)

2020-12-05 Thread kernel test robot
Hi Michael, First bad commit (maybe != root cause): tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: b3298500b23f0b53a8d81e0d5ad98a29db71f4f0 commit: e02152ba2810f7c88cb54e71cda096268dfa9241 powerpc: Drop -me200 addition to build flags date: 3 weeks ag

Re: [PATCH net-next] bonding: correct rr balancing during link failure

2020-12-05 Thread Jakub Kicinski
On Wed, 02 Dec 2020 20:55:57 + Lars Everbrand wrote: > This patch updates the sending algorithm for roundrobin to avoid > over-subscribing interface(s) when one or more interfaces in the bond is > not able to send packets. This happened when order was not random and > more than 2 interfaces wer

Re: crypto: sun4i-ss: error with kmap

2020-12-05 Thread Thomas Gleixner
Corentin, On Sat, Dec 05 2020 at 19:43, Corentin Labbe wrote: > On Fri, Dec 04, 2020 at 09:58:21PM +0100, Thomas Gleixner wrote: >> Can you please replace the debug patch with the one below and try again? >> That stops the trace right on the condition. > > Hello, the result could be found at http:

Re: [PATCH v10 3/8] powerpc: Move ima buffer functions to drivers/of/kexec.c

2020-12-05 Thread Thiago Jung Bauermann
Lakshmi Ramasubramanian writes: > The functions do_get_kexec_buffer() and get_addr_size_cells(), > defined in arch/powerpc/kexec/ima.c, retrieve the address and size > of the given property from the device tree blob. These functions do > not have architecture specific code, but are currently li

Re: [PATCH v10 4/8] powerpc: Use ima kexec node functions

2020-12-05 Thread Thiago Jung Bauermann
Lakshmi Ramasubramanian writes: > do_get_kexec_buffer() and get_addr_size_cells() are called by > ima_get_kexec_buffer(), ima_free_kexec_buffer, and remove_ima_buffer() > to retrieve the address and size of the buffer used for carrying > forward the IMA measurement log across kexec system call.

Re: [PATCH v11 04/10] memory: tegra124-emc: Make driver modular

2020-12-05 Thread Dmitry Osipenko
04.12.2020 19:41, Thierry Reding пишет: ... >> +bool tegra124_clk_emc_driver_available(struct clk_hw *hw) >> +{ >> +struct tegra_clk_emc *tegra = container_of(hw, struct tegra_clk_emc, >> hw); >> + >> +return tegra->prepare_timing_change && tegra->complete_timing_change; >> +} > > This lo

Re: [PATCH 2/4] dt-bindings: display: Add ABT Y030XX067A panel bindings

2020-12-05 Thread Sam Ravnborg
Hi Paul > > > >> >> + > > > >> >> +maintainers: > > > >> >> + - Paul Cercueil > > > >> >> + > > > >> >> +allOf: > > > >> >> + - $ref: panel-common.yaml# > > > >> >> + > > > >> >> +properties: > > > >> >> + compatible: > > > >> >> +const: abt,y030xx067a > > >

Re: [PATCH] drm/kmb: fix array bounds warning

2020-12-05 Thread Sam Ravnborg
Hi Anitha, On Mon, Nov 30, 2020 at 10:19:13PM +, Chrisanthus, Anitha wrote: > Hi Arnd, > Thanks for your patch. > > > -Original Message- > > From: Arnd Bergmann > > Sent: Sunday, November 29, 2020 12:09 PM > > To: Chrisanthus, Anitha ; Dea, Edmund J > > ; David Airlie ; Daniel Vetter

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

2020-12-05 Thread Enrico Weigelt, metux IT consult
On 05.12.20 20:32, Michael S. Tsirkin wrote: Hi, > It seems a bit of a mess, at this point I'm not entirely sure when > should drivers select VIRTIO and when depend on it. if VIRTIO just enables something that could be seen as library functions, then select should be right, IMHO. > The text nea

[PATCH] habanalabs: Revise comment to align with mirror list name

2020-12-05 Thread Oded Gabbay
From: Tomer Tayar hw_queues_mirror was renamed to cs_mirror, so revise accordingly a comment that refers to this list. Signed-off-by: Tomer Tayar Reviewed-by: Oded Gabbay Signed-off-by: Oded Gabbay --- drivers/misc/habanalabs/common/command_submission.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH] habanalabs/gaudi: do not set EB in collective slave queues

2020-12-05 Thread Oded Gabbay
From: Alon Mizrahi We don't need to set EB on signal packets from collective slave queues as it degrades performance. Because the slaves are the network queues, the engine barrier doesn't actually guarantee that the packet has been sent. Signed-off-by: Alon Mizrahi Reviewed-by: Oded Gabbay Sig

Re: [PATCH v10 5/8] powerpc: Move remove_ima_buffer() to drivers/of/kexec.c

2020-12-05 Thread Thiago Jung Bauermann
Lakshmi Ramasubramanian writes: > remove_ima_buffer() removes the chosen node "linux,ima-kexec-buffer" > from the device tree and frees the memory reserved for carrying forward > the IMA measurement logs on kexec. This function does not have > architecture specific code, but is currently limite

Howto listen to/handle gpio state changes ? Re: [PATCH v2 2/2] drivers: gpio: add virtio-gpio guest driver

2020-12-05 Thread Enrico Weigelt, metux IT consult
On 03.12.20 20:11, Enrico Weigelt, metux IT consult wrote: Friends, I've still got a problem w/ signal/irq handling: The virtio-gpio device/host can raise a signal on line state change. Kinda IRQ, but not actually running through real IRQs, instead by a message running though queue. (hmm, kida M

Re: crypto: sun4i-ss: error with kmap

2020-12-05 Thread Julia Lawall
On Sat, 5 Dec 2020, Thomas Gleixner wrote: > Corentin, > > On Sat, Dec 05 2020 at 19:43, Corentin Labbe wrote: > > On Fri, Dec 04, 2020 at 09:58:21PM +0100, Thomas Gleixner wrote: > >> Can you please replace the debug patch with the one below and try again? > >> That stops the trace right on th

Re: genirq, i2c: Provide and use generic_dispatch_irq()

2020-12-05 Thread kernel test robot
Hi Thomas, I love your patch! Perhaps something to improve: [auto build test WARNING on wsa/i2c/for-next] [also build test WARNING on tip/irq/core linux/master linus/master v5.10-rc6 next-20201204] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch,

Re: genirq, i2c: Provide and use generic_dispatch_irq()

2020-12-05 Thread kernel test robot
Hi Thomas, I love your patch! Perhaps something to improve: [auto build test WARNING on wsa/i2c/for-next] [also build test WARNING on tip/irq/core linux/master linus/master v5.10-rc6 next-20201204] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch,

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

2020-12-05 Thread Christophe Leroy
Le 05/12/2020 à 10:56, Christophe Leroy a écrit : 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

Re: [RFC PATCH v1 07/12] efi: Replace strstarts() by str_has_prefix().

2020-12-05 Thread James Bottomley
On Sat, 2020-12-05 at 20:36 +0100, Ard Biesheuvel wrote: > On Fri, 4 Dec 2020 at 19:02, James Bottomley > wrote: > > On Fri, 2020-12-04 at 18:07 +0100, Ard Biesheuvel wrote: > > > On Fri, 4 Dec 2020 at 18:06, > > > wrote: > > > > From: Francis Laniel > > > > > > > > The two functions indicates

Re: [RFC PATCH v1 07/12] efi: Replace strstarts() by str_has_prefix().

2020-12-05 Thread Rasmus Villemoes
On 05/12/2020 20.36, Ard Biesheuvel wrote: > On Fri, 4 Dec 2020 at 19:02, James Bottomley > wrote: >> >> On Fri, 2020-12-04 at 18:07 +0100, Ard Biesheuvel wrote: >>> On Fri, 4 Dec 2020 at 18:06, >>> wrote: From: Francis Laniel The two functions indicates if a string begins with a

Re: [PATCH v2 bpf-next 0/3] bpf: support module BTF in BTF display helpers

2020-12-05 Thread Yonghong Song
On 12/4/20 10:48 AM, Alan Maguire wrote: This series aims to add support to bpf_snprintf_btf() and bpf_seq_printf_btf() allowing them to store string representations of module-specific types, as well as the kernel-specific ones they currently support. Patch 1 removes the btf_module_mutex, as

Re: [PATCH 4.4 17/70] crypto: arm64/sha - avoid non-standard inline asm tricks

2020-12-05 Thread dann frazier
On Mon, Nov 23, 2020 at 01:49:07PM -0700, dann frazier wrote: > On Mon, Nov 26, 2018 at 11:50:32AM +0100, Greg Kroah-Hartman wrote: > > 4.4-stable review patch. If anyone has any objections, please let me know. > > fyi, I bisected a regression down to this commit. This apparently > causes an ADR_

<    1   2   3   4   5   >