Re: [PATCH] drm/panel/simple: add optronics B101XTN01.0

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

 



On Wed, Aug 06, 2014 at 04:31:30PM -0400, Rob Clark wrote:
> LVDS panel, make/model described as:
> 
> AU Optronics Corporation - B101XTN01.0 (H/W:0A)
> 
> See:
> http://www.encore-electronic.com/media/B101XTN01.0.pdf

I've made it a custom to mention which board a panel is used on in the
commit message and I've requested others to do the same. The idea was
that it would give some kind of indication where to find the hardware if
somebody wanted to test or look at for reference.

> 
> Signed-off-by: Rob Clark <robdclark@xxxxxxxxx>
> ---
>  drivers/gpu/drm/panel/panel-simple.c | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)

This is missing device tree binding documentation.

> 
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> index a251361..16119ba 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -447,6 +447,29 @@ static const struct panel_desc samsung_ltn101nt05 = {
>  	},
>  };
>  
> +static const struct drm_display_mode optronics_lvds_mode = {

Can you please name this according to the model for consistency with
other panels?

> +	.clock = 72000,
> +	.hdisplay = 1366,
> +	.hsync_start = 1366 + 20,
> +	.hsync_end = 1366 + 20 + 70,
> +	.htotal = 1366 + 20 + 70,
> +	.vdisplay = 768,
> +	.vsync_start = 768 + 14,
> +	.vsync_end = 768 + 14 + 42,
> +	.vtotal = 768 + 14 + 42,
> +	.vrefresh = 60,
> +	.flags = DRM_MODE_FLAG_NVSYNC | DRM_MODE_FLAG_NHSYNC,
> +};
> +
> +static const struct panel_desc optronics_lvds = {
> +	.modes = &optronics_lvds_mode,
> +	.num_modes = 1,
> +	.size = {
> +		.width = 1366,
> +		.height = 768,

This needs to be the physical dimension in mm.

> +	},
> +};

There's now also a new field called .bpc which contains the number of
bits per color (usually 6 or 8). Can you provide that as well? According
to the datasheet you linked to it should be 6.

> +
>  static const struct of_device_id platform_of_match[] = {
>  	{
>  		.compatible = "auo,b101aw03",
> @@ -476,6 +499,10 @@ static const struct of_device_id platform_of_match[] = {
>  		.compatible = "samsung,ltn101nt05",
>  		.data = &samsung_ltn101nt05,
>  	}, {
> +		.compatible = "optronics,b101xtn01",

From the commit message this panel is from AU Optronics, so it should
use the auo vendor prefix.

Thierry

Attachment: pgpVLEOxT5efa.pgp
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[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