All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v5 00/17] New uAPI drm properties for color management
@ 2021-06-30 15:10 ` Werner Sembach
  0 siblings, 0 replies; 68+ messages in thread
From: Werner Sembach @ 2021-06-30 15:10 UTC (permalink / raw)
  To: harry.wentland, sunpeng.li, alexander.deucher, christian.koenig,
	airlied, daniel, maarten.lankhorst, mripard, tzimmermann,
	jani.nikula, joonas.lahtinen, rodrigo.vivi, amd-gfx, dri-devel,
	linux-kernel, intel-gfx, emil.l.velikov

Implementation of https://lkml.org/lkml/2021/5/12/764 now feature complete
albeit not fully tested.

I have now corrected the DSC behavior, but still no wait to test it.

Exact dithering behavior remains a mistery so in case dithering is active it's
not 100% clear what "active bpc" means, or where the "max bpc" limit is applied.

I have no DP MST splitter at hand. I tried my best to not break anything,
but if one who has one could test it would be very helpful.

Things on my TODO list:
    - add "min bpc" property
    - rewrite "preferred color format" to "force color format"
    - make "Broadcast RGB" only affect RGB on AMD too
    - remove unreachable enums of "active/preferred/force color format"



^ permalink raw reply	[flat|nested] 68+ messages in thread
* Re: [PATCH v5 03/17] drm/uAPI: Add "active bpc" as feedback channel for "max bpc" drm property
  2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
@ 2021-07-03 12:48 ` Dan Carpenter
  -1 siblings, 0 replies; 68+ messages in thread
From: kernel test robot @ 2021-07-02 21:38 UTC (permalink / raw)
  To: kbuild

[-- Attachment #1: Type: text/plain, Size: 4471 bytes --]

CC: kbuild-all(a)lists.01.org
In-Reply-To: <20210630151018.330354-4-wse@tuxedocomputers.com>
References: <20210630151018.330354-4-wse@tuxedocomputers.com>
TO: Werner Sembach <wse@tuxedocomputers.com>
TO: harry.wentland(a)amd.com
TO: sunpeng.li(a)amd.com
TO: alexander.deucher(a)amd.com
TO: christian.koenig(a)amd.com
TO: airlied(a)linux.ie
TO: daniel(a)ffwll.ch
TO: maarten.lankhorst(a)linux.intel.com
TO: mripard(a)kernel.org
TO: tzimmermann(a)suse.de
TO: jani.nikula(a)linux.intel.com

Hi Werner,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm-tip/drm-tip drm-exynos/exynos-drm-next linus/master next-20210701]
[cannot apply to tegra-drm/drm/tegra/for-next v5.13]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Werner-Sembach/New-uAPI-drm-properties-for-color-management/20210630-231204
base:   git://anongit.freedesktop.org/drm-intel for-linux-next
:::::: branch date: 2 days ago
:::::: commit date: 2 days ago
config: i386-randconfig-m021-20210702 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

smatch warnings:
drivers/gpu/drm/drm_connector.c:2189 drm_connector_attach_active_bpc_property() error: uninitialized symbol 'prop'.

vim +/prop +2189 drivers/gpu/drm/drm_connector.c

47e22ff1a9e0c1 Radhakrishna Sripada 2018-10-12  2163  
3914c7c51a7d7f Werner Sembach       2021-06-30  2164  /**
3914c7c51a7d7f Werner Sembach       2021-06-30  2165   * drm_connector_attach_active_bpc_property - attach "active bpc" property
3914c7c51a7d7f Werner Sembach       2021-06-30  2166   * @connector: connector to attach active bpc property on.
3914c7c51a7d7f Werner Sembach       2021-06-30  2167   * @min: The minimum bit depth supported by the connector.
3914c7c51a7d7f Werner Sembach       2021-06-30  2168   * @max: The maximum bit depth supported by the connector.
3914c7c51a7d7f Werner Sembach       2021-06-30  2169   *
3914c7c51a7d7f Werner Sembach       2021-06-30  2170   * This is used to check the applied bit depth on a connector.
3914c7c51a7d7f Werner Sembach       2021-06-30  2171   *
3914c7c51a7d7f Werner Sembach       2021-06-30  2172   * Returns:
3914c7c51a7d7f Werner Sembach       2021-06-30  2173   * Zero on success, negative errno on failure.
3914c7c51a7d7f Werner Sembach       2021-06-30  2174   */
3914c7c51a7d7f Werner Sembach       2021-06-30  2175  int drm_connector_attach_active_bpc_property(struct drm_connector *connector, int min, int max)
3914c7c51a7d7f Werner Sembach       2021-06-30  2176  {
3914c7c51a7d7f Werner Sembach       2021-06-30  2177  	struct drm_device *dev = connector->dev;
3914c7c51a7d7f Werner Sembach       2021-06-30  2178  	struct drm_property *prop;
3914c7c51a7d7f Werner Sembach       2021-06-30  2179  
3914c7c51a7d7f Werner Sembach       2021-06-30  2180  	if (!connector->active_bpc_property) {
3914c7c51a7d7f Werner Sembach       2021-06-30  2181  		prop = drm_property_create_range(dev, DRM_MODE_PROP_IMMUTABLE, "active bpc",
3914c7c51a7d7f Werner Sembach       2021-06-30  2182  						 min, max);
3914c7c51a7d7f Werner Sembach       2021-06-30  2183  		if (!prop)
3914c7c51a7d7f Werner Sembach       2021-06-30  2184  			return -ENOMEM;
3914c7c51a7d7f Werner Sembach       2021-06-30  2185  
3914c7c51a7d7f Werner Sembach       2021-06-30  2186  		connector->active_bpc_property = prop;
3914c7c51a7d7f Werner Sembach       2021-06-30  2187  	}
3914c7c51a7d7f Werner Sembach       2021-06-30  2188  
3914c7c51a7d7f Werner Sembach       2021-06-30 @2189  	drm_object_attach_property(&connector->base, prop, 0);
3914c7c51a7d7f Werner Sembach       2021-06-30  2190  
3914c7c51a7d7f Werner Sembach       2021-06-30  2191  	return 0;
3914c7c51a7d7f Werner Sembach       2021-06-30  2192  }
3914c7c51a7d7f Werner Sembach       2021-06-30  2193  EXPORT_SYMBOL(drm_connector_attach_active_bpc_property);
3914c7c51a7d7f Werner Sembach       2021-06-30  2194  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 43833 bytes --]

^ permalink raw reply	[flat|nested] 68+ messages in thread
* Re: [PATCH v5 12/17] drm/uAPI: Add "preferred color format" drm property as setting for userspace
  2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
@ 2021-07-03 13:24 ` Dan Carpenter
  -1 siblings, 0 replies; 68+ messages in thread
From: kernel test robot @ 2021-07-03 12:12 UTC (permalink / raw)
  To: kbuild

[-- Attachment #1: Type: text/plain, Size: 4769 bytes --]

CC: kbuild-all(a)lists.01.org
In-Reply-To: <20210630151018.330354-13-wse@tuxedocomputers.com>
References: <20210630151018.330354-13-wse@tuxedocomputers.com>
TO: Werner Sembach <wse@tuxedocomputers.com>
TO: harry.wentland(a)amd.com
TO: sunpeng.li(a)amd.com
TO: alexander.deucher(a)amd.com
TO: christian.koenig(a)amd.com
TO: airlied(a)linux.ie
TO: daniel(a)ffwll.ch
TO: maarten.lankhorst(a)linux.intel.com
TO: mripard(a)kernel.org
TO: tzimmermann(a)suse.de
TO: jani.nikula(a)linux.intel.com

Hi Werner,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm-tip/drm-tip drm-exynos/exynos-drm-next linus/master next-20210701]
[cannot apply to tegra-drm/drm/tegra/for-next v5.13]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Werner-Sembach/New-uAPI-drm-properties-for-color-management/20210630-231204
base:   git://anongit.freedesktop.org/drm-intel for-linux-next
:::::: branch date: 3 days ago
:::::: commit date: 3 days ago
config: i386-randconfig-m021-20210702 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

New smatch warnings:
drivers/gpu/drm/drm_connector.c:2281 drm_connector_attach_preferred_color_format_property() error: uninitialized symbol 'prop'.

Old smatch warnings:
drivers/gpu/drm/drm_connector.c:2238 drm_connector_attach_active_bpc_property() error: uninitialized symbol 'prop'.
drivers/gpu/drm/drm_connector.c:2312 drm_connector_attach_active_color_format_property() error: uninitialized symbol 'prop'.
drivers/gpu/drm/drm_connector.c:2358 drm_connector_attach_active_color_range_property() error: uninitialized symbol 'prop'.

vim +/prop +2281 drivers/gpu/drm/drm_connector.c

3914c7c51a7d7f Werner Sembach 2021-06-30  2256  
fadf17def71e00 Werner Sembach 2021-06-30  2257  /**
fadf17def71e00 Werner Sembach 2021-06-30  2258   * drm_connector_attach_preferred_color_format_property - attach "preferred color format" property
fadf17def71e00 Werner Sembach 2021-06-30  2259   * @connector: connector to attach preferred color format property on.
fadf17def71e00 Werner Sembach 2021-06-30  2260   *
fadf17def71e00 Werner Sembach 2021-06-30  2261   * This is used to add support for selecting a color format on a connector.
fadf17def71e00 Werner Sembach 2021-06-30  2262   *
fadf17def71e00 Werner Sembach 2021-06-30  2263   * Returns:
fadf17def71e00 Werner Sembach 2021-06-30  2264   * Zero on success, negative errno on failure.
fadf17def71e00 Werner Sembach 2021-06-30  2265   */
fadf17def71e00 Werner Sembach 2021-06-30  2266  int drm_connector_attach_preferred_color_format_property(struct drm_connector *connector)
fadf17def71e00 Werner Sembach 2021-06-30  2267  {
fadf17def71e00 Werner Sembach 2021-06-30  2268  	struct drm_device *dev = connector->dev;
fadf17def71e00 Werner Sembach 2021-06-30  2269  	struct drm_property *prop;
fadf17def71e00 Werner Sembach 2021-06-30  2270  
fadf17def71e00 Werner Sembach 2021-06-30  2271  	if (!connector->preferred_color_format_property) {
fadf17def71e00 Werner Sembach 2021-06-30  2272  		prop = drm_property_create_enum(dev, 0, "preferred color format",
fadf17def71e00 Werner Sembach 2021-06-30  2273  						drm_preferred_color_format_enum_list,
fadf17def71e00 Werner Sembach 2021-06-30  2274  						ARRAY_SIZE(drm_preferred_color_format_enum_list));
fadf17def71e00 Werner Sembach 2021-06-30  2275  		if (!prop)
fadf17def71e00 Werner Sembach 2021-06-30  2276  			return -ENOMEM;
fadf17def71e00 Werner Sembach 2021-06-30  2277  
fadf17def71e00 Werner Sembach 2021-06-30  2278  		connector->preferred_color_format_property = prop;
fadf17def71e00 Werner Sembach 2021-06-30  2279  	}
fadf17def71e00 Werner Sembach 2021-06-30  2280  
fadf17def71e00 Werner Sembach 2021-06-30 @2281  	drm_object_attach_property(&connector->base, prop, 0);
fadf17def71e00 Werner Sembach 2021-06-30  2282  	connector->state->preferred_color_format = 0;
fadf17def71e00 Werner Sembach 2021-06-30  2283  
fadf17def71e00 Werner Sembach 2021-06-30  2284  	return 0;
fadf17def71e00 Werner Sembach 2021-06-30  2285  }
fadf17def71e00 Werner Sembach 2021-06-30  2286  EXPORT_SYMBOL(drm_connector_attach_preferred_color_format_property);
fadf17def71e00 Werner Sembach 2021-06-30  2287  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 43833 bytes --]

^ permalink raw reply	[flat|nested] 68+ messages in thread

end of thread, other threads:[~2021-07-14 18:21 UTC | newest]

Thread overview: 68+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-30 15:10 [PATCH v5 00/17] New uAPI drm properties for color management Werner Sembach
2021-06-30 15:10 ` Werner Sembach
2021-06-30 15:10 ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 01/17] drm/amd/display: Remove unnecessary SIGNAL_TYPE_HDMI_TYPE_A check Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 02/17] drm/amd/display: Add missing cases convert_dc_color_depth_into_bpc Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 03/17] drm/uAPI: Add "active bpc" as feedback channel for "max bpc" drm property Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-07-01  3:21   ` kernel test robot
2021-06-30 15:10 ` [PATCH v5 04/17] drm/amd/display: Add handling for new "active bpc" property Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 05/17] drm/i915/display: " Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 06/17] drm/uAPI: Add "active color format" drm property as feedback for userspace Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 07/17] drm/amd/display: Add handling for new "active color format" property Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 08/17] drm/i915/display: " Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 09/17] drm/uAPI: Add "active color range" drm property as feedback for userspace Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 10/17] drm/amd/display: Add handling for new "active color range" property Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 11/17] drm/i915/display: " Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 12/17] drm/uAPI: Add "preferred color format" drm property as setting for userspace Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 13/17] drm/amd/display: Add handling for new "preferred color format" property Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 14/17] drm/i915/display: " Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 15/17] drm/uAPI: Move "Broadcast RGB" property from driver specific to general context Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 16/17] drm/i915/display: Use the general "Broadcast RGB" implementation Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-06-30 15:10 ` [PATCH v5 17/17] drm/amd/display: Add handling for new "Broadcast RGB" property Werner Sembach
2021-06-30 15:10   ` Werner Sembach
2021-06-30 15:10   ` [Intel-gfx] " Werner Sembach
2021-07-14 17:49   ` Werner Sembach
2021-07-14 18:20   ` Werner Sembach
2021-07-14 18:20     ` Werner Sembach
2021-07-14 18:20     ` [Intel-gfx] " Werner Sembach
2021-06-30 18:37 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for New uAPI drm properties for color management (rev3) Patchwork
2021-06-30 18:39 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-06-30 19:09 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-07-01  2:21 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-07-02 21:38 [PATCH v5 03/17] drm/uAPI: Add "active bpc" as feedback channel for "max bpc" drm property kernel test robot
2021-07-03 12:48 ` Dan Carpenter
2021-07-03 12:12 [PATCH v5 12/17] drm/uAPI: Add "preferred color format" drm property as setting for userspace kernel test robot
2021-07-03 13:24 ` Dan Carpenter
2021-07-05 15:51 ` Werner Sembach

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.