Re: [PATCH v2 000/101] fbdev: Fix memory leak in option parsing

2023-03-21 Thread Thomas Zimmermann
Hi Helge, thanks for the feedback. Am 20.03.23 um 20:25 schrieb Helge Deller: Hi Thomas, On 3/20/23 11:07, Thomas Zimmermann wrote: Geert, Helge?  Do you have further comments?  There's not really much for a v3 yet. I understand the motivation and I see you invested a lot of work on it, whi

Re: [PATCH v2 000/101] fbdev: Fix memory leak in option parsing

2023-03-20 Thread Helge Deller
Hi Thomas, On 3/20/23 11:07, Thomas Zimmermann wrote: Geert, Helge?  Do you have further comments?  There's not really much for a v3 yet. I understand the motivation and I see you invested a lot of work on it, which is really appreciated. But I have mixed feelings about that patch itself. Ne

Re: [PATCH v2 000/101] fbdev: Fix memory leak in option parsing

2023-03-20 Thread Thomas Zimmermann
Geert, Helge? Do you have further comments? There's not really much for a v3 yet. Best regards Thomas Am 09.03.23 um 17:00 schrieb Thomas Zimmermann: Introduce struct option_iter and helpers to parse command-line options with comma-separated key-value pairs. Then convert fbdev drivers to the

Re: [PATCH v2 000/101] fbdev: Fix memory leak in option parsing

2023-03-10 Thread Thomas Zimmermann
Hi Geert Am 10.03.23 um 09:24 schrieb Geert Uytterhoeven: Hi Thomas, On Thu, Mar 9, 2023 at 5:02 PM Thomas Zimmermann wrote: Introduce struct option_iter and helpers to parse command-line options with comma-separated key-value pairs. Then convert fbdev drivers to the new interface. Fixes a me

Re: [PATCH v2 000/101] fbdev: Fix memory leak in option parsing

2023-03-10 Thread Geert Uytterhoeven
Hi Thomas, On Thu, Mar 9, 2023 at 5:02 PM Thomas Zimmermann wrote: > Introduce struct option_iter and helpers to parse command-line > options with comma-separated key-value pairs. Then convert fbdev > drivers to the new interface. Fixes a memory leak in the parsing of > the video= option. > > Bef

[PATCH v2 000/101] fbdev: Fix memory leak in option parsing

2023-03-09 Thread Thomas Zimmermann
Introduce struct option_iter and helpers to parse command-line options with comma-separated key-value pairs. Then convert fbdev drivers to the new interface. Fixes a memory leak in the parsing of the video= option. Before commit 73ce73c30ba9 ("fbdev: Transfer video= option strings to caller; clari