Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzk@kernel.org>
To: Jason Li <jason.lee651024@gmail.com>,
	jason.li@cortina-access.com,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jiri Slaby <jirislaby@kernel.org>
Cc: Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>, Arnd Bergmann <arnd@arndb.de>,
	linux-serial@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/3] dt-bindings: serial: Add binding for Cortina-Access UART
Date: Wed, 10 Jun 2026 13:46:55 +0200	[thread overview]
Message-ID: <068a7ba8-5b1e-46e3-9388-ba288163eb10@kernel.org> (raw)
In-Reply-To: <20260610112821.3030099-3-jason.li@cortina-access.com>

On 10/06/2026 13:28, Jason Li wrote:
> Add DT binding schema for the Cortina-Access UART controller.
> This IP is integrated into most CAXXXX SoC family members.
> 
> Also add the vendor prefix for Cortina Access, Inc. and the
> top-level ARM board binding document for the CA8289 (Venus) SoC.
> 
> Signed-off-by: Jason Li <jason.li@cortina-access.com>
> Assisted-by: Claude:claude-opus-4-8
> ---
>  .../bindings/arm/cortina-access.yaml          | 29 ++++++++++++
>  .../serial/cortina-access,serial.yaml         | 46 +++++++++++++++++++
>  .../devicetree/bindings/vendor-prefixes.yaml  |  2 +
>  MAINTAINERS                                   |  7 +++

This is somehow complete mess. serial and arm together?

Please carefully read submitting patches (both documents!) and don't
send AI-assisted slop.

You must not combine independent works together.


>  4 files changed, 84 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/cortina-access.yaml
>  create mode 100644 Documentation/devicetree/bindings/serial/cortina-access,serial.yaml
> 
> diff --git a/Documentation/devicetree/bindings/arm/cortina-access.yaml b/Documentation/devicetree/bindings/arm/cortina-access.yaml
> new file mode 100644
> index 000000000000..ec0320ed0c0b
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/cortina-access.yaml
> @@ -0,0 +1,29 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/cortina-access.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Cortina-Access SoC boards
> +
> +maintainers:
> +  - Jason Li <jason.li@cortina-access.com>
> +
> +description:
> +  Boards based on Cortina-Access ARMv8 SoCs targeting networking and
> +  access applications.
> +
> +properties:
> +  $nodename:
> +    const: /
> +  compatible:
> +    oneOf:
> +      - description: Cortina-Access CA8289 (Venus) engineering board
> +        const: cortina-access,ca8289-engboard
> +
> +      - description: Cortina-Access CA8289 (Venus) reference board
> +        const: cortina-access,ca8289-refboard


Where is the SoC? This looks like very poor contribution. If you opened
any existing recent board binding you would see it is done differently.

Best regards,
Krzysztof


  reply	other threads:[~2026-06-10 11:47 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-10 11:28 [PATCH 0/3] tty: serial: Add Cortina-Access UART driver and platform support Jason Li
2026-06-10 11:28 ` Jason Li
2026-06-10 12:50   ` Arnd Bergmann
2026-06-11  5:27     ` Jason Li
2026-06-11  8:35       ` Arnd Bergmann
2026-06-10 11:28 ` [PATCH 1/3] dt-bindings: serial: Add binding for Cortina-Access UART Jason Li
2026-06-10 11:46   ` Krzysztof Kozlowski [this message]
2026-06-11  5:25     ` Jason Li
2026-06-10 11:51   ` Krzysztof Kozlowski
2026-06-11  5:26     ` Jason Li
2026-06-10 11:28 ` [PATCH 2/3] tty: serial: Add UART driver for Cortina-Access platform Jason Li
2026-06-10 22:49   ` Randy Dunlap
2026-06-10 11:28 ` [PATCH 3/3] arm64: dts: cortina-access: Add DTS for CA8289 SoC and Venus board Jason Li
2026-06-10 11:49   ` Krzysztof Kozlowski
2026-06-11  5:26     ` Jason Li

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=068a7ba8-5b1e-46e3-9388-ba288163eb10@kernel.org \
    --to=krzk@kernel.org \
    --cc=arnd@arndb.de \
    --cc=catalin.marinas@arm.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=jason.lee651024@gmail.com \
    --cc=jason.li@cortina-access.com \
    --cc=jirislaby@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=robh@kernel.org \
    --cc=will@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox