Re: [PATCH v4 1/2] drm/panel: Add support for Truly NT35597 panel
On Wed, May 30, 2018 at 03:37:26PM -0700, abhin...@codeaurora.org wrote: > Hi Sean > > Thanks for your review. > > Some responses below. Please help to check. > > Thanks > > Abhinav Looks like this got stuck in review. Some comments below, which I hope will match Sean's opinion. > On 2018-05-30 12:12, Sean Paul wrote: > > On Fri, May 25, 2018 at 05:27:51PM -0700, Abhinav Kumar wrote: > > > Add support for Truly NT35597 panel used > > > in MSM reference platforms. > > > > > > This panel supports both single DSI and dual DSI > > > modes. > > > > > > However, this patch series adds support only for > > > dual DSI mode. > > > > > > Changes in v4: > > > - Fix the license identifier > > > - Fix formatting issues for the regulator loads > > > - Fix error messages and return code > > > > > > Signed-off-by: Archit Taneja > > > Signed-off-by: Abhinav Kumar > > > --- > > > drivers/gpu/drm/panel/Kconfig | 8 + > > > drivers/gpu/drm/panel/Makefile | 1 + > > > drivers/gpu/drm/panel/panel-truly-nt35597.c | 576 > > > > > > 3 files changed, 585 insertions(+) > > > create mode 100644 drivers/gpu/drm/panel/panel-truly-nt35597.c > > > > > > diff --git a/drivers/gpu/drm/panel/Kconfig > > > b/drivers/gpu/drm/panel/Kconfig > > > index 25682ff..2fcd9b1 100644 > > > --- a/drivers/gpu/drm/panel/Kconfig > > > +++ b/drivers/gpu/drm/panel/Kconfig > > > @@ -177,4 +177,12 @@ config DRM_PANEL_SITRONIX_ST7789V > > > Say Y here if you want to enable support for the Sitronix > > > ST7789V controller for 240x320 LCD panels > > > > > > +config DRM_PANEL_TRULY_NT35597_WQXGA > > > + tristate "Truly WQXGA" > > > + depends on OF > > > + depends on DRM_MIPI_DSI > > > + select VIDEOMODE_HELPERS > > > + help > > > + Say Y here if you want to enable support for Truly NT35597 WQXGA > > > Dual DSI > > > + Video Mode panel > > > endmenu > > > diff --git a/drivers/gpu/drm/panel/Makefile > > > b/drivers/gpu/drm/panel/Makefile > > > index f26efc1..056ea93 100644 > > > --- a/drivers/gpu/drm/panel/Makefile > > > +++ b/drivers/gpu/drm/panel/Makefile > > > @@ -18,3 +18,4 @@ obj-$(CONFIG_DRM_PANEL_SEIKO_43WVF1G) += > > > panel-seiko-43wvf1g.o > > > obj-$(CONFIG_DRM_PANEL_SHARP_LQ101R1SX01) += > > > panel-sharp-lq101r1sx01.o > > > obj-$(CONFIG_DRM_PANEL_SHARP_LS043T1LE01) += > > > panel-sharp-ls043t1le01.o > > > obj-$(CONFIG_DRM_PANEL_SITRONIX_ST7789V) += panel-sitronix-st7789v.o > > > +obj-$(CONFIG_DRM_PANEL_TRULY_NT35597_WQXGA) += panel-truly-nt35597.o > > > diff --git a/drivers/gpu/drm/panel/panel-truly-nt35597.c > > > b/drivers/gpu/drm/panel/panel-truly-nt35597.c > > > new file mode 100644 > > > index 000..a57cbf0 > > > --- /dev/null > > > +++ b/drivers/gpu/drm/panel/panel-truly-nt35597.c > > > @@ -0,0 +1,576 @@ > > > +// SPDX-License-Identifier: GPL-2.0 > > > +/* > > > + * Copyright (c) 2018, The Linux Foundation. All rights reserved. > > > + */ > > > + > > > +#include > > > +#include > > > +#include > > > +#include > > > + > > > +#include > > > +#include > > > +#include > > > + > > > +#include > > > +#include > > > +#include > > > + > > > +static const char * const regulator_names[] = { > > > + "vdda", > > > + "vdispp", > > > + "vdispn" > > > +}; > [Abhinav] Will add a ',' here as per jordan's comment > > > + > > > +static unsigned long const regulator_enable_loads[] = { > > > + 62000, > > > + 10, > > > + 10, > > > +}; > > > + > > > +static unsigned long const regulator_disable_loads[] = { > > > + 80, > > > + 100, > > > + 100, > > > +}; > > > + > > > +struct truly_wqxga { > > > + struct device *dev; > > > + struct drm_panel panel; > > > + > > > + struct regulator_bulk_data supplies[ARRAY_SIZE(regulator_names)]; > > > + > > > + struct gpio_desc *reset_gpio; > > > + struct gpio_desc *mode_gpio; > > > + > > > + struct backlight_device *backlight; > > > + struct videomode vm; > > > + > > > + struct mipi_dsi_device *dsi[2]; > > > + > > > + bool prepared; > > > + bool enabled; > > > +}; > > > + > > > +static inline struct truly_wqxga *panel_to_truly_wqxga(struct > > > drm_panel *panel) > > > +{ > > > + return container_of(panel, struct truly_wqxga, panel); > > > +} > > > + > > > +static int truly_wqxga_power_on(struct truly_wqxga *ctx) > > > +{ > > > + int ret, i; > > > + > > > + for (i = 0; i < ARRAY_SIZE(ctx->supplies); i++) { > > > + ret = regulator_set_load(ctx->supplies[i].consumer, > > > + regulator_enable_loads[i]); > > > + if (ret) > > > + return ret; > > > + } > > > + > > > + ret = regulator_bulk_enable(ARRAY_SIZE(ctx->supplies), > > > ctx->supplies); > > > + if (ret < 0) > > > + return ret; > > > + > > > + msleep(20); > > > + gpiod_set_value(ctx->reset_gpio, 1); > > > + msleep(20); > > > + gpiod_set_value(ctx->reset_gpio, 0); > > > + msleep(20); > > > + gpiod_set_value(ctx->reset_gpio, 1); > > > + msleep(50); > > > > Why is this needed? Could you please comment? > >
Re: [PATCH v4 1/2] drm/panel: Add support for Truly NT35597 panel
Hi Sean Thanks for your review. Some responses below. Please help to check. Thanks Abhinav On 2018-05-30 12:12, Sean Paul wrote: On Fri, May 25, 2018 at 05:27:51PM -0700, Abhinav Kumar wrote: Add support for Truly NT35597 panel used in MSM reference platforms. This panel supports both single DSI and dual DSI modes. However, this patch series adds support only for dual DSI mode. Changes in v4: - Fix the license identifier - Fix formatting issues for the regulator loads - Fix error messages and return code Signed-off-by: Archit Taneja Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/panel/Kconfig | 8 + drivers/gpu/drm/panel/Makefile | 1 + drivers/gpu/drm/panel/panel-truly-nt35597.c | 576 3 files changed, 585 insertions(+) create mode 100644 drivers/gpu/drm/panel/panel-truly-nt35597.c diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig index 25682ff..2fcd9b1 100644 --- a/drivers/gpu/drm/panel/Kconfig +++ b/drivers/gpu/drm/panel/Kconfig @@ -177,4 +177,12 @@ config DRM_PANEL_SITRONIX_ST7789V Say Y here if you want to enable support for the Sitronix ST7789V controller for 240x320 LCD panels +config DRM_PANEL_TRULY_NT35597_WQXGA + tristate "Truly WQXGA" + depends on OF + depends on DRM_MIPI_DSI + select VIDEOMODE_HELPERS + help + Say Y here if you want to enable support for Truly NT35597 WQXGA Dual DSI + Video Mode panel endmenu diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile index f26efc1..056ea93 100644 --- a/drivers/gpu/drm/panel/Makefile +++ b/drivers/gpu/drm/panel/Makefile @@ -18,3 +18,4 @@ obj-$(CONFIG_DRM_PANEL_SEIKO_43WVF1G) += panel-seiko-43wvf1g.o obj-$(CONFIG_DRM_PANEL_SHARP_LQ101R1SX01) += panel-sharp-lq101r1sx01.o obj-$(CONFIG_DRM_PANEL_SHARP_LS043T1LE01) += panel-sharp-ls043t1le01.o obj-$(CONFIG_DRM_PANEL_SITRONIX_ST7789V) += panel-sitronix-st7789v.o +obj-$(CONFIG_DRM_PANEL_TRULY_NT35597_WQXGA) += panel-truly-nt35597.o diff --git a/drivers/gpu/drm/panel/panel-truly-nt35597.c b/drivers/gpu/drm/panel/panel-truly-nt35597.c new file mode 100644 index 000..a57cbf0 --- /dev/null +++ b/drivers/gpu/drm/panel/panel-truly-nt35597.c @@ -0,0 +1,576 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (c) 2018, The Linux Foundation. All rights reserved. + */ + +#include +#include +#include +#include + +#include +#include +#include + +#include +#include +#include + +static const char * const regulator_names[] = { + "vdda", + "vdispp", + "vdispn" +}; [Abhinav] Will add a ',' here as per jordan's comment + +static unsigned long const regulator_enable_loads[] = { + 62000, + 10, + 10, +}; + +static unsigned long const regulator_disable_loads[] = { + 80, + 100, + 100, +}; + +struct truly_wqxga { + struct device *dev; + struct drm_panel panel; + + struct regulator_bulk_data supplies[ARRAY_SIZE(regulator_names)]; + + struct gpio_desc *reset_gpio; + struct gpio_desc *mode_gpio; + + struct backlight_device *backlight; + struct videomode vm; + + struct mipi_dsi_device *dsi[2]; + + bool prepared; + bool enabled; +}; + +static inline struct truly_wqxga *panel_to_truly_wqxga(struct drm_panel *panel) +{ + return container_of(panel, struct truly_wqxga, panel); +} + +static int truly_wqxga_power_on(struct truly_wqxga *ctx) +{ + int ret, i; + + for (i = 0; i < ARRAY_SIZE(ctx->supplies); i++) { + ret = regulator_set_load(ctx->supplies[i].consumer, + regulator_enable_loads[i]); + if (ret) + return ret; + } + + ret = regulator_bulk_enable(ARRAY_SIZE(ctx->supplies), ctx->supplies); + if (ret < 0) + return ret; + + msleep(20); + gpiod_set_value(ctx->reset_gpio, 1); + msleep(20); + gpiod_set_value(ctx->reset_gpio, 0); + msleep(20); + gpiod_set_value(ctx->reset_gpio, 1); + msleep(50); Why is this needed? Could you please comment? Also, it seems like this is active low? You should specify that in the dt, and let gpiod translate the value. [Abhinav] This is the panel's reset sequence which demands this delay. I can comment the same. Its active HIGH. I can leave a comment in the bindings mentioning the same. Let me know if this is sufficient. + + return 0; +} + +static int truly_wqxga_power_off(struct truly_wqxga *ctx) +{ + int ret, i; + + gpiod_set_value(ctx->reset_gpio, 0); + + for (i = 0; i < ARRAY_SIZE(ctx->supplies); i++) { + ret = regulator_set_load(ctx->supplies[i].consumer, + regulator_disable_loads[i]); + if (ret) + return ret; + } + + return
Re: [PATCH v4 1/2] drm/panel: Add support for Truly NT35597 panel
On Wed, May 30, 2018 at 09:47:25AM -0600, Jordan Crouse wrote: > On Fri, May 25, 2018 at 05:27:51PM -0700, Abhinav Kumar wrote: > > Add support for Truly NT35597 panel used > > in MSM reference platforms. > > > > This panel supports both single DSI and dual DSI > > modes. > > > > However, this patch series adds support only for > > dual DSI mode. > > > > Changes in v4: > > - Fix the license identifier > > - Fix formatting issues for the regulator loads > > - Fix error messages and return code > > > > Signed-off-by: Archit Taneja > > Signed-off-by: Abhinav Kumar > > --- > > drivers/gpu/drm/panel/Kconfig | 8 + > > drivers/gpu/drm/panel/Makefile | 1 + > > drivers/gpu/drm/panel/panel-truly-nt35597.c | 576 > > > > 3 files changed, 585 insertions(+) > > create mode 100644 drivers/gpu/drm/panel/panel-truly-nt35597.c > > > > diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig > > index 25682ff..2fcd9b1 100644 > > --- a/drivers/gpu/drm/panel/Kconfig > > +++ b/drivers/gpu/drm/panel/Kconfig > > @@ -177,4 +177,12 @@ config DRM_PANEL_SITRONIX_ST7789V > > Say Y here if you want to enable support for the Sitronix > > ST7789V controller for 240x320 LCD panels > > > > +config DRM_PANEL_TRULY_NT35597_WQXGA > > + tristate "Truly WQXGA" > > + depends on OF > > + depends on DRM_MIPI_DSI > > + select VIDEOMODE_HELPERS > > + help > > + Say Y here if you want to enable support for Truly NT35597 WQXGA Dual > > DSI > > + Video Mode panel > > endmenu > > diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile > > index f26efc1..056ea93 100644 > > --- a/drivers/gpu/drm/panel/Makefile > > +++ b/drivers/gpu/drm/panel/Makefile > > @@ -18,3 +18,4 @@ obj-$(CONFIG_DRM_PANEL_SEIKO_43WVF1G) += > > panel-seiko-43wvf1g.o > > obj-$(CONFIG_DRM_PANEL_SHARP_LQ101R1SX01) += panel-sharp-lq101r1sx01.o > > obj-$(CONFIG_DRM_PANEL_SHARP_LS043T1LE01) += panel-sharp-ls043t1le01.o > > obj-$(CONFIG_DRM_PANEL_SITRONIX_ST7789V) += panel-sitronix-st7789v.o > > +obj-$(CONFIG_DRM_PANEL_TRULY_NT35597_WQXGA) += panel-truly-nt35597.o > > diff --git a/drivers/gpu/drm/panel/panel-truly-nt35597.c > > b/drivers/gpu/drm/panel/panel-truly-nt35597.c > > new file mode 100644 > > index 000..a57cbf0 > > --- /dev/null > > +++ b/drivers/gpu/drm/panel/panel-truly-nt35597.c > > @@ -0,0 +1,576 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > I guess it is up to Sean and Rob if they want to accept // comments for SPDX. > I'm not sure there is a hard and fast rule about it. > Hard meh from me. Looks like there are plenty of examples of both in drm. Sean > > +/* > > + * Copyright (c) 2018, The Linux Foundation. All rights reserved. > > + */ > > + > > +#include > > +#include > > +#include > > +#include > > + > > +#include > > +#include > > +#include > > + > > +#include > > +#include > > +#include > > + > > +static const char * const regulator_names[] = { > > + "vdda", > > + "vdispp", > > + "vdispn" > > The reason why the coding style insists on commas for the last member of an > listy is that if you added another item the resulting patch would > only need one line instead of three. For example: > > + "foo", > > instead of: > > - "vdispn" > + "vdispn", > + "foo", > > > > -- > The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, > a Linux Foundation Collaborative Project -- Sean Paul, Software Engineer, Google / Chromium OS ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v4 1/2] drm/panel: Add support for Truly NT35597 panel
On Fri, May 25, 2018 at 05:27:51PM -0700, Abhinav Kumar wrote: > Add support for Truly NT35597 panel used > in MSM reference platforms. > > This panel supports both single DSI and dual DSI > modes. > > However, this patch series adds support only for > dual DSI mode. > > Changes in v4: > - Fix the license identifier > - Fix formatting issues for the regulator loads > - Fix error messages and return code > > Signed-off-by: Archit Taneja > Signed-off-by: Abhinav Kumar > --- > drivers/gpu/drm/panel/Kconfig | 8 + > drivers/gpu/drm/panel/Makefile | 1 + > drivers/gpu/drm/panel/panel-truly-nt35597.c | 576 > > 3 files changed, 585 insertions(+) > create mode 100644 drivers/gpu/drm/panel/panel-truly-nt35597.c > > diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig > index 25682ff..2fcd9b1 100644 > --- a/drivers/gpu/drm/panel/Kconfig > +++ b/drivers/gpu/drm/panel/Kconfig > @@ -177,4 +177,12 @@ config DRM_PANEL_SITRONIX_ST7789V > Say Y here if you want to enable support for the Sitronix > ST7789V controller for 240x320 LCD panels > > +config DRM_PANEL_TRULY_NT35597_WQXGA > + tristate "Truly WQXGA" > + depends on OF > + depends on DRM_MIPI_DSI > + select VIDEOMODE_HELPERS > + help > + Say Y here if you want to enable support for Truly NT35597 WQXGA Dual > DSI > + Video Mode panel > endmenu > diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile > index f26efc1..056ea93 100644 > --- a/drivers/gpu/drm/panel/Makefile > +++ b/drivers/gpu/drm/panel/Makefile > @@ -18,3 +18,4 @@ obj-$(CONFIG_DRM_PANEL_SEIKO_43WVF1G) += > panel-seiko-43wvf1g.o > obj-$(CONFIG_DRM_PANEL_SHARP_LQ101R1SX01) += panel-sharp-lq101r1sx01.o > obj-$(CONFIG_DRM_PANEL_SHARP_LS043T1LE01) += panel-sharp-ls043t1le01.o > obj-$(CONFIG_DRM_PANEL_SITRONIX_ST7789V) += panel-sitronix-st7789v.o > +obj-$(CONFIG_DRM_PANEL_TRULY_NT35597_WQXGA) += panel-truly-nt35597.o > diff --git a/drivers/gpu/drm/panel/panel-truly-nt35597.c > b/drivers/gpu/drm/panel/panel-truly-nt35597.c > new file mode 100644 > index 000..a57cbf0 > --- /dev/null > +++ b/drivers/gpu/drm/panel/panel-truly-nt35597.c > @@ -0,0 +1,576 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (c) 2018, The Linux Foundation. All rights reserved. > + */ > + > +#include > +#include > +#include > +#include > + > +#include > +#include > +#include > + > +#include > +#include > +#include > + > +static const char * const regulator_names[] = { > + "vdda", > + "vdispp", > + "vdispn" > +}; > + > +static unsigned long const regulator_enable_loads[] = { > + 62000, > + 10, > + 10, > +}; > + > +static unsigned long const regulator_disable_loads[] = { > + 80, > + 100, > + 100, > +}; > + > +struct truly_wqxga { > + struct device *dev; > + struct drm_panel panel; > + > + struct regulator_bulk_data supplies[ARRAY_SIZE(regulator_names)]; > + > + struct gpio_desc *reset_gpio; > + struct gpio_desc *mode_gpio; > + > + struct backlight_device *backlight; > + struct videomode vm; > + > + struct mipi_dsi_device *dsi[2]; > + > + bool prepared; > + bool enabled; > +}; > + > +static inline struct truly_wqxga *panel_to_truly_wqxga(struct drm_panel > *panel) > +{ > + return container_of(panel, struct truly_wqxga, panel); > +} > + > +static int truly_wqxga_power_on(struct truly_wqxga *ctx) > +{ > + int ret, i; > + > + for (i = 0; i < ARRAY_SIZE(ctx->supplies); i++) { > + ret = regulator_set_load(ctx->supplies[i].consumer, > + regulator_enable_loads[i]); > + if (ret) > + return ret; > + } > + > + ret = regulator_bulk_enable(ARRAY_SIZE(ctx->supplies), ctx->supplies); > + if (ret < 0) > + return ret; > + > + msleep(20); > + gpiod_set_value(ctx->reset_gpio, 1); > + msleep(20); > + gpiod_set_value(ctx->reset_gpio, 0); > + msleep(20); > + gpiod_set_value(ctx->reset_gpio, 1); > + msleep(50); Why is this needed? Could you please comment? Also, it seems like this is active low? You should specify that in the dt, and let gpiod translate the value. > + > + return 0; > +} > + > +static int truly_wqxga_power_off(struct truly_wqxga *ctx) > +{ > + int ret, i; > + > + gpiod_set_value(ctx->reset_gpio, 0); > + > + for (i = 0; i < ARRAY_SIZE(ctx->supplies); i++) { > + ret = regulator_set_load(ctx->supplies[i].consumer, > + regulator_disable_loads[i]); > + if (ret) > + return ret; > + } > + > + return regulator_bulk_disable(ARRAY_SIZE(ctx->supplies), ctx->supplies); > +} > + > +static int truly_wqxga_disable(struct drm_panel *panel) > +{ > + struct truly_wqxga *ctx = panel_to_truly_wqxga(panel); > + > + if (!ctx->enabled) > +
Re: [PATCH v4 1/2] drm/panel: Add support for Truly NT35597 panel
On Fri, May 25, 2018 at 05:27:51PM -0700, Abhinav Kumar wrote: > Add support for Truly NT35597 panel used > in MSM reference platforms. > > This panel supports both single DSI and dual DSI > modes. > > However, this patch series adds support only for > dual DSI mode. > > Changes in v4: > - Fix the license identifier > - Fix formatting issues for the regulator loads > - Fix error messages and return code > > Signed-off-by: Archit Taneja > Signed-off-by: Abhinav Kumar > --- > drivers/gpu/drm/panel/Kconfig | 8 + > drivers/gpu/drm/panel/Makefile | 1 + > drivers/gpu/drm/panel/panel-truly-nt35597.c | 576 > > 3 files changed, 585 insertions(+) > create mode 100644 drivers/gpu/drm/panel/panel-truly-nt35597.c > > diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig > index 25682ff..2fcd9b1 100644 > --- a/drivers/gpu/drm/panel/Kconfig > +++ b/drivers/gpu/drm/panel/Kconfig > @@ -177,4 +177,12 @@ config DRM_PANEL_SITRONIX_ST7789V > Say Y here if you want to enable support for the Sitronix > ST7789V controller for 240x320 LCD panels > > +config DRM_PANEL_TRULY_NT35597_WQXGA > + tristate "Truly WQXGA" > + depends on OF > + depends on DRM_MIPI_DSI > + select VIDEOMODE_HELPERS > + help > + Say Y here if you want to enable support for Truly NT35597 WQXGA Dual > DSI > + Video Mode panel > endmenu > diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile > index f26efc1..056ea93 100644 > --- a/drivers/gpu/drm/panel/Makefile > +++ b/drivers/gpu/drm/panel/Makefile > @@ -18,3 +18,4 @@ obj-$(CONFIG_DRM_PANEL_SEIKO_43WVF1G) += > panel-seiko-43wvf1g.o > obj-$(CONFIG_DRM_PANEL_SHARP_LQ101R1SX01) += panel-sharp-lq101r1sx01.o > obj-$(CONFIG_DRM_PANEL_SHARP_LS043T1LE01) += panel-sharp-ls043t1le01.o > obj-$(CONFIG_DRM_PANEL_SITRONIX_ST7789V) += panel-sitronix-st7789v.o > +obj-$(CONFIG_DRM_PANEL_TRULY_NT35597_WQXGA) += panel-truly-nt35597.o > diff --git a/drivers/gpu/drm/panel/panel-truly-nt35597.c > b/drivers/gpu/drm/panel/panel-truly-nt35597.c > new file mode 100644 > index 000..a57cbf0 > --- /dev/null > +++ b/drivers/gpu/drm/panel/panel-truly-nt35597.c > @@ -0,0 +1,576 @@ > +// SPDX-License-Identifier: GPL-2.0 I guess it is up to Sean and Rob if they want to accept // comments for SPDX. I'm not sure there is a hard and fast rule about it. > +/* > + * Copyright (c) 2018, The Linux Foundation. All rights reserved. > + */ > + > +#include > +#include > +#include > +#include > + > +#include > +#include > +#include > + > +#include > +#include > +#include > + > +static const char * const regulator_names[] = { > + "vdda", > + "vdispp", > + "vdispn" The reason why the coding style insists on commas for the last member of an listy is that if you added another item the resulting patch would only need one line instead of three. For example: + "foo", instead of: - "vdispn" + "vdispn", + "foo", -- The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel