Re: [PATCH 12/13] drm/i2c: tda998x: add bridge timing information - Russell King

From: Russell King - ARM Linux admin <linux@armlinux.org.uk>
To: Sven Van Asbroeck <thesven73@gmail.com>
Cc: David Airlie <airlied@linux.ie>,
	dri-devel@lists.freedesktop.org,
	Peter Ujfalusi <peter.ujfalusi@ti.com>,
	Mark Brown <broonie@kernel.org>, Jyri Sarha <jsarha@ti.com>
Subject: Re: [PATCH 12/13] drm/i2c: tda998x: add bridge timing information
Date: Thu, 13 Jun 2019 11:01:37 +0100	[thread overview]
Message-ID: <20190613100136.krfozwahi3ca73li@shell.armlinux.org.uk> (raw)
In-Reply-To: <CAGngYiUMCY7Mx30jNxznrUqyPp4D98GJbMOjtoQBOE4qJqf96w@mail.gmail.com>

On Wed, Jun 12, 2019 at 11:38:06AM -0400, Sven Van Asbroeck wrote:
> On Tue, Jun 11, 2019 at 7:04 AM Russell King <rmk+kernel@armlinux.org.uk> wrote:
> >
> > Add bridge timing information so that bridge users can figure out the
> > timing parameters that are necessary for TDA998x.
> >
> > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
> > ---
> 
> +static const struct drm_bridge_timings tda9989_timings = {
> +       .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE,
> +       .setup_time_ps = 1500,
> +       .hold_time_ps = 1000,
> +};
> +
> +static const struct drm_bridge_timings tda19988_timings = {
> +       .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE,
> +       .setup_time_ps = 1600,
> +       .hold_time_ps = 1200,
> +};
> 
> Need to port these to 5.1 kernel: sampling_edge -> input_bus_flags ?

5.1 still has it as "sampling_edge", but 5.2-rc changes this, so
I'll drop this patch from this series for now.  Thanks for pointing
that out.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel