Received-SPF: pass (google.com: domain of linux-kernel+bounces-46308-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <471d9438-e2c0-4881-8ace-778c9d14669c@amd.com>
Date: Wed, 31 Jan 2024 11:03:25 +0000
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 2/3] dt-bindings: fpga: xlnx,fpga-slave-selectmap: add DT
 schema
Content-Language: en-GB
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
 Charles Perry <charles.perry@savoirfairelinux.com>, mdf@kernel.org,
 "michal.simek@amd.com" <michal.simek@amd.com>
Cc: hao.wu@intel.com, yilun.xu@intel.com, trix@redhat.com,
 krzysztof.kozlowski+dt@linaro.org, bcody@markem-imaje.com,
 avandiver@markem-imaje.com, linux-fpga@vger.kernel.org,
 devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20240129225602.3832449-1-charles.perry@savoirfairelinux.com>
 <20240129225602.3832449-2-charles.perry@savoirfairelinux.com>
 <9b0680b6-1952-41d3-82f4-88c60469dc3a@linaro.org>
From: Kris Chaplin <kris.chaplin@amd.com>
In-Reply-To: <9b0680b6-1952-41d3-82f4-88c60469dc3a@linaro.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?TUlPWDduSEV1WmpMbjhOZlZmcnBrZ1N0Lzg5WEhEL3c1alA5VHpWSWdtNzU2?=
 =?utf-8?B?akVyemFKYWJ6dFpzNjJoQVFsVjBDN0VWN0tXK1J1VmxLM2hqVVpTV3UxczNV?=
 =?utf-8?B?MndrNHBoTkE2N0VYQktnWjMxMXZTVXhFQ3B5bW13bWpPNkhwSklNOGQyMFN3?=
 =?utf-8?B?R1dKOUc1aS8xMFE0SHhMUktWdFgzeXB5dUhwTzR5OENudjRuU1B6S1pDaE0w?=
 =?utf-8?B?MUtieS9KdkdxUmM0L244SDlOaUtDdW5ZTStlU1p2K1dUaDQwak84aXJhTFFk?=
 =?utf-8?B?djc2TForVE5JZ0d1ZmoyTW9ZMmlaVDlsbmNHYTA5Z0RyNDRFeUMxc0ZjMElW?=
 =?utf-8?B?d1N3UzgwRllaU2FROVVqTFVTY1BOemxxbXNoT0U0aDYrVkFQYUdudEROc0kv?=
 =?utf-8?B?bjJnSW04dGJRRUo5VWpWU29TV0VYYzgvdUtwazA1blEvcUdwQlZKZEhZQ1R4?=
 =?utf-8?B?YVdiYjdWUVk0eGdKRDFKWE5xK2EzTFh2TFNGd3Y4V0daM3RZWWI3Wmgwekhp?=
 =?utf-8?B?QnNSR05RM3hIbzhYUy9MOTB4UlF4bjgwYy80RDlTZVdOeUxQeXgyQnRHSTE2?=
 =?utf-8?B?RlR5azhxL2lRNWpyOWxMRllvM0d5MU84QkwwbGNncHZMNlZQTmhhdTVVM1NV?=
 =?utf-8?B?eVhseGlzbjBZWUZTZXZjS2VHSEJVWlVNYTkrWGZGMFAydVdsU3pjVC9mMUtJ?=
 =?utf-8?B?NnFQQjQweFlhWklzRnZjMlcrM09YbzB6RHd3NktXeTVKa3phaStUcVlvRUxZ?=
 =?utf-8?B?K213cVY4OUdlcjBBZC9EZ09FTmRrbG83ZC91OXd4RW5FU29hUktlTmJQbW1z?=
 =?utf-8?B?UUZiaE5QQW53L2pqVHprajVJemtHak8vckVtU2EzbGxUeVowaCttM2lEVW5R?=
 =?utf-8?B?dTNVMmdIam5IRm52cGhxVG9OeVRzM055eXRNUWtqUXh6UTVUSEVVSDJlRDFC?=
 =?utf-8?B?V2xqNVhCbjhGb2F1WnYzdHByZUFOYi81dXlTKzJwWmtMenhNSUZyK0VsQ1h1?=
 =?utf-8?B?Y09RRnRub3VkNjdRQjJKQTZmOGhZOWJMbzI4R0FLS0UwRTFCeEszMGc1elBW?=
 =?utf-8?B?Ylg0Tjk3NFBQTFNUNVkxTDdLdGlKUm95aml0RmNxMTR5RDRvTXExcHd0bndU?=
 =?utf-8?B?bktjMEVjN3hZM0QwQzM4NHovVm45bE8zamtMRFZkYjh4cnBnZWJpTjd5R1c0?=
 =?utf-8?B?L3lrZjdKVWN2bEJ3RmcvVWsrTitUVFM5bzREK0dveUxhWG5tMGdMUEFES3M0?=
 =?utf-8?B?T1ZjMEVEVU9Wa2JCb3dCaUF6WXRweVZXa3N6RWlKcE05VGliSkYxUE5aMUx3?=
 =?utf-8?B?dmRDaGNlSEJBRmtja2NETm5nY0tESDFhdFhmR3pUNVpRb0RhajZnNjVyY3ZK?=
 =?utf-8?B?QmdNc2NHMFZYK3NESkNVT09oU3VNZEV1SkVpTE5aV0lKUEdPYVZFc1pZTDQ1?=
 =?utf-8?B?YlFESzVNeWdxWVEwQzM1K0hqeUVyMktqcHJEUitJb3VCL2QxMzBBQlJaWnIw?=
 =?utf-8?B?cVRhQjFFUFdtZXZJR3IwYjFUK285aHN5RnMrd0xHeWNyT3F0TnZlMldSRkpw?=
 =?utf-8?B?aVVXRTJvYzlGTXFhYityZ2N3ekdRcnVQaGVIYUVkaUx1YU83cXl4c1NGN3Jx?=
 =?utf-8?B?WnNmRWFnQTFwbVphdlBQOXI5Qk8vQ0xUa2VRY2ttb2xwVmswRE0xVGtrWThE?=
 =?utf-8?B?THNycjZXOC9zeExzSW4zcGZrNU9NR2M3ajd5Z1RMTU54UnJuaVQ5clltcEh6?=
 =?utf-8?B?RVJmUUphdnlyT0NtbDNJVWp6OVpmanlNcVZodGZqVTlONXRXeGNWU04vVEJJ?=
 =?utf-8?B?SGh1U3ZOQXVMRFArUkRJSFFaMElIZkw3L0dISFk3cjlaemNXUVhsYzR5SFhK?=
 =?utf-8?B?b0htek5PcDhOWE9HLzloNVQ1MDc3YVVwV3I0UWY1elhKOThJR1lpY092M2tF?=
 =?utf-8?B?N1BCOUxDNm5NUHVJRkZKRkRpUkNUT000UTJydTQ2SFhtak1DSzh0czZUOVFY?=
 =?utf-8?B?WFNNK1hvVkRIVzB0bHpwbzgxSUFPQkRJS05jYU90WUJoNGlXeU1IaFU3d0dB?=
 =?utf-8?B?QjJHQkFTZzJPbGFkWVVuUTRjN0dtbEFGZ2oyaUZUakdFd1RTU0pjVFhnRjBT?=
 =?utf-8?Q?B0lY=3D?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: dfae9429-972f-48ef-017f-08dc224c75e5
X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB3407.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Jan 2024 11:04:57.2225
 (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: NT28Ey1ibUsQx3XBjidIU6xi0IllIWsaHu7oTuGlpe5HnLciQFbbU9PoOku++b7Z
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CYYPR12MB9015

Hello Krzysztof,

On 30/01/2024 16:09, Krzysztof Kozlowski wrote:

>> +
>> +description: |
>> +  Xilinx 7 Series FPGAs support a method of loading the bitstream over a
>> +  parallel port named the slave SelectMAP interface in the documentation. Only
>> +  the x8 mode is supported where data is loaded at one byte per rising edge of
>> +  the clock, with the MSB of each byte presented to the D0 pin.
>> +
>> +  Datasheets:
>> +    https://www.xilinx.com/support/documentation/user_guides/ug470_7Series_Config.pdf
> 
> I am surprised that AMD/Xilinx still did not update the document to
> modern naming (slave->secondary).

Thank you for bringing this up.

We are moving away from using non-inclusive technical terminology and 
are removing non-inclusive language from our products and related 
collateral.  You will for some time find examples of non-inclusive 
language, especially in our older products as we work to make these 
changes and align with industry standards.  For new IP we're ensuring 
that we switch and stick to inclusive terminology, as you may have seen 
with my recent w1 driver submission.

SelectMAP is a decades-old interface and as such it is unlikely that we 
will update this in all documentation dating back this time.  I shall 
however look to understand what is planned here for active documentation 
and new driver submissions.

regards
Kris