On Tue, Jan 17, 2017 at 01:33:57PM +0200, Krzysztof Kozlowski wrote: > On Tue, Jan 17, 2017 at 12:22 PM, Neil Armstrong > <narmstrong@xxxxxxxxxxxx> wrote: > > On 01/17/2017 10:38 AM, Maxime Ripard wrote: > >> Hi, > >> > >> On Mon, Jan 16, 2017 at 08:49:06PM +0200, Krzysztof Kozlowski wrote: > >>> On Mon, Jan 16, 2017 at 02:24:23PM +0100, Maxime Ripard wrote: > >>>> The ARM Mali Utgard GPU family is embedded into a number of SoCs from > >>>> Allwinner, Amlogic, Mediatek or Rockchip. > >>>> > >>>> Add a binding for the GPU of that family. > >>>> > >>>> Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> > >>>> --- > >>>> .../devicetree/bindings/gpu/arm,mali-utgard.txt | 76 ++++++++++++++++++++++ > >>>> 1 file changed, 76 insertions(+) > >>>> create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt > >>> > >>> Do you have a driver in kernel which will implement these bindings? > >> > >> No, but we have bindings for out-of-tree drivers already. > >> > >>> Defining them for out-of-tree driver does not bring any benefits > >>> (3rd party driver will not respect them anyway). > >> > >> You could see it the other way around too. The out-of-tree drivers > >> don't respect it at the moment because there's no binding to respect. > >> > >> And at least for us, we definitely plan on doing that. > >> > >> Maxime > > > > Hi Maxime, Krzysztof, > > > > We hope this will be accepted so it will solve the same issue we have on Amlogic SoCs > > and all the other mali powered SoCs. > > It will be helpful also for other SoCs using Mali 400 (e.g. > Exynos3250, Exynos4412). Yep, hence why you were in Cc :) > > Having mainline bindings will forcre out-of-tree driver to respect those bindings > > and remove a dts out-of-tree patch aswell. > > I would argue here over the word "force". Having bindings defined here > does not force anyone into anything. The out-of-tree can do whatever > they want. It is a wish from kernel side - it might be respected but > it might not. Well, that statement can be made for each line of the kernel itself, nothing forces you to use the code as is, or to make any kind of modifications, including to its ABI. You just have an incentive to respect the various frameworks and ABI because it's the path of least resistance and all the tools, libraries and applications already know how to use those interfaces. But that's just it, an incentive, that anyone is free or not to comply to. > Just to be sure - I am not opposed against. Some time ago I wanted > Mali400 to be upstreamed but with current policy about user-space side > it is not possible. While I would also very much like to see the kernel driver upstreamed, this is unreasonable at this point. But this patch is here only to add a binding to it, so that the out-of-tree drivers can comply to it, and this is something we already have done. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
Attachment:
signature.asc
Description: PGP signature