Thomas Zimmermann
2021-Apr-15 07:31 UTC
[PATCH v2 08/10] drm/simpledrm: Acquire clocks from DT device node
Hi Am 08.04.21 um 10:13 schrieb Maxime Ripard:> Hi, > > On Thu, Mar 18, 2021 at 11:29:19AM +0100, Thomas Zimmermann wrote: >> Make sure required hardware clocks are enabled while the firmware >> framebuffer is in use. >> >> The basic code has been taken from the simplefb driver and adapted >> to DRM. Clocks are released automatically via devres helpers. >> >> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de> >> Tested-by: nerdopolis <bluescreen_avenger at verizon.net> > > Even though it's definitely simpler to review, merging the driver first > and then the clocks and regulators will break bisection on the platforms > that rely on themI'd like to keep the patches separate for now, but can squash patches 6 to 8 them into one before pushing them. OK?> > Another thing worth considering is also that both drivers will probe if > they are enabled (which is pretty likely), which is not great :) > > I guess we should make them mutually exclusive through KconfigWe already have several drivers in fbdev and DRM that handle the same hardware. We don't do this for any other pair, why bother now? Best regards Thomas> > Maxime >-- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 N?rnberg, Germany (HRB 36809, AG N?rnberg) Gesch?ftsf?hrer: Felix Imend?rffer -------------- next part -------------- A non-text attachment was scrubbed... Name: OpenPGP_signature Type: application/pgp-signature Size: 840 bytes Desc: OpenPGP digital signature URL: <http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20210415/a470a6e9/attachment.sig>