Skip to content

publish ext image extensions#141

Merged
bashbaug merged 1 commit intoKhronosGroup:mainfrom
bashbaug:publish-ext-image-extensions
Jun 14, 2023
Merged

publish ext image extensions#141
bashbaug merged 1 commit intoKhronosGroup:mainfrom
bashbaug:publish-ext-image-extensions

Conversation

@bashbaug
Copy link
Copy Markdown
Contributor

@bashbaug bashbaug requested a review from kpet June 13, 2023 18:00
@bashbaug bashbaug marked this pull request as draft June 13, 2023 18:01
@bashbaug
Copy link
Copy Markdown
Contributor Author

bashbaug commented Jun 13, 2023

@kpet heads up, I've converted this PR to a "draft" because there are a few issue I think we'll want to fix in the extension source before publishing:

  • The biggest of these is that the cl_ext_image_requirements_info spec still says "draft spec, not approved", and is still described as being version 0.5. I think these extension are shipping, in which case the status and the version should be updated.
  • There are a few minor editorial issues with mismatched parentheses and slightly odd indentation. Nothing critical, but if we're updating the specs anyhow it would be nice to fix these.
  • I did a quick review of the cl_ext_image_from_buffer spec and it mostly looks OK to me, but it might be worth a quick read through also.

Thanks!

@bashbaug bashbaug changed the title public ext image extensions publish ext image extensions Jun 13, 2023
kpet added a commit to kpet/OpenCL-Docs that referenced this pull request Jun 14, 2023
…age_from_buffer

- Fix status of cl_ext_image_requirements_info
- Use macros for all commands and enums
- Fix mismatched parenthesis

See KhronosGroup/OpenCL-Registry#141

Signed-off-by: Kevin Petit <kevin.petit@arm.com>
Change-Id: I36ae9926e1da3fcf9c74d221888d30ab61c75b4e
@kpet
Copy link
Copy Markdown
Contributor

kpet commented Jun 14, 2023

@bashbaug I've created KhronosGroup/OpenCL-Docs#942 to tidy up the sources. I see what you mean regarding the indentation but I've exhausted the time I could afford to spend convincing asciidoctor to not do that. I don't think it's ugly enough to require blocking publication but if you happen to know the magic runes to align those paragraphs, any pointers would be welcome! :)

bashbaug pushed a commit to KhronosGroup/OpenCL-Docs that referenced this pull request Jun 14, 2023
…age_from_buffer (#942)

- Fix status of cl_ext_image_requirements_info
- Use macros for all commands and enums
- Fix mismatched parenthesis

See KhronosGroup/OpenCL-Registry#141


Change-Id: I36ae9926e1da3fcf9c74d221888d30ab61c75b4e

Signed-off-by: Kevin Petit <kevin.petit@arm.com>
cl_ext_image_from_buffer
cl_ext_image_requirements_info
@bashbaug bashbaug force-pushed the publish-ext-image-extensions branch from f052b1b to 4c95309 Compare June 14, 2023 16:53
@bashbaug bashbaug marked this pull request as ready for review June 14, 2023 16:53
@bashbaug
Copy link
Copy Markdown
Contributor Author

I've rebuilt the specs after merging KhronosGroup/OpenCL-Docs#942. I think this is ready to go now.

For the record, I think it's a little weird to publish a shipping spec as v0.5, but if this is intended I don't think it should block publication.

Thanks!

Copy link
Copy Markdown
Contributor

@kpet kpet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, all looking good!

@bashbaug bashbaug merged commit b2ae9cb into KhronosGroup:main Jun 14, 2023
@bashbaug bashbaug deleted the publish-ext-image-extensions branch June 14, 2023 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants