[PATCH 2/3] pinctrl: get rid of some ifdeffery

[Marek Vasut]

On 1/18/23 13:12, Michael Walle wrote:

[...]

> @@ -411,12 +405,11 @@ static int __maybe_unused pinctrl_post_bind(struct udevice *dev)
>   	}
>   
>   	/*
> -	 * If set_state callback is set, we assume this pinctrl driver is the
> -	 * full implementation.  In this case, its child nodes should be bound
> -	 * so that peripheral devices can easily search in parent devices
> -	 * during later DT-parsing.
> +	 * If the pinctrl driver has the full implementation, its child nodes
> +	 * should be bound so that peripheral devices can easily search in
> +	 * parent devices during later DT-parsing.
>   	 */
> -	if (ops->set_state)
> +	if (CONFIG_IS_ENABLED(PINCTRL_FULL))
>   		return pinconfig_post_bind(dev);

Is it correct to drop the ops->set_state non-NULL check ?