[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c4c73918-edbc-8cea-329b-51be967b26ba@intel.com>
Date: Tue, 19 Jul 2022 13:40:08 +0200
From: Cezary Rojewski <cezary.rojewski@...el.com>
To: Andy Shevchenko <andy.shevchenko@...il.com>
CC: Péter Ujfalusi <peter.ujfalusi@...ux.intel.com>,
"Andy Shevchenko" <andy@...nel.org>,
Mark Brown <broonie@...nel.org>,
"ALSA Development Mailing List" <alsa-devel@...a-project.org>,
Takashi Iwai <tiwai@...e.com>,
Jaroslav Kysela <perex@...ex.cz>,
<amadeuszx.slawinski@...ux.intel.com>,
Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>,
Hans de Goede <hdegoede@...hat.com>,
Ranjani Sridharan <ranjani.sridharan@...ux.intel.com>,
"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>,
Liam Girdwood <lgirdwood@...il.com>,
"Kai Vehmanen" <kai.vehmanen@...ux.intel.com>,
Bard Liao <yung-chuan.liao@...ux.intel.com>
Subject: Re: [PATCH 1/2] lib/string_helpers: Introduce strsplit_u32()
On 2022-07-12 4:24 PM, Andy Shevchenko wrote:
> On Tue, Jul 12, 2022 at 03:51:04PM +0200, Cezary Rojewski wrote:
>> On 2022-07-09 10:42 PM, Andy Shevchenko wrote:
>
> ...
>
>> I still believe that casting blindly is not the way to go. I did explicitly
>> ask about int vs u32, not int vs unsigned int. Please note that these values
>> are later passed to the IPC handlers, and this changes the context a bit. If
>> hw expects u32, then u32 it shall be.
>
> What you can do is probably utilize _Generic() which will reduce the code base
> and allow to use the same template for different types.
Writing to let you know that the feedback is not ignored, just my TODO
queue is large. Will come back once _Generic() idea is verified on my
side or when I have more questions. Thanks once again for your input Andy!
Regards,
Czarek
Powered by blists - more mailing lists