// SPDX-License-Identifier: GPL-2.0+
 /*
- * MIPI-DSI Sony ACX424AKP panel driver. This is a 480x864
- * AMOLED panel with a command-only DSI interface.
+ * MIPI-DSI Novatek NT35560-based panel controller.
  *
- * Copyright (C) Linaro Ltd. 2019
+ * Supported panels include:
+ * Sony ACX424AKM - a 480x854 AMOLED DSI panel
+ * Sony ACX424AKP - a 480x864 AMOLED DSI panel
+ *
+ * Copyright (C) Linaro Ltd. 2019-2021
  * Author: Linus Walleij
  * Based on code and know-how from Marcus Lorentzon
  * Copyright (C) ST-Ericsson SA 2010
 #include <drm/drm_modes.h>
 #include <drm/drm_panel.h>
 
-#define ACX424_DCS_READ_ID1            0xDA
-#define ACX424_DCS_READ_ID2            0xDB
-#define ACX424_DCS_READ_ID3            0xDC
-#define ACX424_DCS_SET_MDDI            0xAE
+#define NT35560_DCS_READ_ID1           0xDA
+#define NT35560_DCS_READ_ID2           0xDB
+#define NT35560_DCS_READ_ID3           0xDC
+#define NT35560_DCS_SET_MDDI           0xAE
 
 /*
  * Sony seems to use vendor ID 0x81
  */
 #define DISPLAY_SONY_ACX424AKP_ID3     0x8000
 
-struct acx424akp {
+struct nt35560 {
        struct drm_panel panel;
        struct device *dev;
        struct regulator *supply;
        .height_mm = 84,
 };
 
-static inline struct acx424akp *panel_to_acx424akp(struct drm_panel *panel)
+static inline struct nt35560 *panel_to_nt35560(struct drm_panel *panel)
 {
-       return container_of(panel, struct acx424akp, panel);
+       return container_of(panel, struct nt35560, panel);
 }
 
 #define FOSC                   20 /* 20Mhz */
 #define SCALE_FACTOR_NS_DIV_MHZ        1000
 
-static int acx424akp_set_brightness(struct backlight_device *bl)
+static int nt35560_set_brightness(struct backlight_device *bl)
 {
-       struct acx424akp *acx = bl_get_data(bl);
-       struct mipi_dsi_device *dsi = to_mipi_dsi_device(acx->dev);
+       struct nt35560 *nt = bl_get_data(bl);
+       struct mipi_dsi_device *dsi = to_mipi_dsi_device(nt->dev);
        int period_ns = 1023;
        int duty_ns = bl->props.brightness;
        u8 pwm_ratio;
                ret = mipi_dsi_dcs_write(dsi, MIPI_DCS_WRITE_CONTROL_DISPLAY,
                                         &par, 1);
                if (ret) {
-                       dev_err(acx->dev, "failed to disable display backlight (%d)\n", ret);
+                       dev_err(nt->dev, "failed to disable display backlight (%d)\n", ret);
                        return ret;
                }
                return 0;
                      SCALE_FACTOR_NS_DIV_MHZ);
 
        /* Set up PWM dutycycle ONE byte (differs from the standard) */
-       dev_dbg(acx->dev, "calculated duty cycle %02x\n", pwm_ratio);
+       dev_dbg(nt->dev, "calculated duty cycle %02x\n", pwm_ratio);
        ret = mipi_dsi_dcs_write(dsi, MIPI_DCS_SET_DISPLAY_BRIGHTNESS,
                                 &pwm_ratio, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "failed to set display PWM ratio (%d)\n", ret);
+               dev_err(nt->dev, "failed to set display PWM ratio (%d)\n", ret);
                return ret;
        }
 
        par = 0xaa;
        ret = mipi_dsi_dcs_write(dsi, 0xf3, &par, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "failed to unlock CMD 2 (%d)\n", ret);
+               dev_err(nt->dev, "failed to unlock CMD 2 (%d)\n", ret);
                return ret;
        }
        par = 0x01;
        ret = mipi_dsi_dcs_write(dsi, 0x00, &par, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "failed to enter page 1 (%d)\n", ret);
+               dev_err(nt->dev, "failed to enter page 1 (%d)\n", ret);
                return ret;
        }
        par = 0x01;
        ret = mipi_dsi_dcs_write(dsi, 0x7d, &par, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "failed to disable MTP reload (%d)\n", ret);
+               dev_err(nt->dev, "failed to disable MTP reload (%d)\n", ret);
                return ret;
        }
        ret = mipi_dsi_dcs_write(dsi, 0x22, &pwm_div, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "failed to set PWM divisor (%d)\n", ret);
+               dev_err(nt->dev, "failed to set PWM divisor (%d)\n", ret);
                return ret;
        }
        par = 0xaa;
        ret = mipi_dsi_dcs_write(dsi, 0x7f, &par, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "failed to lock CMD 2 (%d)\n", ret);
+               dev_err(nt->dev, "failed to lock CMD 2 (%d)\n", ret);
                return ret;
        }
 
        ret = mipi_dsi_dcs_write(dsi, MIPI_DCS_WRITE_CONTROL_DISPLAY,
                                 &par, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "failed to enable display backlight (%d)\n", ret);
+               dev_err(nt->dev, "failed to enable display backlight (%d)\n", ret);
                return ret;
        }
 
        return 0;
 }
 
-static const struct backlight_ops acx424akp_bl_ops = {
-       .update_status = acx424akp_set_brightness,
+static const struct backlight_ops nt35560_bl_ops = {
+       .update_status = nt35560_set_brightness,
 };
 
-static const struct backlight_properties acx424akp_bl_props = {
+static const struct backlight_properties nt35560_bl_props = {
        .type = BACKLIGHT_RAW,
        .brightness = 512,
        .max_brightness = 1023,
 };
 
-static int acx424akp_read_id(struct acx424akp *acx)
+static int nt35560_read_id(struct nt35560 *nt)
 {
-       struct mipi_dsi_device *dsi = to_mipi_dsi_device(acx->dev);
+       struct mipi_dsi_device *dsi = to_mipi_dsi_device(nt->dev);
        u8 vendor, version, panel;
        u16 val;
        int ret;
 
-       ret = mipi_dsi_dcs_read(dsi, ACX424_DCS_READ_ID1, &vendor, 1);
+       ret = mipi_dsi_dcs_read(dsi, NT35560_DCS_READ_ID1, &vendor, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "could not vendor ID byte\n");
+               dev_err(nt->dev, "could not vendor ID byte\n");
                return ret;
        }
-       ret = mipi_dsi_dcs_read(dsi, ACX424_DCS_READ_ID2, &version, 1);
+       ret = mipi_dsi_dcs_read(dsi, NT35560_DCS_READ_ID2, &version, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "could not read device version byte\n");
+               dev_err(nt->dev, "could not read device version byte\n");
                return ret;
        }
-       ret = mipi_dsi_dcs_read(dsi, ACX424_DCS_READ_ID3, &panel, 1);
+       ret = mipi_dsi_dcs_read(dsi, NT35560_DCS_READ_ID3, &panel, 1);
        if (ret < 0) {
-               dev_err(acx->dev, "could not read panel ID byte\n");
+               dev_err(nt->dev, "could not read panel ID byte\n");
                return ret;
        }
 
        if (vendor == 0x00) {
-               dev_err(acx->dev, "device vendor ID is zero\n");
+               dev_err(nt->dev, "device vendor ID is zero\n");
                return -ENODEV;
        }
 
        case DISPLAY_SONY_ACX424AKP_ID1:
        case DISPLAY_SONY_ACX424AKP_ID2:
        case DISPLAY_SONY_ACX424AKP_ID3:
-               dev_info(acx->dev, "MTP vendor: %02x, version: %02x, panel: %02x\n",
+               dev_info(nt->dev, "MTP vendor: %02x, version: %02x, panel: %02x\n",
                         vendor, version, panel);
                break;
        default:
-               dev_info(acx->dev, "unknown vendor: %02x, version: %02x, panel: %02x\n",
+               dev_info(nt->dev, "unknown vendor: %02x, version: %02x, panel: %02x\n",
                         vendor, version, panel);
                break;
        }
        return 0;
 }
 
-static int acx424akp_power_on(struct acx424akp *acx)
+static int nt35560_power_on(struct nt35560 *nt)
 {
        int ret;
 
-       ret = regulator_enable(acx->supply);
+       ret = regulator_enable(nt->supply);
        if (ret) {
-               dev_err(acx->dev, "failed to enable supply (%d)\n", ret);
+               dev_err(nt->dev, "failed to enable supply (%d)\n", ret);
                return ret;
        }
 
        /* Assert RESET */
-       gpiod_set_value_cansleep(acx->reset_gpio, 1);
+       gpiod_set_value_cansleep(nt->reset_gpio, 1);
        udelay(20);
        /* De-assert RESET */
-       gpiod_set_value_cansleep(acx->reset_gpio, 0);
+       gpiod_set_value_cansleep(nt->reset_gpio, 0);
        usleep_range(11000, 20000);
 
        return 0;
 }
 
-static void acx424akp_power_off(struct acx424akp *acx)
+static void nt35560_power_off(struct nt35560 *nt)
 {
        /* Assert RESET */
-       gpiod_set_value_cansleep(acx->reset_gpio, 1);
+       gpiod_set_value_cansleep(nt->reset_gpio, 1);
        usleep_range(11000, 20000);
 
-       regulator_disable(acx->supply);
+       regulator_disable(nt->supply);
 }
 
-static int acx424akp_prepare(struct drm_panel *panel)
+static int nt35560_prepare(struct drm_panel *panel)
 {
-       struct acx424akp *acx = panel_to_acx424akp(panel);
-       struct mipi_dsi_device *dsi = to_mipi_dsi_device(acx->dev);
+       struct nt35560 *nt = panel_to_nt35560(panel);
+       struct mipi_dsi_device *dsi = to_mipi_dsi_device(nt->dev);
        const u8 mddi = 3;
        int ret;
 
-       ret = acx424akp_power_on(acx);
+       ret = nt35560_power_on(nt);
        if (ret)
                return ret;
 
-       ret = acx424akp_read_id(acx);
+       ret = nt35560_read_id(nt);
        if (ret) {
-               dev_err(acx->dev, "failed to read panel ID (%d)\n", ret);
+               dev_err(nt->dev, "failed to read panel ID (%d)\n", ret);
                goto err_power_off;
        }
 
        ret = mipi_dsi_dcs_set_tear_on(dsi,
                                       MIPI_DSI_DCS_TEAR_MODE_VBLANK);
        if (ret) {
-               dev_err(acx->dev, "failed to enable vblank TE (%d)\n", ret);
+               dev_err(nt->dev, "failed to enable vblank TE (%d)\n", ret);
                goto err_power_off;
        }
 
         * this command. Due to the lack of documentation we cannot know for
         * sure.
         */
-       ret = mipi_dsi_dcs_write(dsi, ACX424_DCS_SET_MDDI,
+       ret = mipi_dsi_dcs_write(dsi, NT35560_DCS_SET_MDDI,
                                 &mddi, sizeof(mddi));
        if (ret < 0) {
-               dev_err(acx->dev, "failed to set MDDI (%d)\n", ret);
+               dev_err(nt->dev, "failed to set MDDI (%d)\n", ret);
                goto err_power_off;
        }
 
        /* Exit sleep mode */
        ret = mipi_dsi_dcs_exit_sleep_mode(dsi);
        if (ret) {
-               dev_err(acx->dev, "failed to exit sleep mode (%d)\n", ret);
+               dev_err(nt->dev, "failed to exit sleep mode (%d)\n", ret);
                goto err_power_off;
        }
        msleep(140);
 
        ret = mipi_dsi_dcs_set_display_on(dsi);
        if (ret) {
-               dev_err(acx->dev, "failed to turn display on (%d)\n", ret);
+               dev_err(nt->dev, "failed to turn display on (%d)\n", ret);
                goto err_power_off;
        }
-       if (acx->video_mode) {
+       if (nt->video_mode) {
                /* In video mode turn peripheral on */
                ret = mipi_dsi_turn_on_peripheral(dsi);
                if (ret) {
-                       dev_err(acx->dev, "failed to turn on peripheral\n");
+                       dev_err(nt->dev, "failed to turn on peripheral\n");
                        goto err_power_off;
                }
        }
        return 0;
 
 err_power_off:
-       acx424akp_power_off(acx);
+       nt35560_power_off(nt);
        return ret;
 }
 
-static int acx424akp_unprepare(struct drm_panel *panel)
+static int nt35560_unprepare(struct drm_panel *panel)
 {
-       struct acx424akp *acx = panel_to_acx424akp(panel);
-       struct mipi_dsi_device *dsi = to_mipi_dsi_device(acx->dev);
+       struct nt35560 *nt = panel_to_nt35560(panel);
+       struct mipi_dsi_device *dsi = to_mipi_dsi_device(nt->dev);
        int ret;
 
        ret = mipi_dsi_dcs_set_display_off(dsi);
        if (ret) {
-               dev_err(acx->dev, "failed to turn display off (%d)\n", ret);
+               dev_err(nt->dev, "failed to turn display off (%d)\n", ret);
                return ret;
        }
 
        /* Enter sleep mode */
        ret = mipi_dsi_dcs_enter_sleep_mode(dsi);
        if (ret) {
-               dev_err(acx->dev, "failed to enter sleep mode (%d)\n", ret);
+               dev_err(nt->dev, "failed to enter sleep mode (%d)\n", ret);
                return ret;
        }
        msleep(85);
 
-       acx424akp_power_off(acx);
+       nt35560_power_off(nt);
 
        return 0;
 }
 
 
-static int acx424akp_get_modes(struct drm_panel *panel,
-                              struct drm_connector *connector)
+static int nt35560_get_modes(struct drm_panel *panel,
+                            struct drm_connector *connector)
 {
-       struct acx424akp *acx = panel_to_acx424akp(panel);
+       struct nt35560 *nt = panel_to_nt35560(panel);
        struct drm_display_mode *mode;
 
-       if (acx->video_mode)
+       if (nt->video_mode)
                mode = drm_mode_duplicate(connector->dev,
                                          &sony_acx424akp_vid_mode);
        else
        return 1; /* Number of modes */
 }
 
-static const struct drm_panel_funcs acx424akp_drm_funcs = {
-       .unprepare = acx424akp_unprepare,
-       .prepare = acx424akp_prepare,
-       .get_modes = acx424akp_get_modes,
+static const struct drm_panel_funcs nt35560_drm_funcs = {
+       .unprepare = nt35560_unprepare,
+       .prepare = nt35560_prepare,
+       .get_modes = nt35560_get_modes,
 };
 
-static int acx424akp_probe(struct mipi_dsi_device *dsi)
+static int nt35560_probe(struct mipi_dsi_device *dsi)
 {
        struct device *dev = &dsi->dev;
-       struct acx424akp *acx;
+       struct nt35560 *nt;
        int ret;
 
-       acx = devm_kzalloc(dev, sizeof(struct acx424akp), GFP_KERNEL);
-       if (!acx)
+       nt = devm_kzalloc(dev, sizeof(struct nt35560), GFP_KERNEL);
+       if (!nt)
                return -ENOMEM;
-       acx->video_mode = of_property_read_bool(dev->of_node,
+       nt->video_mode = of_property_read_bool(dev->of_node,
                                                "enforce-video-mode");
 
-       mipi_dsi_set_drvdata(dsi, acx);
-       acx->dev = dev;
+       mipi_dsi_set_drvdata(dsi, nt);
+       nt->dev = dev;
 
        dsi->lanes = 2;
        dsi->format = MIPI_DSI_FMT_RGB888;
        dsi->lp_rate = 19200000;
        dsi->hs_rate = 420160000;
 
-       if (acx->video_mode)
+       if (nt->video_mode)
                /* Burst mode using event for sync */
                dsi->mode_flags =
                        MIPI_DSI_MODE_VIDEO |
                dsi->mode_flags =
                        MIPI_DSI_CLOCK_NON_CONTINUOUS;
 
-       acx->supply = devm_regulator_get(dev, "vddi");
-       if (IS_ERR(acx->supply))
-               return PTR_ERR(acx->supply);
+       nt->supply = devm_regulator_get(dev, "vddi");
+       if (IS_ERR(nt->supply))
+               return PTR_ERR(nt->supply);
 
        /* This asserts RESET by default */
-       acx->reset_gpio = devm_gpiod_get_optional(dev, "reset",
-                                                 GPIOD_OUT_HIGH);
-       if (IS_ERR(acx->reset_gpio))
-               return dev_err_probe(dev, PTR_ERR(acx->reset_gpio),
+       nt->reset_gpio = devm_gpiod_get_optional(dev, "reset",
+                                                GPIOD_OUT_HIGH);
+       if (IS_ERR(nt->reset_gpio))
+               return dev_err_probe(dev, PTR_ERR(nt->reset_gpio),
                                     "failed to request GPIO\n");
 
-       drm_panel_init(&acx->panel, dev, &acx424akp_drm_funcs,
+       drm_panel_init(&nt->panel, dev, &nt35560_drm_funcs,
                       DRM_MODE_CONNECTOR_DSI);
 
-       acx->panel.backlight = devm_backlight_device_register(dev, "acx424akp", dev, acx,
-                                       &acx424akp_bl_ops, &acx424akp_bl_props);
-       if (IS_ERR(acx->panel.backlight))
-               return dev_err_probe(dev, PTR_ERR(acx->panel.backlight),
+       nt->panel.backlight = devm_backlight_device_register(dev, "nt35560", dev, nt,
+                                       &nt35560_bl_ops, &nt35560_bl_props);
+       if (IS_ERR(nt->panel.backlight))
+               return dev_err_probe(dev, PTR_ERR(nt->panel.backlight),
                                     "failed to register backlight device\n");
 
-       drm_panel_add(&acx->panel);
+       drm_panel_add(&nt->panel);
 
        ret = mipi_dsi_attach(dsi);
        if (ret < 0) {
-               drm_panel_remove(&acx->panel);
+               drm_panel_remove(&nt->panel);
                return ret;
        }
 
        return 0;
 }
 
-static int acx424akp_remove(struct mipi_dsi_device *dsi)
+static int nt35560_remove(struct mipi_dsi_device *dsi)
 {
-       struct acx424akp *acx = mipi_dsi_get_drvdata(dsi);
+       struct nt35560 *nt = mipi_dsi_get_drvdata(dsi);
 
        mipi_dsi_detach(dsi);
-       drm_panel_remove(&acx->panel);
+       drm_panel_remove(&nt->panel);
 
        return 0;
 }
 
-static const struct of_device_id acx424akp_of_match[] = {
+static const struct of_device_id nt35560_of_match[] = {
        { .compatible = "sony,acx424akp" },
        { /* sentinel */ }
 };
-MODULE_DEVICE_TABLE(of, acx424akp_of_match);
+MODULE_DEVICE_TABLE(of, nt35560_of_match);
 
-static struct mipi_dsi_driver acx424akp_driver = {
-       .probe = acx424akp_probe,
-       .remove = acx424akp_remove,
+static struct mipi_dsi_driver nt35560_driver = {
+       .probe = nt35560_probe,
+       .remove = nt35560_remove,
        .driver = {
-               .name = "panel-sony-acx424akp",
-               .of_match_table = acx424akp_of_match,
+               .name = "panel-novatek-nt35560",
+               .of_match_table = nt35560_of_match,
        },
 };
-module_mipi_dsi_driver(acx424akp_driver);
+module_mipi_dsi_driver(nt35560_driver);
 
 MODULE_AUTHOR("Linus Wallei <linus.walleij@linaro.org>");
-MODULE_DESCRIPTION("MIPI-DSI Sony acx424akp Panel Driver");
+MODULE_DESCRIPTION("MIPI-DSI Novatek NT35560 Panel Driver");
 MODULE_LICENSE("GPL v2");