From: Sam Ravnborg <sam@ravnborg.org>
To: "Guido Günther" <agx@sigxcpu.org>
Cc: Thierry Reding <thierry.reding@gmail.com>,
David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>,
Rob Herring <robh+dt@kernel.org>, Arnd Bergmann <arnd@arndb.de>,
Linus Walleij <linus.walleij@linaro.org>,
Heiko Stuebner <heiko.stuebner@theobroma-systems.com>,
Daniel Palmer <daniel@0x0f.com>, Lubomir Rintel <lkundrak@v3.sk>,
Mark Brown <broonie@kernel.org>,
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>,
allen <allen.chen@ite.com.tw>,
Mauro Carvalho Chehab <mchehab+huawei@kernel.org>,
"David S. Miller" <davem@davemloft.net>,
dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/3] drm/panel: Add panel driver for the Mantix MLAF057WE51-X DSI panel
Date: Tue, 18 Aug 2020 19:05:56 +0200 [thread overview]
Message-ID: <20200818170556.GA2291850@ravnborg.org> (raw)
In-Reply-To: <cover.1597652012.git.agx@sigxcpu.org>
Hi Guido.
On Mon, Aug 17, 2020 at 10:14:33AM +0200, Guido Günther wrote:
>
> The panel uses a Focaltech FT8006p, the touch part is handled by the already
> existing edt-ft5x06. It can be found in e.g. the Librem 5.
>
> Changes from v2:
> - Due to review comments by Sam Ravnborg, thanks!
> https://lore.kernel.org/lkml/20200815212727.GA1244923@ravnborg.org/
> - Drop unused header
> - Use newline before comment
> - Add Reviewed/Acked-by by Sam Ravnborg, thanks!
> https://lore.kernel.org/lkml/20200815212727.GA1244923@ravnborg.org/
> https://lore.kernel.org/lkml/20200815212750.GB1244923@ravnborg.org/
> https://lore.kernel.org/lkml/20200815212840.GC1244923@ravnborg.org/
>
> Changes from v1:
> - Due to review comments by Sam Ravnborg, thanks!
> https://lore.kernel.org/dri-devel/20200815083917.GA993113@ravnborg.org/
> - Don't preserve newlines with '|' in description
> - Use reset-gpios and backlight from panel-common.yaml
> - Reindent example
> https://lore.kernel.org/dri-devel/20200815093226.GB993113@ravnborg.org/
> - Drop unused includes
> - Use dev_* instead of DRM_* for printing
> - Turn off regulators in reverse order from enable
> - Silence errors in mantix_{shutdown,remove}
> - Drop duplicate mipi_dsi_dcs_enter_sleep_mode()
> https://lore.kernel.org/dri-devel/20200815100230.GA1002374@ravnborg.org/
> - Use dev_err_probe()
> - Add delays when turning off panel as suggested by the data sheet
>
> This series is against next-20200814.
>
> Guido Günther (3):
> dt-bindings: vendor-prefixes: Add mantix vendor prefix
> dt-bindings: Add Mantix MLAF057WE51-X panel bindings
> drm/panel: Add panel driver for the Mantix MLAF057WE51-X DSI panel
Maxime Ripard backmerged -rc1 to drm-misc-next so I went ahead and applied
this series to drm-misc-next, forgetting you got commit rights. As they are
already pushed out too late for you to do push them - sorry!
Sam
>
> .../display/panel/mantix,mlaf057we51-x.yaml | 70 ++++
> .../devicetree/bindings/vendor-prefixes.yaml | 2 +
> MAINTAINERS | 7 +
> drivers/gpu/drm/panel/Kconfig | 11 +
> drivers/gpu/drm/panel/Makefile | 1 +
> .../gpu/drm/panel/panel-mantix-mlaf057we51.c | 328 ++++++++++++++++++
> 6 files changed, 419 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/display/panel/mantix,mlaf057we51-x.yaml
> create mode 100644 drivers/gpu/drm/panel/panel-mantix-mlaf057we51.c
>
> --
> 2.26.2
next prev parent reply other threads:[~2020-08-18 17:06 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-17 8:14 [PATCH v3 0/3] drm/panel: Add panel driver for the Mantix MLAF057WE51-X DSI panel Guido Günther
2020-08-17 8:14 ` [PATCH v3 1/3] dt-bindings: vendor-prefixes: Add mantix vendor prefix Guido Günther
2020-08-17 8:14 ` [PATCH v3 2/3] dt-bindings: Add Mantix MLAF057WE51-X panel bindings Guido Günther
2020-08-17 8:14 ` [PATCH v3 3/3] drm/panel: Add panel driver for the Mantix MLAF057WE51-X DSI panel Guido Günther
2020-08-18 17:05 ` Sam Ravnborg [this message]
2020-08-18 17:33 ` [PATCH v3 0/3] " Guido Günther
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200818170556.GA2291850@ravnborg.org \
--to=sam@ravnborg.org \
--cc=agx@sigxcpu.org \
--cc=airlied@linux.ie \
--cc=allen.chen@ite.com.tw \
--cc=arnd@arndb.de \
--cc=broonie@kernel.org \
--cc=daniel@0x0f.com \
--cc=daniel@ffwll.ch \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=heiko.stuebner@theobroma-systems.com \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=linus.walleij@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkundrak@v3.sk \
--cc=mchehab+huawei@kernel.org \
--cc=robh+dt@kernel.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).