Received-SPF: pass (google.com: domain of linux-kernel+bounces-7512-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <a9b5c452-6131-4c73-a972-566551df381b@intel.com>
Date: Wed, 20 Dec 2023 14:24:55 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH V2 06/12 net-next] qca_spi: Add QCASPI prefix to ring
 defines
Content-Language: en-US
To: Stefan Wahren <wahrenst@gmx.net>, "David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>, "Paolo
 Abeni" <pabeni@redhat.com>
CC: <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>
References: <20231218232639.33327-1-wahrenst@gmx.net>
 <20231218232639.33327-7-wahrenst@gmx.net>
From: Jacob Keller <jacob.e.keller@intel.com>
In-Reply-To: <20231218232639.33327-7-wahrenst@gmx.net>
Content-Type: text/plain; charset="UTF-8"
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?ZG80WUFiSTk1ZHV5TG0wQlZqQVI3Zk51WE5NWHNyTUtONWhpL0pJVklEdjNh?=
 =?utf-8?B?NUp4anlCRk43UXdkNG1VWjBnVFpRcVkvVTh4N3RQcXJtWlVGS2d0bVhaa1F6?=
 =?utf-8?B?UDQ1alltdy8rODgycURRT01EcitxTVRoWTczTlhVSTFkNTVWUTFKaXZZcVJI?=
 =?utf-8?B?bUsyVjV3WW1Ebm05TnZXcndrbWpnb2R5bmxYTW8xRHVZVDRObTJtR3ZDOFJR?=
 =?utf-8?B?Ny9FZXdad2NoOWI5SjVqVjlkQ1lpbU9KTGs4RGgwWTBEOURzMUNQNzlmeXZ2?=
 =?utf-8?B?VlNhd1FscmkzZ3BwanZZWGk0QzVUcFlmOHRNbVVNRTZZRFZZREFVNDNSWW8w?=
 =?utf-8?B?MVQwN2FyRklBSlRVejgzaWxHdk4raUV1dldYWVFtSVVqTXZwZ0RyZXg2R2Nn?=
 =?utf-8?B?MzlSc1QzSEdxMXFlTnFwZjBrdVpRRDAyOHJCU0RrYmg1QWNMclIzZW56akVt?=
 =?utf-8?B?ZUowMGx3Z3B4bEtUVy85WDhRWmZpVXlyWm8wUzdTNmdNcVF0SzlGd0FIQWZT?=
 =?utf-8?B?RzY1bjY1V3ZySVI5bnFlOEhMbWk5QzJmRFNvNlpseXU5K1FreXhNcXl5bUsv?=
 =?utf-8?B?K3NHZnAwR2FaekpVU0I0M1FHV3VaQlZWbXZFVUxmb0tEalJRVDUzM20yUGI5?=
 =?utf-8?B?VHRhQ1pHck53MGhBOU9RMEp1dVhjQUZqeW5hcTE0ZEM0cHlnSGhYa2tvOFpz?=
 =?utf-8?B?dUdkdU1MMmFjOERCbUZaVCtVaDlQVXVnTDRQSzlTeXNya1lGN1h2SUtZdDlX?=
 =?utf-8?B?VFJVQW9TdjZKY1NIeXN1eENWREVNL2tpYnplNldYdm1udk9aeldmMmlVdThJ?=
 =?utf-8?B?V25rWTNoSFM5SWQrcCsxZkNJU1NyamlTeUUvQzNXa2pydmxzM0xvaXRpTmVn?=
 =?utf-8?B?a3ZuVndmL2lZeHJLYWFXM29xN2NFNzVaRE9HZkt2K25vYjQ4ZjNRVC9obytW?=
 =?utf-8?B?dFdiaFUxeVJac0RXSEJaMElOOWdQYWhiSVFnS29tTGQrdXR3MndFWTFpNThW?=
 =?utf-8?B?TnBlMUY0SzByc01FVDZVRExiV3h6bldxQUtwbzg3ZWVxOWR4V3ZiSjdZTTVk?=
 =?utf-8?B?M3ZRL2tGV3dvUlIrYitIY2xkUkVWTWV5SkdoZGl0SElCakxXYTk4ZkhGSGJw?=
 =?utf-8?B?RXFvMksrY1pMc2xrTXlyZ1BVWDdadm1hSjJPc2Zvbi9ZeU53bW52bGlxVW9l?=
 =?utf-8?B?ZVR2Ky8xM3FhRDM4ZlUzNnRRTVphY1oxWU8wa21ZdjhoUmV1cmhneVdTc2Rv?=
 =?utf-8?B?cUNJY1VIb2c1UGEwU3hCZXFCcnNkeTl4ZncwSE1EazgrZGZDZVo1Mk0zL3c0?=
 =?utf-8?B?dm1tL1NVL3krT3ZtK2F5MzVoWnFtZDJEY1V0TUw1QnFCUEkxSmJSdTUyeFNI?=
 =?utf-8?B?bFgxSmt4bkNvVUkrSFJhQWtMNFIvQW1pVUdycnNGSVhGRWpGc1ViWUtaZnpE?=
 =?utf-8?B?REovbWJ6Z01scWtHNGViYkcrcGkvc01FZ05oVFlpWVhoUERObDluUVIyTkFJ?=
 =?utf-8?B?VjBHY2NKQXlqcnluZVplNjR0aWxQVWt3NVBwM3JKZmo0UTc2UkRLZ0R6b0Vl?=
 =?utf-8?B?Rm95UjVJNk8vWEZPc0ZnUVBpQXBVcjFNekF0RUl6WVJKOFNVcHg3MkkzVFQv?=
 =?utf-8?B?YncwcUEwUWVEbUdnakxTWTdxN0E0OVN1UkZCZGxLa1lYR3B3dndBU1ZLc2lW?=
 =?utf-8?B?ckNqVVRSb1VkRVNrTkx5THdYNVZGajRZaEtpdThsUXp6NG11OVN3YWhaelRa?=
 =?utf-8?B?ZUFxbUZkcm1VVEJqcTBNd0Fta1JhTXpEeHZkNlNXc1huZ3hiRGJXZittWHhE?=
 =?utf-8?B?UisvNDdrK29VL2xFTjZSNVlQL0xmRUplc0RPSUVEbDQzRDJSZjZTVVRXVGtj?=
 =?utf-8?B?Y2VubENncU5qL0JBV0pkYmZyYkVUVCtrbmswVTk1NVFscGZacTJXcW9lMGg5?=
 =?utf-8?B?cGwvYXc0QlVOdjZTalpNM1hqbE9jbVIzY1VOQy9lMlBzREJnbU9GVFhnc3Iw?=
 =?utf-8?B?VnBhY1hkdTRacjNQSkhSWlhQS3hBb2JjcUVVaGJ0SVZTQ0tIbCtVREZJUWVv?=
 =?utf-8?B?ZTVBSG5IUnRLcGJrTGN1c1J0MGszL1BLbjV1eWRhVy8vTmVvN3VIY2JIZGRD?=
 =?utf-8?B?MTZKNmo3WW9RS1l4QlNTc2ZjN3BPRlcrNjZ6L2JqZkFocUdUYWFzcXBhSDVt?=
 =?utf-8?B?Q0E9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 4f4ccd45-7bca-46fc-853c-08dc01aa7e72
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5095.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Dec 2023 22:24:55.9233
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: nBSgK6xjub0phuxsy9bdgCOcDyhIL2EMtNIMvKcnTuBOo+dEuTKXkihAH6fL2mWi4lQfGNzL6Fg0JqVTnp2cMO/aGB8CRcrxaFoDpDoQDYw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB4926
X-OriginatorOrg: intel.com


On 12/18/2023 3:26 PM, Stefan Wahren wrote:
> All defines in qca_spi.h except of the two ring limit defines have
> a QCASPI prefix. Since the name is quite generic add the QCASPI prefix
> to avoid possible name conflicts.
> 
> Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
>

Makes sense. This is in the header too, not just a single .c file.

Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>