Re: [Patch v9 0/4] ledtrig-tty: add additional tty state evaluation

From: Greg KH
Date: Mon Nov 27 2023 - 08:54:23 EST


On Mon, Nov 27, 2023 at 12:03:07PM +0100, Florian Eckert wrote:
> Changes in v9:
> ==============
> - As requested, I have removed the memory leak fix patch from the previous
> patch series v8 and have send this patch directly to the stable
> mailinglist [10].
> - I also have removed the not needed wait for completion code path on
> sysfs attr store and show, for the new configuration options, as
> requested by greg k-h [11].
> - Patch v8 3/6 dropped, because the change is not correct as mentiond
> by greg k-h [12] and this part of the code will be removed anyway
> in this patch series.
> - Update naming of the function ledtrig_tty_waitforcompletion() to
> ledtrig_tty_wait_for_completion() which match better with the API call
> function wait_for_completion_timeout().
>
> Thanks to all reviewers for the suggestions for improvement :+1: of this
> patch set.

Nice work, thanks for sticking with it:

Reviewed-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>