[PATCH 2/2] drm: Add ASPEED GFX driver

Joel Stanley joel at jms.id.au
Tue Apr 2 13:09:19 AEDT 2019


On Thu, 28 Mar 2019 at 07:53, Daniel Vetter <daniel at ffwll.ch> wrote:

> > +static int aspeed_gfx_get_modes(struct drm_connector *connector)
> > +{
> > +     return drm_add_modes_noedid(connector, 800, 600);
>
> Is this the only mode you do, or just a default? Iirc if you report
> "connected", you'll get this as one of the fallback modes already.

I this is the only mode we can do with the clock that we have available.

> Very nice driver!
>
> Acked-by: Daniel Vetter <daniel.vetter at ffwll.ch>

Thanks!

> For merging/maintenance, do you want to put this into drm-misc?
>
> https://drm.pages.freedesktop.org/maintainer-tools/committer-drm-misc.html#small-drivers

That makes sense. I'll send out a v2 with the changes you and Noralf
suggested, and a maintainers patch, once I've given it a spin on
hardware.

> We could also officially put drm/ast in there, if you want to maintain
> that too.

I haven't had much to do with that driver (it's physically inside the
BMC, but a different piece of hardware). I can probably be included so
I'm on cc for changes.

Cheers,

Joel


>
> Either way, patch also needs a MAINTAINERS entry.
> -Daniel
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel


More information about the Linux-aspeed mailing list