Re: [PATCH] selftests: helpers: handle multi line in test_sysfs_prop_optional_list

[borissh1983]

On Thursday, 25 June 2026 15:52:37 Israel Daylight Time Boris Shtrasman wrote:
> Modify test_sysfs_prop_optional_list to allow multi line parameters, in
> order to comply with checkpatch output that limit line by 100 chars.
> 
> Allow use cases like:
> 
> test_sysfs_prop_optional_list \
>         property_name "value1","value2","val3","very_long_option"\
>         ,"extra_long_option" \
> 	,"even_more_data"
> 
> Signed-off-by: Boris Shtrasman <borissh1983@xxxxxxxxx>
> ---
>  tools/testing/selftests/power_supply/helpers.sh | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tools/testing/selftests/power_supply/helpers.sh b/tools/testing/selftests/power_supply/helpers.sh
> index 1ec90d7c9108..8535cb7ed094 100644
> --- a/tools/testing/selftests/power_supply/helpers.sh
> +++ b/tools/testing/selftests/power_supply/helpers.sh
> @@ -108,7 +108,7 @@ test_sysfs_prop_optional_range() {
>  
>  test_sysfs_prop_optional_list() {
>  	PROP=$1
> -	LIST=$2
> +	shift
>  
>  	TEST_NAME="$DEVNAME".sysfs."$PROP"
>  
> @@ -118,9 +118,9 @@ test_sysfs_prop_optional_list() {
>  	valid=0
>  
>  	OLDIFS=$IFS
> -	IFS=","
> -	for item in $LIST; do
> -		if [ "$DATA" = "$item" ]; then
> +	IFS=",IFS"
> +	for item in $*; do
> +		if [ "$item" ] && [ "$DATA" = "$item" ]; then
>  			valid=1
>  			break
>  		fi
> 

A new version of this patch now at 
https://lore.kernel.org/linux-pm/20260625160556.54830-1-borissh1983@xxxxxxxxx/