On Wed, Jun 07, 2023 at 12:22:33AM +0200, Heiko Stübner wrote: > Am Dienstag, 6. Juni 2023, 20:41:17 CEST schrieb Shengyu Qu: > > > On Fri, Jun 02, 2023 at 03:40:35PM +0800, Keith Zhao wrote: > > >> Add bindings for JH7110 display subsystem which > > >> has a display controller verisilicon dc8200 > > >> and an HDMI interface. > > >> +description: > > >> + The StarFive SoC uses the HDMI signal transmiter based on innosilicon IP > > > Is innosilicon the same thing as verisilicon? Also > > > s/transmiter/transmitter/, both here and in the title. > > > > I think that is not the same, I remember Rockchip has used a HDMI > > transmitter from > > > > Innosilicon, and there is a existing driver for that in mainline. > > Yep, I think Innosilicon is the company you turn to when you want to save > a bit of money ;-) . In the bigger SoCs Rockchip most of the time uses > Designware hdmi blocks and looking at the history only the rk3036 ever > used an Innosilicon block. > > Looking at the history, 2016 really was a long time ago :-D. > > > So Keith, if that's true, I think it is better to seperate the HDMI > > stuff and reuse existing driver. > > I'm not so sure about that - at least from a cursory glance :-) . > > The registers do look slightly different and I don't know how much > the IP changed between the rk3036-version and the jh7110 version. > > At the very least, I know my rk3036 board isn't booting right now, so > I can't really provide help for generalizing the rockchip-driver. > > At the very least both the binding and driver could drop the "starfive-hdmi" > and actually use the Innosilicon in the naming somewhere, so that it's > clear for future developers :-) Seeing "based on" always makes me a little bit nervous to be honest when it comes to using a compatible from the IP. Is it the IP? What version is it? etc. Perhaps "starfive,jh7110-hdmi" & falling back to some sort of "innosilicon,hdmi" would be more future/IP-silliness proof. Driver can always be generic & bind against "innosilicon,hdmi" until that becomes impossible. Cheers, Conor.
Attachment:
signature.asc
Description: PGP signature
