Hi Andrew, On Mon, Sep 03, 2018 at 03:27:56PM +0200, Andrew Lunn wrote: > On Mon, Sep 03, 2018 at 10:48:50AM +0200, Quentin Schulz wrote: > > In the DT binding, it is specified nowhere that 'vsc8531,edge-slowdown' > > is an u8, even though it's read as an u8 in the driver. > > Hi Quentin > > of_property_read_u8() will perform bounds checking. Is that important > here? > Just to be sure, we're talking here about making sure the value stored in the DT is not bigger than the specified value (here an u8)? If so, that isn't the reason why I'm suggesting those two patches. Without /bits 8/ in the DT property, whatever were the values I put in the property, I'd always get a 0. So I need to fix it either in the DT (but Rob does not really like it) or in the driver. Hope that makes sense. Quentin
Attachment:
signature.asc
Description: PGP signature
