mbox series

[0/3] riscv: Kconfig: Style cleanups

Message ID 20220520120232.148310-1-juergh@canonical.com (mailing list archive)
Headers show
Series riscv: Kconfig: Style cleanups | expand

Message

Juerg Haefliger May 20, 2022, 12:02 p.m. UTC
The majority of the Kconfig files use a single tab for basic indentation
and a single tab followed by two whitespaces for help text indentation.
Fix the lines that don't follow this convention.

While at it:
  - Add trailing comments to endif/endmenu statements for better
    readability.

Juerg Haefliger (3):
  riscv: Kconfig: Fix indentation and add comments
  riscv: Kconfig.erratas: Add comments
  riscv: Kconfig.socs: Add comments

 arch/riscv/Kconfig         | 19 +++++++++----------
 arch/riscv/Kconfig.erratas |  2 +-
 arch/riscv/Kconfig.socs    |  4 ++--
 3 files changed, 12 insertions(+), 13 deletions(-)

Comments

Palmer Dabbelt July 1, 2022, 4:07 a.m. UTC | #1
On Fri, 20 May 2022 05:02:29 PDT (-0700), juerg.haefliger@canonical.com wrote:
> The majority of the Kconfig files use a single tab for basic indentation
> and a single tab followed by two whitespaces for help text indentation.
> Fix the lines that don't follow this convention.
>
> While at it:
>   - Add trailing comments to endif/endmenu statements for better
>     readability.
>
> Juerg Haefliger (3):
>   riscv: Kconfig: Fix indentation and add comments
>   riscv: Kconfig.erratas: Add comments
>   riscv: Kconfig.socs: Add comments
>
>  arch/riscv/Kconfig         | 19 +++++++++----------
>  arch/riscv/Kconfig.erratas |  2 +-
>  arch/riscv/Kconfig.socs    |  4 ++--
>  3 files changed, 12 insertions(+), 13 deletions(-)

Thanks, this is on for-next.  There were a few merge conflicts, 
everything looks pretty straight-forward but let me know if there were 
any issues.