Re: [PATCH net-next 01/11] tools: ynl-gen: fix enum index in _decode_enum(..)
From: Vadim Fedorenko <vadim.fedorenko@linux.dev>
Date: 2023-07-20 14:54:46
Also in:
linux-arm-kernel, linux-clk
From: Vadim Fedorenko <vadim.fedorenko@linux.dev>
Date: 2023-07-20 14:54:46
Also in:
linux-arm-kernel, linux-clk
On 20.07.2023 14:58, Kubalewski, Arkadiusz wrote:
quoted
-----Original Message----- From: Jiri Pirko <jiri@resnulli.us> Sent: Thursday, July 20, 2023 3:40 PM Thu, Jul 20, 2023 at 11:18:53AM CEST, vadim.fedorenko@linux.dev wrote:quoted
From: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com> Remove wrong index adjustement, which is leftover from addings/adjustement/adjustment/Sure will fix, Although those "tools: ynl" patches were not intended to be a part of dpll Series, they are being discussed on the other thread: https://lore.kernel.org/netdev/20230718162225.231775-1-arkadiusz.kubalewski@intel.com/ (local) I think Vadim have sent them, because I included them in the branch candidate for next version, seems was not clear enough on that.. I think we can skip them for next submission.
Yeah, I just realised that these patches have been sent earlier as separate series and are still under review. Once they are committed I'll remove them from DPLL series, but for now they are needed for spec generation of DPLL yaml, so it's good to have them anyway.
Thank you! Arkadiuszquoted
quoted
support for sparse enums. enum.entries_by_val() function shall not subtract the start-value, as it is indexed with real enum value. Fixes: c311aaa74ca1 ("tools: ynl: fix enum-as-flags in the generic CLI") Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com> Signed-off-by: Vadim Fedorenko <vadim.fedorenko@linux.dev>Reviewed-by: Jiri Pirko <redacted>