U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/1] doc: .clang-format description
@ 2026-04-24 13:05 Heinrich Schuchardt
       [not found] ` <0b958275-d387-4cb8-ad68-55f3e2d0ac82@cherry.de>
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Heinrich Schuchardt @ 2026-04-24 13:05 UTC (permalink / raw)
  To: Tom Rini
  Cc: Quentin Schulz, Mattijs Korpershoek, Javier Tia, Neil Armstrong,
	u-boot, Heinrich Schuchardt

We cannot use .clang-format without modification. For instance U-Boot
has some for each macros that Linux does not have.

Adjust the description.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
---
 doc/develop/codingstyle.rst | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/doc/develop/codingstyle.rst b/doc/develop/codingstyle.rst
index 2a69162fa95..26881cf3900 100644
--- a/doc/develop/codingstyle.rst
+++ b/doc/develop/codingstyle.rst
@@ -80,8 +80,10 @@ Code Formatting with clang-format
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 U-Boot provides a `.clang-format` configuration file that was copied directly
-from the Linux kernel without any modifications, ensuring complete compatibility
-with kernel coding standards. Here are common ways to use clang-format:
+from the Linux kernel, ensuring complete compatibility with kernel coding
+standards.
+
+Here are common ways to use clang-format:
 
 **Basic usage for single files:**
 
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] doc: .clang-format description
       [not found] ` <0b958275-d387-4cb8-ad68-55f3e2d0ac82@cherry.de>
@ 2026-04-24 13:27   ` Heinrich Schuchardt
  0 siblings, 0 replies; 4+ messages in thread
From: Heinrich Schuchardt @ 2026-04-24 13:27 UTC (permalink / raw)
  To: Quentin Schulz, Tom Rini
  Cc: Mattijs Korpershoek, Javier Tia, Neil Armstrong, u-boot

On 4/24/26 15:10, Quentin Schulz wrote:
> Hi Heinrich,
> 
> On 4/24/26 3:05 PM, Heinrich Schuchardt wrote:
>> We cannot use .clang-format without modification. For instance U-Boot
>> has some for each macros that Linux does not have.
>>
> 
> Uh?
> 
> sha256sum ../u-boot/.clang-format <(git show v6.16:.clang-format)
> a7a2d4f21d9ff4696498077b71b5589a44b95697eb80c013a94768d05f7d8b54 ../u- 
> boot/.clang-format
> a7a2d4f21d9ff4696498077b71b5589a44b95697eb80c013a94768d05f7d8b54 /proc/ 
> self/fd/11
> 
> Cheers,
> Quentin

Try to format anything with one of these macros:

+  # U-Boot specific
+  - 'alist_for_each'
+  - 'alist_for_each_filter'
+  - 'for_each_bl_params_node'
+  - 'ofnode_for_each_prop'
+  - 'ofnode_for_each_subnode'
+  - 'ofnode_for_each_compatible_node'
+  - 'dev_for_each_subnode'
+  - 'dev_for_each_property'
+  - 'fdt_for_each_node_by_compatible'
+  - 'for_each_console_dev'
+  - 'mtd_for_each_device'
+  - 'for_each_tpm_device'

I have created a separate patch for .clang-format but unfortunately our 
list server is down.

Best regards

Heinrich

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] doc: .clang-format description
  2026-04-24 13:05 [PATCH 1/1] doc: .clang-format description Heinrich Schuchardt
       [not found] ` <0b958275-d387-4cb8-ad68-55f3e2d0ac82@cherry.de>
@ 2026-04-27 16:54 ` Quentin Schulz
  2026-04-30  8:04 ` Mattijs Korpershoek
  2 siblings, 0 replies; 4+ messages in thread
From: Quentin Schulz @ 2026-04-27 16:54 UTC (permalink / raw)
  To: Heinrich Schuchardt, Tom Rini
  Cc: Mattijs Korpershoek, Javier Tia, Neil Armstrong, u-boot

Hi Heinrich,

On 4/24/26 3:05 PM, Heinrich Schuchardt wrote:
> We cannot use .clang-format without modification. For instance U-Boot
> has some for each macros that Linux does not have.
> 
> Adjust the description.
> 

Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>

Thanks!
Quentin

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] doc: .clang-format description
  2026-04-24 13:05 [PATCH 1/1] doc: .clang-format description Heinrich Schuchardt
       [not found] ` <0b958275-d387-4cb8-ad68-55f3e2d0ac82@cherry.de>
  2026-04-27 16:54 ` Quentin Schulz
@ 2026-04-30  8:04 ` Mattijs Korpershoek
  2 siblings, 0 replies; 4+ messages in thread
From: Mattijs Korpershoek @ 2026-04-30  8:04 UTC (permalink / raw)
  To: Heinrich Schuchardt, Tom Rini
  Cc: Quentin Schulz, Mattijs Korpershoek, Javier Tia, Neil Armstrong,
	u-boot, Heinrich Schuchardt

Hi Heinrich,

Thank you for the patch.

On Fri, Apr 24, 2026 at 15:05, Heinrich Schuchardt <heinrich.schuchardt@canonical.com> wrote:

> We cannot use .clang-format without modification. For instance U-Boot
> has some for each macros that Linux does not have.
>
> Adjust the description.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>

Reviewed-by: Mattijs Korpershoek <mkorpershoek@kernel.org>

> ---
>  doc/develop/codingstyle.rst | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-04-30  8:04 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-24 13:05 [PATCH 1/1] doc: .clang-format description Heinrich Schuchardt
     [not found] ` <0b958275-d387-4cb8-ad68-55f3e2d0ac82@cherry.de>
2026-04-24 13:27   ` Heinrich Schuchardt
2026-04-27 16:54 ` Quentin Schulz
2026-04-30  8:04 ` Mattijs Korpershoek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox