On 16/03/2020 10:53, Sam Ravnborg wrote:
We have panel-simple-dsi for DSI simple based panels.
This binding includes the reg property.
If we have included DSI panels in panel-simple.yaml, and we likely have
by accident, then they should be moved to panel-simple-dsi.yaml.
If they requires anything else then they shall have their
own binding.
panel-simple.yaml and panel-simple.dsi.yaml are on purpose
only for the simple panels and they have:
"additionalProperties: false" to avoid that a lot
of extra sneaks in.
I actually considered shortly a panel-simple-spi.yaml,
but the few panels I looked at had different names
for the power-supply so that did not fly.
I did not check them all - we have today (with this patch-set)
9 bindings that references spi-slave.yaml.
Okay, I understand now. Makes sense.
panel-simple.c has dsi_of_match, which lists DSI panels. I was looking at that when I said
panel-simple binding has DSI panels. At least auo,b080uan01 and osddisplays,osd101t2045-53ts are
there, and earlier in this series you moved osddisplays,osd101t2587-53ts to panel-simple bindings.
Tomi
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
