[PATCH v4 45/47] net/wan/fsl_ucc_hdlc: reject muram offsets above 64K

Timur Tabi timur at kernel.org
Sat Nov 16 01:33:52 AEDT 2019


On 11/15/19 1:44 AM, Rasmus Villemoes wrote:
> I can change it, sure, but it's a matter of taste. To me the above asks
> "does the value change when it is truncated to a u16" which makes
> perfect sense when the value is next used with iowrite16be(). Using a
> comparison to U16_MAX takes more brain cycles for me, because I have to
> think whether it should be > or >=, and are there some
> signedness/integer promotion business interfering with that test.

Ok.


More information about the Linuxppc-dev mailing list