Received-SPF: pass (google.com: domain of linux-kernel+bounces-158399-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <ab5cfd8c-0e88-4194-a77e-5ffbb6890319@amperemail.onmicrosoft.com>
Date: Thu, 25 Apr 2024 17:33:38 +0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 3/3] dt-bindings: hwmon: max31790: Add
 maxim,pwmout-pin-as-tach-input property
To: Conor Dooley <conor@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
 Chanh Nguyen <chanh@os.amperecomputing.com>, Jean Delvare
 <jdelvare@suse.com>, Guenter Roeck <linux@roeck-us.net>,
 Rob Herring <robh+dt@kernel.org>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
 Conor Dooley <conor+dt@kernel.org>, Justin Ledford
 <justinledford@google.com>, devicetree@vger.kernel.org,
 linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org,
 OpenBMC Maillist <openbmc@lists.ozlabs.org>,
 Open Source Submission <patches@amperecomputing.com>,
 Phong Vo <phong@os.amperecomputing.com>,
 Thang Nguyen <thang@os.amperecomputing.com>,
 Quan Nguyen <quan@os.amperecomputing.com>
References: <20240414042246.8681-1-chanh@os.amperecomputing.com>
 <20240414042246.8681-4-chanh@os.amperecomputing.com>
 <13b195e6-cbbd-4f74-a6fa-d874cb4aaa45@linaro.org>
 <065243cc-09cf-4087-8842-bd4394fb324f@amperemail.onmicrosoft.com>
 <d549cf2b-a7fa-4644-8fcb-3c420503ee01@amperemail.onmicrosoft.com>
 <20240423-gallantly-slurp-24adbfbd6f09@spud>
Content-Language: en-US
From: Chanh Nguyen <chanh@amperemail.onmicrosoft.com>
In-Reply-To: <20240423-gallantly-slurp-24adbfbd6f09@spud>
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?N3diR0RRWU05blFHY05VS01uK2EwTjNNcTFsa0Nublp6ZGJCTnpCRUg0b0JJ?=
 =?utf-8?B?MWFpeXVFQ1hLMitkaWpFM001Y0FZbmRIL0hubnpBQlFUazlRbG91eG1BUjZI?=
 =?utf-8?B?eXE5ZjAzTDQ1azlsa3FyTnVGN0VxSUlVclRpN3RtdGxqRHpNYVVHVGxyV2pE?=
 =?utf-8?B?UStQQnlNeUcyT2tmelpiMnBmbDNCeU9sWVBJdXp4WXRzUVluVjRrYUJWRkhL?=
 =?utf-8?B?anhJN21MbkhqUlBSeE5lMUxaSnc0ZXl5WUxRMU5hd0wzVm1hRlpIRk5LT09S?=
 =?utf-8?B?UHRaVHpqcjgweEprZTRmRlFOcTNVWU5zZzh2SW55K2dqSG5RdEYxb21OZXVF?=
 =?utf-8?B?YlluNmxybTVVd1laNDRrTHdLNkVLK241VEdHc29EbEtzLytML0JpZjFsSEN0?=
 =?utf-8?B?MDYvTndMa0lOWU9sQzZzOWViMmZkSGlFb1RDRDFMVVVPN01DUHBJMll0cDhC?=
 =?utf-8?B?MVpaSzh4MjFzdnVyb0wxSmlNQ2QzaG5hbnYrZVh6SWlvQy9oMmt1NnhCMjNs?=
 =?utf-8?B?WFJzbnFJNG0rS2tXbGgvczZNcFRKbnNIaEIzVVBLNU90VUFSd1R2N2N5cXdO?=
 =?utf-8?B?TGtNSWFQRGpiVGRIKzZkcmVQdUlZbUlZYVNTZms0bzVGbDJSemUxeGJTQzNv?=
 =?utf-8?B?eGp4TkZYRFp0TkVITGI4ZndPdjdwUnNvMkZmT09wK3U2MHAzcnBLOHVUVFNr?=
 =?utf-8?B?VXdxYTFWUEkwWXJ6V251RXhtSHdpQ214N3ZrYUFISVRJemI3UjR2UmVwK3RJ?=
 =?utf-8?B?Y0U3MlFQWXNLM1U5eElqb3lxaHhrSVFPQVlDbXphYTc2M3VFR2xOK3pTT0Q2?=
 =?utf-8?B?TElIZ2w3OVlndmpMQXBZWGIxU0N0SWI5U1hNZ2dQOWVCeGlxODBBZ2NzQWhq?=
 =?utf-8?B?M3FOMFRTU2RvUnBzdUJQVzYyWDNBT1hLczAxb0dzQjNpMlEzbmlaSklhYXRw?=
 =?utf-8?B?WksyR1NaL0FoNzI0WlJKaVd2WHdVOU9YQVpSdEl1YVpza1E2WWthZkZJc1Fu?=
 =?utf-8?B?TDdKaUtaMDJPaXRtYnFQSXJ4eU9jMmxNVXhpblF1bExpOU51QkVLU21ST0RV?=
 =?utf-8?B?a1lHakFmREM5RFQzZ1BYR2hRbmFnLyt5a25pSStYRnNoRkNZT2RiYWorSHFi?=
 =?utf-8?B?NUxsWktUdWNIeThlV0swcWxwQlVMN1Z4U1pYclJaNVV3N1Q5blk5aE9rNmIx?=
 =?utf-8?B?ZVNWdGpzdkU1K0R3SmVGSDhvUk9TandZQ1F5NWltYmdoRVE2emJNc3RlT1Nm?=
 =?utf-8?B?eW1XNG9ZRmJuRXVuYktWbGFLZzllSzNjcHpIWmdEcW1ZVVlCMjNlMGkzMEI4?=
 =?utf-8?B?V2ZxTmpVaW5BQ2NjWGFINzZTdHYrTGk2eVprMGNiWVN4aEpIeFp0MHhINnho?=
 =?utf-8?B?cTQ1bi82WXd4cklxejdjRVBIR0RRMDgvREVFMUJGZjVoOUd0N3VYNXc3b2M4?=
 =?utf-8?B?MVZ0aHl5MkV1ZlhtcFIwNlNXZDBPTFAzQTlta3YxdVI4VHZOelk2YTd2c21k?=
 =?utf-8?B?WVA0cEM1VnZKOW5rbU1PWkhOdHhZck5HOXFvanFHM0pYeWpFeWQwNU1ZOGRE?=
 =?utf-8?B?OTFvcGlwRzgzZ251a0trL21XYUZhTHRhS2tzYThHdTE3ai9hb3VCaHh5TldK?=
 =?utf-8?B?N1N0S1ZVTzJlZUsrOXpWTFhpZ09oNFJVRWVvNmFGOWlwSmVQUk4wN1FidWxC?=
 =?utf-8?B?ZkxKZEpPUXozV3d0VVhNUkNuQ1RyNHNwWnhJUmJ5Yk9Ub2Y1aWtJOERpdW1E?=
 =?utf-8?B?MXBSaHMzNWhnTUdNZGVWRW5sbkdzemxQdGtPWWxTZ3hMSythbTNZeDZ5aVdj?=
 =?utf-8?B?MEQ4MldFeVFzQkRUczlFUXlES1lEbXBiODllV2hXQVI2ak9TdVdSeE5PMDdm?=
 =?utf-8?B?aHlsSGt1djNlaTR1SDY3c05uWlhHSEhSVUFIVk9aUXovdzhuU1MwckI2U2RR?=
 =?utf-8?B?RmREWnpSZHFnS0liMVJrMVZUejFqSFZmcUgvU3Nsb1dPcnpuT2FIZ0huSE5Y?=
 =?utf-8?B?dzhVYXZqd1J0ZDZDRGZHTjFyYTNqRzg5WUtrOWxoMExJT1RqWTRWNmR2alow?=
 =?utf-8?B?SHh5azFCZkJwYndTcEFsd3FFTnZiZllPeTZaeVROWjZ2VllxUkZxcXdNdHZF?=
 =?utf-8?B?TXA5QXNSa3ArRmNVbmVuam5LWmtjQkk0MElnT2xrbDgyT0JJYjlPc09IcjFr?=
 =?utf-8?Q?8TWvXP1vUOcCazY3CBnag5w=3D?=
X-OriginatorOrg: amperemail.onmicrosoft.com
X-MS-Exchange-CrossTenant-Network-Message-Id: db8759cf-795e-497d-f799-08dc651332b6
X-MS-Exchange-CrossTenant-AuthSource: DM6PR01MB5947.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Apr 2024 10:33:51.4939
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: kxPyNfo7cMTxnl1SNdzS2QJnb5S7DQ2c0MxhE2ZPP6VwT50GZl+pMfEEyqm5e2d/pDucTJT9wN3fGaVWtZPVEclWlB5XbD4MKDuze8ildkUYrB8sGFr9gAAUpCFL02+l
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV2PR01MB7600


On 24/04/2024 00:02, Conor Dooley wrote:
> [EXTERNAL EMAIL NOTICE: This email originated from an external sender. Please be mindful of safe email handling and proprietary information protection practices.]
> 

Sorry Conor, there may be confusion here. I mean the mapping of the PWM 
output to the TACH input, which is on the MAX31790, and it is not sure a 
common feature on all fan controllers.

I would like to open a discussion about whether we should use the 
tach-ch property on the fan-common.yaml

I'm looking forward to hearing comments from everyone. For me, both 
tach-ch and vendor property are good.

Thank you, Conor!