linux-kernel - Re: [PATCH RFC 3/6] ARM: qcom_defconfig: add display-related options

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CACRpkdZFK9EGptXbtowUMai6M-jdh6OSTU2=X9A-N3R7hcvXiQ@mail.gmail.com>
Date:   Mon, 6 May 2019 09:04:35 +0200
From:   Linus Walleij <linus.walleij@...aro.org>
To:     Brian Masney <masneyb@...tation.org>
Cc:     Rob Clark <robdclark@...il.com>, Sean Paul <sean@...rly.run>,
        "open list:DRM PANEL DRIVERS" <dri-devel@...ts.freedesktop.org>,
        MSM <linux-arm-msm@...r.kernel.org>,
        freedreno@...ts.freedesktop.org, Dave Airlie <airlied@...ux.ie>,
        Daniel Vetter <daniel@...ll.ch>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH RFC 3/6] ARM: qcom_defconfig: add display-related options

On Sun, May 5, 2019 at 3:04 PM Brian Masney <masneyb@...tation.org> wrote:

> Add the CMA (Contiguous Memory Allocator) for the MSM DRM driver, the
> simple panel, and the TI LM3630A driver in order to support the display
> on the LG Nexus 5 (hammerhead) phone.
>
> Signed-off-by: Brian Masney <masneyb@...tation.org>

Reviewed-by: Linus Walleij <linus.walleij@...aro.org>

> The panel and backlight are currently compiled into the kernel, but will
> be moved to be modules once the display is fully working in a later
> verison of this patch series.

I don't see why we would want to do that, the FB console is
traditionally (x86) good to alwaýs get up, as serial console
might not always be available. For example for people
who can't solder special cables but still want to boot a
custom ROM on their Nexus. So I'd say keep it like this.

Yours,
Linus Walleij