| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <c4a90873-d209-4c87-8875-9344d533fd9e@suse.de>
Date: Fri, 17 Jan 2025 11:39:56 +0100
From: Thomas Zimmermann <tzimmermann@...e.de>
To: fnkl.kernel@...il.com, Hector Martin <marcan@...can.st>,
Sven Peter <sven@...npeter.dev>, Alyssa Rosenzweig <alyssa@...enzweig.io>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, David Airlie <airlied@...il.com>,
Simona Vetter <simona@...ll.ch>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, Neil Armstrong <neil.armstrong@...aro.org>,
Jessica Zhang <quic_jesszhan@...cinc.com>, asahi@...ts.linux.dev
Cc: linux-arm-kernel@...ts.infradead.org, dri-devel@...ts.freedesktop.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Alyssa Ross <hi@...ssa.is>, Janne Grunau <j@...nau.net>
Subject: Re: [PATCH v4 2/5] drm: adp: Add Apple Display Pipe driver
Hi
Am 14.01.25 um 22:38 schrieb Sasha Finkelstein via B4 Relay:
[...]
> +
> +static int adp_setup_mode_config(struct adp_drv_private *adp)
> +{
> + struct drm_device *drm = &adp->drm;
> + int ret;
> + u32 size;
> +
> + ret = drmm_mode_config_init(drm);
> + if (ret)
> + return ret;
> +
> + /*
> + * Query screen size restrict the frame buffer size to the screen size
> + * aligned to the next multiple of 64. This is not necessary but can be
> + * used as simple check for non-desktop devices.
> + * Xorg's modesetting driver does not care about the connector
> + * "non-desktop" property. The max frame buffer width or height can be
> + * easily checked and a device can be reject if the max width/height is
> + * smaller than 120 for example.
> + * Any touchbar daemon is not limited by this small framebuffer size.
> + */
> + size = readl(adp->fe + ADP_SCREEN_SIZE);
> +
> + drm->mode_config.min_width = 32;
> + drm->mode_config.min_height = 32;
> + drm->mode_config.max_width = ALIGN(FIELD_GET(ADP_SCREEN_HSIZE, size), 64);
> + drm->mode_config.max_height = ALIGN(FIELD_GET(ADP_SCREEN_VSIZE, size), 64);
> + drm->mode_config.preferred_depth = 24;
> + drm->mode_config.prefer_shadow = 0;
> + drm->mode_config.funcs = &adp_mode_config_funcs;
> +
> + ret = adp_setup_crtc(adp);
> + if (ret) {
> + drm_err(drm, "failed to create crtc");
> + return ret;
> + }
> +
> + adp->encoder.possible_crtcs = ALL_CRTCS;
> + ret = drm_simple_encoder_init(drm, &adp->encoder, DRM_MODE_ENCODER_DSI);
Please do not use drm_simple_encoder_init(). It was a bad idea and is
deprecated. Open coding it in your driver should resolve the problem.
Best regards
Thomas
> + if (ret) {
> + drm_err(drm, "failed to init encoder");
> + return ret;
> + }
> +
> + ret = drm_bridge_attach(&adp->encoder, adp->next_bridge, NULL,
> + DRM_BRIDGE_ATTACH_NO_CONNECTOR);
> + if (ret) {
> + drm_err(drm, "failed to init bridge chain");
> + return ret;
> + }
> +
> + adp->connector = drm_bridge_connector_init(drm, &adp->encoder);
> + if (IS_ERR(adp->connector))
> + return PTR_ERR(adp->connector);
> +
> + drm_connector_attach_encoder(adp->connector, &adp->encoder);
> +
> + ret = drm_vblank_init(drm, drm->mode_config.num_crtc);
> + if (ret < 0) {
> + drm_err(drm, "failed to initialize vblank");
> + return ret;
> + }
> +
> + drm_mode_config_reset(drm);
> +
> + return 0;
> +}
> +
> +static int adp_parse_of(struct platform_device *pdev, struct adp_drv_private *adp)
> +{
> + struct device *dev = &pdev->dev;
> +
> + adp->be = devm_platform_ioremap_resource_byname(pdev, "be");
> + if (IS_ERR(adp->be)) {
> + dev_err(dev, "failed to map display backend mmio");
> + return PTR_ERR(adp->be);
> + }
> +
> + adp->fe = devm_platform_ioremap_resource_byname(pdev, "fe");
> + if (IS_ERR(adp->fe)) {
> + dev_err(dev, "failed to map display pipe mmio");
> + return PTR_ERR(adp->fe);
> + }
> +
> + adp->be_irq = platform_get_irq_byname(pdev, "be");
> + if (adp->be_irq < 0) {
> + dev_err(dev, "failed to find be irq");
> + return adp->be_irq;
> + }
> +
> + adp->fe_irq = platform_get_irq_byname(pdev, "fe");
> + if (adp->fe_irq < 0) {
> + dev_err(dev, "failed to find fe irq");
> + return adp->fe_irq;
> + }
> +
> + adp->next_bridge = devm_drm_of_get_bridge(dev, dev->of_node, 0, 0);
> + if (IS_ERR(adp->next_bridge)) {
> + if (PTR_ERR(adp->next_bridge) != EPROBE_DEFER)
> + dev_err(dev, "failed to find next bridge");
> + return PTR_ERR(adp->next_bridge);
> + }
> +
> + return 0;
> +}
> +
> +static irqreturn_t adp_fe_irq(int irq, void *arg)
> +{
> + struct adp_drv_private *adp = (struct adp_drv_private *)arg;
> + u32 int_status;
> + u32 int_ctl;
> +
> + spin_lock(&adp->irq_lock);
> +
> + int_status = readl(adp->fe + ADP_INT_STATUS);
> + if (int_status & ADP_INT_STATUS_VBLANK) {
> + drm_crtc_handle_vblank(&adp->crtc);
> + spin_lock(&adp->crtc.dev->event_lock);
> + if (adp->event) {
> + int_ctl = readl(adp->fe + ADP_CTRL);
> + if ((int_ctl & 0xF00) == 0x600) {
> + drm_crtc_send_vblank_event(&adp->crtc, adp->event);
> + adp->event = NULL;
> + drm_crtc_vblank_put(&adp->crtc);
> + }
> + }
> + spin_unlock(&adp->crtc.dev->event_lock);
> + }
> +
> + writel(int_status, adp->fe + ADP_INT_STATUS);
> +
> + spin_unlock(&adp->irq_lock);
> +
> + return IRQ_HANDLED;
> +}
> +
> +static int adp_probe(struct platform_device *pdev)
> +{
> + struct adp_drv_private *adp;
> + int err;
> +
> + adp = devm_drm_dev_alloc(&pdev->dev, &adp_driver, struct adp_drv_private, drm);
> + if (IS_ERR(adp))
> + return PTR_ERR(adp);
> +
> + spin_lock_init(&adp->irq_lock);
> +
> + dev_set_drvdata(&pdev->dev, &adp->drm);
> +
> + err = adp_parse_of(pdev, adp);
> + if (err < 0)
> + return err;
> +
> + adp_disable_vblank(adp);
> + writel(ADP_CTRL_FIFO_ON | ADP_CTRL_VBLANK_ON, adp->fe + ADP_CTRL);
> +
> + err = adp_setup_mode_config(adp);
> + if (err < 0)
> + return err;
> +
> + err = devm_request_irq(&pdev->dev, adp->fe_irq, adp_fe_irq, 0,
> + "adp-fe", adp);
> + if (err)
> + return err;
> +
> + err = drm_dev_register(&adp->drm, 0);
> + if (err)
> + return err;
> + return 0;
> +}
> +
> +static void adp_remove(struct platform_device *pdev)
> +{
> + struct device *dev = &pdev->dev;
> + struct drm_device *drm = dev_get_drvdata(dev);
> + struct adp_drv_private *adp = to_adp(drm);
> +
> + adp_disable_vblank(adp);
> + drm_dev_unregister(drm);
> + dev_set_drvdata(dev, NULL);
> + drm_atomic_helper_shutdown(drm);
> +}
> +
> +static const struct of_device_id adp_of_match[] = {
> + { .compatible = "apple,h7-display-pipe", },
> + { },
> +};
> +MODULE_DEVICE_TABLE(of, adp_of_match);
> +
> +static struct platform_driver adp_platform_driver = {
> + .driver = {
> + .name = "adp",
> + .of_match_table = adp_of_match,
> + },
> + .probe = adp_probe,
> + .remove = adp_remove,
> +};
> +
> +module_platform_driver(adp_platform_driver);
> +
> +MODULE_DESCRIPTION("Apple Display Pipe DRM driver");
> +MODULE_LICENSE("GPL");
>
--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
Powered by blists - more mailing lists