Erwan Gouriou

Results 37 comments of Erwan Gouriou

> This "Not implemented" issue is about to get fixed by #36848 it is at stale because of some part of the PR that is not generic enough to cover...

@MaureenHelm Should CODEOWNERS/MAINTAINERS be updated ?

> @erwango how? From recent examples, I think that CODEOWNER is still in use. So if confirmed, this file should be updated.

@spectrum70 Thanks for this change but we don't merge new features in this fork. However, you can submit them in https://github.com/STMicroelectronics/STM32CubeH7. If approved, we can merge them here.

> AFAYK, if i cannot have ST to accept such change, can i implement the feature by direct register access in our driver ? Well, indeed, if HAL support is...

Thanks for this fix, but these files are generated from a data base, see [README.rst#dtsi-files](https://github.com/zephyrproject-rtos/hal_stm32/blob/d46453dfb710bcc2ea70f47af610d3a225c72316/README.rst#dtsi-files). So either the generating script or the database should be updated, not the generated files.

LGTM, but it's expected that you generate a new batch of -pinctrl.dtsi files by yourself (so we can see if the result is as expected). Instructions are available in the...

SHA1, is the one provided in `dts/README.rst`. There are actually some lines in https://github.com/zephyrproject-rtos/hal_stm32#dtsi-files, but this can be improved and suggestions are welcome.

> Oh weird - I assumed that the legacy header definitions would be deprecated at some point. I think that was the point initially, but it's being carried out from...

Thanks for this patch, but in current repo we're only merging fixing waiting to be delivered in main Cube packages (or Zephyr specific). For this kind of issue, please raise...