Hi Maxime, On Tue, Aug 25, 2015 at 12:26 AM, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > Hi, > > On Fri, Aug 21, 2015 at 09:53:07AM +0200, Hans de Goede wrote: >> I just realized that I never answered your question about the need >> for a tvencoder simplefb node in sun5i-a13.dtsi, like the one from >> the "ARM: dts: sun5i: Add simplefb node for tvencoder output" >> >> I think the real question is do we want to use sun5i-a13.dtsi for >> the r8 ? > > So far, the only information I got about the R8 was that it was > strictly identical to the A13, and it was making sense to re-use it > entirely. Does having the tvout pin change the pinmux options or is that pin strictly single use? (And if it is strictly single use, is adding that pin the entirety of the software visible hardware changes?) >> I would expect us to use a sun5i-r8.dtsi, this one could then >> include sun5i-a13.dtsi, and have the extra simplefb node for >> the tvencoder. Putting the simplefb node for the tvencoder >> in sun5i-a13.dtsi feels wrong because the A13 package does >> not have a tvout pin. > > But if it turns out it's not, then yeah, we'll have to make another > DTSI. IMHO that's enough of a change to warrant a new DTSI, I feel that someone's going to use the lack of DTSI as "proof" that we don't support the R8 or try to use the tv encoder on an A13. Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ -- 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
