> > I agree that we need to support disabling rx-copybreak. > > What about 0 ~ 64 means to disable rx-copybreak? > > Or should only 0 be allowed to disable rx-copybreak? > > > > I think a single value of 0 that means disable RX copybreak is more > clear and intuitive. Also, I think we can allow 64 to be a valid > value. > > So, 0 means to disable. 1 to 63 are -EINVAL and 64 to 1024 are valid. Thanks. Please spend a little time and see what other drivers do. Ideally we want one consistent behaviour for all drivers that allow copybreak to be disabled. Andrew
