Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
Message-ID: <aa66b946-8d97-db50-bc3b-7f1694de2cda@amd.com>
Date:   Fri, 13 Oct 2023 16:36:06 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.15.1
Subject: Re: [PATCH 1/2] dt-bindings: w1: Add YAML DT Schema for AMD w1 master
 and MAINTAINERS entry
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Conor Dooley <conor@kernel.org>
Cc:     thomas.delev@amd.com, michal.simek@amd.com, robh+dt@kernel.org,
        conor+dt@kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, git@amd.com
References: <20231013093109.37165-1-kris.chaplin@amd.com>
 <20231013093109.37165-2-kris.chaplin@amd.com>
 <f864dd17-7848-4a83-bd8b-2093d11a153a@linaro.org>
 <20231013-january-caliber-2e7acbee15ec@spud>
 <c58fbddb-cda1-e0de-8ad9-e4701c2c73cb@amd.com>
 <6fa74ee3-a2ef-4d76-964f-51249f53cbaa@linaro.org>
From:   Kris Chaplin <kris.chaplin@amd.com>
In-Reply-To: <6fa74ee3-a2ef-4d76-964f-51249f53cbaa@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?akUvazd5RkdOMVI3TjBTdVlYWjRxT01OUlFRZEgyektaY3FNbEExK0lja1ZK?=
 =?utf-8?B?T0lXQ2VRRHR0c1hEUThadlhERWVPMzllQXZzMEVrcHNPeVRPM2F0cDNMUHUr?=
 =?utf-8?B?K3BQNFd3MVoxL0JySzNGTG82eFRtNlRJc0hiN1hSRmJBdzRLSzBhaE9hOWxz?=
 =?utf-8?B?LzVpT2swQm05M2hhNU5GbDV5UGFuODFDUjBUeGhHRHBKTVo3ZDZ5VlViRzYx?=
 =?utf-8?B?QnJRWFJESkxVbFd1aHVWSUsrZU9CM0NQK2RFTC90MFBodFo4T2U3YjZXa0Vt?=
 =?utf-8?B?N0xxUkFBZUNFR0dZY3VQemt3NjlzSjNLUWF0dlNyOTJaNVM5SERyZGlVeWha?=
 =?utf-8?B?ZmgrYlJIdzZyK3puSUdlUHh3UVBqSWNWb3piU0QvYlhLdkJsTDk4RnR0U2Nx?=
 =?utf-8?B?L2d2VTVmL0hjRnQyalpaYlVMTnprTUhPVDRkVmF6TFpwRXFnTCtZUlc3U0Iz?=
 =?utf-8?B?Zm1ra2diYXNnUVA3MEJUVzBFR1VjNlRrSFBseUp4UWtXNFc0R1ZMSDNsK2tZ?=
 =?utf-8?B?b2ZsN0tBem5BdVFMNHBydVI0bHB0VkpXVFFlYm8zZHQwemdsZmZOcUVYQ2Iv?=
 =?utf-8?B?VGFqbHVnYUJJWEdLV21oZktUV3Zyd1dyNFN4Ukk3bkFiZmwwNTd0ZHlLY2ZB?=
 =?utf-8?B?MEJtSWhBOGNtM3ZjdFNVVkZQdVpRVzlVZUwweTN3THMxeW8vK0xNSnYrNFM1?=
 =?utf-8?B?OVlVZXhxY3p0TWtrU2ZnSXk4VEtpeU9sZERXL29jOCtVUzkwZkZ2MGVEa2NK?=
 =?utf-8?B?clZOTlZBWkY2Z2JObzBMQWI0eU02SkJNaS9SelhaOE5lWTJHRmlnaTBXSWtU?=
 =?utf-8?B?TlJ1d3MvMndOWENiRG1KYlhZNlEva0RBczUyakNmbXJDK04zRlFpUm95MWlO?=
 =?utf-8?B?K202SWZid2doNzRsdWo3bHFJUTdaci9Od0RmamRsQVFJNUZJZWdhTkxNeHRj?=
 =?utf-8?B?OG5NTy9CT09oaGlkaWRYemwzVGcvdENYRW1zZGs5bSt5UUdua1BkZmNBYXZa?=
 =?utf-8?B?em9NbVJ1RjJ3V2ZJU3NFM0luTzJFOEM3NWJnVmQ4NXMvWXk3Wmg2d0kwd1ZE?=
 =?utf-8?B?dWplTHVTeHVSME4vdERQem5HcUJpWnc0UEZoT2ZHMitzN2dydFBvSytuMCs2?=
 =?utf-8?B?TEZWQ2NVT2FZNG45RE51dGVsYWN5K1l6WklsbC8xalJCMGlocE5xMER0bSt4?=
 =?utf-8?B?OEF1R0UzQzdvWW1WeFdTdUZKVDIyRHRWWGRFZ1YyMUZFemhVS0dmRG5GOWp6?=
 =?utf-8?B?Y2FzVU5Ud1RFYUZ2R2tTdWZBVnc2ZlBSNUZpQUVseWZodEZReTh2UWVXSzMv?=
 =?utf-8?B?UlFDS3FFSWF0cFd1T25tMnlXT0dRZzdPWkdDa2JXMGpxam54MGc1Q3dud0J3?=
 =?utf-8?B?bm84a3ZCRWNSUm82ZGRDNzNSTWFwclpzdDB4YnJqd2k1NjFxVjBkYWkza0ZW?=
 =?utf-8?B?MFY4UlZVYkU2OExCaitKV0g3cUZGL3JCQWQ5WE52NlZGejlVUCsxNllUUGpN?=
 =?utf-8?B?c3JXUGhWZnNFcGlNc1NOdzg5TDdiWmlseFRUTURqMkM2ZW4yWEFLWmYwQjFT?=
 =?utf-8?B?b1dqTFdPZHVaV3l3cXZGT0dEQTAzYVExT08zTFZaVHlTbzlvMjZDYnNrNU1w?=
 =?utf-8?B?c04wSEphMWt5eStwL1BZMkcySEM3dGtXMWllaVg0NGVvNzcraXVuRGJDSHBP?=
 =?utf-8?B?VE9jVDhRcktzazZUMXFtbSt6MVcyWjNiZDNEa0pjOHJXRnRpN2lDZ0p5ZGxS?=
 =?utf-8?B?djFkenJDWmYxOGN2b0dESDVneEQ0amV2dTdJc0pzeU9PbEMvVkpYZmgzMXNn?=
 =?utf-8?B?bkRYRDU5b05EU1N3UWhiUXdLdXVsVEZXdnBtcHloSkhNSEF3WkppZ05vdmRX?=
 =?utf-8?B?dm1YRXhMbzZCSEpCVndwMXlQY3BRZkFqejNIMzNKTXh0RU8xOTBGUS9kN3lx?=
 =?utf-8?B?ZEpyTUh5NnQ3dUROR0ZyMUtPNzNZLy9pQkJ6WTBsNnA0cVJ0cWdtVzJnNmRu?=
 =?utf-8?B?a0x4eFM3RnRYek9LbTJUakFMa1FSTldINzBEczhuVG9ka1JXc1plT2NmYVFZ?=
 =?utf-8?B?QlZqci84a0lMRHZSa1lNUU53a0hBSjd0aldHdTM0cmVWRGJkR2NYNkM1Rloy?=
 =?utf-8?Q?VwIBaK4y4UTPgjQICFb2es5cl?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6db9254c-7268-4cea-c1be-08dbcc022031
X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB3407.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Oct 2023 15:36:10.6177
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: YLJ95x7EYw657TxJcBL1yhLlFxC4Ojg13+xC095HStbZvD3z3TYTtM6wdERiTjUT
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR12MB8480
X-Spam-Status: No, score=-4.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Fri, 13 Oct 2023 08:36:29 -0700 (PDT)

On 13/10/2023 16:29, Krzysztof Kozlowski wrote:
>
> AMD product managers are highly skilled in naming things. Sigh.
>
> Go ahead with AXI 1-wire master. Any future - from now to next 100 years
> - product from AMD which will be different but sold under the same name,
> thus creating conflict in compatible naming, should be rejected because
> of that conflict or renamed to something else. If that happen I will
> propose a name like "banana-wire".
>
> Best regards,
> Krzysztof

To be fair on the product managers the IP naming from a hardware 
perspective is in the context of the ex-Xilinx development tools, and as 
such there is no risk of namespace overlap with the rest of the 
business.  I'm not against changing the binding name for clarity in the 
kernel. Would adding pl within the binding (Programmable Logic) assist? 
Ie: amd,pl-axi-1wire-master

Regards
Kris