From: Daniel Vetter <daniel@ffwll.ch> To: Hans Verkuil <hverkuil@xs4all.nl> Cc: linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH 0/5] Rename AdobeRGB to opRGB Date: Thu, 13 Sep 2018 15:46:24 +0200 [thread overview] Message-ID: <20180913134624.GG11082@phenom.ffwll.local> (raw) In-Reply-To: <20180913114731.16500-1-hverkuil@xs4all.nl> On Thu, Sep 13, 2018 at 01:47:26PM +0200, Hans Verkuil wrote: > From: Hans Verkuil <hans.verkuil@cisco.com> > > This patch series replaces all AdobeRGB references by opRGB references. > > In November last year all references to the AdobeRGB colorspace were removed > from the CTA-861 standards (all versions) and replaced with the corresponding > international opRGB standard (IEC 61966-2-5) due to trademark issues. > > This patch series makes the same change in the kernel because: > > 1) it makes sense to keep in sync with the CTA-861 standard, > 2) using an international standard is always preferable to a company standard, > 3) avoid possible future trademark complaints. > > The first two patches can go through the media subsystem. The third patch > changes hdmi.c/h, but since the renamed defines from hdmi.h are only used > in the media subsystem I would prefer to merge this via the media subsystem > as well. So if I can get an Ack, then that would be great. > > The fourth patch I can push to drm-misc when it's reviewed, and the final > patch can go through the AMD GPU maintainers. > > There are only two references to the old name left since they are part of > the V4L2 public API, so I can't remove them. Does what it says on the tin. For the series: Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> But definitely needs an ack from amdgpu folks too (plus media ofc). -Daniel > > Regards, > > Hans > > > Hans Verkuil (5): > media: replace ADOBERGB by OPRGB > media colorspaces*.rst: rename AdobeRGB to opRGB > hdmi.h: rename ADOBE_RGB to OPRGB and ADOBE_YCC to OPYCC > drm/bridge/synopsys/dw-hdmi.h: rename ADOBE to OP > drm/amd: rename ADOBE to OP > > Documentation/media/uapi/v4l/biblio.rst | 10 - > .../media/uapi/v4l/colorspaces-defs.rst | 8 +- > .../media/uapi/v4l/colorspaces-details.rst | 13 +- > .../media/videodev2.h.rst.exceptions | 2 + > .../drm/amd/display/dc/core/dc_hw_sequencer.c | 4 +- > .../gpu/drm/amd/display/dc/core/dc_resource.c | 4 +- > drivers/gpu/drm/amd/display/dc/dc_hw_types.h | 2 +- > .../amd/display/dc/dce/dce_stream_encoder.c | 2 +- > .../amd/display/dc/dcn10/dcn10_hw_sequencer.c | 2 +- > .../display/dc/dcn10/dcn10_stream_encoder.c | 2 +- > .../gpu/drm/amd/display/dc/inc/hw/transform.h | 4 +- > drivers/gpu/drm/bridge/synopsys/dw-hdmi.h | 4 +- > .../media/common/v4l2-tpg/v4l2-tpg-colors.c | 262 +++++++++--------- > drivers/media/i2c/adv7511.c | 6 +- > drivers/media/i2c/adv7604.c | 2 +- > drivers/media/i2c/tc358743.c | 4 +- > drivers/media/platform/vivid/vivid-core.h | 2 +- > drivers/media/platform/vivid/vivid-ctrls.c | 6 +- > drivers/media/platform/vivid/vivid-vid-out.c | 2 +- > drivers/media/v4l2-core/v4l2-dv-timings.c | 12 +- > drivers/video/hdmi.c | 8 +- > include/linux/hdmi.h | 4 +- > include/uapi/linux/videodev2.h | 16 +- > 23 files changed, 190 insertions(+), 191 deletions(-) > > -- > 2.18.0 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Vetter <daniel@ffwll.ch> To: Hans Verkuil <hverkuil@xs4all.nl> Cc: dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org Subject: Re: [PATCH 0/5] Rename AdobeRGB to opRGB Date: Thu, 13 Sep 2018 15:46:24 +0200 [thread overview] Message-ID: <20180913134624.GG11082@phenom.ffwll.local> (raw) In-Reply-To: <20180913114731.16500-1-hverkuil@xs4all.nl> On Thu, Sep 13, 2018 at 01:47:26PM +0200, Hans Verkuil wrote: > From: Hans Verkuil <hans.verkuil@cisco.com> > > This patch series replaces all AdobeRGB references by opRGB references. > > In November last year all references to the AdobeRGB colorspace were removed > from the CTA-861 standards (all versions) and replaced with the corresponding > international opRGB standard (IEC 61966-2-5) due to trademark issues. > > This patch series makes the same change in the kernel because: > > 1) it makes sense to keep in sync with the CTA-861 standard, > 2) using an international standard is always preferable to a company standard, > 3) avoid possible future trademark complaints. > > The first two patches can go through the media subsystem. The third patch > changes hdmi.c/h, but since the renamed defines from hdmi.h are only used > in the media subsystem I would prefer to merge this via the media subsystem > as well. So if I can get an Ack, then that would be great. > > The fourth patch I can push to drm-misc when it's reviewed, and the final > patch can go through the AMD GPU maintainers. > > There are only two references to the old name left since they are part of > the V4L2 public API, so I can't remove them. Does what it says on the tin. For the series: Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> But definitely needs an ack from amdgpu folks too (plus media ofc). -Daniel > > Regards, > > Hans > > > Hans Verkuil (5): > media: replace ADOBERGB by OPRGB > media colorspaces*.rst: rename AdobeRGB to opRGB > hdmi.h: rename ADOBE_RGB to OPRGB and ADOBE_YCC to OPYCC > drm/bridge/synopsys/dw-hdmi.h: rename ADOBE to OP > drm/amd: rename ADOBE to OP > > Documentation/media/uapi/v4l/biblio.rst | 10 - > .../media/uapi/v4l/colorspaces-defs.rst | 8 +- > .../media/uapi/v4l/colorspaces-details.rst | 13 +- > .../media/videodev2.h.rst.exceptions | 2 + > .../drm/amd/display/dc/core/dc_hw_sequencer.c | 4 +- > .../gpu/drm/amd/display/dc/core/dc_resource.c | 4 +- > drivers/gpu/drm/amd/display/dc/dc_hw_types.h | 2 +- > .../amd/display/dc/dce/dce_stream_encoder.c | 2 +- > .../amd/display/dc/dcn10/dcn10_hw_sequencer.c | 2 +- > .../display/dc/dcn10/dcn10_stream_encoder.c | 2 +- > .../gpu/drm/amd/display/dc/inc/hw/transform.h | 4 +- > drivers/gpu/drm/bridge/synopsys/dw-hdmi.h | 4 +- > .../media/common/v4l2-tpg/v4l2-tpg-colors.c | 262 +++++++++--------- > drivers/media/i2c/adv7511.c | 6 +- > drivers/media/i2c/adv7604.c | 2 +- > drivers/media/i2c/tc358743.c | 4 +- > drivers/media/platform/vivid/vivid-core.h | 2 +- > drivers/media/platform/vivid/vivid-ctrls.c | 6 +- > drivers/media/platform/vivid/vivid-vid-out.c | 2 +- > drivers/media/v4l2-core/v4l2-dv-timings.c | 12 +- > drivers/video/hdmi.c | 8 +- > include/linux/hdmi.h | 4 +- > include/uapi/linux/videodev2.h | 16 +- > 23 files changed, 190 insertions(+), 191 deletions(-) > > -- > 2.18.0 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2018-09-13 18:56 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-09-13 11:47 [PATCH 0/5] Rename AdobeRGB to opRGB Hans Verkuil 2018-09-13 11:47 ` Hans Verkuil 2018-09-13 11:47 ` [PATCH 1/5] media: replace ADOBERGB by OPRGB Hans Verkuil 2018-09-13 11:47 ` Hans Verkuil 2018-09-13 14:29 ` Mauro Carvalho Chehab 2018-09-13 14:29 ` Mauro Carvalho Chehab 2018-09-13 14:35 ` Hans Verkuil 2018-09-13 14:35 ` Hans Verkuil 2018-09-14 8:58 ` [PATCHv2 " Hans Verkuil 2018-09-14 8:58 ` Hans Verkuil 2018-09-13 11:47 ` [PATCH 2/5] media colorspaces*.rst: rename AdobeRGB to opRGB Hans Verkuil 2018-09-13 11:47 ` Hans Verkuil 2018-09-13 11:47 ` [PATCH 3/5] hdmi.h: rename ADOBE_RGB to OPRGB and ADOBE_YCC to OPYCC Hans Verkuil 2018-09-13 11:47 ` Hans Verkuil 2018-09-13 11:47 ` [PATCH 4/5] drm/bridge/synopsys/dw-hdmi.h: rename ADOBE to OP Hans Verkuil 2018-09-13 11:47 ` Hans Verkuil 2018-09-13 11:47 ` [PATCH 5/5] drm/amd: " Hans Verkuil 2018-09-13 11:47 ` Hans Verkuil 2018-09-13 13:46 ` Daniel Vetter [this message] 2018-09-13 13:46 ` [PATCH 0/5] Rename AdobeRGB to opRGB Daniel Vetter
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20180913134624.GG11082@phenom.ffwll.local \ --to=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=hverkuil@xs4all.nl \ --cc=linux-media@vger.kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.