Message ID | 20240411234021.14534-1-greg.malysa@timesys.com |
---|---|
Headers | show |
Series | arm: Add Analog Devices SC5xx Machine Type | expand |
On Thu, Apr 11, 2024 at 07:37:27PM -0400, Greg Malysa wrote: > > This series adds support for the ADI SC5xx machine type and includes two > core drivers that are required for being able to boot any board--a UART > driver and the clock tree driver. Our corresponding Linux support relies > on u-boot configuring the clocks correctly before booting, so it is not > possible to boot any board without the CGU/CDU configuration happening > here. The clock tree itself is only used by the UART in this minimal > patch set, but is of course broadly useful for all other peripheral > drivers to be submitted in future patch sets. There are also no board > files or defconfigs included here, but some common definitions that will > be used to build board files currently are. > > Some of the configuration code in dmcinit and clkinit is quite scary and > causes a lot of checkpatch violations. It is modified from code > initially provided by ADI, but it has not been fully rewritten. There's > a question of how important it is to clean up this code--it has some > quality violations, but it has been in use (including in production) for > over two years and is known to work for performing the low level SoC > initialization, while a rewrite might introduce bugs that could take a > significant amount of time to detect in the future. > > Please provide any feedback or comments that will be helpful in creating > a v2 of this patch that will be ready to submit for inclusion into the > main tree, as well as general observations that we should consider in > other driver patches before submitting them. Does this series pass CI currently? Thanks.
I'm afraid I have to admit I don't know. I'll work with our IT team to make sure we can run CI locally, and when v2 comes around the answer will be yes. On Thu, Apr 11, 2024 at 7:52 PM Tom Rini <trini@konsulko.com> wrote: > > On Thu, Apr 11, 2024 at 07:37:27PM -0400, Greg Malysa wrote: > > > > This series adds support for the ADI SC5xx machine type and includes two > > core drivers that are required for being able to boot any board--a UART > > driver and the clock tree driver. Our corresponding Linux support relies > > on u-boot configuring the clocks correctly before booting, so it is not > > possible to boot any board without the CGU/CDU configuration happening > > here. The clock tree itself is only used by the UART in this minimal > > patch set, but is of course broadly useful for all other peripheral > > drivers to be submitted in future patch sets. There are also no board > > files or defconfigs included here, but some common definitions that will > > be used to build board files currently are. > > > > Some of the configuration code in dmcinit and clkinit is quite scary and > > causes a lot of checkpatch violations. It is modified from code > > initially provided by ADI, but it has not been fully rewritten. There's > > a question of how important it is to clean up this code--it has some > > quality violations, but it has been in use (including in production) for > > over two years and is known to work for performing the low level SoC > > initialization, while a rewrite might introduce bugs that could take a > > significant amount of time to detect in the future. > > > > Please provide any feedback or comments that will be helpful in creating > > a v2 of this patch that will be ready to submit for inclusion into the > > main tree, as well as general observations that we should consider in > > other driver patches before submitting them. > > Does this series pass CI currently? Thanks. > > -- > Tom
On Thu, 2024-04-11 at 20:18 -0400, Greg Malysa wrote: > I'm afraid I have to admit I don't know. I'll work with our IT team to > make sure we can run CI locally, and when v2 comes around the answer > will be yes. > AFAIR, I think you the only exception to open a PR in the github fork is to run CI. - Nuno Sá