public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 1/5] dt-bindings: vendor-prefixes: Add 'qst' for QST Corporation
@ 2025-05-08 12:08 Brajesh Patil
  2025-05-08 12:13 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 3+ messages in thread
From: Brajesh Patil @ 2025-05-08 12:08 UTC (permalink / raw)
  To: robh, krzk+dt, conor+dt
  Cc: devicetree, linux-kernel, jic23, marcelo.schmitt1, dlechner,
	Brajesh Patil

Signed-off-by: Brajesh Patil <brajeshpatil11@gmail.com>
---
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
index 86f6a19b28ae..b306950ebea8 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1236,6 +1236,8 @@ patternProperties:
     description: Shenzhen QiShenglong Industrialist Co., Ltd.
   "^qnap,.*":
     description: QNAP Systems, Inc.
+  "^qst,.*":
+    description: QST Corporation Ltd.
   "^quanta,.*":
     description: Quanta Computer Inc.
   "^radxa,.*":
--
2.39.5


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

* Re: [PATCH v2 1/5] dt-bindings: vendor-prefixes: Add 'qst' for QST Corporation
  2025-05-08 12:08 [PATCH v2 1/5] dt-bindings: vendor-prefixes: Add 'qst' for QST Corporation Brajesh Patil
@ 2025-05-08 12:13 ` Krzysztof Kozlowski
       [not found]   ` <CALJe6R14mpMCyzrmfN1w8DrDe-=xvH=6HpF5Hc=6tVEX4p+5Hw@mail.gmail.com>
  0 siblings, 1 reply; 3+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-08 12:13 UTC (permalink / raw)
  To: Brajesh Patil, robh, krzk+dt, conor+dt
  Cc: devicetree, linux-kernel, jic23, marcelo.schmitt1, dlechner

On 08/05/2025 14:08, Brajesh Patil wrote:
> Signed-off-by: Brajesh Patil <brajeshpatil11@gmail.com>
> ---

Please respond to individual comments instead of ignoring them. For
example: how did you solve that comment:

"Please run scripts/checkpatch.pl on the patches and fix reported
warnings. After that, run also 'scripts/checkpatch.pl --strict' on the
patches and (probably) fix more warnings. Some warnings can be ignored,
especially from --strict run, but the code here looks like it needs a
fix. Feel free to get in touch if the warning is not clear."

?

Best regards,
Krzysztof

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

* Re: [PATCH v2 1/5] dt-bindings: vendor-prefixes: Add 'qst' for QST Corporation
       [not found]   ` <CALJe6R14mpMCyzrmfN1w8DrDe-=xvH=6HpF5Hc=6tVEX4p+5Hw@mail.gmail.com>
@ 2025-05-08 15:49     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-08 15:49 UTC (permalink / raw)
  To: Brajesh Patil
  Cc: robh, krzk+dt, conor+dt, devicetree, linux-kernel, jic23,
	marcelo.schmitt1, dlechner

On 08/05/2025 15:09, Brajesh Patil wrote:

...

Do not top post.

> 2. I ran scripts/checkpatch.pl on the patches, which reported some warnings
> related to trailing whitespaces. I have fixed those manually. Additionally,
> I used the --strict flag, but no errors or warnings were generated during
> this run.

I have doubts about this. I clearly see warning:

$ scripts/checkpatch.pl
0001-dt-bindings-vendor-prefixes-Add-qst-for-QST-Corporat.patch
WARNING: Missing commit description - Add an appropriate one
total: 0 errors, 1 warnings, 8 lines checked

Best regards,
Krzysztof

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

end of thread, other threads:[~2025-05-08 15:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-08 12:08 [PATCH v2 1/5] dt-bindings: vendor-prefixes: Add 'qst' for QST Corporation Brajesh Patil
2025-05-08 12:13 ` Krzysztof Kozlowski
     [not found]   ` <CALJe6R14mpMCyzrmfN1w8DrDe-=xvH=6HpF5Hc=6tVEX4p+5Hw@mail.gmail.com>
2025-05-08 15:49     ` Krzysztof Kozlowski

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