Re: [PATCH] dts: rockchip: rk3066: add qos_hdmi and HCLK_HDMI to pmu node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi all,

Thanks Tomasz for adding all the mailing lists.
I prefer to ask first if a qos_hdmi exists before sending it in for
public review.

All the clocks in the pmu node seem to have a "quality-of-service" (QoS)
block.
So I added one for hdmi too with the question if it exists and which
address it might have.
SoC documentation still isn't fully public, not that I'm aware off.

>From Heiko's response I learned that the rk3066 manual does
not list any hdmi-related QoS blocks.
We now only add the HCLK_HDMI clock to pmu node in our patch.

After a small step forward people immediately ask to do a real patch serie.
The rk3066_hdmi.c file also handles audio, but this is not tested yet.
The rk3066a-rayeager.dts file gives a idea how it's done,
but I haven't had the time to get it working with .config and dts.
Let me know if you can.

That's it for now.


On 12/17/18 6:46 AM, Tomasz Figa wrote:

> Thanks for the patch. Unfortunately, it looks like you didn't add the
> necessary mailing lists to the recipient list. For reference, the
> ./scripts/get_maintainer.pl script in the kernel source tree should be
> able to give you a reasonable recipient list. For now, I added the
> mailing lists on CC and replied without snipping, so people should be
> still able to review the patch.
> 
> Other than that, It looks reasonable to me, but we need someone with
> access to SoC documentation to check it. Heiko, Sandy, is that
> something you would be able to help with?

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux