| Message ID | 20211111094103.494831-1-jagan@amarulasolutions.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/panel: panel-simple: Fix proper bpc for AM-1280800N3TZQW-T00H | expand |
Hi Sam, On Thu, Nov 11, 2021 at 3:11 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per > recent testing in i.MX8MM platform. > > Fix it. > > Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > --- > drivers/gpu/drm/panel/panel-simple.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > index eb475a3a774b..cf3f21f649cb 100644 > --- a/drivers/gpu/drm/panel/panel-simple.c > +++ b/drivers/gpu/drm/panel/panel-simple.c > @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { > static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { > .modes = &ire_am_1280800n3tzqw_t00h_mode, > .num_modes = 1, > - .bpc = 6, > + .bpc = 8, Any response on this? Thanks, Jagan.
Hi Jagan, On Sun, Dec 19, 2021 at 10:10:10PM +0530, Jagan Teki wrote: > Hi Sam, > > On Thu, Nov 11, 2021 at 3:11 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per > > recent testing in i.MX8MM platform. > > > > Fix it. > > > > Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > --- > > drivers/gpu/drm/panel/panel-simple.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > > index eb475a3a774b..cf3f21f649cb 100644 > > --- a/drivers/gpu/drm/panel/panel-simple.c > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { > > static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { > > .modes = &ire_am_1280800n3tzqw_t00h_mode, > > .num_modes = 1, > > - .bpc = 6, > > + .bpc = 8, > > Any response on this? I am too busy with other stuff at the moment to spend time on Linux stuff, but expect to re-surface sometime after xmas. Sam
Hi Sam, On Mon, Dec 20, 2021 at 1:45 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > Hi Jagan, > > On Sun, Dec 19, 2021 at 10:10:10PM +0530, Jagan Teki wrote: > > Hi Sam, > > > > On Thu, Nov 11, 2021 at 3:11 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per > > > recent testing in i.MX8MM platform. > > > > > > Fix it. > > > > > > Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") > > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > > --- > > > drivers/gpu/drm/panel/panel-simple.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > > > index eb475a3a774b..cf3f21f649cb 100644 > > > --- a/drivers/gpu/drm/panel/panel-simple.c > > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > > @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { > > > static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { > > > .modes = &ire_am_1280800n3tzqw_t00h_mode, > > > .num_modes = 1, > > > - .bpc = 6, > > > + .bpc = 8, > > > > Any response on this? > > I am too busy with other stuff at the moment to spend time on Linux > stuff, but expect to re-surface sometime after xmas. Any further comments? Thanks, Jagan.
On Mon, Feb 7, 2022 at 6:34 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > Hi Sam, > > On Mon, Dec 20, 2021 at 1:45 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > > > Hi Jagan, > > > > On Sun, Dec 19, 2021 at 10:10:10PM +0530, Jagan Teki wrote: > > > Hi Sam, > > > > > > On Thu, Nov 11, 2021 at 3:11 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > > > AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per > > > > recent testing in i.MX8MM platform. > > > > > > > > Fix it. > > > > > > > > Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") > > > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > > > --- > > > > drivers/gpu/drm/panel/panel-simple.c | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > > > > index eb475a3a774b..cf3f21f649cb 100644 > > > > --- a/drivers/gpu/drm/panel/panel-simple.c > > > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > > > @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { > > > > static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { > > > > .modes = &ire_am_1280800n3tzqw_t00h_mode, > > > > .num_modes = 1, > > > > - .bpc = 6, > > > > + .bpc = 8, > > > > > > Any response on this? > > > > I am too busy with other stuff at the moment to spend time on Linux > > stuff, but expect to re-surface sometime after xmas. > > Any further comments? Gentle Ping.
+ Robert On Tue, Feb 22, 2022 at 12:17 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > On Mon, Feb 7, 2022 at 6:34 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > Hi Sam, > > > > On Mon, Dec 20, 2021 at 1:45 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > > > > > Hi Jagan, > > > > > > On Sun, Dec 19, 2021 at 10:10:10PM +0530, Jagan Teki wrote: > > > > Hi Sam, > > > > > > > > On Thu, Nov 11, 2021 at 3:11 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > > > > > AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per > > > > > recent testing in i.MX8MM platform. > > > > > > > > > > Fix it. > > > > > > > > > > Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") > > > > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > > > > --- > > > > > drivers/gpu/drm/panel/panel-simple.c | 2 +- > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > > > > > index eb475a3a774b..cf3f21f649cb 100644 > > > > > --- a/drivers/gpu/drm/panel/panel-simple.c > > > > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > > > > @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { > > > > > static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { > > > > > .modes = &ire_am_1280800n3tzqw_t00h_mode, > > > > > .num_modes = 1, > > > > > - .bpc = 6, > > > > > + .bpc = 8, > > > > > > > > Any response on this? > > > > > > I am too busy with other stuff at the moment to spend time on Linux > > > stuff, but expect to re-surface sometime after xmas. > > > > Any further comments? > > Gentle Ping.
On Thu, 31 Mar 2022 at 16:50, Jagan Teki <jagan@amarulasolutions.com> wrote: > > + Robert > > On Tue, Feb 22, 2022 at 12:17 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > On Mon, Feb 7, 2022 at 6:34 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > Hi Sam, > > > > > > On Mon, Dec 20, 2021 at 1:45 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > > > > > > > Hi Jagan, > > > > > > > > On Sun, Dec 19, 2021 at 10:10:10PM +0530, Jagan Teki wrote: > > > > > Hi Sam, > > > > > > > > > > On Thu, Nov 11, 2021 at 3:11 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > > > > > > > AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per > > > > > > recent testing in i.MX8MM platform. > > > > > > > > > > > > Fix it. > > > > > > > > > > > > Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") > > > > > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > > > > > --- > > > > > > drivers/gpu/drm/panel/panel-simple.c | 2 +- > > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > > > > > > index eb475a3a774b..cf3f21f649cb 100644 > > > > > > --- a/drivers/gpu/drm/panel/panel-simple.c > > > > > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > > > > > @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { > > > > > > static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { > > > > > > .modes = &ire_am_1280800n3tzqw_t00h_mode, > > > > > > .num_modes = 1, > > > > > > - .bpc = 6, > > > > > > + .bpc = 8, > > > > > > > > > > Any response on this? Reviewed-by: Robert Foss <robert.foss@linaro.org>
Hi Robert, Can you apply this? On Thu, Mar 31, 2022 at 8:22 PM Robert Foss <robert.foss@linaro.org> wrote: > > On Thu, 31 Mar 2022 at 16:50, Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > + Robert > > > > On Tue, Feb 22, 2022 at 12:17 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > On Mon, Feb 7, 2022 at 6:34 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > > > Hi Sam, > > > > > > > > On Mon, Dec 20, 2021 at 1:45 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > > > > > > > > > Hi Jagan, > > > > > > > > > > On Sun, Dec 19, 2021 at 10:10:10PM +0530, Jagan Teki wrote: > > > > > > Hi Sam, > > > > > > > > > > > > On Thu, Nov 11, 2021 at 3:11 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > > > > > > > > > AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per > > > > > > > recent testing in i.MX8MM platform. > > > > > > > > > > > > > > Fix it. > > > > > > > > > > > > > > Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") > > > > > > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > > > > > > --- > > > > > > > drivers/gpu/drm/panel/panel-simple.c | 2 +- > > > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > > > > > > > index eb475a3a774b..cf3f21f649cb 100644 > > > > > > > --- a/drivers/gpu/drm/panel/panel-simple.c > > > > > > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > > > > > > @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { > > > > > > > static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { > > > > > > > .modes = &ire_am_1280800n3tzqw_t00h_mode, > > > > > > > .num_modes = 1, > > > > > > > - .bpc = 6, > > > > > > > + .bpc = 8, > > > > > > > > > > > > Any response on this? > > Reviewed-by: Robert Foss <robert.foss@linaro.org>
On Fri, 29 Apr 2022 at 11:49, Jagan Teki <jagan@amarulasolutions.com> wrote: > > Hi Robert, > > Can you apply this? > Thanks for the reminder. Applied to drm-misc-next. > On Thu, Mar 31, 2022 at 8:22 PM Robert Foss <robert.foss@linaro.org> wrote: > > > > On Thu, 31 Mar 2022 at 16:50, Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > + Robert > > > > > > On Tue, Feb 22, 2022 at 12:17 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > > > On Mon, Feb 7, 2022 at 6:34 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > > > > > Hi Sam, > > > > > > > > > > On Mon, Dec 20, 2021 at 1:45 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > > > > > > > > > > > Hi Jagan, > > > > > > > > > > > > On Sun, Dec 19, 2021 at 10:10:10PM +0530, Jagan Teki wrote: > > > > > > > Hi Sam, > > > > > > > > > > > > > > On Thu, Nov 11, 2021 at 3:11 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > > > > > > > > > > > AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per > > > > > > > > recent testing in i.MX8MM platform. > > > > > > > > > > > > > > > > Fix it. > > > > > > > > > > > > > > > > Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") > > > > > > > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > > > > > > > --- > > > > > > > > drivers/gpu/drm/panel/panel-simple.c | 2 +- > > > > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > > > > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > > > > > > > > index eb475a3a774b..cf3f21f649cb 100644 > > > > > > > > --- a/drivers/gpu/drm/panel/panel-simple.c > > > > > > > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > > > > > > > @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { > > > > > > > > static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { > > > > > > > > .modes = &ire_am_1280800n3tzqw_t00h_mode, > > > > > > > > .num_modes = 1, > > > > > > > > - .bpc = 6, > > > > > > > > + .bpc = 8, > > > > > > > > > > > > > > Any response on this? > > > > Reviewed-by: Robert Foss <robert.foss@linaro.org> > > > > -- > Jagan Teki, > Amarula Solutions India Pvt. Ltd. > Co-Founder & Embedded Linux Architect > 405/E-Block, Sri Lakshmi Shubham Arcade, Chandanagar, Hyderabad - 500050, India > M. (+91) 910 009 0959 > [`as] http://www.amarulasolutions.com
diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index eb475a3a774b..cf3f21f649cb 100644 --- a/drivers/gpu/drm/panel/panel-simple.c +++ b/drivers/gpu/drm/panel/panel-simple.c @@ -719,7 +719,7 @@ static const struct drm_display_mode ampire_am_1280800n3tzqw_t00h_mode = { static const struct panel_desc ampire_am_1280800n3tzqw_t00h = { .modes = &ire_am_1280800n3tzqw_t00h_mode, .num_modes = 1, - .bpc = 6, + .bpc = 8, .size = { .width = 217, .height = 136,
AM-1280800N3TZQW-T00H panel support 8 bpc not 6 bpc as per recent testing in i.MX8MM platform. Fix it. Fixes: bca684e69c4c ("drm/panel: simple: Add AM-1280800N3TZQW-T00H") Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> --- drivers/gpu/drm/panel/panel-simple.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)