Am Mittwoch, den 13.05.2020, 23:41 -0300 schrieb Fabio Estevam: > On Wed, May 13, 2020 at 2:09 PM Fabio Estevam <festevam@xxxxxxxxx> wrote: > > > The binding doc Documentation/devicetree/bindings/gpu/vivante,gc.yaml > > says that only the 'reg' clock could be optional, the others are > > required. > > arch/arm/boot/dts/dove.dtsi only uses the 'core' clock. > arch/arm/boot/dts/stm32mp157.dtsi uses 'bus' and 'core' > > Maybe the binding needs to be updated and it seems that using > devm_clk_get_optional() like you propose is safe. The binding is correct as-is. We want to require those clocks to be present, but the dove DT was added before the binding was finalized, so the driver still treats the clocks as optional to not break compatibility with old DTs. Maybe this warrants a comment in the code... Regards, Lucas _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
