On 6/4/20 10:35 PM, Tom Rini wrote: > In lan78xx_read_otp() we want to know if sig is LAN78XX_OTP_INDICATOR_1 > or LAN78XX_OTP_INDICATOR_2. In the case of matching the first one we > set offset to itself, and clang warns about this. Rework the logic so > that if sig is the second indicator we adjust the offset as today and if > it does not match the first indicator we return -EINVAL > > Cc: Marek Vasut <ma...@denx.de> > Signed-off-by: Tom Rini <tr...@konsulko.com>
Reviewed-by: Marek Vasut <ma...@denx.de>
