Received-SPF: pass (google.com: domain of linux-kernel+bounces-166434-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <a3720150-e920-48b2-85f4-8b99abdd274e@theobroma-systems.com>
Date: Thu, 2 May 2024 14:21:02 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 2/7] dt-bindings: i2c: mux: mule: add dt-bindings for mule
 i2c multiplexer
To: Rob Herring <robh@kernel.org>
Cc: Andi Shyti <andi.shyti@kernel.org>, linux-rockchip@lists.infradead.org,
 Wolfram Sang <wsa+renesas@sang-engineering.com>,
 Quentin Schulz <quentin.schulz@theobroma-systems.com>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
 Heiko Stuebner <heiko@sntech.de>, linux-i2c@vger.kernel.org,
 devicetree@vger.kernel.org, Peter Rosin <peda@axentia.se>,
 linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
 Conor Dooley <conor+dt@kernel.org>
References: <20240426-dev-mule-i2c-mux-v1-0-045a482f6ffb@theobroma-systems.com>
 <20240426-dev-mule-i2c-mux-v1-2-045a482f6ffb@theobroma-systems.com>
 <171415576010.2476476.2469508869846775606.robh@kernel.org>
Content-Language: en-US
From: Farouk Bouabid <farouk.bouabid@theobroma-systems.com>
In-Reply-To: <171415576010.2476476.2469508869846775606.robh@kernel.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?b0M1bU1jaElzc3hTUHFzMmg4VzFBS0JCeEVCb3BOMjdsNCtlR1dBMzFITUtT?=
 =?utf-8?B?WEJ3M2c1N25EVFVJNVIwQ0hBY0JEQWZ5N25wcUFmWHJFaTNvSUpxK2o1Y0tJ?=
 =?utf-8?B?SkRDeU1oczlkeHNjdCsyeHB4WjcwKzRiWlhyMUh4V2xEaDM3TTc3VVRGY3M4?=
 =?utf-8?B?alQwTjB6clNGTU5Ra3ZrWGc2WURJVFczNlA4S3dqVkl2Um93Z3FORmtFazg2?=
 =?utf-8?B?dkhqSlNxcjJzaGJFZG5ESlp6NW5TckFUcUJyZis4b0QyV2NNV252cFl1NHd5?=
 =?utf-8?B?N3RPeHdOUUtSRlkzdytRUFNoWWhWdERXTkt3U3hZMXh5MHNLakRURnNMenBq?=
 =?utf-8?B?NkdRckl4Mm0zOHA4bGJxMU9NVXd4QXgvQVRhWVNDaUpEMDU5cFNlRjI0Q2FY?=
 =?utf-8?B?R2dmQUQzeCsrTXA4OGg2eWdPZWdCNlFkTm5KRkFaQnpzS3lhRFBQZDNGUmxT?=
 =?utf-8?B?K1A4RllpUzAvRWJ1SnhwZ0w2UWlNLzVwWUlkSFIzaU44QWxJWGp2VGx6VlN2?=
 =?utf-8?B?UmZsdzFTK1hFWkJPMWpKbWpBTEVKZXlKUTVYVlA3UXBtR1dxVmdXMXZnMUlt?=
 =?utf-8?B?RXgyYmc2ai9uRWVBOWFBQThsQkw0K2l6MWRlU2VmNUV1aHFEdVQyQTY4UEo4?=
 =?utf-8?B?NnUrR0ZDcmY0SUQ2VGtURy91eG03MklrTHZ4WVdEZkRFdGl5SXlEZ3hBRnZX?=
 =?utf-8?B?V09QVW5UK1ZnQkdwdjcxYXZkVERPOUgxRzhkcGhqT1o5cDdLS0h1RTd5bk1K?=
 =?utf-8?B?ZTZkRTdUakZweGlIWlkybkR3Rno1MTBaMTdka3hGUlg5MUVqNzZ3WXc2NVdR?=
 =?utf-8?B?cEZsRDhyUDZ2Q3VseHF1RzIrMFI3TkNmQkZmYkYrdVl1TjBYaVlHUnNpMGNv?=
 =?utf-8?B?YU9TZDFZUkxOTUV5SEtZSjcvWi9RKzZSNm0wbDVBRENzUVBLcXU4Wm9mNnRS?=
 =?utf-8?B?OHowOWdCWXo1Z3BORzYzc2IreC9iNUxvTnVxdXE4d1h1MGhzOWVPeXdMNVR3?=
 =?utf-8?B?WDFCNzNGZ0hvRUF3RW5HOHhrTFF0YTYxTXhyYnpudk55cmYyOWIzeEYyaGw5?=
 =?utf-8?B?RXVtSkY1akZCUHNxOFZya000VEl5NU44MHdJMkhFMVpGRzdPZFRtejFKa1Fr?=
 =?utf-8?B?dlMwQWhPODV2NjViSEtxdG94akFNMGR6NUNjTW0yelNtS201NFY1Ky9LSk1S?=
 =?utf-8?B?a1hwWS9XWG41K0c2TCtYcm1YOUJoLzFGOXU0UXVZYXdwTnB4WjRRZnh0T292?=
 =?utf-8?B?UXZZZkRvMDRvVUlWMkhCd0JacVp6blRHSzA0L0gvVTdCZTNTYXNHVHpPUWww?=
 =?utf-8?B?WGFKOHZsNHdpcFQ5L1hzZDE0UGhsdFEyS3k2M2QzemRna21wY09hN0N0YVVy?=
 =?utf-8?B?NTFSR2s4SzRsekFSM215L1YzQi9EamZ4MUlReWl1alZRaSs1NlJIS0lwR1Vu?=
 =?utf-8?B?RzdOcVZoQ05Qc21NMDJsb3pVWEEwSGtMbG12OEczUVI0Qy8yY0pGTThadzFM?=
 =?utf-8?B?UjFIcWJMTXZod1pUVGZnTWNPTnNWWTk1MVRiaVVPQXpma2Z5a3g2N2h3ZTBk?=
 =?utf-8?B?RDEwanArQVlOM2ExK1Q4YmtDVDExUTc3QzVWMjNCajZVM0QwbVpNNmlYYW9m?=
 =?utf-8?B?aTIxRW5aelg4VXF0VG1oNnVHNlFld09pVFBrMHkxNk9yaS9Ec0VSb251dUk2?=
 =?utf-8?B?UXdDZjZJYjdCZFFtaVhkK1lBaXpsMisvcW96T09od3RlaE1TTWJXM1NsL3Ny?=
 =?utf-8?B?MWhpZ1FFakRqUmNWVENZU0hIUytKK0p4WlpDS05JbXMvb3hybTJyVTA4ajhu?=
 =?utf-8?B?RmJpR2JMQ2N5bWtmdURRM0tCMDI2enF1UlUzT1BxdFQ1N2dxZkxNQjk5Vmgz?=
 =?utf-8?B?M2hQRSszbUN2bFVtL0JPaUQwcjJ0cVZ5aXFDbURsRGdRMFBnbGFzajJtWWF3?=
 =?utf-8?B?QlcwSHdiKzRvS1FRZ1U2Z0hxSE5ueTRTdWxGYjJtQjRZTy8yRC9MTXpldUF3?=
 =?utf-8?B?TFdxT2RmdTNuRHlWeTRiZnNvWWdObmJ6bmhtNHRPRkY1WmU3MlJIaEdiYzA4?=
 =?utf-8?B?SkVjNGQyazViME1SNllOMWlvNjR5V0hHR3Vkc3JIekVJQ1l0bHF3UFdqVk9L?=
 =?utf-8?B?OFB2cG1CeGNnZjliWTY5bHRWUzZJQjVoOHFqOVRjaDRGNm1saDl3WUNNSmVY?=
 =?utf-8?B?TEE9PQ==?=
X-OriginatorOrg: theobroma-systems.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3720f838-f3a0-44e3-dc0b-08dc6aa25643
X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6382.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 May 2024 12:21:04.5258
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5e0e1b52-21b5-4e7b-83bb-514ec460677e
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: qq/7BQlpi+mcmRd+nVllhvnBElCmZMzk1JTEIxeDoPYRfnUSMkr/PXEFZNueicxNHTo1KbEBCxaEJJO/1DS4rOBPRx8qKG3IQnvvQi2sPas=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU4PR04MB10404

Hi Rob,

On 26.04.24 20:22, Rob Herring wrote:
> On Fri, 26 Apr 2024 18:49:33 +0200, Farouk Bouabid wrote:
>> This patch adds support for the Mule I2C multiplexer.
>>
>> Mule is an mcu that emulates a set of i2c devices which are reacheable
>> through an i2c-mux.
>>
>> The emulated devices share a single i2c address with the mux itself where
>> the requested register is what determines which logic is executed (mux or
>> device).
>>
>> Signed-off-by: Farouk Bouabid <farouk.bouabid@theobroma-systems.com>
>> ---
>>   .../devicetree/bindings/i2c/i2c-mux-mule.yaml      | 80 ++++++++++++++++++++++
>>   1 file changed, 80 insertions(+)
>>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/i2c/i2c-mux-mule.example.dtb: fan@18: '#cooling-cells' does not match any of the regexes: 'pinctrl-[0-9]+'


Currently ti,amc6821 uses trivial devices dt-bindings which does not 
support "#cooling-cells". We can fix this in a different patch though.


Best regards

Farouk