Re-using sun5i-a13.dtsi for the r8 ?
Hi Maxime,
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 ?
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.
Does that sound like a good solution to you ?
Regards,
Hans
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
A message that is not in the archive (sent off-list or not yet received): [email protected]
Hi,
Quoted 6 lines · Hans de Goede
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.dtsifor 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.
Quoted 5 lines
I would expect us to use a
sun5i-r8.dtsi, this one could then includesun5i-a13.dtsi, and have the extra simplefb node for the tvencoder. Putting the simplefb node for the tvencoder insun5i-a13.dtsifeels 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.
Maxime
Signature · 3 lines
Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
Hi Maxime,
Quoted 7 lines · Maxime Ripard
Hi,
Quoted 4 lines · Hans de Goede
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"
Quoted 6 lines
Quoted 2 lines
I think the real question is do we want to use
sun5i-a13.dtsifor 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?)
Quoted 8 lines
Quoted 5 lines
I would expect us to use a
sun5i-r8.dtsi, this one could then includesun5i-a13.dtsi, and have the extra simplefb node for the tvencoder. Putting the simplefb node for the tvencoder insun5i-a13.dtsifeels 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,
Signature · 4 lines
Julian Calaby Email: [email protected] Profile: http://www.google.com/profiles/julian.calaby/
A message that is not in the archive (sent off-list or not yet received): CAGRGNgW06-znXSnPKS3nfU_FseTZ0OcMea5qEzHVaTkpRfXHUg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org
Quoted 18 lines · Julian Calaby
Hi Maxime,
Quoted 7 lines · Maxime Ripard
Hi,
Quoted 4 lines · Hans de Goede
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"Quoted 6 lines
Quoted 2 lines
I think the real question is do we want to use
sun5i-a13.dtsifor 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.
Quoted 5 lines
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 have no idea.
Quoted 10 lines
Quoted 8 lines
Quoted 5 lines
I would expect us to use a
sun5i-r8.dtsi, this one could then includesun5i-a13.dtsi, and have the extra simplefb node for the tvencoder. Putting the simplefb node for the tvencoder insun5i-a13.dtsifeels 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,
Yep.
Quoted 2 lines
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.
But that argument is dubious. That's not how it works. The list of DTSIs has never been comprehensive list of the supported SoCs.
We support SoCs that don't have a DTSI (like we can see with the R8), and we also support, and even support DT that don't even have a matching machine. It's never been that way, and will never be.
Plus, you're never actually using the DTSI directly, just the board DT, that will be there.
Maxime
Signature · 3 lines
Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
Hi Maxime,
Quoted 38 lines · Maxime Ripard
Quoted 18 lines · Julian Calaby
Hi Maxime,
Quoted 7 lines · Maxime Ripard
Hi,
Quoted 4 lines · Hans de Goede
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"Quoted 6 lines
Quoted 2 lines
I think the real question is do we want to use
sun5i-a13.dtsifor 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.
Quoted 5 lines
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 have no idea.
Quoted 10 lines
Quoted 8 lines
Quoted 5 lines
I would expect us to use a
sun5i-r8.dtsi, this one could then includesun5i-a13.dtsi, and have the extra simplefb node for the tvencoder. Putting the simplefb node for the tvencoder insun5i-a13.dtsifeels 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,
Quoted 7 lines
Yep.
Quoted 2 lines
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.
But that argument is dubious. That's not how it works. The list of DTSIs has never been comprehensive list of the supported SoCs.
Most definitely. I'm not talking about sensible people here. =)
Thanks,
Signature · 4 lines
Julian Calaby Email: [email protected] Profile: http://www.google.com/profiles/julian.calaby/