From: You Kangren <youkangren@xxxxxxxx> > Sent: 27 June 2023 05:32 > > Replace the ternary conditional operator with min_t() to simplify the code > > Signed-off-by: You Kangren <youkangren@xxxxxxxx> > --- > drivers/net/wireless/legacy/ray_cs.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/legacy/ray_cs.c b/drivers/net/wireless/legacy/ray_cs.c > index 8ace797ce951..25edbc655738 100644 > --- a/drivers/net/wireless/legacy/ray_cs.c > +++ b/drivers/net/wireless/legacy/ray_cs.c > @@ -2086,8 +2086,7 @@ static void ray_rx(struct net_device *dev, ray_dev_t *local, > rx_data(dev, prcs, pkt_addr, rx_len); > > copy_from_rx_buff(local, (UCHAR *) &local->last_bcn, pkt_addr, > - rx_len < sizeof(struct beacon_rx) ? > - rx_len : sizeof(struct beacon_rx)); > + min_t(size_t, rx_len, sizeof(struct beacon_rx)); You should really consider changing the type of rx_len before using min_t(). David > > local->beacon_rxed = 1; > /* Get the statistics so the card counters never overflow */ > -- > 2.39.0 - Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK Registration No: 1397386 (Wales)
