es on pre-Linux software(U-Boot) to do
> M7 TCM ECC initialization. In future, we could add the support in Linux
> to decouple U-Boot and Linux.
>
> Signed-off-by: Peng Fan
Reviewed-by: Daniel Baluta
On Fri, Jun 6, 2025 at 4:57 AM Peng Fan (OSS) wrote:
>
> From: Peng Fan
>
> i.MX95 has a System Manager(SM) core runs the System Control Management
> Interface(SCMI) firmware. Vendor extenions are implemented in the
> firmware, Logical Machine Management(LMM) protocol to manage
> Logical Machines
> This seems to be off by one. For instance, if the loop reaches the last
> valid type in mod_mem_type, MOD_INIT_RODATA, and successfully allocates
> its memory, the variable t gets set to MOD_INIT_RODATA. Subsequently, if
> an error occurs later in move_module() and control is transferred to
> out
ed-off-by: Petr Pavlu
Reviewed-by: Daniel Gomez
> ---
> kernel/module/main.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/module/main.c b/kernel/module/main.c
> index 322b38c0a782..06511607075c 100644
> --- a/kernel/module/main.c
> +++ b/kernel
On 07/06/2025 18.16, Petr Pavlu wrote:
> The function move_module() uses the variable t to track how many memory
> types it has allocated and consequently how many should be freed if an
> error occurs.
>
> The variable is initially set to 0 and is updated when a call to
> module_memory_alloc() fai
Hi Peng,
Thanks a lot for the patches. Comments inline:
On Wed, Jun 4, 2025 at 5:36 AM Peng Fan (OSS) wrote:
>
> From: Peng Fan
>
> i.MX95 features a Cortex-M33 core, six Cortex-A55 cores, and
> one Cortex-M7 core. The System Control Management Interface(SCMI)
> firmware runs on the M33 core. T
On Wed, Jun 4, 2025 at 5:37 AM Peng Fan (OSS) wrote:
>
> From: Peng Fan
>
> Add imx_rproc_cfg_imx95_m7 and address(TCM and DDR) mapping
> Add i.MX95 of_device_id entry
>
> Signed-off-by: Peng Fan
Reviewed-by: Daniel Baluta
("net: add UAPI to the header guard in various network
headers")
Reported-by: Venkat Rao Bagalkote
Closes:
https://lore.kernel.org/all/c2bc466d-dff2-4d0d-a797-9af7f676c...@linux.ibm.com/
Tested-by: Venkat Rao Bagalkote
Signed-off-by: Saket Kumar Bhaskar
Acked-by: Daniel Borkmann
On Thu, May 1, 2025, at 7:28 AM, Lorenz Bauer wrote:
> I'd like to cut down the memory usage of parsing vmlinux BTF in ebpf-go.
> With some upcoming changes the library is sitting at 5MiB for a parse.
> Most of that memory is simply copying the BTF blob into user space.
> By allowing vmlinux BTF to
Hi Shawn,
Gentle ping.
On Thu, Mar 20, 2025 at 2:08 PM Daniel Baluta wrote:
>
> DSP found in i.MX8MP SOC can be used by multiple frameworks in order to
> enable various applications:
> - rproc/rpmsg framework, used to load for example Zephyr samples
> - Soun
ed) ChromeOS kernel trees, so the
> > compatibility impact should be minimal. I will also try to clean up
> > the leftover bits on the crosvm side just to clear things up.
>
> Thanks for your reply, and thanks for clarifying+cleaning it up.
>
[...]
> >> IIRC, in that com
bpf_check(), as it currently exists, will soon be going away to make way
for loadable BPF verifier support. Fixup selftests so they fentry attach
to a more reliable location.
Signed-off-by: Daniel Xu
---
tools/testing/selftests/bpf/progs/exceptions_assert.c | 2 +-
tools/testing/selftests/bpf
> >
> > Which does not seem to reflect reality ...
Please feel free to disregard these features and reuse their bits and
queue indexes; as far as I know, they are not actually enabled
anywhere currently and the corresponding guest patches were only
applied to some (no-longer-used) Chrome
eset controller API (changes merged ->
https://patchwork.kernel.org/project/linux-arm-kernel/cover/20250311085812.1296243-1-daniel.bal...@nxp.com/)
- drop patches related to DSP run/stall/reset via syscon
- picked up patch related to using run_stall via reset
contr
With imx8mp-evk board we are now configuring 'dsp' node for rproc usage,
so add rproc specific memory regions.
Also, enable dsp node because it is ready to be used.
Reviewed-by: Ahmad Fatoum
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/
11085812.1296243-1-daniel.bal...@nxp.com/)
- drop patches related to DSP run/stall/reset via syscon
- picked up patch related to using run_stall via reset
controller API.
Daniel Baluta (5):
arm64: dts: imx8mp: Use resets property
arm64: dts: imx8mp: Add mu2 root clock
ware) will need to
create a separate dts file (or an overlay).
This change follows the approach taken for other i.MX8 boards
in commit 391a319c81f6d7 ("arm64: dts: imx8-ss-audio: configure dsp node
for rproc usage")
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by:
e patch matches the spec, so from that
perspective, looks fine:
Reviewed-by: Daniel Verkamp
Since the driver code has been using the wrong order since support for
this message was added in 2010, but there is no support for sending
this message in the current qemu device implementation, I wondered
wh
be writable - tainting kernel\n",
+ pr_warn(
+"%s: ro_after_init failed with %d, data might be writable - tainting kernel\n",
mod->name, ret);
add_taint_module(mod, TAINT_BAD_PAGE, LOCKDEP_STILL_OK);
}
Daniel
DSP core needs ocram, core and debug clocks.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
b/arch/arm64/boot/dts
ware) will need to
create a separate dts file (or an overlay).
This change follows the approach taken for other i.MX8 boards
in commit 391a319c81f6d7 ("arm64: dts: imx8-ss-audio: configure dsp node
for rproc usage")
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by:
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch
Enable all relevant nodes to support remoteproc with imx8mp-evk board.
- add rproc specific memory regions
- enable dsp_reserved node
- enable mu2 node
- enable dsp node
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 14
Enable MU2 node and add mu2 root clock.
MU2 is used to communicate with DSP core.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/freescale
oject/linux-arm-kernel/cover/20250311085812.1296243-1-daniel.bal...@nxp.com/)
- drop patches related to DSP run/stall/reset via syscon
- picked up patch related to using run_stall via reset
controller API.
Daniel Baluta (5):
arm64: dts: imx8mp: Use resets property
arm6
: 0||
1024 -> 2047 : 10748||
Tested-by: Daniel Gomez
Andreas, Petr, Miguel,
Based on the discussion in v7, it seems that all these patches will go through
the Rust tree. Is that correct? What would be missing from t
-862b-fd77552ae...@suse.com/ [1]
> Signed-off-by: Andreas Hindborg
And this is what we signed up for here:
https://lore.kernel.org/all/zspanzx4-5dro...@bombadil.infradead.org/
Acked-by: Daniel Gomez
> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/
Enable MU2 node and add mu2 root clock.
MU2 is used to communicate with DSP core.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 4
arch/arm64/boot/dts/freescale/imx8mp.dtsi| 1 +
2 files changed, 5
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch
DSP core needs ocram, core and debug clocks.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
b/arch/arm64/boot/dts
lback to handle resource table parsing and to
> process DSP-specific resource, to determine if waiting is needed.
>
> Update imx_dsp_rproc_start() to handle this condition accordingly.
>
> Signed-off-by: Iuliana Prodan
Reviewed-by: Daniel Baluta
rved_mem_region_to_resource() which is checked for an error.
>
> Signed-off-by: Rob Herring (Arm)
For IMX part:
Reviewed-by: Daniel Baluta
" and then get
> the region's address and size. As this is a standard property, it should
> have common functions for drivers to use. Add new functions to count the
> number of regions and retrieve the region's address as a resource.
>
> Signed-off-by: Rob Herring (Arm)
Reviewed-by: Daniel Baluta
On Mon, Mar 17, 2025 at 5:30 PM Alexander Stein
wrote:
>
> Am Montag, 17. März 2025, 14:33:03 CET schrieb Daniel Baluta:
> > Enable MU2 node and add mu2 root clock.
> > MU2 is used to communicate with DSP core.
> >
> > Reviewed-by: Iuliana Prodan
> > Revi
Enable MU2 node and add mu2 root clock.
MU2 is used to communicate with DSP core.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot
DSP core needs ocram, core and debug clocks.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
b/arch/arm64/boot/dts
ware) will need to
create a separate dts file (or an overlay).
This change follows the approach taken for other i.MX8 boards
in commit 391a319c81f6d7 ("arm64: dts: imx8-ss-audio: configure dsp node
for rproc usage")
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by:
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch
With imx8mp-evk board we are now configuring 'dsp' node for rproc usage,
so add rproc specific memory regions.
Also, enable dsp node because it is ready to be used.
Reviewed-by: Ahmad Fatoum
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/
trol the
DSP")
Reported-by: kernel test robot
Closes:
https://lore.kernel.org/oe-kbuild-all/202503142125.ie33scto-...@intel.com/
Signed-off-by: Daniel Baluta
---
drivers/remoteproc/imx_dsp_rproc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/remoteproc/imx_dsp_rproc.c
b/
https://patchwork.kernel.org/project/imx/patch/20250212085222.107102-6-daniel.bal...@nxp.com/
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/remoteproc/imx_dsp_rproc.c | 25 +
drivers/remoteproc/imx_rproc.h | 2 ++
2 files changed, 19 insertions(
ff-by: Daniel Baluta
---
.../devicetree/bindings/dsp/fsl,dsp.yaml | 24 ++-
1 file changed, 23 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml
b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml
index ab93ffd3d2e5..b8693e4b4b0d 1
For EARC and EARC PHY the reset happens when clearing the reset bits.
Refactor assert/deassert function in order to take into account the
active_low configuration option.
Reviewed-by: Philipp Zabel
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset
We can Run/Stall the DSP via audio block control bits found in audiomix.
Implement this functionality using the reset controller and use assert
for Stall and deassert for Run.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c
On Mon, Mar 10, 2025 at 5:43 PM Mathieu Poirier
wrote:
>
> Good day,
>
> On Wed, Mar 05, 2025 at 12:00:36PM +0200, Daniel Baluta wrote:
> > DSP on i.MX8MP doesn't have a direct reset line so according to hardware
> > design team in order to handle assert/deassert/re
Current code supports EARC PHY Software Reset and EARC Software Reset
but it is not easily extensible to more reset bits.
So, refactor the code in order to easily allow more reset bits in the
future.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset
This adds IMX8MP_AUDIOMIX_ prefix to internal macros in order to show
that specific macros are related to audiomix.
Reviewed-by: Philipp Zabel
Reviewed-by: Frank Li
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 14 +++---
1 file
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch
Add reset ids used for EARC and DSP on i.MX8MP platform.
Acked-by: Rob Herring (Arm)
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
include/dt-bindings/reset/imx8mp-reset-audiomix.h | 13 +
1 file changed, 13 insertions(+)
create mode 100644 include/dt-bindings/reset
- add correct include in the yaml dts snippet example
Changes since v1:
(https://lore.kernel.org/imx/20250219030809.GD6537@nxa18884-linux/T/)
- addresed comments received on v1
- picked up R-b and A-b tags
Daniel Baluta (8):
dt-bindings: reset: audiomix: Add reset ids for EA
This adds IMX8MP_AUDIOMIX_ prefix to internal macros in order to show
that specific macros are related to audiomix.
Reviewed-by: Philipp Zabel
Reviewed-by: Frank Li
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 14 +++---
1 file
stall control now, but softreset will
be added in the future when we will convert the softreset functionality
to use reset controller API.
[1]
https://patchwork.kernel.org/project/imx/patch/20250212085222.107102-6-daniel.bal...@nxp.com/
Reviewed-by: Frank Li
Signed-off-by: Daniel B
07102-6-daniel.bal...@nxp.com/
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/remoteproc/imx_dsp_rproc.c | 25 +
drivers/remoteproc/imx_rproc.h | 2 ++
2 files changed, 19 insertions(+), 8 deletions(-)
diff --git a/drivers/remot
Current code supports EARC PHY Software Reset and EARC Software Reset
but it is not easily extensible to more reset bits.
So, refactor the code in order to easily allow more reset bits in the
future.
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c
We can Run/Stall the DSP via audio block control bits found in audiomix.
Implement this functionality using the reset controller and use assert
for Stall and deassert for Run.
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 8
1 file
For EARC and EARC PHY the reset happens when clearing the reset bits.
Refactor assert/deassert function in order to take into account the
active_low configuration option.
Reviewed-by: Philipp Zabel
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale
emove 'resets' description as it is a common property
- add correct include in the yaml dts snippet example
Changes since v1:
(https://lore.kernel.org/imx/20250219030809.GD6537@nxa18884-linux/T/)
- addresed comments received on v1
- picked up R-b and A-b tags
Daniel B
Add reset ids used for EARC and DSP on i.MX8MP platform.
Acked-by: Rob Herring (Arm)
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
include/dt-bindings/reset/imx8mp-reset-audiomix.h | 13 +
1 file changed, 13 insertions(+)
create mode 100644 include/dt-bindings/reset
Hello Philipp,
Thanks for your comments!
> The DAP core reset is mentioned in the commit message. Why is it
> missing here? After reading the discussion in [1], I'd expect both the
> stall and the (core) reset signal to be documented, something like:
There is no reset controller driver for DAP a
On Tue, Feb 25, 2025 at 3:18 PM Philipp Zabel wrote:
>
> On Di, 2025-02-25 at 12:19 +0200, Daniel Baluta wrote:
> > Add reset ids used for EARC and DSP on i.MX8MP platform.
> >
> > Signed-off-by: Daniel Baluta
> > Acked-by: Rob Herring (Arm)
> > Reviewed-b
Current code supports EARC PHY Software Reset and EARC Software Reset
but it is not easily extensible to more reset bits.
So, refactor the code in order to easily allow more reset bits in the
future.
Signed-off-by: Daniel Baluta
Reviewed-by: Frank Li
---
drivers/reset/reset-imx8mp-audiomix.c
This adds IMX8MP_AUDIOMIX_ prefix to internal macros in order to show
that specific macros are related to audiomix.
Signed-off-by: Daniel Baluta
Reviewed-by: Philipp Zabel
Reviewed-by: Frank Li
Reviewed-by: Peng Fan
---
drivers/reset/reset-imx8mp-audiomix.c | 14 +++---
1 file
07102-6-daniel.bal...@nxp.com/
Signed-off-by: Daniel Baluta
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
---
drivers/remoteproc/imx_dsp_rproc.c | 25 +
drivers/remoteproc/imx_rproc.h | 2 ++
2 files changed, 19 insertions(+), 8 deletions(-)
diff --git a/drivers/remot
We can Run/Stall the DSP via audio block control bits found in audiomix.
Implement this functionality using the reset controller and use assert
for Stall and deassert for Run.
Signed-off-by: Daniel Baluta
Reviewed-by: Frank Li
---
drivers/reset/reset-imx8mp-audiomix.c | 8
1 file
For EARC and EARC PHY the reset happens when clearing the reset bits.
Refactor assert/deassert function in order to take into account the
active_low configuration option.
Signed-off-by: Daniel Baluta
Reviewed-by: Philipp Zabel
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
---
drivers/reset
Add reset ids used for EARC and DSP on i.MX8MP platform.
Signed-off-by: Daniel Baluta
Acked-by: Rob Herring (Arm)
Reviewed-by: Frank Li
---
include/dt-bindings/reset/imx8mp-reset-audiomix.h | 13 +
1 file changed, 13 insertions(+)
create mode 100644 include/dt-bindings/reset
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Signed-off-by: Daniel Baluta
Reviewed-by: Frank Li
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale
l.org/project/imx/patch/20250212085222.107102-6-daniel.bal...@nxp.com/
Signed-off-by: Daniel Baluta
Reviewed-by: Frank Li
---
.../devicetree/bindings/dsp/fsl,dsp.yaml | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/ds
ppet example
Changes since v1:
(https://lore.kernel.org/imx/20250219030809.GD6537@nxa18884-linux/T/)
- addresed comments received on v1
- picked up R-b and A-b tags
Daniel Baluta (8):
dt-bindings: reset: audiomix: Add reset ids for EARC and DSP
dt-bindings: dsp: fsl,dsp: Add res
"params" => info.params = Some(expect_params(it)),
> _ => panic!(
> "Unknown key \"{}\". Valid keys are: {:?}.",
> key, EXPECTED_KEYS
> @@ -183,28 +333,30 @@ pub(crate) fn module(ts: TokenStream) -> TokenStream {
> let info = ModuleInfo::parse(&mut it);
>
> let mut modinfo = ModInfoBuilder::new(info.name.as_ref());
> -if let Some(author) = info.author {
> -modinfo.emit("author", &author);
> +if let Some(author) = &info.author {
> +modinfo.emit("author", author);
> }
> -if let Some(description) = info.description {
> -modinfo.emit("description", &description);
> +if let Some(description) = &info.description {
> +modinfo.emit("description", description);
> }
> modinfo.emit("license", &info.license);
> -if let Some(aliases) = info.alias {
> +if let Some(aliases) = &info.alias {
> for alias in aliases {
> -modinfo.emit("alias", &alias);
> +modinfo.emit("alias", alias);
> }
> }
> -if let Some(firmware) = info.firmware {
> +if let Some(firmware) = &info.firmware {
> for fw in firmware {
> -modinfo.emit("firmware", &fw);
> +modinfo.emit("firmware", fw);
> }
> }
These seem a bit unrelated?
>
> // Built-in modules also export the `file` modinfo string.
> let file =
> std::env::var("RUST_MODFILE").expect("Unable to fetch RUST_MODFILE
> environmental variable");
> -modinfo.emit_only_builtin("file", &file);
> +modinfo.emit_only_builtin("file", &file, false);
> +
> +modinfo.emit_params(&info);
>
> format!(
> "
> @@ -362,14 +514,17 @@ unsafe fn __exit() {{
> __MOD.assume_init_drop();
> }}
> }}
> -
> {modinfo}
> }}
> }}
> +mod module_parameters {{
> +{params}
> +}}
> ",
> type_ = info.type_,
> name = info.name,
> modinfo = modinfo.buffer,
> +params = modinfo.param_buffer,
> initcall_section = ".initcall6.init"
> )
> .parse()
> diff --git a/samples/rust/rust_minimal.rs b/samples/rust/rust_minimal.rs
> index 4aaf117bf8e3c..d999a77c6eb9a 100644
I wonder if the changes to rust_minimal.rs should be a separate patch.
> --- a/samples/rust/rust_minimal.rs
> +++ b/samples/rust/rust_minimal.rs
> @@ -10,6 +10,12 @@
> author: "Rust for Linux Contributors",
> description: "Rust minimal sample",
> license: "GPL",
> +params: {
> +test_parameter: i64 {
> +default: 1,
> +description: "This parameter has a default of 1",
> +},
> +},
> }
>
> struct RustMinimal {
> @@ -20,6 +26,10 @@ impl kernel::Module for RustMinimal {
> fn init(_module: &'static ThisModule) -> Result {
> pr_info!("Rust minimal sample (init)\n");
> pr_info!("Am I built-in? {}\n", !cfg!(MODULE));
> +pr_info!(
> +"My parameter: {}\n",
> +*module_parameters::test_parameter.get()
> +);
>
> let mut numbers = KVec::new();
> numbers.push(72, GFP_KERNEL)?;
>
> --
> 2.47.0
>
>
>
— Daniel
Hi Andreas,
> On 18 Feb 2025, at 10:00, Andreas Hindborg wrote:
>
> Add the trait `ParseInt` for parsing string representations of integers
> where the string representations are optionally prefixed by a radix
> specifier. Implement the trait for the primitive integer types.
>
> Signed-off-by:
> On 21 Feb 2025, at 13:01, Daniel Almeida wrote:
>
>
>
>> On 18 Feb 2025, at 10:00, Andreas Hindborg wrote:
>>
>> Implement `AsRef` for `[u8]` and `BStr` so these can be used
>> interchangeably for operations on `BStr`.
>>
>> Signed-off-by
quot;)));
> +/// assert_eq!(Some(b_str!("")),
> b_str!("foobar").strip_prefix(b_str!("foobar")));
> +/// ```
This is passing.
> +pub fn strip_prefix(&self, pattern: impl AsRef) -> Option<&BStr> {
> +self.deref()
> +.strip_prefix(pattern.as_ref().deref())
> +.map(Self::from_bytes)
> +}
> }
>
> impl fmt::Display for BStr {
>
> --
> 2.47.0
>
>
Reviewed-by: Daniel Almeida
> On 18 Feb 2025, at 10:00, Andreas Hindborg wrote:
>
> Implement `AsRef` for `[u8]` and `BStr` so these can be used
> interchangeably for operations on `BStr`.
>
> Signed-off-by: Andreas Hindborg
> ---
> rust/kernel/str.rs | 12
> 1 file changed, 12 insertions(+)
>
> diff --git
BStr::from_bytes(&self.0[index])
> +}
> +}
> +
> /// Creates a new [`BStr`] from a string literal.
> ///
> /// `b_str!` converts the supplied string literal to byte string, so non-ASCII
>
> --
> 2.47.0
>
>
>
Reviewed-by: Daniel Almeida
t; +impl PartialEq for BStr {
> +fn eq(&self, other: &Self) -> bool {
> +self.deref().eq(other.deref())
> +}
> +}
> +
> /// Creates a new [`BStr`] from a string literal.
> ///
> /// `b_str!` converts the supplied string literal to byte string, so non-ASCII
>
> --
> 2.47.0
>
>
>
Reviewed-by: Daniel Almeida
y for `rnull`, the Rust
> null block driver [2].
>
```
$ sudo modprobe rust_minimal test_parameter=2
[ 251.384125] rust_minimal: Rust minimal sample (init)
[ 251.384600] rust_minimal: Am I built-in? false
[ 251.385010] rust_minimal: My parameter: 2
```
Tested-by: Daniel Almeida
I
> > + case IMX_RPROC_RESET_CONTROLLER:
> > + priv->reset = devm_reset_control_get_optional_exclusive(dev,
> > NULL);
>
> Is this optional on purpose? There is no mention of it in the commit
> message. Where is this resets property documented in the dt-bindings?
For this particular
On Tue, Feb 18, 2025 at 5:56 PM Frank Li wrote:
>
> On Tue, Feb 18, 2025 at 10:57:09AM +0200, Daniel Baluta wrote:
> > Current code supports EARC PHY Software Reset and EARCSoftware
> > Reset but it is not easily extensible to more reset bits.
> >
> > So,
For EARC and EARC PHY the reset happens when clearing the reset bits.
Refactor assert/deassert function in order to take into account
the active_low configuratin option.
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 45 ++-
1 file changed, 23
We can Run/Stall the DSP via audio block control bits found in audiomix.
Implement this functionality using the reset controller and use assert
for Stall and deassert for Run.
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 12 ++--
1 file changed, 10 insertions
Use the reset controller API to control the DSP on i.MX8MP. This way
we can have a better control of the resources and avoid using a syscon
to access the audiomix bits.
Signed-off-by: Daniel Baluta
---
drivers/remoteproc/imx_dsp_rproc.c | 25 +
drivers/remoteproc
://patchwork.kernel.org/project/imx/patch/20241210125338.104959-6-daniel.bal...@nxp.com/
Daniel Baluta (5):
reset: imx8mp-audiomix: Add prefix for internal macro
reset: imx8mp-audiomix: Prepare the code for more reset bits
reset: imx8mp-audiomix: Introduce active_low configuration option
reset
Current code supports EARC PHY Software Reset and EARC Software
Reset but it is not easily extensible to more reset bits.
So, refactor the code in order to easily allow more reset bits
in the future.
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 53
This adds IMX8MP_AUDIOMIX_ prefix to internal macros
in order to show that specific macros are related to
audiomix.
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/reset/reset
Test that very high constant map keys are not interpreted as an error
value by the verifier. This would previously fail.
Acked-by: Eduard Zingerman
Signed-off-by: Daniel Xu
---
.../selftests/bpf/progs/verifier_array_access.c | 15 +++
1 file changed, 15 insertions(+)
diff --git
Two fixes for nullness elision. See commits for more details.
=== Changelog ===
Changes from v1:
* Reword commit message in patch 1
* Add tags
Daniel Xu (3):
bpf: verifier: Do not extract constant map keys for irrelevant maps
bpf: selftests: Test constant key extraction on irrelevant maps
Test that very high constant map keys are not interpreted as an error
value by the verifier. This would previously fail.
Signed-off-by: Daniel Xu
---
.../selftests/bpf/progs/verifier_array_access.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/tools/testing/selftests
Two fixes for nullness elision. See commits for more details.
Daniel Xu (3):
bpf: verifier: Do not extract constant map keys for irrelevant maps
bpf: selftests: Test constant key extraction on irrelevant maps
bpf: verifier: Disambiguate get_constant_map_key() errors
kernel/bpf/verifier.c
On Thu, Jan 30, 2025, at 7:41 PM, Daniel Xu wrote:
> Hi Ilya,
>
> On Thu, Jan 30, 2025, at 2:06 AM, Ilya Leoshkevich wrote:
>> On Wed, 2025-01-29 at 10:45 -0700, Daniel Xu wrote:
>>> On Wed, Jan 29, 2025 at 09:49:12AM -0700, Daniel Xu wrote:
>>> > Hi Ilya,
Hi Yonghong,
On Thu, Jan 30, 2025 at 10:28:11PM -0800, Yonghong Song wrote:
>
>
>
> On 1/30/25 2:33 PM, Daniel Xu wrote:
> > Since 67ab80a01886 ("selftests/bpf: Prefer static linking for LLVM
> > libraries"), only statically linking test_progs is support
e. If both options are available, static linking
is chosen.
Signed-off-by: Daniel Xu
---
tools/testing/selftests/bpf/Makefile | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/tools/testing/selftests/bpf/Makefile
b/tools/testing/selftests/bpf/Makefile
index 67
Hi Ilya,
On Thu, Jan 30, 2025, at 2:06 AM, Ilya Leoshkevich wrote:
> On Wed, 2025-01-29 at 10:45 -0700, Daniel Xu wrote:
>> On Wed, Jan 29, 2025 at 09:49:12AM -0700, Daniel Xu wrote:
>> > Hi Ilya,
>> >
>> > On Wed, Jan 29, 2025 at 03:58:54PM +0100, Ilya Leosh
On Wed, Jan 29, 2025 at 09:49:12AM -0700, Daniel Xu wrote:
> Hi Ilya,
>
> On Wed, Jan 29, 2025 at 03:58:54PM +0100, Ilya Leoshkevich wrote:
> > On Tue, 2025-01-14 at 13:28 -0700, Daniel Xu wrote:
> > > This commit allows progs to elide a null check on statically known
>
Hi Ilya,
On Wed, Jan 29, 2025 at 03:58:54PM +0100, Ilya Leoshkevich wrote:
> On Tue, 2025-01-14 at 13:28 -0700, Daniel Xu wrote:
> > This commit allows progs to elide a null check on statically known
> > map
> > lookup keys. In other words, if the verifier can statically p
On 1/15/25 2:47 AM, Li Zhijian wrote:
Fixes an issue where out-of-tree kselftest builds fail when building
the BPF and bpftools components. The failure occurs because the top-level
Makefile passes a relative srctree path to its sub-Makefiles, which
leads to errors in locating necessary files.
Fo
ointer. So check that we disallow
both.
Signed-off-by: Daniel Xu
---
.../bpf/progs/verifier_array_access.c | 188 ++
1 file changed, 188 insertions(+)
diff --git a/tools/testing/selftests/bpf/progs/verifier_array_access.c
b/tools/testing/selftests/bpf/progs/verifier_arra
.
An additional benefit is the verifier logs are also more precise. For
this particular error, users will enjoy a slightly clearer message. See
included selftest updates for examples.
Acked-by: Eduard Zingerman
Signed-off-by: Daniel Xu
---
kernel/bpf/verifier.c | 45
d in sync with these changes, as the
verifier is more efficient with this change. Notable, iters.c tests had
to be changed to use a map type that still requires null checks, as it's
exercising verifier tracking logic w.r.t iterators.
Signed-off-by: Daniel Xu
---
kernel/bpf/verifier.c
1 - 100 of 6362 matches
Mail list logo
