Re: [PATCH V9 03/16] rv/include: Add helper functions for deterministic automata
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH V9 03/16] rv/include: Add helper functions for deterministic automata
- From: Tao Zhou <tao.zhou@xxxxxxxxx>
- Date: Mon, 1 Aug 2022 02:17:49 +0800
- Cc: Steven Rostedt <rostedt@xxxxxxxxxxx>, Wim Van Sebroeck <wim@xxxxxxxxxxxxxxxxxx>, Guenter Roeck <linux@xxxxxxxxxxxx>, Jonathan Corbet <corbet@xxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Will Deacon <will@xxxxxxxxxx>, Catalin Marinas <catalin.marinas@xxxxxxx>, Marco Elver <elver@xxxxxxxxxx>, Dmitry Vyukov <dvyukov@xxxxxxxxxx>, "Paul E. McKenney" <paulmck@xxxxxxxxxx>, Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>, Gabriele Paoloni <gpaoloni@xxxxxxxxxx>, Juri Lelli <juri.lelli@xxxxxxxxxx>, Clark Williams <williams@xxxxxxxxxx>, Randy Dunlap <rdunlap@xxxxxxxxxxxxx>, linux-doc@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-trace-devel@xxxxxxxxxxxxxxx, Tao Zhou <tao.zhou@xxxxxxxxx>
- In-reply-to: <7b3f7ec1-a479-f3ed-42b3-ddead0f9b427@kernel.org>
- References: <cover.1659052063.git.bristot@kernel.org> <563234f2bfa84b540f60cf9e39c2d9f0eea95a55.1659052063.git.bristot@kernel.org> <YuacJsPya8PSE8qt@geo.homenetwork> <7b3f7ec1-a479-f3ed-42b3-ddead0f9b427@kernel.org>
On Sun, Jul 31, 2022 at 06:02:47PM +0200, Daniel Bristot de Oliveira wrote:
> On 7/31/22 17:13, Tao Zhou wrote:
> > On Fri, Jul 29, 2022 at 11:38:42AM +0200, Daniel Bristot de Oliveira wrote:
> >
> > [...]
> >
> >> +static inline type model_get_next_state_##name(enum states_##name curr_state, \
> >> + enum events_##name event) \
> >> +{ \
> >> + if ((curr_state < 0) || (curr_state >= state_max_##name)) \
> >> + return INVALID_STATE; \
> >> + \
> >> + if ((event < 0) || (event >= event_max_##name)) \
> >> + return INVALID_STATE; \
> >
> > Should define the INVALID_EVENT corresponding to event invalid case.
>
> no.
Absolutly I lost here, this is for *get next state*, must return
a state info.
>
> -- Daniel
[Index of Archives]
[Linux USB Development]
[Linux USB Development]
[Linux Audio Users]
[Yosemite Hiking]
[Linux Kernel]
[Linux SCSI]