* [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS @ 2021-11-10 22:00 Jiri Vanek 2021-11-18 14:50 ` Robert Foss 2022-01-06 19:00 ` [PATCH v2] " Jiri Vanek 0 siblings, 2 replies; 13+ messages in thread From: Jiri Vanek @ 2021-11-10 22:00 UTC (permalink / raw) To: Andrzej Hajda, Neil Armstrong, Robert Foss Cc: Jernej Skrabec, Jonas Karlman, David Airlie, Jiri Vanek, linux-kernel, dri-devel, Laurent Pinchart Fixed wrong register shift for single/dual link LVDS output. Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> --- drivers/gpu/drm/bridge/tc358775.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c index 2272adcc5b4a..1d6ec1baeff2 100644 --- a/drivers/gpu/drm/bridge/tc358775.c +++ b/drivers/gpu/drm/bridge/tc358775.c @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) } #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 -#define TC358775_LVCFG_LVDLINK__SHIFT 0 +#define TC358775_LVCFG_LVDLINK__SHIFT 1 static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) { return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & -- 2.30.2 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS 2021-11-10 22:00 [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS Jiri Vanek @ 2021-11-18 14:50 ` Robert Foss 2021-12-14 8:13 ` Vinay Simha B N 2022-01-06 19:00 ` [PATCH v2] " Jiri Vanek 1 sibling, 1 reply; 13+ messages in thread From: Robert Foss @ 2021-11-18 14:50 UTC (permalink / raw) To: Jiri Vanek, Zheng Bin, Vinay Simha BN Cc: Jonas Karlman, David Airlie, dri-devel, Neil Armstrong, linux-kernel, Jernej Skrabec, Andrzej Hajda, Laurent Pinchart + Zhen & Vinay This patch looks good in itself, but I would like to see a tested by tag. At the very least testing for regression in single-link LVDS but ideally some third party verification of this patch. On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <jirivanek1@gmail.com> wrote: > > Fixed wrong register shift for single/dual link LVDS output. > > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> > --- > drivers/gpu/drm/bridge/tc358775.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c > index 2272adcc5b4a..1d6ec1baeff2 100644 > --- a/drivers/gpu/drm/bridge/tc358775.c > +++ b/drivers/gpu/drm/bridge/tc358775.c > @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) > } > > #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) > { > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & > -- > 2.30.2 > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS 2021-11-18 14:50 ` Robert Foss @ 2021-12-14 8:13 ` Vinay Simha B N 2022-01-04 13:50 ` Robert Foss 0 siblings, 1 reply; 13+ messages in thread From: Vinay Simha B N @ 2021-12-14 8:13 UTC (permalink / raw) To: robert.foss Cc: Jiri Vanek, Jonas Karlman, David Airlie, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Jernej Skrabec, Zheng Bin, Andrzej Hajda, Laurent Pinchart [-- Attachment #1: Type: text/plain, Size: 1396 bytes --] Robert, I do not have the hardware to test this feature. Sorry for the late response. On Thu, Nov 18, 2021 at 8:20 PM Robert Foss <robert.foss@linaro.org> wrote: > + Zhen & Vinay > > This patch looks good in itself, but I would like to see a tested by > tag. At the very least testing for regression in single-link LVDS but > ideally some third party verification of this patch. > > On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <jirivanek1@gmail.com> wrote: > > > > Fixed wrong register shift for single/dual link LVDS output. > > > > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> > > --- > > drivers/gpu/drm/bridge/tc358775.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/bridge/tc358775.c > b/drivers/gpu/drm/bridge/tc358775.c > > index 2272adcc5b4a..1d6ec1baeff2 100644 > > --- a/drivers/gpu/drm/bridge/tc358775.c > > +++ b/drivers/gpu/drm/bridge/tc358775.c > > @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t > val) > > } > > > > #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 > > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 > > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 > > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) > > { > > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & > > -- > > 2.30.2 > > > -- regards, vinaysimha [-- Attachment #2: Type: text/html, Size: 2122 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS 2021-12-14 8:13 ` Vinay Simha B N @ 2022-01-04 13:50 ` Robert Foss 2022-01-04 14:29 ` Jiří Vaněk 2022-01-04 16:56 ` Vinay Simha B N 0 siblings, 2 replies; 13+ messages in thread From: Robert Foss @ 2022-01-04 13:50 UTC (permalink / raw) To: Vinay Simha B N Cc: Jiri Vanek, Jonas Karlman, David Airlie, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Jernej Skrabec, Zheng Bin, Andrzej Hajda, Laurent Pinchart Jiri: Are you able to test this patch? Vinay: Could you supply a R-b tag, if you feel that it is warranted? On Tue, 14 Dec 2021 at 09:13, Vinay Simha B N <simhavcs@gmail.com> wrote: > > Robert, > I do not have the hardware to test this feature. Sorry for the late response. > > On Thu, Nov 18, 2021 at 8:20 PM Robert Foss <robert.foss@linaro.org> wrote: >> >> + Zhen & Vinay >> >> This patch looks good in itself, but I would like to see a tested by >> tag. At the very least testing for regression in single-link LVDS but >> ideally some third party verification of this patch. >> >> On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <jirivanek1@gmail.com> wrote: >> > >> > Fixed wrong register shift for single/dual link LVDS output. >> > >> > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> >> > --- >> > drivers/gpu/drm/bridge/tc358775.c | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c >> > index 2272adcc5b4a..1d6ec1baeff2 100644 >> > --- a/drivers/gpu/drm/bridge/tc358775.c >> > +++ b/drivers/gpu/drm/bridge/tc358775.c >> > @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) >> > } >> > >> > #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 >> > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 >> > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 >> > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) >> > { >> > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & >> > -- >> > 2.30.2 >> > > > > > -- > regards, > vinaysimha ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS 2022-01-04 13:50 ` Robert Foss @ 2022-01-04 14:29 ` Jiří Vaněk 2022-01-04 14:36 ` Robert Foss 2022-01-04 17:04 ` Vinay Simha B N 2022-01-04 16:56 ` Vinay Simha B N 1 sibling, 2 replies; 13+ messages in thread From: Jiří Vaněk @ 2022-01-04 14:29 UTC (permalink / raw) To: Robert Foss Cc: Jonas Karlman, David Airlie, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Jernej Skrabec, Zheng Bin, Andrzej Hajda, Laurent Pinchart, Vinay Simha B N [-- Attachment #1: Type: text/plain, Size: 2025 bytes --] Actually, this patch is based on testing with a real HW with dual-link LVDS display (full HD) and it also matches with a datasheet. Without this fix it does not work at all. út 4. 1. 2022 v 14:51 odesílatel Robert Foss <robert.foss@linaro.org> napsal: > Jiri: Are you able to test this patch? > > Vinay: Could you supply a R-b tag, if you feel that it is warranted? > > On Tue, 14 Dec 2021 at 09:13, Vinay Simha B N <simhavcs@gmail.com> wrote: > > > > Robert, > > I do not have the hardware to test this feature. Sorry for the late > response. > > > > On Thu, Nov 18, 2021 at 8:20 PM Robert Foss <robert.foss@linaro.org> > wrote: > >> > >> + Zhen & Vinay > >> > >> This patch looks good in itself, but I would like to see a tested by > >> tag. At the very least testing for regression in single-link LVDS but > >> ideally some third party verification of this patch. > >> > >> On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <jirivanek1@gmail.com> wrote: > >> > > >> > Fixed wrong register shift for single/dual link LVDS output. > >> > > >> > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> > >> > --- > >> > drivers/gpu/drm/bridge/tc358775.c | 2 +- > >> > 1 file changed, 1 insertion(+), 1 deletion(-) > >> > > >> > diff --git a/drivers/gpu/drm/bridge/tc358775.c > b/drivers/gpu/drm/bridge/tc358775.c > >> > index 2272adcc5b4a..1d6ec1baeff2 100644 > >> > --- a/drivers/gpu/drm/bridge/tc358775.c > >> > +++ b/drivers/gpu/drm/bridge/tc358775.c > >> > @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t > val) > >> > } > >> > > >> > #define TC358775_LVCFG_LVDLINK__MASK > 0x00000002 > >> > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 > >> > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 > >> > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) > >> > { > >> > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & > >> > -- > >> > 2.30.2 > >> > > > > > > > > > -- > > regards, > > vinaysimha > [-- Attachment #2: Type: text/html, Size: 3044 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS 2022-01-04 14:29 ` Jiří Vaněk @ 2022-01-04 14:36 ` Robert Foss 2022-01-04 17:04 ` Vinay Simha B N 1 sibling, 0 replies; 13+ messages in thread From: Robert Foss @ 2022-01-04 14:36 UTC (permalink / raw) To: Jiří Vaněk Cc: Jonas Karlman, David Airlie, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Jernej Skrabec, Zheng Bin, Andrzej Hajda, Laurent Pinchart, Vinay Simha B N Excellent. Jiri, can you add your Tested-by tag to this patch? On Tue, 4 Jan 2022 at 15:29, Jiří Vaněk <jirivanek1@gmail.com> wrote: > > Actually, this patch is based on testing with a real HW with dual-link LVDS display (full HD) and it also matches with a datasheet. Without this fix it does not work at all. > > út 4. 1. 2022 v 14:51 odesílatel Robert Foss <robert.foss@linaro.org> napsal: >> >> Jiri: Are you able to test this patch? >> >> Vinay: Could you supply a R-b tag, if you feel that it is warranted? >> >> On Tue, 14 Dec 2021 at 09:13, Vinay Simha B N <simhavcs@gmail.com> wrote: >> > >> > Robert, >> > I do not have the hardware to test this feature. Sorry for the late response. >> > >> > On Thu, Nov 18, 2021 at 8:20 PM Robert Foss <robert.foss@linaro.org> wrote: >> >> >> >> + Zhen & Vinay >> >> >> >> This patch looks good in itself, but I would like to see a tested by >> >> tag. At the very least testing for regression in single-link LVDS but >> >> ideally some third party verification of this patch. >> >> >> >> On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <jirivanek1@gmail.com> wrote: >> >> > >> >> > Fixed wrong register shift for single/dual link LVDS output. >> >> > >> >> > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> >> >> > --- >> >> > drivers/gpu/drm/bridge/tc358775.c | 2 +- >> >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> >> > >> >> > diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c >> >> > index 2272adcc5b4a..1d6ec1baeff2 100644 >> >> > --- a/drivers/gpu/drm/bridge/tc358775.c >> >> > +++ b/drivers/gpu/drm/bridge/tc358775.c >> >> > @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) >> >> > } >> >> > >> >> > #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 >> >> > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 >> >> > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 >> >> > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) >> >> > { >> >> > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & >> >> > -- >> >> > 2.30.2 >> >> > >> > >> > >> > >> > -- >> > regards, >> > vinaysimha ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS 2022-01-04 14:29 ` Jiří Vaněk 2022-01-04 14:36 ` Robert Foss @ 2022-01-04 17:04 ` Vinay Simha B N 1 sibling, 0 replies; 13+ messages in thread From: Vinay Simha B N @ 2022-01-04 17:04 UTC (permalink / raw) To: Jiří Vaněk Cc: Jernej Skrabec, Jonas Karlman, David Airlie, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Robert Foss, Zheng Bin, Andrzej Hajda, Laurent Pinchart Jiri Vanek, Could you please share the part number or datasheet of the dual-link LVDS display used. On Tue, Jan 4, 2022 at 7:59 PM Jiří Vaněk <jirivanek1@gmail.com> wrote: > > Actually, this patch is based on testing with a real HW with dual-link LVDS display (full HD) and it also matches with a datasheet. Without this fix it does not work at all. > > út 4. 1. 2022 v 14:51 odesílatel Robert Foss <robert.foss@linaro.org> napsal: >> >> Jiri: Are you able to test this patch? >> >> Vinay: Could you supply a R-b tag, if you feel that it is warranted? >> >> On Tue, 14 Dec 2021 at 09:13, Vinay Simha B N <simhavcs@gmail.com> wrote: >> > >> > Robert, >> > I do not have the hardware to test this feature. Sorry for the late response. >> > >> > On Thu, Nov 18, 2021 at 8:20 PM Robert Foss <robert.foss@linaro.org> wrote: >> >> >> >> + Zhen & Vinay >> >> >> >> This patch looks good in itself, but I would like to see a tested by >> >> tag. At the very least testing for regression in single-link LVDS but >> >> ideally some third party verification of this patch. >> >> >> >> On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <jirivanek1@gmail.com> wrote: >> >> > >> >> > Fixed wrong register shift for single/dual link LVDS output. >> >> > >> >> > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> >> >> > --- >> >> > drivers/gpu/drm/bridge/tc358775.c | 2 +- >> >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> >> > >> >> > diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c >> >> > index 2272adcc5b4a..1d6ec1baeff2 100644 >> >> > --- a/drivers/gpu/drm/bridge/tc358775.c >> >> > +++ b/drivers/gpu/drm/bridge/tc358775.c >> >> > @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) >> >> > } >> >> > >> >> > #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 >> >> > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 >> >> > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 >> >> > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) >> >> > { >> >> > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & >> >> > -- >> >> > 2.30.2 >> >> > >> > >> > >> > >> > -- >> > regards, >> > vinaysimha -- regards, vinaysimha ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS 2022-01-04 13:50 ` Robert Foss 2022-01-04 14:29 ` Jiří Vaněk @ 2022-01-04 16:56 ` Vinay Simha B N 2022-01-04 17:09 ` Robert Foss 1 sibling, 1 reply; 13+ messages in thread From: Vinay Simha B N @ 2022-01-04 16:56 UTC (permalink / raw) To: Robert Foss Cc: Jiri Vanek, Jonas Karlman, David Airlie, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Jernej Skrabec, Zheng Bin, Andrzej Hajda, Laurent Pinchart Robert, What is R-b tag? On Tue, Jan 4, 2022 at 7:21 PM Robert Foss <robert.foss@linaro.org> wrote: > > Jiri: Are you able to test this patch? > > Vinay: Could you supply a R-b tag, if you feel that it is warranted? > > On Tue, 14 Dec 2021 at 09:13, Vinay Simha B N <simhavcs@gmail.com> wrote: > > > > Robert, > > I do not have the hardware to test this feature. Sorry for the late response. > > > > On Thu, Nov 18, 2021 at 8:20 PM Robert Foss <robert.foss@linaro.org> wrote: > >> > >> + Zhen & Vinay > >> > >> This patch looks good in itself, but I would like to see a tested by > >> tag. At the very least testing for regression in single-link LVDS but > >> ideally some third party verification of this patch. > >> > >> On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <jirivanek1@gmail.com> wrote: > >> > > >> > Fixed wrong register shift for single/dual link LVDS output. > >> > > >> > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> > >> > --- > >> > drivers/gpu/drm/bridge/tc358775.c | 2 +- > >> > 1 file changed, 1 insertion(+), 1 deletion(-) > >> > > >> > diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c > >> > index 2272adcc5b4a..1d6ec1baeff2 100644 > >> > --- a/drivers/gpu/drm/bridge/tc358775.c > >> > +++ b/drivers/gpu/drm/bridge/tc358775.c > >> > @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) > >> > } > >> > > >> > #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 > >> > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 > >> > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 > >> > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) > >> > { > >> > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & > >> > -- > >> > 2.30.2 > >> > > > > > > > > > -- > > regards, > > vinaysimha -- regards, vinaysimha ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS 2022-01-04 16:56 ` Vinay Simha B N @ 2022-01-04 17:09 ` Robert Foss 0 siblings, 0 replies; 13+ messages in thread From: Robert Foss @ 2022-01-04 17:09 UTC (permalink / raw) To: Vinay Simha B N Cc: Jiri Vanek, Jonas Karlman, David Airlie, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Jernej Skrabec, Zheng Bin, Andrzej Hajda, Laurent Pinchart [-- Attachment #1: Type: text/plain, Size: 2208 bytes --] On Tue, Jan 4, 2022, 17:56 Vinay Simha B N <simhavcs@gmail.com> wrote: > Robert, > What is R-b tag? > It looks like this Reviewed-by: Name Lastname <email address> Maybe have a quick look at this document. https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html > > On Tue, Jan 4, 2022 at 7:21 PM Robert Foss <robert.foss@linaro.org> wrote: > > > > Jiri: Are you able to test this patch? > > > > Vinay: Could you supply a R-b tag, if you feel that it is warranted? > > > > On Tue, 14 Dec 2021 at 09:13, Vinay Simha B N <simhavcs@gmail.com> > wrote: > > > > > > Robert, > > > I do not have the hardware to test this feature. Sorry for the late > response. > > > > > > On Thu, Nov 18, 2021 at 8:20 PM Robert Foss <robert.foss@linaro.org> > wrote: > > >> > > >> + Zhen & Vinay > > >> > > >> This patch looks good in itself, but I would like to see a tested by > > >> tag. At the very least testing for regression in single-link LVDS but > > >> ideally some third party verification of this patch. > > >> > > >> On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <jirivanek1@gmail.com> > wrote: > > >> > > > >> > Fixed wrong register shift for single/dual link LVDS output. > > >> > > > >> > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> > > >> > --- > > >> > drivers/gpu/drm/bridge/tc358775.c | 2 +- > > >> > 1 file changed, 1 insertion(+), 1 deletion(-) > > >> > > > >> > diff --git a/drivers/gpu/drm/bridge/tc358775.c > b/drivers/gpu/drm/bridge/tc358775.c > > >> > index 2272adcc5b4a..1d6ec1baeff2 100644 > > >> > --- a/drivers/gpu/drm/bridge/tc358775.c > > >> > +++ b/drivers/gpu/drm/bridge/tc358775.c > > >> > @@ -241,7 +241,7 @@ static inline u32 > TC358775_LVCFG_PCLKDIV(uint32_t val) > > >> > } > > >> > > > >> > #define TC358775_LVCFG_LVDLINK__MASK > 0x00000002 > > >> > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 > > >> > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 > > >> > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) > > >> > { > > >> > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & > > >> > -- > > >> > 2.30.2 > > >> > > > > > > > > > > > > > -- > > > regards, > > > vinaysimha > > > > -- > regards, > vinaysimha > [-- Attachment #2: Type: text/html, Size: 4015 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH v2] drm/bridge/tc358775: Fix for dual-link LVDS 2021-11-10 22:00 [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS Jiri Vanek 2021-11-18 14:50 ` Robert Foss @ 2022-01-06 19:00 ` Jiri Vanek 2022-01-06 19:22 ` Vinay Simha B N 1 sibling, 1 reply; 13+ messages in thread From: Jiri Vanek @ 2022-01-06 19:00 UTC (permalink / raw) To: Andrzej Hajda, Neil Armstrong, Robert Foss Cc: Jernej Skrabec, Jonas Karlman, David Airlie, Jiri Vanek, linux-kernel, dri-devel, Laurent Pinchart, Vinay Simha B N Fixed wrong register shift for single/dual link LVDS output. Tested-by: Jiri Vanek <jirivanek1@gmail.com> Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> --- v1: * Initial version v2: * Tested-by tag added --- drivers/gpu/drm/bridge/tc358775.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c index 2272adcc5b4a..1d6ec1baeff2 100644 --- a/drivers/gpu/drm/bridge/tc358775.c +++ b/drivers/gpu/drm/bridge/tc358775.c @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) } #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 -#define TC358775_LVCFG_LVDLINK__SHIFT 0 +#define TC358775_LVCFG_LVDLINK__SHIFT 1 static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) { return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & -- 2.30.2 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH v2] drm/bridge/tc358775: Fix for dual-link LVDS 2022-01-06 19:00 ` [PATCH v2] " Jiri Vanek @ 2022-01-06 19:22 ` Vinay Simha B N 2022-01-07 16:39 ` Robert Foss 2022-01-26 21:41 ` Jiří Vaněk 0 siblings, 2 replies; 13+ messages in thread From: Vinay Simha B N @ 2022-01-06 19:22 UTC (permalink / raw) To: Jiri Vanek Cc: Jonas Karlman, David Airlie, Robert Foss, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Jernej Skrabec, Andrzej Hajda, Laurent Pinchart [-- Attachment #1: Type: text/plain, Size: 1234 bytes --] Reviewed-by: Vinay Simha BN <simhavcs@gmail.com> Jiri Vanek, Could you please share the part number or datasheet of the dual-link LVDS display/panel used. On Fri, Jan 7, 2022 at 12:30 AM Jiri Vanek <jirivanek1@gmail.com> wrote: > Fixed wrong register shift for single/dual link LVDS output. > > Tested-by: Jiri Vanek <jirivanek1@gmail.com> > Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> > > --- > v1: > * Initial version > > v2: > * Tested-by tag added > > --- > drivers/gpu/drm/bridge/tc358775.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/bridge/tc358775.c > b/drivers/gpu/drm/bridge/tc358775.c > index 2272adcc5b4a..1d6ec1baeff2 100644 > --- a/drivers/gpu/drm/bridge/tc358775.c > +++ b/drivers/gpu/drm/bridge/tc358775.c > @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) > } > > #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 > -#define TC358775_LVCFG_LVDLINK__SHIFT 0 > +#define TC358775_LVCFG_LVDLINK__SHIFT 1 > static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) > { > return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & > -- > 2.30.2 > > -- regards, vinaysimha [-- Attachment #2: Type: text/html, Size: 1966 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] drm/bridge/tc358775: Fix for dual-link LVDS 2022-01-06 19:22 ` Vinay Simha B N @ 2022-01-07 16:39 ` Robert Foss 2022-01-26 21:41 ` Jiří Vaněk 1 sibling, 0 replies; 13+ messages in thread From: Robert Foss @ 2022-01-07 16:39 UTC (permalink / raw) To: Vinay Simha B N Cc: Jiri Vanek, Jonas Karlman, David Airlie, open list:DRM PANEL DRIVERS, Neil Armstrong, open list, Jernej Skrabec, Andrzej Hajda, Laurent Pinchart On Thu, 6 Jan 2022 at 20:22, Vinay Simha B N <simhavcs@gmail.com> wrote: > > Reviewed-by: Vinay Simha BN <simhavcs@gmail.com> > > Jiri Vanek, > Could you please share the part number or datasheet of the dual-link LVDS display/panel used. > > > On Fri, Jan 7, 2022 at 12:30 AM Jiri Vanek <jirivanek1@gmail.com> wrote: >> >> Fixed wrong register shift for single/dual link LVDS output. >> >> Tested-by: Jiri Vanek <jirivanek1@gmail.com> >> Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> >> >> --- >> v1: >> * Initial version >> >> v2: >> * Tested-by tag added >> >> --- >> drivers/gpu/drm/bridge/tc358775.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c >> index 2272adcc5b4a..1d6ec1baeff2 100644 >> --- a/drivers/gpu/drm/bridge/tc358775.c >> +++ b/drivers/gpu/drm/bridge/tc358775.c >> @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) >> } >> >> #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 >> -#define TC358775_LVCFG_LVDLINK__SHIFT 0 >> +#define TC358775_LVCFG_LVDLINK__SHIFT 1 >> static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) >> { >> return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & >> -- Applied to drm-misc-next ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v2] drm/bridge/tc358775: Fix for dual-link LVDS 2022-01-06 19:22 ` Vinay Simha B N 2022-01-07 16:39 ` Robert Foss @ 2022-01-26 21:41 ` Jiří Vaněk 1 sibling, 0 replies; 13+ messages in thread From: Jiří Vaněk @ 2022-01-26 21:41 UTC (permalink / raw) To: Vinay Simha B N; +Cc: open list, open list:DRM PANEL DRIVERS [-- Attachment #1: Type: text/plain, Size: 1461 bytes --] AUO P215HVN01.0 / AUO G215HVN01.0 čt 6. 1. 2022 v 20:22 odesílatel Vinay Simha B N <simhavcs@gmail.com> napsal: > Reviewed-by: Vinay Simha BN <simhavcs@gmail.com> > > Jiri Vanek, > Could you please share the part number or datasheet of the dual-link LVDS > display/panel used. > > > On Fri, Jan 7, 2022 at 12:30 AM Jiri Vanek <jirivanek1@gmail.com> wrote: > >> Fixed wrong register shift for single/dual link LVDS output. >> >> Tested-by: Jiri Vanek <jirivanek1@gmail.com> >> Signed-off-by: Jiri Vanek <jirivanek1@gmail.com> >> >> --- >> v1: >> * Initial version >> >> v2: >> * Tested-by tag added >> >> --- >> drivers/gpu/drm/bridge/tc358775.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/bridge/tc358775.c >> b/drivers/gpu/drm/bridge/tc358775.c >> index 2272adcc5b4a..1d6ec1baeff2 100644 >> --- a/drivers/gpu/drm/bridge/tc358775.c >> +++ b/drivers/gpu/drm/bridge/tc358775.c >> @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val) >> } >> >> #define TC358775_LVCFG_LVDLINK__MASK 0x00000002 >> -#define TC358775_LVCFG_LVDLINK__SHIFT 0 >> +#define TC358775_LVCFG_LVDLINK__SHIFT 1 >> static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val) >> { >> return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) & >> -- >> 2.30.2 >> >> > > -- > regards, > vinaysimha > [-- Attachment #2: Type: text/html, Size: 2433 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2022-01-26 21:41 UTC | newest] Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-11-10 22:00 [PATCH] drm/bridge/tc358775: Fix for dual-link LVDS Jiri Vanek 2021-11-18 14:50 ` Robert Foss 2021-12-14 8:13 ` Vinay Simha B N 2022-01-04 13:50 ` Robert Foss 2022-01-04 14:29 ` Jiří Vaněk 2022-01-04 14:36 ` Robert Foss 2022-01-04 17:04 ` Vinay Simha B N 2022-01-04 16:56 ` Vinay Simha B N 2022-01-04 17:09 ` Robert Foss 2022-01-06 19:00 ` [PATCH v2] " Jiri Vanek 2022-01-06 19:22 ` Vinay Simha B N 2022-01-07 16:39 ` Robert Foss 2022-01-26 21:41 ` Jiří Vaněk
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).