[PATCH net-next v3 01/10] net: freescale: ucc_geth: Drop support for the "interface" DT property

2024-12-03 Thread Maxime Chevallier
In april 2007, ucc_geth was converted to phylib with : commit 728de4c927a3 ("ucc_geth: migrate ucc_geth to phylib"). In that commit, the device-tree property "interface", that could be used to retrieve the PHY interface mode was deprecated. DTS files that still used that property were converted

[PATCH net-next v3 07/10] net: freescale: ucc_geth: Hardcode the preamble length to 7 bytes

2024-12-03 Thread Maxime Chevallier
The preamble length can be configured in ucc_geth, however it just ends-up always being configured to 7 bytes, as nothing ever changes the default value of 7. Make that value the default value when the MACCFG2 register gets initialized, and remove the code to configure that value altogether. Revi

[PATCH net-next v3 00/10] net: freescale: ucc_geth: Phylink conversion

2024-12-03 Thread Maxime Chevallier
Hello everyone, This is V3 of the phylink conversion for ucc_geth. The main changes in this V3 are related to error handling in the patches 1 and 10 to report an error when the deprecated "interface" property is found in DT. Doing so, I found and addressed some issues with the jump labels in the

[PATCH net-next v3 06/10] net: freescale: ucc_geth: Simplify frame length check

2024-12-03 Thread Maxime Chevallier
The frame length check is configured when the phy interface is setup. However, it's configured according to an internal flag that is always false. So, just make so that we disable the relevant bit in the MACCFG2 register upon accessing it for other MAC configuration operations. Reviewed-by: Andrew

[PATCH net-next v3 10/10] net: freescale: ucc_geth: phylink conversion

2024-12-03 Thread Maxime Chevallier
ucc_geth is quite capable in terms of supported interfaces, and even includes an externally controlled PCS (well, TBI). Port that driver to phylink. Signed-off-by: Maxime Chevallier --- V3: After adjusting patch 1, I also had to adjust the jump label in this patch for the probe error handling.

[PATCH net-next v3 03/10] net: freescale: ucc_geth: Use netdev->phydev to access the PHY

2024-12-03 Thread Maxime Chevallier
As this driver pre-dates phylib, it uses a private pointer to get a reference to the attached phy_device. Drop that pointer and use the netdev's pointer instead. Reviewed-by: Andrew Lunn Signed-off-by: Maxime Chevallier --- V3: No changes drivers/net/ethernet/freescale/ucc_geth.c | 27

[PATCH net-next v3 04/10] net: freescale: ucc_geth: Fix WOL configuration

2024-12-03 Thread Maxime Chevallier
The get/set_wol ethtool ops rely on querying the PHY for its WoL capabilities, checking for the presence of a PHY and a PHY interrupts isn't enough. Address that by cleaning up the WoL configuration sequence. Signed-off-by: Maxime Chevallier --- V3: No changes drivers/net/ethernet/freescale/ucc

[PATCH net-next v3 08/10] net: freescale: ucc_geth: Move the serdes configuration around

2024-12-03 Thread Maxime Chevallier
The uec_configure_serdes() function deals with serialized linkmodes settings. It's used during the link bringup sequence. It is planned to be used during the phylink conversion for mac configuration, but it needs to me moved around in the process. To make the phylink port clearer, this commit moves

[PATCH net-next v3 02/10] net: freescale: ucc_geth: split adjust_link for phylink conversion

2024-12-03 Thread Maxime Chevallier
Preparing the phylink conversion, split the adjust_link callbaclk, by clearly separating the mac configuration, link_up and link_down phases. Reviewed-by: Andrew Lunn Signed-off-by: Maxime Chevallier --- V3: No changes drivers/net/ethernet/freescale/ucc_geth.c | 180 +++--- 1 f

[PATCH net-next v3 09/10] net: freescale: ucc_geth: Introduce a helper to check Reduced modes

2024-12-03 Thread Maxime Chevallier
A number of parallel MII interfaces also exist in a "Reduced" mode, usually with higher clock rates and fewer data lines, to ease the hardware design. This is what the 'R' stands for in RGMII, RMII, RTBI, RXAUI, etc. The UCC Geth controller has a special configuration bit that needs to be set when

[PATCH net-next v3 05/10] net: freescale: ucc_geth: Use the correct type to store WoL opts

2024-12-03 Thread Maxime Chevallier
The WoL opts are represented through a bitmask stored in a u32. As this mask is copied as-is in the driver, make sure we use the exact same type to store them internally. Signed-off-by: Maxime Chevallier --- V3: No changes drivers/net/ethernet/freescale/ucc_geth.h | 2 +- 1 file changed, 1 inse

Re: [PATCH RESEND v2 2/6] mm/page_isolation: don't pass gfp flags to start_isolate_page_range()

2024-12-03 Thread Vlastimil Babka
On 12/3/24 10:47, David Hildenbrand wrote: > The parameter is unused, so let's stop passing it. > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand Reviewed-by: Vlastimil Babka

Re: [PATCH RESEND v2 3/6] mm/page_alloc: make __alloc_contig_migrate_range() static

2024-12-03 Thread Vlastimil Babka
On 12/3/24 10:47, David Hildenbrand wrote: > The single user is in page_alloc.c. > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand Reviewed-by: Vlastimil Babka > --- > mm/internal.h | 4 > mm/page_alloc.c | 5 ++--- > 2 files changed, 2 insertions(+), 7 deletions(-) > > diff

RE: [PATCH v5 1/2] kexec: Consolidate machine_kexec_mask_interrupts() implementation

2024-12-03 Thread Farber, Eliav
On 12/3/2024 1:04 PM, Thomas Gleixner wrote: >> + >> +config GENERIC_IRQ_KEXEC_CLEAR_VM_FORWARD >> + bool "Clear forwarded VM interrupts during kexec" > > This should not be user selectable. Just keep it as: > > config GENERIC_IRQ_KEXEC_CLEAR_VM_FORWARD > bool > > which defaults to 'n'.

Re: [PATCH RESEND v2 1/6] mm/page_isolation: don't pass gfp flags to isolate_single_pageblock()

2024-12-03 Thread Vlastimil Babka
On 12/3/24 10:47, David Hildenbrand wrote: > The flags are no longer used, we can stop passing them to > isolate_single_pageblock(). > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand Reviewed-by: Vlastimil Babka > --- > mm/page_isolation.c | 8 +++- > 1 file changed, 3 insertion

[PATCH] genirq: Remove leading space from .irq_print_chip() callbacks

2024-12-03 Thread Geert Uytterhoeven
The space separator was factored out from the multiple chip name prints, but several irq_chip.irq_print_chip() callbacks still print a leading space. Remove the superfluous double spaces. Fixes: 9d9f204bdf7243bf ("genirq/proc: Add missing space separator back") Signed-off-by: Geert Uytterhoeven

Re: [PATCH RESEND v2 2/6] mm/page_isolation: don't pass gfp flags to start_isolate_page_range()

2024-12-03 Thread Oscar Salvador
On Tue, Dec 03, 2024 at 10:47:28AM +0100, David Hildenbrand wrote: > The parameter is unused, so let's stop passing it. > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand Reviewed-by: Oscar Salvador -- Oscar Salvador SUSE Labs

Re: [PATCH RESEND v2 3/6] mm/page_alloc: make __alloc_contig_migrate_range() static

2024-12-03 Thread Oscar Salvador
On Tue, Dec 03, 2024 at 10:47:29AM +0100, David Hildenbrand wrote: > The single user is in page_alloc.c. > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand Reviewed-by: Oscar Salvador -- Oscar Salvador SUSE Labs

Re: [PATCH RESEND v2 1/6] mm/page_isolation: don't pass gfp flags to isolate_single_pageblock()

2024-12-03 Thread Oscar Salvador
On Tue, Dec 03, 2024 at 10:47:27AM +0100, David Hildenbrand wrote: > The flags are no longer used, we can stop passing them to > isolate_single_pageblock(). > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand Reviewed-by: Oscar Salvador -- Oscar Salvador SUSE Labs

Re:[PATCH] genirq: Remove leading space from .irq_print_chip() callbacks

2024-12-03 Thread David Wang
HI, At 2024-12-03 23:02:31, "Geert Uytterhoeven" wrote: >The space separator was factored out from the multiple chip name prints, >but several irq_chip.irq_print_chip() callbacks still print a leading >space. Remove the superfluous double spaces. > >Fixes: 9d9f204bdf7243bf ("genirq/proc: Add mis

Re: [PATCH RESEND v2 4/6] mm/page_alloc: sort out the alloc_contig_range() gfp flags mess

2024-12-03 Thread Zi Yan
On 3 Dec 2024, at 9:24, Vlastimil Babka wrote: > On 12/3/24 15:12, David Hildenbrand wrote: >> On 03.12.24 14:55, Vlastimil Babka wrote: >>> On 12/3/24 10:47, David Hildenbrand wrote: It's all a bit complicated for alloc_contig_range(). For example, we don't support many flags, so let's

Re: [PATCH v1 0/6] mm/page_alloc: gfp flags cleanups for alloc_contig_*()

2024-12-03 Thread David Hildenbrand
On 02.12.24 23:07, Zi Yan wrote: On 2 Dec 2024, at 7:58, David Hildenbrand wrote: Let's clean up the gfp flags handling, and support __GFP_ZERO, such that we can finally remove the TODO in memtrace code. I did some alloc_contig_*() testing with virtio-mem and hugetlb; I did not test powernv/me

[PATCH RESEND v2 4/6] mm/page_alloc: sort out the alloc_contig_range() gfp flags mess

2024-12-03 Thread David Hildenbrand
It's all a bit complicated for alloc_contig_range(). For example, we don't support many flags, so let's start bailing out on unsupported ones -- ignoring the placement hints, as we are already given the range to allocate. While we currently set cc.gfp_mask, in __alloc_contig_migrate_range() we sim

[PATCH RESEND v2 3/6] mm/page_alloc: make __alloc_contig_migrate_range() static

2024-12-03 Thread David Hildenbrand
The single user is in page_alloc.c. Reviewed-by: Zi Yan Signed-off-by: David Hildenbrand --- mm/internal.h | 4 mm/page_alloc.c | 5 ++--- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/mm/internal.h b/mm/internal.h index 74713b44bedb..4bd3685c33ef 100644 --- a/mm/interna

[PATCH RESEND v2 2/6] mm/page_isolation: don't pass gfp flags to start_isolate_page_range()

2024-12-03 Thread David Hildenbrand
The parameter is unused, so let's stop passing it. Reviewed-by: Zi Yan Signed-off-by: David Hildenbrand --- include/linux/page-isolation.h | 2 +- mm/memory_hotplug.c| 3 +-- mm/page_alloc.c| 2 +- mm/page_isolation.c| 4 +--- 4 files changed, 4 insertion

[PATCH RESEND v2 0/6] mm/page_alloc: gfp flags cleanups for alloc_contig_*()

2024-12-03 Thread David Hildenbrand
For some reason v1 didn't make it to linux-mm; same with v2. Let's try a different SMTP server (gmail), maybe there are some issues with the RH one ... Let's clean up the gfp flags handling, and support __GFP_ZERO, such that we can finally remove the TODO in memtrace code. I did some alloc_contig

[PATCH RESEND v2 6/6] powernv/memtrace: use __GFP_ZERO with alloc_contig_pages()

2024-12-03 Thread David Hildenbrand
alloc_contig_pages()->alloc_contig_range() now supports __GFP_ZERO, so let's use that instead to resolve our TODO. Signed-off-by: David Hildenbrand --- arch/powerpc/platforms/powernv/memtrace.c | 31 +-- 1 file changed, 6 insertions(+), 25 deletions(-) diff --git a/arch/powe

[PATCH RESEND v2 1/6] mm/page_isolation: don't pass gfp flags to isolate_single_pageblock()

2024-12-03 Thread David Hildenbrand
The flags are no longer used, we can stop passing them to isolate_single_pageblock(). Reviewed-by: Zi Yan Signed-off-by: David Hildenbrand --- mm/page_isolation.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/mm/page_isolation.c b/mm/page_isolation.c index 7e040479

[PATCH RESEND v2 5/6] mm/page_alloc: forward the gfp flags from alloc_contig_range() to post_alloc_hook()

2024-12-03 Thread David Hildenbrand
In the __GFP_COMP case, we already pass the gfp_flags to prep_new_page()->post_alloc_hook(). However, in the !__GFP_COMP case, we essentially pass only hardcoded __GFP_MOVABLE to post_alloc_hook(), preventing some action modifiers from being effective.. Let's pass our now properly adjusted gfp fla

Re: [RESEND] macintosh: Fix spelling mistake "toogle" to "toggle"

2024-12-03 Thread Christophe Leroy
Hi, Le 03/12/2024 à 09:42, Zhu Jun a écrit : There is a spelling mistake, Please fix it. This text doesn't belong to a commit message, the patch cannot be applied with such a message. By the way, can you explain why it is so urgent to fix a minor typo like this on a so old driver that hasn

Re: [PATCH RESEND v2 4/6] mm/page_alloc: sort out the alloc_contig_range() gfp flags mess

2024-12-03 Thread Vlastimil Babka
On 12/3/24 10:47, David Hildenbrand wrote: > It's all a bit complicated for alloc_contig_range(). For example, we don't > support many flags, so let's start bailing out on unsupported > ones -- ignoring the placement hints, as we are already given the range > to allocate. > > While we currently se

Re: [PATCH 1/3] powerpc: copy preempt.h into arch/include/asm

2024-12-03 Thread Shrikanth Hegde
On 12/2/24 23:47, Christophe Leroy wrote: Le 02/12/2024 à 15:05, Shrikanth Hegde a écrit : On 11/27/24 12:07, Christophe Leroy wrote: Le 25/11/2024 à 05:22, Shrikanth Hegde a écrit : PowerPC uses asm-generic preempt definitions as of now. Copy that into arch/asm so that arch specific

Re: [PATCH RESEND v2 4/6] mm/page_alloc: sort out the alloc_contig_range() gfp flags mess

2024-12-03 Thread David Hildenbrand
On 03.12.24 14:55, Vlastimil Babka wrote: On 12/3/24 10:47, David Hildenbrand wrote: It's all a bit complicated for alloc_contig_range(). For example, we don't support many flags, so let's start bailing out on unsupported ones -- ignoring the placement hints, as we are already given the range to

Re: [PATCH RESEND v2 4/6] mm/page_alloc: sort out the alloc_contig_range() gfp flags mess

2024-12-03 Thread Vlastimil Babka
On 12/3/24 15:12, David Hildenbrand wrote: > On 03.12.24 14:55, Vlastimil Babka wrote: >> On 12/3/24 10:47, David Hildenbrand wrote: >>> It's all a bit complicated for alloc_contig_range(). For example, we don't >>> support many flags, so let's start bailing out on unsupported >>> ones -- ignoring

Re: [PATCH RESEND v2 5/6] mm/page_alloc: forward the gfp flags from alloc_contig_range() to post_alloc_hook()

2024-12-03 Thread Vlastimil Babka
On 12/3/24 10:47, David Hildenbrand wrote: > In the __GFP_COMP case, we already pass the gfp_flags to > prep_new_page()->post_alloc_hook(). However, in the !__GFP_COMP case, we > essentially pass only hardcoded __GFP_MOVABLE to post_alloc_hook(), > preventing some action modifiers from being effect

[PATCH v2 5/6] mm/page_alloc: forward the gfp flags from alloc_contig_range() to post_alloc_hook()

2024-12-03 Thread David Hildenbrand
In the __GFP_COMP case, we already pass the gfp_flags to prep_new_page()->post_alloc_hook(). However, in the !__GFP_COMP case, we essentially pass only hardcoded __GFP_MOVABLE to post_alloc_hook(), preventing some action modifiers from being effective.. Let's pass our now properly adjusted gfp fla

[PATCH v2 6/6] powernv/memtrace: use __GFP_ZERO with alloc_contig_pages()

2024-12-03 Thread David Hildenbrand
alloc_contig_pages()->alloc_contig_range() now supports __GFP_ZERO, so let's use that instead to resolve our TODO. Signed-off-by: David Hildenbrand --- arch/powerpc/platforms/powernv/memtrace.c | 31 +-- 1 file changed, 6 insertions(+), 25 deletions(-) diff --git a/arch/powe

[PATCH v2 4/6] mm/page_alloc: sort out the alloc_contig_range() gfp flags mess

2024-12-03 Thread David Hildenbrand
It's all a bit complicated for alloc_contig_range(). For example, we don't support many flags, so let's start bailing out on unsupported ones -- ignoring the placement hints, as we are already given the range to allocate. While we currently set cc.gfp_mask, in __alloc_contig_migrate_range() we sim

[PATCH v2 3/6] mm/page_alloc: make __alloc_contig_migrate_range() static

2024-12-03 Thread David Hildenbrand
The single user is in page_alloc.c. Reviewed-by: Zi Yan Signed-off-by: David Hildenbrand --- mm/internal.h | 4 mm/page_alloc.c | 5 ++--- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/mm/internal.h b/mm/internal.h index 74713b44bedb..4bd3685c33ef 100644 --- a/mm/interna

[PATCH v2 2/6] mm/page_isolation: don't pass gfp flags to start_isolate_page_range()

2024-12-03 Thread David Hildenbrand
The parameter is unused, so let's stop passing it. Reviewed-by: Zi Yan Signed-off-by: David Hildenbrand --- include/linux/page-isolation.h | 2 +- mm/memory_hotplug.c| 3 +-- mm/page_alloc.c| 2 +- mm/page_isolation.c| 4 +--- 4 files changed, 4 insertion

[PATCH v2 1/6] mm/page_isolation: don't pass gfp flags to isolate_single_pageblock()

2024-12-03 Thread David Hildenbrand
The flags are no longer used, we can stop passing them to isolate_single_pageblock(). Reviewed-by: Zi Yan Signed-off-by: David Hildenbrand --- mm/page_isolation.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/mm/page_isolation.c b/mm/page_isolation.c index 7e040479

[PATCH v2 0/6] mm/page_alloc: gfp flags cleanups for alloc_contig_*()

2024-12-03 Thread David Hildenbrand
For some reason v1 didn't make it to linux-mm; let's see if the problem persists. Let's clean up the gfp flags handling, and support __GFP_ZERO, such that we can finally remove the TODO in memtrace code. I did some alloc_contig_*() testing with virtio-mem and hugetlb; I did not test powernv/memtr

[RESEND] macintosh: Fix spelling mistake "toogle" to "toggle"

2024-12-03 Thread Zhu Jun
There is a spelling mistake, Please fix it. Signed-off-by: Zhu Jun --- drivers/macintosh/adbhid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/macintosh/adbhid.c b/drivers/macintosh/adbhid.c index b2fe7a3dc471..943a7123786a 100644 --- a/drivers/macintosh/adbhid.c +

Re: [PATCH v5 1/2] kexec: Consolidate machine_kexec_mask_interrupts() implementation

2024-12-03 Thread Thomas Gleixner
On Sat, Nov 30 2024 at 20:11, Eliav Farber wrote: > + > +config GENERIC_IRQ_KEXEC_CLEAR_VM_FORWARD > + bool "Clear forwarded VM interrupts during kexec" This should not be user selectable. Just keep it as: config GENERIC_IRQ_KEXEC_CLEAR_VM_FORWARD bool which defaults to 'n'. Just add

Re: [PATCH RESEND v2 6/6] powernv/memtrace: use __GFP_ZERO with alloc_contig_pages()

2024-12-03 Thread Vlastimil Babka
On 12/3/24 10:47, David Hildenbrand wrote: > alloc_contig_pages()->alloc_contig_range() now supports __GFP_ZERO, > so let's use that instead to resolve our TODO. > > Signed-off-by: David Hildenbrand Reviewed-by: Vlastimil Babka

Re: [PATCH v2 09/10] sysfs: bin_attribute: add const read/write callback variants

2024-12-03 Thread James Bottomley
> diff --git a/include/linux/sysfs.h b/include/linux/sysfs.h > index > d17c473c1ef292875475bf3bdf62d07241c13882..d713a6445a6267145a7014f308d > f3bb25b8c3287 100644 > --- a/include/linux/sysfs.h > +++ b/include/linux/sysfs.h > @@ -305,8 +305,12 @@ struct bin_attribute { > struct address_space

RE: [PATCH v5 1/2] kexec: Consolidate machine_kexec_mask_interrupts() implementation

2024-12-03 Thread Thomas Gleixner
On Tue, Dec 03 2024 at 12:55, Eliav Farber wrote: > On 12/3/2024 1:04 PM, Thomas Gleixner wrote: >>> + >>> +config GENERIC_IRQ_KEXEC_CLEAR_VM_FORWARD >>> + bool "Clear forwarded VM interrupts during kexec" >> >> This should not be user selectable. Just keep it as: >> >> config GENERIC_IRQ_KEXEC

Re: [PATCH v2 09/10] sysfs: bin_attribute: add const read/write callback variants

2024-12-03 Thread Thomas Weißschuh
On 2024-12-03 11:06:16-0500, James Bottomley wrote: > > diff --git a/include/linux/sysfs.h b/include/linux/sysfs.h > > index > > d17c473c1ef292875475bf3bdf62d07241c13882..d713a6445a6267145a7014f308d > > f3bb25b8c3287 100644 > > --- a/include/linux/sysfs.h > > +++ b/include/linux/sysfs.h > > @@ -305

Re: [PATCH] dt-bindings: soc: fsl: cpm_qe: Limit matching to nodes with "fsl,qe"

2024-12-03 Thread Conor Dooley
On Mon, Dec 02, 2024 at 12:57:55PM +0800, Chen-Yu Tsai wrote: > Otherwise the binding matches against random nodes with "simple-bus" > giving out all kinds of invalid warnings: > > $ make CHECK_DTBS=y mediatek/mt8188-evb.dtb > SYNCinclude/config/auto.conf.cmd > UPD include/

Re: [PATCH] dt-bindings: soc: fsl: cpm_qe: Limit matching to nodes with "fsl,qe"

2024-12-03 Thread Rob Herring
On Mon, Dec 02, 2024 at 12:57:55PM +0800, Chen-Yu Tsai wrote: > Otherwise the binding matches against random nodes with "simple-bus" > giving out all kinds of invalid warnings: > > $ make CHECK_DTBS=y mediatek/mt8188-evb.dtb > SYNCinclude/config/auto.conf.cmd > UPD include/

Re: [PATCH RESEND v2 1/6] mm/page_isolation: don't pass gfp flags to isolate_single_pageblock()

2024-12-03 Thread Vishal Moola
On Tue, Dec 03, 2024 at 10:47:27AM +0100, David Hildenbrand wrote: > The flags are no longer used, we can stop passing them to > isolate_single_pageblock(). > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand > --- Reviewed-by: Vishal Moola (Oracle)

Re: [PATCH 2/3] powerpc: support dynamic preemption

2024-12-03 Thread Christophe Leroy
Le 01/12/2024 à 20:45, Shrikanth Hegde a écrit : On 11/27/24 12:14, Christophe Leroy wrote: Le 25/11/2024 à 05:22, Shrikanth Hegde a écrit : Once the lazy preemption is supported, it would be desirable to change the preemption models at runtime. So this change adds support for dynamic

Re: [PATCH RESEND v2 3/6] mm/page_alloc: make __alloc_contig_migrate_range() static

2024-12-03 Thread Vishal Moola
On Tue, Dec 03, 2024 at 10:47:29AM +0100, David Hildenbrand wrote: > The single user is in page_alloc.c. > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand Reviewed-by: Vishal Moola (Oracle)

Re: [PATCH v2] selftests/ftrace: adjust offset for kprobe syntax error test

2024-12-03 Thread Steven Rostedt
On Tue, 3 Dec 2024 18:01:06 -0700 Shuah Khan wrote: > On 12/2/24 12:41, Steven Rostedt wrote: > > On Sat, 30 Nov 2024 01:56:21 +0530 > > Hari Bathini wrote: > > > >> In 'NOFENTRY_ARGS' test case for syntax check, any offset X of > >> `vfs_read+X` except function entry offset (0) fits the crit

Re: [PATCH net-next v3 08/10] net: freescale: ucc_geth: Move the serdes configuration around

2024-12-03 Thread Andrew Lunn
On Tue, Dec 03, 2024 at 01:43:19PM +0100, Maxime Chevallier wrote: > The uec_configure_serdes() function deals with serialized linkmodes > settings. It's used during the link bringup sequence. It is planned to > be used during the phylink conversion for mac configuration, but it > needs to me moved

Re: [PATCH RESEND v2 4/6] mm/page_alloc: sort out the alloc_contig_range() gfp flags mess

2024-12-03 Thread David Hildenbrand
On 03.12.24 16:49, Zi Yan wrote: On 3 Dec 2024, at 9:24, Vlastimil Babka wrote: On 12/3/24 15:12, David Hildenbrand wrote: On 03.12.24 14:55, Vlastimil Babka wrote: On 12/3/24 10:47, David Hildenbrand wrote: It's all a bit complicated for alloc_contig_range(). For example, we don't support m

[PATCH v4 2/4] objtool/powerpc: Add support for decoding all types of uncond branches

2024-12-03 Thread Christophe Leroy
Add support for 'bla' instruction. This is done by 'flagging' the address as an absolute address so that arch_jump_destination() can calculate it as expected. Because code is _always_ 4 bytes aligned, use bit 30 as flag. Also add support for 'b' and 'ba' instructions. Objtool call them jumps. Si

Re: [PATCH v2] selftests/ftrace: adjust offset for kprobe syntax error test

2024-12-03 Thread Shuah Khan
On 12/2/24 12:41, Steven Rostedt wrote: On Sat, 30 Nov 2024 01:56:21 +0530 Hari Bathini wrote: In 'NOFENTRY_ARGS' test case for syntax check, any offset X of `vfs_read+X` except function entry offset (0) fits the criterion, even if that offset is not at instruction boundary, as the parser come

Re: [PATCH 3/3] powerpc: print right preemption model in die

2024-12-03 Thread Christophe Leroy
Le 25/11/2024 à 05:22, Shrikanth Hegde a écrit : Preemption models can change at runtime with dynamic preemption in place. So need to use the right methods instead of relying on CONFIG_PREEMPT to decide whether its full preemption or not. Then this patch should go _before_ activating dynamic

[PATCH v4 4/4] powerpc/static_call: Implement inline static calls

2024-12-03 Thread Christophe Leroy
Implement inline static calls: - Put a 'bl' to the destination function ('b' if tail call) - Put a 'nop' when the destination function is NULL ('blr' if tail call) - Put a 'li r3,0' when the destination is the RET0 function and not a tail call. If the destination is too far (over the 32Mb limit),

[PATCH v4 1/4] static_call_inline: Provide trampoline address when updating sites

2024-12-03 Thread Christophe Leroy
In preparation of support of inline static calls on powerpc, provide trampoline address when updating sites, so that when the destination function is too far for a direct function call, the call site is patched with a call to the trampoline. Signed-off-by: Christophe Leroy --- arch/x86/kernel/st

[PATCH v4 0/4] Implement inline static calls on PPC32 - v4

2024-12-03 Thread Christophe Leroy
This series implements inline static calls on PPC32. First patch adds to static_call core the ability to pass the trampoline address at the same time as the site address to arch_static_call_transform() so that it can fallback on branching to the trampoline when the site is too far (Max distance fo

[PATCH v4 3/4] powerpc: Prepare arch_static_call_transform() for supporting inline static calls

2024-12-03 Thread Christophe Leroy
Reorganise arch_static_call_transform() in order to ease the support of inline static calls in following patch: - remove 'target' to nhide whether it is a 'return 0' or not. - Don't bail out if 'tramp' is NULL, just do nothing until next patch. Note that 'target' was 'tramp + PPC_SCT_RET0', is_sho

Re: [PATCH RESEND v2 4/6] mm/page_alloc: sort out the alloc_contig_range() gfp flags mess

2024-12-03 Thread David Hildenbrand
On 03.12.24 15:24, Vlastimil Babka wrote: On 12/3/24 15:12, David Hildenbrand wrote: On 03.12.24 14:55, Vlastimil Babka wrote: On 12/3/24 10:47, David Hildenbrand wrote: It's all a bit complicated for alloc_contig_range(). For example, we don't support many flags, so let's start bailing out on

Re: [PATCH 1/3] powerpc: copy preempt.h into arch/include/asm

2024-12-03 Thread Christophe Leroy
Le 03/12/2024 à 15:00, Shrikanth Hegde a écrit : On 12/2/24 23:47, Christophe Leroy wrote: Le 02/12/2024 à 15:05, Shrikanth Hegde a écrit : On 11/27/24 12:07, Christophe Leroy wrote: Le 25/11/2024 à 05:22, Shrikanth Hegde a écrit : PowerPC uses asm-generic preempt definitions as of

Re: [PATCH RESEND v2 2/6] mm/page_isolation: don't pass gfp flags to start_isolate_page_range()

2024-12-03 Thread Vishal Moola
On Tue, Dec 03, 2024 at 10:47:28AM +0100, David Hildenbrand wrote: > The parameter is unused, so let's stop passing it. > > Reviewed-by: Zi Yan > Signed-off-by: David Hildenbrand Reviewed-by: Vishal Moola (Oracle)

[PATCH v5 RESEND 2/6] ASoC: fsl_asrc: define functions for memory to memory usage

2024-12-03 Thread Shengjiu Wang
ASRC can be used on memory to memory case, define several functions for m2m usage. m2m_prepare: prepare for the start step m2m_start: the start step m2m_unprepare: unprepare for stop step, optional m2m_stop: stop step m2m_check_format: check format is supported or not m2m_calc_out_len: calculate o

[PATCH v5 RESEND 4/6] ASoC: fsl_asrc_m2m: Add memory to memory function

2024-12-03 Thread Shengjiu Wang
Implement the ASRC memory to memory function using the compress framework, user can use this function with compress ioctl interface. This feature can be shared by ASRC and EASRC drivers Signed-off-by: Shengjiu Wang Acked-by: Jaroslav Kysela --- sound/soc/fsl/Kconfig | 1 + sound/so

[PATCH v5 RESEND 1/6] ALSA: compress: Add output rate and output format support

2024-12-03 Thread Shengjiu Wang
Add 'pcm_format' for struct snd_codec, add 'pcm_formats' for struct snd_codec_desc, these are used for accelerator usage. Current accelerator example is sample rate converter (SRC). Define struct snd_codec_desc_src for descript minmum and maxmum sample rates. And add 'src_d' in union snd_codec_opt

[PATCH v5 RESEND 5/6] ASoC: fsl_asrc: register m2m platform device

2024-12-03 Thread Shengjiu Wang
Register m2m platform device, that user can use M2M feature. Signed-off-by: Shengjiu Wang Acked-by: Jaroslav Kysela --- sound/soc/fsl/fsl_asrc.c | 37 + 1 file changed, 33 insertions(+), 4 deletions(-) diff --git a/sound/soc/fsl/fsl_asrc.c b/sound/soc/fsl/fs

Re: [PATCH 0/2] ASoC: fsl_xcvr: Add suspend and resume support

2024-12-03 Thread Mark Brown
On Wed, 27 Nov 2024 11:00:33 +0800, Shengjiu Wang wrote: > Define regmap for PHY and PLL registers, the PHY and PLL > registers are accessed by AI interface in controller. > Then driver can use regcache to recover registers > after suspend and resume. > > Shengjiu Wang (2): > ASoC: fsl_xcvr: Us

Re: [PATCH 09/22] drm/etnaviv: Convert timeouts to secs_to_jiffies()

2024-12-03 Thread Lucas Stach
Am Freitag, dem 15.11.2024 um 21:22 + schrieb Easwar Hariharan: > Changes made with the following Coccinelle rules: > > @@ constant C; @@ > > - msecs_to_jiffies(C * 1000) > + secs_to_jiffies(C) > > @@ constant C; @@ > > - msecs_to_jiffies(C * MSEC_PER_SEC) > + secs_to_jiffies(C) > Thanks,

Re: [PATCH net-next v3 09/10] net: freescale: ucc_geth: Introduce a helper to check Reduced modes

2024-12-03 Thread Andrew Lunn
> +static bool phy_interface_mode_is_reduced(phy_interface_t interface) > +{ > + return phy_interface_mode_is_rgmii(interface) || > +interface == PHY_INTERFACE_MODE_RMII || > +interface == PHY_INTERFACE_MODE_RTBI; > +} I wounder if this is useful anywhere else? Did you

Re: [PATCH net-next v3 04/10] net: freescale: ucc_geth: Fix WOL configuration

2024-12-03 Thread Andrew Lunn
On Tue, Dec 03, 2024 at 01:43:15PM +0100, Maxime Chevallier wrote: > The get/set_wol ethtool ops rely on querying the PHY for its WoL > capabilities, checking for the presence of a PHY and a PHY interrupts > isn't enough. Address that by cleaning up the WoL configuration > sequence. > > Signed-off

Re: [PATCH net-next v3 05/10] net: freescale: ucc_geth: Use the correct type to store WoL opts

2024-12-03 Thread Andrew Lunn
On Tue, Dec 03, 2024 at 01:43:16PM +0100, Maxime Chevallier wrote: > The WoL opts are represented through a bitmask stored in a u32. As this > mask is copied as-is in the driver, make sure we use the exact same type > to store them internally. > > Signed-off-by: Maxime Chevallier Reviewed-by: An

Re: [PATCH] dt-bindings: soc: fsl: cpm_qe: Limit matching to nodes with "fsl,qe"

2024-12-03 Thread Chen-Yu Tsai
On Wed, Dec 4, 2024 at 1:20 AM Rob Herring wrote: > > On Mon, Dec 02, 2024 at 12:57:55PM +0800, Chen-Yu Tsai wrote: > > Otherwise the binding matches against random nodes with "simple-bus" > > giving out all kinds of invalid warnings: > > > > $ make CHECK_DTBS=y mediatek/mt8188-evb.dtb > >

Re: [PATCH] tools/perf/tests/expr: Make the system_tsc_freq test only for intel

2024-12-03 Thread Namhyung Kim
On Tue, Dec 03, 2024 at 10:42:45AM -0800, Namhyung Kim wrote: > On Tue, Dec 03, 2024 at 10:16:06AM -0800, Namhyung Kim wrote: > > Hello, > > > > On Fri, Nov 08, 2024 at 10:50:10AM +0530, Athira Rajeev wrote: > > > > > > > > > > On 7 Nov 2024, at 7:26 PM, Leo Yan wrote: > > > > > > > > Hi Athir

Re: [PATCH] tools/perf/tests/expr: Make the system_tsc_freq test only for intel

2024-12-03 Thread Namhyung Kim
Hello, On Fri, Nov 08, 2024 at 10:50:10AM +0530, Athira Rajeev wrote: > > > > On 7 Nov 2024, at 7:26 PM, Leo Yan wrote: > > > > Hi Athira, > > > > On Wed, Nov 06, 2024 at 03:04:57PM +0530, Athira Rajeev wrote: > > > > [...] > > > >>> Hi Athira, > >>> > >>> sorry for the breakage and thank

Re: [PATCH] tools/perf/tests/expr: Make the system_tsc_freq test only for intel

2024-12-03 Thread Namhyung Kim
On Tue, Dec 03, 2024 at 10:16:06AM -0800, Namhyung Kim wrote: > Hello, > > On Fri, Nov 08, 2024 at 10:50:10AM +0530, Athira Rajeev wrote: > > > > > > > On 7 Nov 2024, at 7:26 PM, Leo Yan wrote: > > > > > > Hi Athira, > > > > > > On Wed, Nov 06, 2024 at 03:04:57PM +0530, Athira Rajeev wrote: >

[PATCH v5 RESEND 6/6] ASoC: fsl_easrc: register m2m platform device

2024-12-03 Thread Shengjiu Wang
Register m2m platform device,that user can use M2M feature. Signed-off-by: Shengjiu Wang Acked-by: Jaroslav Kysela --- sound/soc/fsl/fsl_easrc.c | 33 +++-- 1 file changed, 31 insertions(+), 2 deletions(-) diff --git a/sound/soc/fsl/fsl_easrc.c b/sound/soc/fsl/fsl_e

[PATCH v5 RESEND 0/6] ASoC: fsl: add memory to memory function for ASRC

2024-12-03 Thread Shengjiu Wang
This function is base on the accelerator implementation for compress API: https://patchwork.kernel.org/project/alsa-devel/patch/20241002093904.1809799-1-pe...@perex.cz/ Audio signal processing also has the requirement for memory to memory similar as Video. This asrc memory to memory (memory ->asr

[PATCH v5 RESEND 3/6] ASoC: fsl_easrc: define functions for memory to memory usage

2024-12-03 Thread Shengjiu Wang
ASRC can be used on memory to memory case, define several functions for m2m usage and export them as function pointer. Signed-off-by: Shengjiu Wang Acked-by: Jaroslav Kysela --- sound/soc/fsl/fsl_easrc.c | 228 ++ sound/soc/fsl/fsl_easrc.h | 4 + 2 files ch