Re: [PATCH net-next RFC v4] net: hdlc_x25: Queue outgoing LAPB frames

From: Xie He
Date: Thu Feb 18 2021 - 14:09:24 EST


On Thu, Feb 18, 2021 at 2:37 AM Leon Romanovsky <leon@xxxxxxxxxx> wrote:
>
> It is not me who didn't explain, it is you who didn't want to write clear
> comment that describes the headroom size without need of "3 - 1".

Why do I need to write unnecessary comments when "3 - 1" and the
current comment already explains everything?

> So in current situation, you added two things: comment and assignment.
> Both of them aren't serve their goals.

Why?

> Your comment doesn't explain
> enough and needs extra help

Why? My comment already explains everything.

> and your assignment is useless without
> comment.

My assignment is already very clear with my current comment. My
comment explains very clearly what this assignment means.