Re: [PATCH v1 0/2] iio: core: remove iio_validate_own_trigger() function

[Matti Vaittinen <mazziesaccount@gmail.com>]

On 9/21/24 23:07, Vasileios Amoiridis wrote:
> On Sat, Sep 21, 2024 at 12:23:39PM -0700, Lars-Peter Clausen wrote:
>> On 9/21/24 11:19, Vasileios Amoiridis wrote:
>>> The iio_validate_own_trigger() function was added in this commit [1] but it is
>>> the same with the below function called iio_trigger_validate_own_device(). The
>>> bodies of the functions can be found in [2], [3].
>>>
>>> [1]: https://lore.kernel.org/all/51cd3e3e74a6addf8d333f4a109fb9c5a11086ee.1683541225.git.mazziesaccount@gmail.com/
>>> [2]: https://elixir.bootlin.com/linux/v6.11/source/drivers/iio/industrialio-trigger.c#L732
>>> [3]: https://elixir.bootlin.com/linux/v6.11/source/drivers/iio/industrialio-trigger.c#L752
>>
>> The signature of the two functions are different, the order of the
>> parameters is switched. So you can't just swap them out for the
>> `validate_trigger` callback since the signature is not compatible. But maybe
>> you can update the implementation of one of the functions to calling the
>> other function.
>>
> 
> Hi Lars,
> 
> Hmm, I see what you mean. Still though, do you think that we could do some
> cleaning here? I can see 3 approaches:
> 
> 1) One of the 2 functions calls the other internally and nothing else has
> to change.

I would go with this. Changing the signatures to be the same would be 
(in my, not always humble enough, opinion) wrong. The different order of 
parameters reflects the different idea. One checks if device for trigger 
is the right one, the other checks if the trigger for the device is the 
right one. Thus, the order of parameters should be different.

Calling the same implementation internally is fine with me. Maybe 
Jonathan will share his opinion when recovers from all the plumbing in 
Vienna ;)

Yours,
	-- Matti

-- 
Matti Vaittinen
Linux kernel developer at ROHM Semiconductors
Oulu Finland

~~ When things go utterly wrong vim users can always type :help! ~~