<div dir="ltr"><pre style="word-wrap:break-word"><font color="#000000"><span style="white-space:pre-wrap">Hi Sergey,

He are my wine-staging patches you included in your patches:

Patch 1:
- d3dx9_36: Align texture dimensions to block size for compressed textures in D3DXCheckTextureRequirements.
- d3dx9_36: Add support for FOURCC surface to save_dds_surface_to_memory
- d3dx9_36: Fix several issues in save_dds_surface_to_memory.
- d3dx9_36: Improve D3DXSaveTextureToFile to save simple texture to dds file.

Patch 2:
- d3dx9_36: Add dxtn support.

It seems also that patch 2 uses some code from Michael Muller's patches and it seems you intend to use them for the dxtn conversion in wined3d. 
- wined3d: Add support for DXTn software decoding through libtxc_dxtn. [rev 3]
- wined3d: add DXT1 to B4G4R4A4, DXT1 to B5G5R5A1 and DXT3 to B4G4R4A4 conversion.
- wined3d: Improve DXTn support and export conversion functions for d3dx9_36.

When using code from others, you need to mention its origin and and add the proper references to the authors.
Moreover, you squashed different changes into single patches while the rule is to have one patch per change. I would like also the granularity of my patches to be kept.
This is much better to discuss how to upstream these patches with the authors at #wine-staging.
Note that before doing some DXTN work you should first implement the low level functions based on libtxc_dxtn and make sure they can be accepted (wined3d exports, legal issue, ...).

Christian</span></font></pre><div class="gmail_extra"><br><div class="gmail_quote">2015-06-08 9:09 GMT+02:00 Nikolay Sivov <span dir="ltr"><<a href="mailto:bunglehead@gmail.com" target="_blank">bunglehead@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 08.06.2015 10:01, Sergey Isakov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
It is hard to distill what part of codes comes from where. My patches depends on previous patches and so they must be included too.<br>
OK, all copylefts must be included if I will know how to do this.<br>
If Christian Costa will say what to do I will do this.<br>
</blockquote>
<br>
Sure, let's hear his opinion.<div><div><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On 08.06.2015, at 9:43, Nikolay Sivov wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 07.06.2015 23:02, Sergey Isakov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I decided to improve builtin d3dx9_36 library with support of more file<br>
formats, texture formats, volume textures.<br>
</blockquote>
<br>
It's not your work though, it looks to me like 3 or so patches from Christian Costa squashed together. Why didn't you mention it?<br>
</blockquote>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div></div>