On Mon, 18 May 2020 at 13:48, Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> wrote: > > > On 5/18/20 1:19 PM, Emil Velikov wrote: > > Hi Michael, > > > > On Mon, 18 May 2020 at 08:30, Michael Ellerman <m...@ellerman.id.au> wrote: > >> > >> Emil Velikov <emil.l.veli...@gmail.com> writes: > >>> As mentioned in earlier commit, the riva and nvidia fbdev drivers have > >>> seen no love over the years, are short on features and overall below par > >>> > >>> Users are encouraged to switch to the nouveau drm driver instead. > >>> > >>> v2: Split configs to separate patch, enable nouveau (Bartlomiej) > >>> > >>> Cc: Antonino Daplas <adap...@gmail.com> > >>> Cc: Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> > >>> Cc: linux-fb...@vger.kernel.org > >>> Cc: dri-de...@lists.freedesktop.org > >>> Cc: Michael Ellerman <m...@ellerman.id.au> > >>> Cc: Benjamin Herrenschmidt <b...@kernel.crashing.org> > >>> Cc: Paul Mackerras <pau...@samba.org> > >>> Cc: linuxppc-dev@lists.ozlabs.org > >>> Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> > >>> Acked-by: Daniel Vetter <daniel.vet...@ffwll.ch> (v1) > >>> --- > >>> Hi all unless, there are objections I would prefer to merge this via > >>> the drm tree. > >> > >> Have you tested that the resulting kernels work on the relevant > >> hardware? > >> > > Sadly, no I haven't. I'm updating the defconfigs as requested by the > > fbdev maintainer. > > I've just noticed that v1 (patch #1/1) & v2 (patch #1/2) lack > Cc: to powerpc Maintainers so they cannot see the context of > changes in this patch. > Haven't see any instances of fbdev/drm patches being Cc to arch/ developers. As such I made sure the commit message illustrates 1/2 as clearly as possible.
> Also you've proposed v1 yourself and it has already contained > modifications to defconfigs (removal of setting the config > options for the old drivers) in addition to marking the old > drivers as BROKEN. > > It now turns out that v1 has also never been tested. :( > > Please don't submit untested patches without marking them > as such. > Does x86 box with nv34 GPU count as testing? I suspect not :-( It was been using nouveau ever since I bought it 15+ years ago. Will be more careful in the future. -Emil