linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] scripts/dtc: Fix errors in yamltree.c
       [not found] <tencent_C2E29E92E4A5FCCF6F6BB14E2ED98DC6C309@qq.com>
@ 2023-07-18  1:38 ` shijie001
  2023-07-18 13:42   ` Rob Herring
  0 siblings, 1 reply; 2+ messages in thread
From: shijie001 @ 2023-07-18  1:38 UTC (permalink / raw)
  To: robh+dt, frowand.list; +Cc: devicetree, linux-kernel

The following checkpatch errors are removed:
ERROR: "(foo*)" should be "(foo *)"

Signed-off-by: Jie Shi <shijie001@208suo.com>
---
  scripts/dtc/yamltree.c | 12 ++++++------
  1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/scripts/dtc/yamltree.c b/scripts/dtc/yamltree.c
index 55908c829c98..dd22858d1bb1 100644
--- a/scripts/dtc/yamltree.c
+++ b/scripts/dtc/yamltree.c
@@ -57,7 +57,7 @@ static void yaml_propval_int(yaml_emitter_t *emitter, 
struct marker *markers,

          switch(width) {
          case 1:
-            sprintf(buf, "0x%"PRIx8, *(uint8_t*)(data + off));
+            sprintf(buf, "0x%"PRIx8, *(uint8_t *)(data + off));
              break;
          case 2:
              sprintf(buf, "0x%"PRIx16, dtb_ld16(data + off));
@@ -80,11 +80,11 @@ static void yaml_propval_int(yaml_emitter_t 
*emitter, struct marker *markers,

          if (is_phandle)
              yaml_scalar_event_initialize(&event, NULL,
-                (yaml_char_t*)"!phandle", (yaml_char_t *)buf,
+                (yaml_char_t *)"!phandle", (yaml_char_t *)buf,
                  strlen(buf), 0, 0, YAML_PLAIN_SCALAR_STYLE);
          else
              yaml_scalar_event_initialize(&event, NULL,
-                (yaml_char_t*)YAML_INT_TAG, (yaml_char_t *)buf,
+                (yaml_char_t *)YAML_INT_TAG, (yaml_char_t *)buf,
                  strlen(buf), 1, 1, YAML_PLAIN_SCALAR_STYLE);
          yaml_emitter_emit_or_die(emitter, &event);
      }
@@ -119,7 +119,7 @@ static void yaml_propval(yaml_emitter_t *emitter, 
struct property *prop)

      /* Emit the property name */
      yaml_scalar_event_initialize(&event, NULL,
-        (yaml_char_t *)YAML_STR_TAG, (yaml_char_t*)prop->name,
+        (yaml_char_t *)YAML_STR_TAG, (yaml_char_t *)prop->name,
          strlen(prop->name), 1, 1, YAML_PLAIN_SCALAR_STYLE);
      yaml_emitter_emit_or_die(emitter, &event);

@@ -127,7 +127,7 @@ static void yaml_propval(yaml_emitter_t *emitter, 
struct property *prop)
      if (len == 0) {
          yaml_scalar_event_initialize(&event, NULL,
              (yaml_char_t *)YAML_BOOL_TAG,
-            (yaml_char_t*)"true",
+            (yaml_char_t *)"true",
              strlen("true"), 1, 0, YAML_PLAIN_SCALAR_STYLE);
          yaml_emitter_emit_or_die(emitter, &event);
          return;
@@ -194,7 +194,7 @@ static void yaml_tree(struct node *tree, 
yaml_emitter_t *emitter)
      /* Loop over all the children, emitting them into the map */
      for_each_child(tree, child) {
          yaml_scalar_event_initialize(&event, NULL,
-            (yaml_char_t *)YAML_STR_TAG, (yaml_char_t*)child->name,
+            (yaml_char_t *)YAML_STR_TAG, (yaml_char_t *)child->name,
              strlen(child->name), 1, 0, YAML_PLAIN_SCALAR_STYLE);
          yaml_emitter_emit_or_die(emitter, &event);
          yaml_tree(child, emitter);

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

* Re: [PATCH] scripts/dtc: Fix errors in yamltree.c
  2023-07-18  1:38 ` [PATCH] scripts/dtc: Fix errors in yamltree.c shijie001
@ 2023-07-18 13:42   ` Rob Herring
  0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring @ 2023-07-18 13:42 UTC (permalink / raw)
  To: shijie001; +Cc: frowand.list, devicetree, linux-kernel

On Mon, Jul 17, 2023 at 7:38 PM <shijie001@208suo.com> wrote:
>
> The following checkpatch errors are removed:
> ERROR: "(foo*)" should be "(foo *)"
>
> Signed-off-by: Jie Shi <shijie001@208suo.com>
> ---
>   scripts/dtc/yamltree.c | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)

This is a copy of upstream dtc imported as-is into the kernel. If you
want to make changes, you have to do them on the upstream dtc.

The YAML support is likely going to be removed, so I wouldn't bother
with style fixes.

Rob

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

end of thread, other threads:[~2023-07-18 13:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <tencent_C2E29E92E4A5FCCF6F6BB14E2ED98DC6C309@qq.com>
2023-07-18  1:38 ` [PATCH] scripts/dtc: Fix errors in yamltree.c shijie001
2023-07-18 13:42   ` Rob Herring

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).