On Fri, Feb 14, 2025 at 08:04:41PM -0500, Alex Lanzano wrote: > On Fri, Feb 14, 2025 at 10:29:29AM +0100, Josef Luštický wrote: > > Hello Alex, > > there is a bug in mipi_dbi_hw_reset() function that implements the logic of > > display reset contrary. > > It keeps the reset line activated which keeps displays in reset state. > > > > I reported the bug to > > https://gitlab.freedesktop.org/drm/misc/kernel/-/issues/63 > > > > Unfortunately, fixing the bug would mean current DTB-ABI breakage and > > device-trees modification would be needed. > > I mainly write this email to let you and other people know about it, so > > hopefully it can be found easier. > > What are your thoughts? > Thanks for making me aware. I'll dig into over the weekend and get back > to you
Alright so I looked into a bit more. Looks like the MIPI Specification says that the reset line is active low. So, if we want dt entries to be correct the logic for setting the reset line in mipi_dbi_hw_reset() should be flipped. However, like you said, this is going to cause a some confused developers to break out their oscilloscopes to figure out why their display isn't working. Best regards, Alex