Hi Pavel, 2017-12-15 21:57 GMT+01:00 Pavel Machek <pavel@xxxxxx>: > Hi! > >> Yes, I think that how you describe luminance and lightness is right, >> and sounds good improve the doc. >> >> To be clear the correction table for PWM values can be calculated with >> this code. >> >> OUTPUT_SIZE = 65535 # Output integer size >> INPUT_SIZE = 2047 >> >> def cie1931(L): >> L = L*100.0 >> if L <= 8: >> return (L/902.3) >> else: >> return ((L+16.0)/116.0)**3 >> >> x = range(0,int(INPUT_SIZE+1)) >> y = [int(round(cie1931(float(L)/INPUT_SIZE)*(OUTPUT_SIZE))) for L in x] > > Can we just generate the table on the fly? Should not be hard to do in > fixed point, right? This was discussed a bit in previous RFC which had the code to generate the table on the fly, see [1]. The use of a fixed table or an on the fly table is something that I'll let the maintainers to decide. I've no strong opinion on use the on the fly table if someone takes care to review deeply the fixed point maths :) [1] https://lkml.org/lkml/2017/9/4/335 Regards, Enric > Pavel > -- > (english) http://www.livejournal.com/~pavelmachek > (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
