Received-SPF: pass (google.com: domain of linux-kernel+bounces-169149-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <cf87f691-c2db-4a24-bf33-af26adffcafa@nvidia.com>
Date: Sun, 5 May 2024 13:57:47 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] selftests/alsa: fix a build warning: return a value in
 all cases
To: Mark Brown <broonie@kernel.org>
Cc: Shuah Khan <shuah@kernel.org>, Jaroslav Kysela <perex@perex.cz>,
 Takashi Iwai <tiwai@suse.com>, Ivan Orlov <ivan.orlov0322@gmail.com>,
 linux-sound@vger.kernel.org, Valentin Obst <kernel@valentinobst.de>,
 linux-kselftest@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
 llvm@lists.linux.dev
References: <20240504021330.33429-1-jhubbard@nvidia.com>
 <ZjedcAW6ITmo9pXp@finisterre.sirena.org.uk>
Content-Language: en-US
From: John Hubbard <jhubbard@nvidia.com>
In-Reply-To: <ZjedcAW6ITmo9pXp@finisterre.sirena.org.uk>
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?SGdJWGJJZ2k3ak0wTkQ0UWFENzVmdEdoTHo3bmIwaUdUeEZ1dWZ6R1FlRnRW?=
 =?utf-8?B?WGJjUXFyMkMyS3NaZGVXYVdlazMxMThRT2hUOFF3NXVtTXJIc2wzMTZIR0ZT?=
 =?utf-8?B?cDBLT1RtYWFkRks3ZUJlOUM1eGxna1FWRk93cXdVUHNmR1l3c1Y5ZFllNm96?=
 =?utf-8?B?NjJLbDYrVi9rQUVoVW54cmRjTXVNVWZOQjlZcDZKSGsrV3NUOWJsTkNscTI4?=
 =?utf-8?B?UHBMb1FsQklJRThGRFJwNUFsVmlUN1VIbzhsK3piMnRCYnpNVUZROHl2czRi?=
 =?utf-8?B?cFNFaTRZUmZNbVZiVXFYVFZJMEJkV09uYzZlT0hyamQzRmFFalVjTG8rdHJq?=
 =?utf-8?B?LzB2TitYbHYzYWREM1Z4b3QvS1Z4VkU4RHVBN05IbDNHMlNpc0VpK1FRcmJw?=
 =?utf-8?B?Qnh1b1UrQTZwRGI2dzh2RDB5WDdpY0ZuTlJ6cTQrajRBYnNTeGo2VUZ6VFdK?=
 =?utf-8?B?RzdzemI2aEprYk1MamxuYXZrNStxQTVubWlrSlA1cXdjNGxEYy9HYlFWR3VN?=
 =?utf-8?B?bEhNbmhIUXFhbS9pNmh1UWVSVEtrK1JXTmtWVHFNQXpjWjNLSzU0WUUySjdl?=
 =?utf-8?B?L0xJbjNSSXBsc0FNZTMrT3VTaElNK3E4ZkxmdklzN2ZGQ0pUSFV2RStidEYy?=
 =?utf-8?B?Vk1peWlmL1RncVZHakdYT0hwSVZkMHR5Sk05L1BLUDlRWW1WWWcwdlp1aWdz?=
 =?utf-8?B?eXVPYVRBZ2xkdGRGL3d2dlVjYy9CdGtmRjBCMWlzMzJFNWh0eWtIMUM0Z1RR?=
 =?utf-8?B?cTljalN5SG9ROGF0ZHU4R3RmRjlNOVAvNng1UTFkNW53b2QwOHQ1K1lPL1lw?=
 =?utf-8?B?MSt0dU9CM3NOdFhMa2s4Ym5YNk82K0FjTTBaRFZKa3AxSHZRbzRVSUg2aUkv?=
 =?utf-8?B?UC9vQnJLaGNtdmUxS3UrZ09SNDI3ZzZRdjZ1RGc3T1ZWSkc2S3Z2L3lxVGQ2?=
 =?utf-8?B?ZWFtUUo0RENJSnhWQlBXbllOdFdmcUs3MHBEeFJjekswaW5MYUpkM3lhT1c0?=
 =?utf-8?B?Si9TVzFoL1dHVHpRc1hJVXJ4OUlnQy9ZUk5rQmtuL0hMb203aXlHY21IRk9P?=
 =?utf-8?B?RjlnMktOcWYzYVdjS2htTC9YUEZwWms3ek9VYlI1V0hWM0RWTU1rUjdmZm5a?=
 =?utf-8?B?WkJWNFBOWkZSbUFqamFJNjkwU0FuelNLNG1tak5ZcUwxQU5HQjJkdmVKKzV4?=
 =?utf-8?B?QkhZaE94N3c0RDE2YmZGN050L1p0VTFVampRaGRSSzAyMTZHRHFmbUpEdHhB?=
 =?utf-8?B?WXdLNlVyRllkWVNBOGRVOU9FQlcxcFNINjRuajMwbUJTV1ZyYkN1MWdicGZo?=
 =?utf-8?B?M3FiZzJmbFE4bGlzYy9GdExDZFN1ODNjMlI4VHA5RFc3NUthb0pJSkp6QjMv?=
 =?utf-8?B?MHA4WXE0SzNXVXdvd3pCZDJWZDdjNWV4d3N0c1ZjUEdkVklBcjUvVlNQRXJZ?=
 =?utf-8?B?ZS9XbU52cDl0QnRxbXNOWkkzYS9OSVpxK1FwY0xpYTl2SGdiYzRJSlVabjEx?=
 =?utf-8?B?QktnQUVPVkRzY0prZU5Yd0xjOG04SUcxcG9KMHJWVWVPVTM1RXpJeVRKdnFZ?=
 =?utf-8?B?bWc1MTZQcUpoSktZdUI0djFGckJERm1qQmJaOU8vOExuVTM3VkcxZnJuRFM0?=
 =?utf-8?B?ajdpa0lXU2Y4S1poL094akhybit2azdVVmsweFIxdFU3NXQ0Qm5HRGt2elNq?=
 =?utf-8?B?ckNWNE5PZXF3dmRnYkVvMlJqUjAxN29SS1NvOTNTZUZ6dE1yOG54VlFQTUd2?=
 =?utf-8?B?d0pmYnpoNXc3WFlvLzVUYUpQSlBlYi9CL05JRmRLTXF5VHRacmZ1S095RHhi?=
 =?utf-8?B?VjI2aGxPTGNmYlpHOHgxR0J4b1VkNHB0YkE4eHE2VXpyOXVqY1RBODdReVpR?=
 =?utf-8?B?VE0wY1REUExpUjN4ckdqYWFvTTJtbWI1ZHNHZFFZQnIzbUE3MEhPVzd4andI?=
 =?utf-8?B?YlkrZGlmdklOMnhLMWtuSEpUQXo1NWViQ0JjSmxnelJoZGJ2RU9vZHZCMlFo?=
 =?utf-8?B?Ky9YbXJreXRUNk5TbEJrcUpsby9kNS9rQlcwaW9XN1d0Ujl3SXZoeFRLOXhl?=
 =?utf-8?B?VThzVi9rMW4wcU1DRmxpYzcvb0dmZWVzWUs0Ymd6SlhNaTR3eDBKYkZXSXhO?=
 =?utf-8?B?RTNEdlZDZVpPSlgzeEVIM0VRTjJoemFoejFIL21FMlZ4ZHNlUzlUYkFUbDNY?=
 =?utf-8?B?R1E9PQ==?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f3e8170e-fde8-4e5c-74b0-08dc6d460a9e
X-MS-Exchange-CrossTenant-AuthSource: BY5PR12MB4130.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 May 2024 20:57:57.4378
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 6V6pMa7FfcEAo5MiE+3wEjQv2sIUzBaDG4xHY4uQCgQ7XPZDpVRYZju1zCR68sMuRt6HHEyhWdL+YCttBDemtg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB7710

On 5/5/24 7:53 AM, Mark Brown wrote:
> On Fri, May 03, 2024 at 07:13:30PM -0700, John Hubbard wrote:
>> dump_config_tree() is declared to return an int, but the compiler cannot
>> prove that it always returns any value at all. This leads to a clang
>> warning, when building via:
> 
> Though given that the function isn't referenced we could also just
> delete it.

Even better, thanks for spotting that. I'll send a v2 to delete it.

thanks,
-- 
John Hubbard
NVIDIA