Re: [PATCH] staging: sm750fb: convert nDirection to snake_case

From: Dan Carpenter

Date: Thu Feb 12 2026 - 00:36:05 EST


On Thu, Feb 12, 2026 at 12:24:44AM +0200, Rachel Ovrani wrote:
> Thank you for the review!
>
> The 'n_' prefix stands for "new" - the variable holds a newly calculated
> direction value based on the overlap between source and destination regions.
>
> That said, if you think 'direction' would be clearer, I'm happy to submit a
> v2 removing the prefix.

I would have never guessed it was new_. Normally n_ means number and I
couldn't think of any meaning in Hungarian notation. Naming it
"new_direction" is even better.

regards,
dan carpenter