Re: [PATCH v2] dt-bindings: bridge: Convert Samsung MIPI DSIM bridge to yaml

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 3, 2023 at 7:13 PM Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxx>
> >>> Signed-off-by: Fabio Estevam <festevam@xxxxxxx>
> >>> ---
> >>> 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@xxxxxxxxxx/#t
> >
> > Jagan pointed out an earlier submission he did in 2021:
> >
> > https://lore.kernel.org/all/20210704090230.26489-9-jagan@xxxxxxxxxxxxxxxxxxxx/
> >
> > 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.




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux