All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: Jagan Teki <jagan@amarulasolutions.com>,
	Fabio Estevam <festevam@gmail.com>,
	Dave Stevenson <dave.stevenson@raspberrypi.com>,
	Maxime Ripard <maxime@cerno.tech>
Cc: Andrzej Hajda <andrzej.hajda@intel.com>,
	Inki Dae <inki.dae@samsung.com>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Frieder Schrempf <frieder.schrempf@kontron.de>,
	Fancy Fang <chen.fang@nxp.com>,
	Tim Harvey <tharvey@gateworks.com>,
	Michael Nazzareno Trimarchi <michael@amarulasolutions.com>,
	Adam Ford <aford173@gmail.com>,
	Neil Armstrong <narmstrong@linaro.org>,
	Robert Foss <robert.foss@linaro.org>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
	Tommaso Merciai <tommaso.merciai@amarulasolutions.com>,
	linux-samsung-soc@vger.kernel.org,
	Matteo Lisi <matteo.lisi@engicam.com>,
	dri-devel@lists.freedesktop.org,
	NXP Linux Team <linux-imx@nxp.com>,
	linux-amarula <linux-amarula@amarulasolutions.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v10 00/18] drm: Add Samsung MIPI DSIM bridge
Date: Fri, 20 Jan 2023 16:06:09 +0100	[thread overview]
Message-ID: <ea400d44-a6cb-2f26-9c03-ee1ede2cdf09@denx.de> (raw)
In-Reply-To: <CAMty3ZAc=t5FEphQkd=O1eaA70-779zhESwPFqoiGs8x569H2w@mail.gmail.com>

On 1/20/23 15:41, Jagan Teki wrote:
> Hi Fabio,

Hello all,

> On Fri, Jan 20, 2023 at 5:36 PM Fabio Estevam <festevam@gmail.com> wrote:
>>
>> Hi Jagan,
>>
>> On Thu, Jan 19, 2023 at 2:59 PM Jagan Teki <jagan@amarulasolutions.com> wrote:
>>
>>> There are two patch series prior to this need to apply.
>>>
>>> https://patchwork.kernel.org/project/dri-devel/patch/20221212145745.15387-1-jagan@amarulasolutions.com/
>>> https://patchwork.kernel.org/project/dri-devel/cover/20221212182923.29155-1-jagan@amarulasolutions.com/
>>
>> Would it make sense to re-submit these two patches as part of your series?
> 
> The previous version's comment was to separate them from the DSIM series.

Hmmmmm, seems like those first two patches got stuck. I fixed up the 
malformed Fixes: line (it was split across two lines and had angular 
brackets around it) and picked the first series via drm-misc-next .

Can you send a subsequent patch to convert the DSIM_* macros to BIT() 
macro , since checkpatch --strict complains about it ?

For the second series, you likely want a RB from Maxime Ripard and Dave 
Stevenson first about the probe order handling.

WARNING: multiple messages have this Message-ID (diff)
From: Marek Vasut <marex@denx.de>
To: Jagan Teki <jagan@amarulasolutions.com>,
	Fabio Estevam <festevam@gmail.com>,
	Dave Stevenson <dave.stevenson@raspberrypi.com>,
	Maxime Ripard <maxime@cerno.tech>
Cc: dri-devel@lists.freedesktop.org,
	linux-samsung-soc@vger.kernel.org,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	Tommaso Merciai <tommaso.merciai@amarulasolutions.com>,
	linux-amarula <linux-amarula@amarulasolutions.com>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	Neil Armstrong <narmstrong@linaro.org>,
	Frieder Schrempf <frieder.schrempf@kontron.de>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Matteo Lisi <matteo.lisi@engicam.com>,
	Robert Foss <robert.foss@linaro.org>,
	Andrzej Hajda <andrzej.hajda@intel.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	Fancy Fang <chen.fang@nxp.com>,
	Michael Nazzareno Trimarchi <michael@amarulasolutions.com>,
	Adam Ford <aford173@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	Marek Szyprowski <m.szyprowski@samsung.com>
Subject: Re: [PATCH v10 00/18] drm: Add Samsung MIPI DSIM bridge
Date: Fri, 20 Jan 2023 16:06:09 +0100	[thread overview]
Message-ID: <ea400d44-a6cb-2f26-9c03-ee1ede2cdf09@denx.de> (raw)
In-Reply-To: <CAMty3ZAc=t5FEphQkd=O1eaA70-779zhESwPFqoiGs8x569H2w@mail.gmail.com>

On 1/20/23 15:41, Jagan Teki wrote:
> Hi Fabio,

Hello all,

> On Fri, Jan 20, 2023 at 5:36 PM Fabio Estevam <festevam@gmail.com> wrote:
>>
>> Hi Jagan,
>>
>> On Thu, Jan 19, 2023 at 2:59 PM Jagan Teki <jagan@amarulasolutions.com> wrote:
>>
>>> There are two patch series prior to this need to apply.
>>>
>>> https://patchwork.kernel.org/project/dri-devel/patch/20221212145745.15387-1-jagan@amarulasolutions.com/
>>> https://patchwork.kernel.org/project/dri-devel/cover/20221212182923.29155-1-jagan@amarulasolutions.com/
>>
>> Would it make sense to re-submit these two patches as part of your series?
> 
> The previous version's comment was to separate them from the DSIM series.

Hmmmmm, seems like those first two patches got stuck. I fixed up the 
malformed Fixes: line (it was split across two lines and had angular 
brackets around it) and picked the first series via drm-misc-next .

Can you send a subsequent patch to convert the DSIM_* macros to BIT() 
macro , since checkpatch --strict complains about it ?

For the second series, you likely want a RB from Maxime Ripard and Dave 
Stevenson first about the probe order handling.

WARNING: multiple messages have this Message-ID (diff)
From: Marek Vasut <marex@denx.de>
To: Jagan Teki <jagan@amarulasolutions.com>,
	Fabio Estevam <festevam@gmail.com>,
	Dave Stevenson <dave.stevenson@raspberrypi.com>,
	Maxime Ripard <maxime@cerno.tech>
Cc: Andrzej Hajda <andrzej.hajda@intel.com>,
	Inki Dae <inki.dae@samsung.com>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Frieder Schrempf <frieder.schrempf@kontron.de>,
	Fancy Fang <chen.fang@nxp.com>,
	Tim Harvey <tharvey@gateworks.com>,
	Michael Nazzareno Trimarchi <michael@amarulasolutions.com>,
	Adam Ford <aford173@gmail.com>,
	Neil Armstrong <narmstrong@linaro.org>,
	Robert Foss <robert.foss@linaro.org>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
	Tommaso Merciai <tommaso.merciai@amarulasolutions.com>,
	linux-samsung-soc@vger.kernel.org,
	Matteo Lisi <matteo.lisi@engicam.com>,
	dri-devel@lists.freedesktop.org,
	NXP Linux Team <linux-imx@nxp.com>,
	linux-amarula <linux-amarula@amarulasolutions.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v10 00/18] drm: Add Samsung MIPI DSIM bridge
Date: Fri, 20 Jan 2023 16:06:09 +0100	[thread overview]
Message-ID: <ea400d44-a6cb-2f26-9c03-ee1ede2cdf09@denx.de> (raw)
In-Reply-To: <CAMty3ZAc=t5FEphQkd=O1eaA70-779zhESwPFqoiGs8x569H2w@mail.gmail.com>

On 1/20/23 15:41, Jagan Teki wrote:
> Hi Fabio,

Hello all,

> On Fri, Jan 20, 2023 at 5:36 PM Fabio Estevam <festevam@gmail.com> wrote:
>>
>> Hi Jagan,
>>
>> On Thu, Jan 19, 2023 at 2:59 PM Jagan Teki <jagan@amarulasolutions.com> wrote:
>>
>>> There are two patch series prior to this need to apply.
>>>
>>> https://patchwork.kernel.org/project/dri-devel/patch/20221212145745.15387-1-jagan@amarulasolutions.com/
>>> https://patchwork.kernel.org/project/dri-devel/cover/20221212182923.29155-1-jagan@amarulasolutions.com/
>>
>> Would it make sense to re-submit these two patches as part of your series?
> 
> The previous version's comment was to separate them from the DSIM series.

Hmmmmm, seems like those first two patches got stuck. I fixed up the 
malformed Fixes: line (it was split across two lines and had angular 
brackets around it) and picked the first series via drm-misc-next .

Can you send a subsequent patch to convert the DSIM_* macros to BIT() 
macro , since checkpatch --strict complains about it ?

For the second series, you likely want a RB from Maxime Ripard and Dave 
Stevenson first about the probe order handling.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2023-01-20 15:06 UTC|newest]

Thread overview: 169+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-14 12:58 [PATCH v10 00/18] drm: Add Samsung MIPI DSIM bridge Jagan Teki
2022-12-14 12:58 ` Jagan Teki
2022-12-14 12:58 ` Jagan Teki
2022-12-14 12:58 ` [PATCH v10 01/18] drm: of: Lookup if child node has DSI panel or bridge Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58 ` [PATCH v10 02/18] drm: bridge: panel: Add devm_drm_of_dsi_get_bridge helper Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58 ` [PATCH v10 03/18] drm: exynos: dsi: Drop explicit call to bridge detach Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58 ` [PATCH v10 04/18] drm: exynos: dsi: Switch to devm_drm_of_dsi_get_bridge Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58 ` [PATCH v10 05/18] drm: exynos: dsi: Mark PHY as optional Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-15  8:33   ` Frieder Schrempf
2022-12-15  8:33     ` Frieder Schrempf
2022-12-15  8:33     ` Frieder Schrempf
2022-12-14 12:58 ` [PATCH v10 06/18] drm: exynos: dsi: Add platform PLL_P (PMS_P) offset Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58 ` [PATCH v10 07/18] drm: exynos: dsi: Introduce hw_type platform data Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-15  8:35   ` Frieder Schrempf
2022-12-15  8:35     ` Frieder Schrempf
2022-12-15  8:35     ` Frieder Schrempf
2022-12-14 12:58 ` [PATCH v10 08/18] drm: exynos: dsi: Handle proper host initialization Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-15  8:37   ` Frieder Schrempf
2022-12-15  8:37     ` Frieder Schrempf
2022-12-15  8:37     ` Frieder Schrempf
2022-12-14 12:58 ` [PATCH v10 09/18] drm: exynos: dsi: Add atomic check Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-15  8:39   ` Frieder Schrempf
2022-12-15  8:39     ` Frieder Schrempf
2022-12-15  8:39     ` Frieder Schrempf
2022-12-15  8:40     ` Frieder Schrempf
2022-12-15  8:40       ` Frieder Schrempf
2022-12-15  8:40       ` Frieder Schrempf
2022-12-14 12:58 ` [PATCH v10 10/18] drm: exynos: dsi: Add input_bus_flags Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-14 12:58   ` Jagan Teki
2022-12-15  8:42   ` Frieder Schrempf
2022-12-15  8:42     ` Frieder Schrempf
2022-12-15  8:42     ` Frieder Schrempf
2022-12-14 12:59 ` [PATCH v10 11/18] drm: exynos: dsi: Add atomic_get_input_bus_fmts Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-15  8:57   ` Frieder Schrempf
2022-12-15  8:57     ` Frieder Schrempf
2022-12-15  8:57     ` Frieder Schrempf
2022-12-14 12:59 ` [PATCH v10 12/18] drm: exynos: dsi: Consolidate component and bridge Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59 ` [PATCH v10 13/18] drm: exynos: dsi: Add Exynos based host irq hooks Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59 ` [PATCH v10 14/18] drm: bridge: Generalize Exynos-DSI driver into a Samsung DSIM bridge Jagan Teki
2022-12-14 12:59 ` [PATCH v10 15/18] dt-bindings: display: exynos: dsim: Add NXP i.MX8M Mini/Nano support Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59 ` [PATCH v10 16/18] drm: bridge: samsung-dsim: Add " Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-15  9:22   ` Frieder Schrempf
2022-12-15  9:22     ` Frieder Schrempf
2022-12-15  9:22     ` Frieder Schrempf
2022-12-15 11:39     ` Robert Foss
2022-12-15 11:39       ` Robert Foss
2022-12-15 11:39       ` Robert Foss
2022-12-14 12:59 ` [PATCH v10 17/18] dt-bindings: display: exynos: dsim: Add NXP i.MX8M Plus support Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-20 17:26   ` Rob Herring
2022-12-20 17:26     ` Rob Herring
2022-12-20 17:26     ` Rob Herring
2022-12-14 12:59 ` [PATCH v10 18/18] drm: bridge: samsung-dsim: Add " Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-14 12:59   ` Jagan Teki
2022-12-15  9:23   ` Frieder Schrempf
2022-12-15  9:23     ` Frieder Schrempf
2022-12-15  9:23     ` Frieder Schrempf
2022-12-15 11:39     ` Robert Foss
2022-12-15 11:39       ` Robert Foss
2022-12-15 11:39       ` Robert Foss
2023-01-05 10:24 ` [PATCH v10 00/18] drm: Add Samsung MIPI DSIM bridge Jagan Teki
2023-01-05 10:24   ` Jagan Teki
2023-01-05 10:24   ` Jagan Teki
2023-01-06 14:34   ` Adam Ford
2023-01-06 14:34     ` Adam Ford
2023-01-06 14:34     ` Adam Ford
2023-01-06 14:42     ` Fabio Estevam
2023-01-06 14:42       ` Fabio Estevam
2023-01-06 14:42       ` Fabio Estevam
2023-01-19 17:27   ` Fabio Estevam
2023-01-19 17:27     ` Fabio Estevam
2023-01-19 17:27     ` Fabio Estevam
2023-01-19 17:58     ` Jagan Teki
2023-01-19 17:58       ` Jagan Teki
2023-01-19 17:58       ` Jagan Teki
2023-01-20 12:06       ` Fabio Estevam
2023-01-20 12:06         ` Fabio Estevam
2023-01-20 12:06         ` Fabio Estevam
2023-01-20 14:41         ` Jagan Teki
2023-01-20 14:41           ` Jagan Teki
2023-01-20 14:41           ` Jagan Teki
2023-01-20 15:06           ` Marek Vasut [this message]
2023-01-20 15:06             ` Marek Vasut
2023-01-20 15:06             ` Marek Vasut
2023-01-20 18:54             ` Jagan Teki
2023-01-20 18:54               ` Jagan Teki
2023-01-20 18:54               ` Jagan Teki
2023-01-20 19:08               ` Marek Vasut
2023-01-20 19:08                 ` Marek Vasut
2023-01-20 19:08                 ` Marek Vasut
2023-01-20 18:59             ` Dave Stevenson
2023-01-20 18:59               ` Dave Stevenson
2023-01-20 18:59               ` Dave Stevenson
2023-01-23 12:22               ` Jagan Teki
2023-01-23 12:22                 ` Jagan Teki
2023-01-23 12:22                 ` Jagan Teki
2023-01-23 15:48             ` Jagan Teki
2023-01-23 15:48               ` Jagan Teki
2023-01-23 15:48               ` Jagan Teki
2022-12-16 12:41 Alexander Stein
2022-12-16 12:41 ` Alexander Stein
2022-12-16 12:41 ` Alexander Stein
2022-12-16 12:58 ` Marek Vasut
2022-12-16 12:58   ` Marek Vasut
2022-12-16 12:58   ` Marek Vasut
2022-12-16 13:25 Alexander Stein
2022-12-16 13:25 ` Alexander Stein
2022-12-16 13:25 ` Alexander Stein
2022-12-17 23:55 ` Marek Vasut
2022-12-17 23:55   ` Marek Vasut
2022-12-17 23:55   ` Marek Vasut
2022-12-18  4:23   ` Adam Ford
2022-12-18  4:23     ` Adam Ford
2022-12-18  4:23     ` Adam Ford
2022-12-18  4:33     ` Marek Vasut
2022-12-18  4:33       ` Marek Vasut
2022-12-18  4:33       ` Marek Vasut
2022-12-18 22:24       ` Adam Ford
2022-12-18 22:24         ` Adam Ford
2022-12-18 22:24         ` Adam Ford
2022-12-18 22:28         ` Marek Vasut
2022-12-18 22:28           ` Marek Vasut
2022-12-18 22:28           ` Marek Vasut
2023-01-03 10:59           ` Alexander Stein
2023-01-03 10:59             ` Alexander Stein
2023-01-03 10:59             ` Alexander Stein
2023-01-04 15:08             ` Marek Vasut
2023-01-04 15:08               ` Marek Vasut
2023-01-04 15:08               ` Marek Vasut
2023-01-20 19:10               ` Maxime Ripard
2023-01-20 19:10                 ` Maxime Ripard
2023-01-20 19:10                 ` Maxime Ripard
2023-01-03  9:51   ` Alexander Stein
2023-01-03  9:51     ` Alexander Stein
2023-01-03  9:51     ` Alexander Stein
2023-01-04 15:07     ` Marek Vasut
2023-01-04 15:07       ` Marek Vasut
2023-01-04 15:07       ` Marek Vasut

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=ea400d44-a6cb-2f26-9c03-ee1ede2cdf09@denx.de \
    --to=marex@denx.de \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=aford173@gmail.com \
    --cc=andrzej.hajda@intel.com \
    --cc=chen.fang@nxp.com \
    --cc=dave.stevenson@raspberrypi.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=festevam@gmail.com \
    --cc=frieder.schrempf@kontron.de \
    --cc=inki.dae@samsung.com \
    --cc=jagan@amarulasolutions.com \
    --cc=jy0922.shim@samsung.com \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-amarula@amarulasolutions.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=matteo.lisi@engicam.com \
    --cc=maxime@cerno.tech \
    --cc=michael@amarulasolutions.com \
    --cc=narmstrong@linaro.org \
    --cc=robert.foss@linaro.org \
    --cc=sw0312.kim@samsung.com \
    --cc=tharvey@gateworks.com \
    --cc=tommaso.merciai@amarulasolutions.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.