Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <b91576d1-a026-3c7e-955b-b36182cbc2c7@solid-run.com>
Date:   Sat, 17 Jun 2023 17:09:42 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.1
Subject: Re: [PATCH 2/3] dt-bindings: arm: Add SolidRun LX2162A SoM & Clearfog
 Board
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org
Cc:     Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Li Yang <leoyang.li@nxp.com>, Marek Vasut <marex@denx.de>,
        Fabio Estevam <festevam@denx.de>,
        Stefan Wahren <stefan.wahren@chargebyte.com>,
        Frieder Schrempf <frieder.schrempf@kontron.de>,
        Marcel Ziswiler <marcel.ziswiler@toradex.com>,
        Andreas Kemnade <andreas@kemnade.info>
References: <20230616110610.32173-1-josua@solid-run.com>
 <20230616110610.32173-3-josua@solid-run.com>
 <a9098664-ac16-eddb-3e2d-78eb08ac973f@linaro.org>
 <5139b72e-cccb-43b3-ba37-35f0011e4a2b@solid-run.com>
 <f6283bf7-2c38-8257-e430-d17dd4dd233b@linaro.org>
From:   Josua Mayer <josua@solid-run.com>
In-Reply-To: <f6283bf7-2c38-8257-e430-d17dd4dd233b@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RGFvQjNjMUs1Qm1xcnJuaElHT3RrMmVFSTFkYVpEMDdLcXVleUk0bVlxbXhW?=
 =?utf-8?B?MldEdmVnZis5a21lZ0V4OUMwR1JSYlVyeXdjWlltcnFRUUlvZ3RTQmIxYXlM?=
 =?utf-8?B?Szh0K01LZzF3VHFjbEZBcHR2cGJUclpuVXhSYytrNGcvNkNIZzhtZVZsTDVV?=
 =?utf-8?B?clFyWU0rY2Y4elZPbzM0dnNGb2lWYUtlOVBKWHRhQXRCNEIxTlZXckduNWlJ?=
 =?utf-8?B?QjN4VHJRZnFQVWFySXkzYXg1ZGJQcVd1aEFHSlpKdHlBMGhJSHBmZHBJUGhq?=
 =?utf-8?B?RHhDMStpbEpReVlkS1FOYnNUd0VjZmZqdUZUM0x3bzVzSDZta1piMVF0WDd3?=
 =?utf-8?B?NjBvVlpmb2luYmhKQ0JHcHNUYkJXM3A1YWh5cWNoUG9EeDZuZmVodmIyMEkw?=
 =?utf-8?B?SEhaOEN3UUV3YW8rV1c3MXRHWXRqL0FUZkY1d094MGNxZ24xcEVtblE0VTgw?=
 =?utf-8?B?dEY4NUR2VEQwbUxoOUNFMHF0N3g2QklOY3R6SzYxRmNtVmFmTzJobzU1TDF5?=
 =?utf-8?B?YW82d1VaY3JrYll0bGlFYzcyUyt6ei9MUVFha3ovZnk0aythL1p0SUVucHc0?=
 =?utf-8?B?SE9VRFV4dXY3L3JZUWhnM2hyVkZzQ2RxdVpZNnlucGlPVnJiWkJML05LRng4?=
 =?utf-8?B?T1BlUzJ2bnJLTGxtU2Y0SlNKQlA4M2pabm5OeWVUSm9oQkc4QUZQa2ptOG5t?=
 =?utf-8?B?ZE1JTjBNUURRK09UcXFrRHNWVC90QXpMYXptbmhpQXlnUGtEcm9UQytCRXIw?=
 =?utf-8?B?Ri9DMUZtcGRoNXU3ekhlYU9veGYxcHVScjhpY2xEWlFudUQvUlp5Z3k5dTln?=
 =?utf-8?B?SFA0bmMzUzBNeEhVOGtwbEhpTW9td0ZIUGhFQ1RpNFdMMUl0Z3pkQm1aTVNV?=
 =?utf-8?B?TzdwcGIyTkxHeVF5K29aemRyOGdsa3R0TWMxek11SnhsQmU1RWx4OVVqNEdk?=
 =?utf-8?B?VS8wa1FWcGZkVVprK2orUFNtdEZnZmVTbjR2ZDcvclpkNVpCbUtVekJmejVv?=
 =?utf-8?B?VHRxdE44NlFDRmVCd0FHYjd4OXpIZEcvUnkwYjVpN0FLQm5JYm05S1hFdHhi?=
 =?utf-8?B?ajdUb3FXRkE4NjVTdmpPVTNqNnJTWUZkc2dZeVZ2cDErV1JwKzkwQzhpaVg0?=
 =?utf-8?B?czNVL2dEdzBhV1FuN01PQnduc3Q0Q25MZU53bS94WmFWT2cwQmc3RmYzNW5U?=
 =?utf-8?B?Smo3bzVhQlZJc0lMaUhOTDlYemRlU0F0RVBQaExPeVF2OXBXU0hIUkFzRnh3?=
 =?utf-8?B?WEg1eXRpV2JiU2s0VmNHTU5STlBRZWtsL0wxcHJTQnN0RmJXakN4ajRWSkZS?=
 =?utf-8?B?V0hLUlR4aUZHWjlNRXNpbjQ1TWdPTExHUVNrMlZ2Um5kZ0lqMmVwZHExTE9D?=
 =?utf-8?B?cWFZSy91SU9KNVA4T2hRRlRDTncyYS84VDhUN1JMQjkzZHVvNXhIdHNqY3VT?=
 =?utf-8?B?SDJWNkxwazM3MEJOM2VSVzd6bVVuNTdZZnpmR1d4WFFiTnNHdkl0YXd1alI0?=
 =?utf-8?B?bFVrWk10b1lxZ1M5Mm5aVnRKNlhZdkp0S0lEVExvN2IrbkxjS0VHODk0cGdI?=
 =?utf-8?B?S203NUF5MzJCR2JrWEM1b1o3QStKME5vMWxCRXlzRVZ6Y0RudHhWcnlzMG5h?=
 =?utf-8?B?SysvVHJ6SWRmdmpIWEh0ZUNXTlBUWUwyeDRsOWUxYnVoQ2ZjMlA2ZGIwWHdP?=
 =?utf-8?B?b2dFUkVHakg3eTh2SThZQ3dZZHRKVHg3WFZRd0w0MEdVQjhEakJLbHBpaDVn?=
 =?utf-8?B?dGtzRHoyZk5rZWJxMGtMQUV0SERjRGlhNERxbUdtUlkvMnFOYm1lR3Y3SXhy?=
 =?utf-8?B?ZmZNYzZpUlhwYU9rOVZkdGx5VkFhVDR5Q29pU3k2WHo0MkdpSWhvRkgzSW1L?=
 =?utf-8?B?bTBnWEd5N1U0OXRDTEZlb2FtaXpQU0dUSWx1NmZqekRsTXRMRFlrYXcraVov?=
 =?utf-8?B?M1AzNE5RaXkrN1R4Y1pSelFCaXpFdDUxemg1Z1BPbTdtWjdqbHN3aDVZTGdJ?=
 =?utf-8?B?S1dGRlM2WThpMmVyMzVabFVzMDFKaXg5elhoNi85Z3F0Q1FLZXFveFo2bVVB?=
 =?utf-8?B?cXFWNGFMNzFwYlg3K21McjZjVHpNQ1dxNThxYzJuOEFjdVJ2YjBqVlk5aUFD?=
 =?utf-8?Q?+tKuV637dzmb4QInIJTmzc3tQ?=
X-OriginatorOrg: solid-run.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3b1260d2-6e10-4de6-50ea-08db6f3c8208
X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB8963.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jun 2023 14:09:47.4485
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: a4a8aaf3-fd27-4e27-add2-604707ce5b82
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: UySjmfJAjeFy8+QFF1oZeNyAi5eTAIxXYElm/tqzrJn4Ag7V/ISbuoo+dUaBLhQfcRch0MjOJz6hlCE2zy3ivw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM8PR04MB7250
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,
        SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Krzysztof,

Am 16.06.23 um 19:59 schrieb Krzysztof Kozlowski:
> Test the binding and test DTS against it:
> Please run `make dtbs_check` (see
> Documentation/devicetree/bindings/writing-schema.rst for instructions).
> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
This was helpful! I ran it a few times and studied the results.
Many complaints related to lx2160a.dtsi, some common among different 
layerscape based dts.
Finally some mistakes of my own.

v2 addresses all that were under my control, and a common one that I 
understood well enough.

> It might point you to answer.
>
> Why do you make solidrun,honeycomb compatible with cex7 and som?

Exactly - unintentional.


> Best regards,
> Krzysztof
- Josua Mayer