On Mon, 29 Sep 2025 13:32:15 +0100 Bruce Richardson <[email protected]> wrote:
> > + first_seg->pkt_len = > > + (uint16_t)(first_seg->pkt_len + > > + pkt_len); > > Since we allow 100 characters per line, does this line really need to be > split into 3? [I realise this is a copy-paste from other drivers, but we > can clean it up as new code here] Also why the cast?

