Date:   Wed, 22 Feb 2023 13:40:41 -0800
From:   Colin Foster <colin.foster@in-advantage.com>
To:     linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org, devicetree@vger.kernel.org
Cc:     Tony Lindgren <tony@atomide.com>,
        =?utf-8?Q?Beno=C3=AEt?= Cousson <bcousson@baylibre.com>,
        soc@kernel.org, Olof Johansson <olof@lixom.net>,
        Arnd Bergmann <arnd@arndb.de>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>, Andrew Davis <afd@ti.com>
Subject: Re: [PATCH v4 1/2] dt-bindings: arm: omap: add phytec pcm-049 som
 and pcm-959 dev board
Message-ID: <Y/aL2fquXe0aJleG@colin-ia-desktop>
References: <20230222203847.2664903-1-colin.foster@in-advantage.com>
 <20230222203847.2664903-2-colin.foster@in-advantage.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230222203847.2664903-2-colin.foster@in-advantage.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?DVkYW3I537pQzMQhhuU9y7BWc7jxQug6ejXf2XuIalYjcuK16QTctJutke3q?=
 =?us-ascii?Q?AGb4c9xdqJvnLuzgiWptvGRwM2Y5xIbABQvTsHFRgqzRuT/YFT6oKlUMHt8I?=
 =?us-ascii?Q?2b3hsCnCiJauL/X/Nf9fz1u4Q2oeo/jX2xlOHpFDPdEqyx/D/2VLgx06WGPE?=
 =?us-ascii?Q?ps9UzndRLxBQ3xZeTgxmhwc8mOYNrB1rGIdRi83okVQT/qmJYXhPZVFoOGET?=
 =?us-ascii?Q?42uGJxL1NCIa8D+XiLWTmKkHuvQ4a7gGo7J21d4zs1q2+okpNo+qpggq2Rkw?=
 =?us-ascii?Q?wGfBr0kgeKtOV4p6SqDgQzC/ViawuzzonrTbEzCmtmI/mEzSdyvbU4avSGVx?=
 =?us-ascii?Q?g7mCTjyDEHyjBTng0ftOVP6AdO/7B53XqCqpEzeS6G4mutVbwvHX+UlOWnwx?=
 =?us-ascii?Q?3qrenni+DYhwXqBEb3Zu9lFFVf2sWaqG+sw75EhgOmU+cWnG7+/sOjb+Qv6U?=
 =?us-ascii?Q?rxYmvHWFm4pjwt+X1fKapPIyxn/jiPC8QBhV8Tg6b+prJI9TIyd0lWTdrgrZ?=
 =?us-ascii?Q?kHVClHy+3yVAcn6x7DZ42ocI27Zn2gX74THiJ6c7qoCiqGgkve1dggYLlxOg?=
 =?us-ascii?Q?u4Wt8Oe5ibLnKrQ6aKjqmjVSTvPUyshLn9RUB4VVhDUmy3tLOFF0ifZ36Mlc?=
 =?us-ascii?Q?qprX0Yj/YyjGVPx/WPXAPepDTMhR+IgIjiaUdK6Wy+VPHaTkpzHs8mqYuJ9t?=
 =?us-ascii?Q?fngEylThBSaqO7WHAWrIU9l/0l87Qk+eEItp8KR2pA/7DhYr1bQ6xTc91yMx?=
 =?us-ascii?Q?QF2CvWJZoZdo5dfq57nDw70Cv0jtKV1RaTnMwEY0sh08WE67r4fqgCeL3A2m?=
 =?us-ascii?Q?+xT1pNb+MpOo4FvhN3n1yayub3f4Bhsw/CzlRMd9oK2t6GwQA7uUVRWx0t9K?=
 =?us-ascii?Q?Uxl0Z+tPMbJdoMEGwJoIrLLhHPhQb1qy7q8a+JBnxY9e5mpkwcP5al3AwuG2?=
 =?us-ascii?Q?oVQdJFAzIBZ8obnf5JLIJ1bjHUq1WoRH9DcGcZkMl2ZVwAgkay/+SMjDfRJZ?=
 =?us-ascii?Q?UMZ+/BknDpyBethkvuDNuxI/0eZevEPUNZvqUDT1p+YARGwWspk0lmwT2HuH?=
 =?us-ascii?Q?b8PyanPHKVNkXF13GZ7tzRgB4ENWVeAGlrey5dc/jlZCHlhpl0LonpUOFaNO?=
 =?us-ascii?Q?3jol6RC002EtWSfMcQ62RewzfTjd70u7Cb93vCpiGrq2dmoFNJZvwEP3IOIY?=
 =?us-ascii?Q?TOz9Iu+IyqDWqLHn5ayfp/p+0XlucNNOhF/MPIbVfgQF0jkjaRq30zUqpwaa?=
 =?us-ascii?Q?Wq1ANp4RdLDwIRw4p/27Uc/dfOZ51zc3Ra/2FIfmpz8/AX4iaOdCfRJsgODf?=
 =?us-ascii?Q?9RUot12x+WKTJqZStr+O3+2yweY9nNwP4sp0jg/Ivx0zQL7pdGAXeKyt/J89?=
 =?us-ascii?Q?uSdnh3KMMgtuGBnA5VwnlaT7SVQYUykEZl9CXo6jvhd3RjCp3R6u+cWaLZwP?=
 =?us-ascii?Q?pZcIOYTWhVRT054do2ZObgAbwVgVpgmuz5RYGkGIo4GlgJLh/9/mHnC6E/UF?=
 =?us-ascii?Q?ykrB/8Iin9gNk708U5MJWPvLD0p2QlAd7A9ojxy0TAqnc6KY7+i7CiA+qXus?=
 =?us-ascii?Q?Ko9ft4chjclsMZf2gceDLJUiBBdfSI9vJBxk5OfGbrfAndKbdvM/D3LbImO2?=
 =?us-ascii?Q?7e4lPByz19EEyePOKy2vZ4s=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e81494fa-02a7-48ed-4d44-08db151d7452
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Feb 2023 21:40:45.4717
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: BbKGp9AYiWuvnu1eWidGTsdSnZW1PMnh/7A1GpYEYg0Bf24/+cbbA3CWQhuWkUpNVE65cM9ZvexHeMXvVLNzW25eKy6Fj8FHPlkfilDi88E=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR10MB6446
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Feb 22, 2023 at 12:38:46PM -0800, Colin Foster wrote:
> Add yaml documentation for TI OMAP devices for the Phytec PCM-049 SoM and
> the Phytec PCM-959 development kit.
> 
> Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
> ---
> 
> v4
>     * New patch, based on a WIP from Andrew Davis <afd@ti.com>
> 
> ---
>  .../devicetree/bindings/arm/ti,omap.yaml      | 29 +++++++++++++++++++
>  1 file changed, 29 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/ti,omap.yaml
> 
> diff --git a/Documentation/devicetree/bindings/arm/ti,omap.yaml b/Documentation/devicetree/bindings/arm/ti,omap.yaml
> new file mode 100644
> index 000000000000..9c0e08f74fee
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/ti,omap.yaml
> @@ -0,0 +1,29 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/ti,omap.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Texas Instruments OMAP SoC architecture
> +
> +maintainers:
> +  - Tony Lindgren <tony@atomide.com>
> +
> +description:
> +  Platforms based on Texas Instruments OMAP SoC architecture.
> +
> +properties:
> +  $nodename:
> +    const: '/'
> +  compatibnle:

Oops. Caught this typo too late. I can wait until after the merge window
or resend next week. There's probably other feedback.

> +    oneOf:
> +
> +      - description: TI OMAP 4460 SoC based platforms
> +        items:
> +          - enum:
> +              - phytec,pcm049   # Phytec OMAP 4460 SoM
> +              - phytec,pcm959   # Phytec development board for the pcm049
> +          - const: ti,omap4460
> +          - const: ti,omap4
> +
> +additionalProperties: true
> -- 
> 2.25.1
>