From: Jagan Teki <jagan@amarulasolutions.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Fabio Estevam <festevam@denx.de>,
Fabio Estevam <festevam@gmail.com>,
neil.armstrong@linaro.org, marex@denx.de, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org,
dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
inki.dae@samsung.com
Subject: Re: [PATCH v2] dt-bindings: bridge: Convert Samsung MIPI DSIM bridge to yaml
Date: Mon, 3 Apr 2023 20:06:48 +0530 [thread overview]
Message-ID: <CAMty3ZDKTpTAHxqt6320NRVoABwUMOJjBPpUmPpZeP264wLeFg@mail.gmail.com> (raw)
In-Reply-To: <2ed6078f-7982-a5aa-863b-e61f8e9f846a@linaro.org>
On Mon, Apr 3, 2023 at 7:13 PM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 03/04/2023 15:25, Fabio Estevam wrote:
> > Hi Krzysztof,
> >
> > On 03/04/2023 09:49, Krzysztof Kozlowski wrote:
> >
> >>> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> >>> Signed-off-by: Fabio Estevam <festevam@denx.de>
> >>> ---
> >>> Changes since v1:
> >>> - Added samsung,mipi-dsim.yaml entry to MAINTAINERS file (Jagan)
> >>> - Added Marek Szyprowski entry to the samsung,mipi-dsim.yaml
> >>> maintainers section (Jagan)
> >>> - Mention that i.MX8M Plus is also supported (Marek)
> >>> - Remove endpoint@0 description as it only has one endpoint (Marek)
> >>
> >> Where is the changelog from original submission? How your v1 differs
> >> form it? Or did you just ignore all the feedback?
> >
> > I'm sorry, but it was not my intention to ignore any feedback.
> >
> > Which feedback are you referring to specifically?
> >
> > Some more context: last week I sent a patch adding a new property
> > for exynos_dsim.txt and you asked me to convert it to yaml first:
> >
> > https://lore.kernel.org/all/ff66c8b9-c7f7-1eb2-c730-4812b7ff6824@linaro.org/#t
> >
> > Jagan pointed out an earlier submission he did in 2021:
> >
> > https://lore.kernel.org/all/20210704090230.26489-9-jagan@amarulasolutions.com/
> >
> > That was my starting point.
>
> I think I saw v11 but maybe not. I cannot find it now. But anyway if
> this was the newest submission, it already got feedback which we do not
> need to repeat...
Are you referring to v11 based on DSIM series versioning? If so, the
DSIM series from v1 onwards never had this patch of converting .txt to
.yaml that series instead added the new dt-bindings on top of existing
.txt. The actual conversion patch from .txt to .yaml was initially
sent as RFC, so Fabio's versioning seems correct to me.
Thanks,
Jagan.
prev parent reply other threads:[~2023-04-03 14:37 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-03 12:44 [PATCH v2] dt-bindings: bridge: Convert Samsung MIPI DSIM bridge to yaml Fabio Estevam
2023-04-03 12:49 ` Krzysztof Kozlowski
2023-04-03 13:25 ` Fabio Estevam
2023-04-03 13:43 ` Krzysztof Kozlowski
2023-04-03 14:36 ` Jagan Teki [this message]
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=CAMty3ZDKTpTAHxqt6320NRVoABwUMOJjBPpUmPpZeP264wLeFg@mail.gmail.com \
--to=jagan@amarulasolutions.com \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=festevam@denx.de \
--cc=festevam@gmail.com \
--cc=inki.dae@samsung.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=marex@denx.de \
--cc=neil.armstrong@linaro.org \
--cc=robh+dt@kernel.org \
/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).