[PATCH 0/3] riscv: Kconfig: Style cleanups

Juerg Haefliger posted 3 patches 3 years, 11 months ago
arch/riscv/Kconfig         | 19 +++++++++----------
arch/riscv/Kconfig.erratas |  2 +-
arch/riscv/Kconfig.socs    |  4 ++--
3 files changed, 12 insertions(+), 13 deletions(-)
[PATCH 0/3] riscv: Kconfig: Style cleanups
Posted by Juerg Haefliger 3 years, 11 months ago
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(-)

-- 
2.32.0
Re: [PATCH 0/3] riscv: Kconfig: Style cleanups
Posted by Palmer Dabbelt 3 years, 10 months ago
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.