On 3/11/26 5:10 AM, Thomas Zimmermann wrote:
> Replace simple-display helpers with regular atomic helpers. Store the
> pipeline elements in struct ili9163_device and initialize them as part
> of probing the device. Use mipi-dbi's existing helpers and initializer
> macros where possible.
>
> Effectively open-codes the modesetting code in the initializer helpers
> of mipi-dbi and simple-display. Ili9163 requires a custom helper for
> CRTC enablement, and non-freeing cleanup of the pipeline.
>
...
> @@ -139,19 +198,22 @@ MODULE_DEVICE_TABLE(spi, ili9163_id);
> static int ili9163_probe(struct spi_device *spi)
> {
> struct device *dev = &spi->dev;
> + struct ili9163_device *ili9163;
> struct mipi_dbi_dev *dbidev;
> struct drm_device *drm;
> struct mipi_dbi *dbi;
> struct gpio_desc *dc;
> + struct drm_plane *plane;
> + struct drm_crtc *crtc;
> + struct drm_encoder *encoder;
> + struct drm_connector *connector;
> u32 rotation = 0;
> int ret;
>
> - dbidev = devm_drm_dev_alloc(dev, &ili9163_driver,
> - struct mipi_dbi_dev, drm);
> - if (IS_ERR(dbidev))
> - return PTR_ERR(dbidev);
> -
> - dbi = &dbidev->dbi;
Looks like this line was accidentally removed. It will result
in dereferencing an unintalized pointer.
