On Tue, Sep 06, 2016 at 11:12:24AM +0200, Thierry Reding wrote: > On Mon, Sep 05, 2016 at 10:02:48PM +0200, Maxime Ripard wrote: > > Hi, > > > > On Mon, Sep 05, 2016 at 01:03:03AM +0800, Icenowy Zheng wrote: > > > Hi Everyone, > > > > > > 01.09.2016, 23:40, "Maxime Ripard" <maxime.ripard@xxxxxxxxxxxxxxxxxx>: > > > > The SinA33 has an unidentified panel. Add the timings for it under a new > > > > compatible. > > > > > > > > > > > > Excuse me... > > > I will ask a question which is not fully related to the patch here... > > > If I want to add a generic panel for Q8 tablets, what should it be called? > > > "allwinner,q8-lcd-panel-800x480"? > > > > I guess it's more of a question for Thierry, but it seems like the > > trend is to put the diagonal rather than the resolution in the > > compatibles. > > Compatible strings should contain the model number of the panel. There > is no such thing as a "generic panel for Q8 tablets". > > Also, how is it that these panels are unidentified? Has nobody tried to > open them up and look at the panel to find a model number? It is not always possible without breaking the device apart. And I can understand not a lot of people want to do that. But on that particular patch, I had a bunch of spare ones, and ended up doing just that. The reference is from an unknown brand, but at least the part number shows up on Google, I'll use that. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
