[PATCH v1 1/2] nvme/rc: Parse optional arguments in _nvme_connect_subsys()

Chaitanya Kulkarni chaitanyak at nvidia.com
Tue Mar 21 22:17:06 PDT 2023


On 3/21/23 00:20, Daniel Wagner wrote:
> On Tue, Mar 21, 2023 at 05:16:25AM +0000, Chaitanya Kulkarni wrote:
>>> +	set -- "${positional_args[@]}"
>>> +
>>>    	local trtype="$1"
>>>    	local subsysnqn="$2"
>>>    	local traddr="${3:-$def_traddr}"
>> can we please have all variable declarations at the start of the
>> function then add the actual code instead of adding in between
>> different variable declarations ??
> Sure, though the assigment can't be at the beginning, we need to
> go through the parser step first.


that's fine, if possible can you just create a helper for that ?




More information about the Linux-nvme mailing list