Re: [PATCH V7 0/6] drm: bridge: samsung-dsim: Support variable clocking

From: Adam Ford
Date: Thu May 25 2023 - 23:11:48 EST


On Thu, May 25, 2023 at 11:19 AM Neil Armstrong
<neil.armstrong@xxxxxxxxxx> wrote:
>
> On 25/05/2023 18:15, neil.armstrong@xxxxxxxxxx wrote:
> > On 25/05/2023 17:57, Adam Ford wrote:
> >> On Thu, May 25, 2023 at 10:39 AM Neil Armstrong
> >> <neil.armstrong@xxxxxxxxxx> wrote:
> >>>
> >>> On 24/05/2023 14:49, Adam Ford wrote:
> >>>> On Wed, May 24, 2023 at 7:45 AM Fabio Estevam <festevam@xxxxxxxxx> wrote:
> >>>>>
> >>>>> Hi Adam,
> >>>>>
> >>>>> On Tue, May 23, 2023 at 8:49 PM Adam Ford <aford173@xxxxxxxxx> wrote:
> >>>>>
> >>>>>> Inki,
> >>>>>>
> >>>>>> I haven't heard back from you on whether or not you want the bindings
> >>>>>> patch to be included with me resending the series as V7 or if you're
> >>>>>> OK with a single, stand-alone patch.
> >>>>>> Will you let me know? I have the patch standing by waiting for
> >>>>>> instructions. If you're not the right person to ask, please let me
> >>>>>> know who the right person is.
> >>>>>
> >>>>> Neil has also been collecting samsung-dsim patches. Maybe he can clarify.
> >>>>
> >>>> If it matters, my preference all along was to do the bindings as a
> >>>> separate thing once the driver updates were merged into the tree.
> >>>> Since the bindings can be done in different ways, I was hoping to have
> >>>> a separate discussion on the right way to do the bindings. If they
> >>>> need to be part of the series, I can do that.
> >>>
> >>> If you don't introduce compatibles, no need to send bindings, it can
> >>> be send separately.
> >>
> >> This series doesn't change any compatibility.
> >>
> >>>
> >>> Can I apply this serie and 20230503163313.2640898-2-frieder@xxxxxxx ? seems all has been reviewed.
> >>
> >> Looking at the driver, it looks like linux-next has some newer
> >> features added into the driver since I started, so this series might
> >> need a re-base. If that's the case, let me know, and I'll do the
> >> re-base.
> >
> > Ok I'll pull the other bits and let you know if this one needs a rebase.
>
> Indeed, starting at patch 3 it fails to apply, a rebase on drm-misc-next is welcome!

Neil,

I rebased and I added the dt-bindings as an additional patch to the
series. If people are unhappy with the bindings, I am hoping you can
apply the first 6 since they don't seem to break any backwards
compatibility, and we can discuss the bindings separately if
necessary.

adam

>
> Neil
>
> >
> > Neil
> >
> >>
> >> adam
> >>
> >>>
> >>> Neil
> >>>
> >>>>
> >>>> adam
> >>>
> >
>