Re: [libgpiod][PATCH v2 2/2] bindings: python: fix line request by name with multiple entries

[Kent Gibson <warthog618@gmail.com>]

On Thu, Jun 27, 2024 at 11:31:03PM +0800, Chuang Zhu wrote:
> From: Chuang Zhu <git@chuang.cz>
>
> When multiple entries are requested using line names in
> Chip.request_lines, only the the last entry is added to
> LineRequest._name_map, causing a ValueError when trying to use functions
> like LineRequest.set_value on any former entries.
>

Chip.request_lines()

> This patch fixes that by moving the required variables to the correct
> scope.
>

Use imperitive, "Move the required...."

> Signed-off-by: Chuang Zhu <git@chuang.cz>

Use the same address to sign off and send the submission email.

With a multi-patch series you should include a cover letter that
describes the series the changes between versions.

e.g. here you have fixed my review comments, but also fixed not moving the
offsets list into the outer scope.

Cheers,
Kent.

> ---
>  bindings/python/gpiod/chip.py | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/bindings/python/gpiod/chip.py b/bindings/python/gpiod/chip.py
> index 19c62cd..55f0c3e 100644
> --- a/bindings/python/gpiod/chip.py
> +++ b/bindings/python/gpiod/chip.py
> @@ -279,11 +279,12 @@ class Chip:
>          else:
>              mapped_output_values = None
>
> +        name_map = dict()
> +        offset_map = dict()
> +        global_output_values = list()
> +
>          for lines, settings in config.items():
>              offsets = list()
> -            name_map = dict()
> -            offset_map = dict()
> -            global_output_values = list()
>
>              if isinstance(lines, int) or isinstance(lines, str):
>                  lines = (lines,)
> --
> 2.44.0
>