* Drew Fustini <drew@xxxxxxxxxxxxxxx> [200608 12:52]:
> This patch causes pcs_parse_pinconf() to return -ENOTSUPP when no
> pinctrl_map is added. The current behavior is to return 0 when
> !PCS_HAS_PINCONF or !nconfs. Thus pcs_parse_one_pinctrl_entry()
> incorrectly assumes that a map was added and sets num_maps = 2.
...
> Fixes: 9dddb4df90d1 ("pinctrl: single: support generic pinconf")
It would be good to get an ack/tested-by from Haojian for this.
The patch looks right to me:
Acked-by: Tony Lindgren <tony@xxxxxxxxxxx>
Re: [PATCH v2] pinctrl-single: fix pcs_parse_pinconf() return value
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH v2] pinctrl-single: fix pcs_parse_pinconf() return value
- From: Tony Lindgren <tony@xxxxxxxxxxx>
- Date: Tue, 9 Jun 2020 11:06:31 -0700
- In-reply-to: <20200608125143.GA2789203@x1>
- References: <20200608125143.GA2789203@x1>
- References:
- [PATCH v2] pinctrl-single: fix pcs_parse_pinconf() return value
- From: Drew Fustini
- [PATCH v2] pinctrl-single: fix pcs_parse_pinconf() return value
- Prev by Date: Generic way for verifying GPIO wakeup capability
- Next by Date: [PATCH v1 4/6] pinctrl: intel: Allow drivers to define total amount of IRQs per community
- Previous by thread: [PATCH v2] pinctrl-single: fix pcs_parse_pinconf() return value
- Next by thread: Re: [PATCH v2] pinctrl-single: fix pcs_parse_pinconf() return value
- Index(es):
 |