Re: [PATCH v4 0/6] reset: gpio: ASoC: shared GPIO resets

From: Krzysztof Kozlowski
Date: Wed Jan 24 2024 - 02:25:39 EST


On 23/01/2024 20:28, Chris Packham wrote:
>
> On 24/01/24 03:13, Krzysztof Kozlowski wrote:
>> Hi,
>>
>> Patch #2 (cpufreq: do not open-code of_phandle_args_equal()) and patch #4
>> (reset: Instantiate reset GPIO controller for shared reset-gpios) depend on OF
>> change (patch #1).
>>
>> Changes in v4
>> =============
>> 1. New patches:
>> of: add of_phandle_args_equal() helper
>> cpufreq: do not open-code of_phandle_args_equal()
>>
>> 2. reset-gpio.c:
>> - Drop unneeded comment (Bartosz), add Rb tag.
>> - Do not assign of_node.
>>
>> 3. reset/core.c:
>> - Implement most of Bartosz feedback (I responded to one which I did not
>> implement) and comments from Philipp.
>> - Expect either rcdev->of_args or rcdev->of_node.
>> - Drop __reset_gpios_args_match() and use common helper (Philipp).
>> - Move declarations of automatic-cleanup variables in
>> __reset_add_reset_gpio_lookup() to place of use (Bartosz).
>> - Separate gpio_device_get_label() and kstrdup() (Philipp).
>> - Correct doc for __reset_add_reset_gpio_device(), rewrite few comments.
>> - Drop unneeded "r" variable in __reset_find_rcdev() (Philipp).
>> - Drop of_phandle_args initialization in __of_reset_control_get (Philipp).
>> - Check if CONFIG_RESET_GPIO is enabled before trying to look up reset-gpios.
>>
>> 4. Drop Chris' patch: "i2c: muxes: pca954x: Allow sharing reset GPIO", because
>> discussion is on going.
>
> I actually think it would have been OK as-is with your latest change to
> return NULL when CONFIG_RESET_GPIO is not enabled. But I'm happy to
> submit it independently after this series lands. It'll give me a chance
> to do a bit more testing.

Yeah, I wasn't sure if the concerns are really resolved. I think you can
send your patch separately because there is no clear dependency, so also
no clear benefits of me taking your patch into this patchset.

Best regards,
Krzysztof