On 24/02/2022 10:24, Daniel Vetter wrote: > Some things changed, and add two useful links. > > v2: Also include a link to the QR encoding work. Plus review from > Javier. > > Reviewed-by: Javier Martinez Canillas <javi...@redhat.com> > Cc: Javier Martinez Canillas <javi...@redhat.com> > Cc: Pekka Paalanen <ppaala...@gmail.com> > Cc: gpicc...@igalia.com > Signed-off-by: Daniel Vetter <daniel.vet...@intel.com> > ---
Hi Daniel, thanks for the improvement - much appreciated! Below a comment inline; other than that, feel free to add my: Reviewed-by: Guilherme G. Piccoli <gpicc...@igalia.com> Cheers, Guilherme > [...] > > -* There's also proposal for a simplied DRM console instead of the full-blown > - fbcon and DRM fbdev emulation. Any kind of panic handling tricks should > - obviously work for both console, in case we ever get kmslog merged. > +* Encoding the actual oops and preceeding dmesg in a QR might help with the Email client complains about "preceeding" word - misspelled maybe? > + dread "important stuff scrolled away" problem. See `[RFC][PATCH] Oops > messages > + transfer using QR codes > + > <https://lore.kernel.org/lkml/1446217392-11981-1-git-send-email-alexandru.murt...@intel.com/>`_ > + for some example code that could be reused. > > Contact: Daniel Vetter >