Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-03 Thread Jan Ekström
On Mon, Aug 3, 2020 at 10:34 AM Carl Eugen Hoyos wrote: > > > > > Am 02.08.2020 um 23:16 schrieb Jan Ekström : > > > >> On Sat, Aug 1, 2020 at 4:29 PM Carl Eugen Hoyos wrote: > >> > >>> Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : > >>> > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-03 Thread Hendrik Leppkes
On Mon, Aug 3, 2020 at 9:34 AM Carl Eugen Hoyos wrote: > > > > > Am 02.08.2020 um 23:16 schrieb Jan Ekström : > > > >> On Sat, Aug 1, 2020 at 4:29 PM Carl Eugen Hoyos wrote: > >> > >>> Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : > >>> > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen H

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-03 Thread Carl Eugen Hoyos
> Am 02.08.2020 um 23:16 schrieb Jan Ekström : > >> On Sat, Aug 1, 2020 at 4:29 PM Carl Eugen Hoyos wrote: >> >>> Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : >>> On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen Hoyos wrote: > Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-02 Thread Jan Ekström
On Sat, Aug 1, 2020 at 4:29 PM Carl Eugen Hoyos wrote: > > Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : > > > > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen Hoyos wrote: > > > > > > Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan Ekström : > > > > > > > Additionally, the spec only mentions

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-02 Thread James Almer
On 8/2/2020 11:22 AM, Jan Ekström wrote: > On Sat, Aug 1, 2020 at 4:44 PM Jan Ekström wrote: >> At this point I would not like to attempt to figure out how to skip channel coding units in the AAC decoder >>> >>> Just throw them away? >>> >> >> Very easy to say in words, but I am not sure

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-02 Thread Nicolas George
Jan Ekström (12020-08-02): > I think I've seen it mentioned recently when I've been discussing this > patch set, so I have a feeling we might see a new version of that at > some point. We already had multiple channels which were outside of > WAVEFORMATEXTENSIBLE (such as LFE2), so a more defining A

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-02 Thread Jan Ekström
On Sat, Aug 1, 2020 at 4:44 PM Jan Ekström wrote: > > > > At this point I would not like to attempt to figure out how to skip > > > channel coding units in the AAC decoder > > > > Just throw them away? > > > > Very easy to say in words, but I am not sure how simple this is in the > code. That said

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-02 Thread Jan Ekström
On Sat, Aug 1, 2020 at 2:46 PM Paul B Mahol wrote: > > On 6/16/20, Jan Ekström wrote: > > Requires some extraneous top side and bottom front channels to be > > defined. > > > > According to STD-B59v2, the defined channel layout is: > > - FL > > - FR > > - FC > > - LFE1 > > - BL > > - BR > > - FLc

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread Jan Ekström
On Sat, Aug 1, 2020 at 4:42 PM James Almer wrote: > > On 8/1/2020 9:17 AM, Jan Ekström wrote: > > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen Hoyos wrote: > >> > >> Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan Ekström : > >> > >>> Additionally, the spec only mentions stereo/5.1 for direct mapping,

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread Jan Ekström
On Sat, Aug 1, 2020 at 4:29 PM Carl Eugen Hoyos wrote: > > Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : > > > > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen Hoyos wrote: > > > > > > Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan Ekström : > > > > > > > Additionally, the spec only mentions

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread James Almer
On 8/1/2020 9:17 AM, Jan Ekström wrote: > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen Hoyos wrote: >> >> Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan Ekström : >> >>> Additionally, the spec only mentions stereo/5.1 for direct mapping, so >>> I've kept to 5.1 since we have the capability for that on

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread Carl Eugen Hoyos
Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : > > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen Hoyos wrote: > > > > Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan Ekström : > > > > > Additionally, the spec only mentions stereo/5.1 for direct mapping, so > > > I've kept to 5.1 since we have t

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread Jan Ekström
On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen Hoyos wrote: > > Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan Ekström : > > > Additionally, the spec only mentions stereo/5.1 for direct mapping, so > > I've kept to 5.1 since we have the capability for that one. > > And you still believe it would be a di

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread Carl Eugen Hoyos
Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan Ekström : > Additionally, the spec only mentions stereo/5.1 for direct mapping, so > I've kept to 5.1 since we have the capability for that one. And you still believe it would be a disadvantage if the decoder outputs this 5.1 mapping by default if not

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread Jan Ekström
On Sat, Aug 1, 2020 at 2:51 PM Carl Eugen Hoyos wrote: > > Am Di., 16. Juni 2020 um 23:11 Uhr schrieb Jan Ekström : > > > > Requires some extraneous top side and bottom front channels to be > > defined. > > > > According to STD-B59v2, the defined channel layout is: > > - FL > > - FR > > - FC > > -

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread Carl Eugen Hoyos
Am Di., 16. Juni 2020 um 23:11 Uhr schrieb Jan Ekström : > > Requires some extraneous top side and bottom front channels to be > defined. > > According to STD-B59v2, the defined channel layout is: > - FL > - FR > - FC > - LFE1 > - BL > - BR > - FLc > - FRc > - BC > - LFE2 > - SiL > - SiR No way t

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-01 Thread Paul B Mahol
On 6/16/20, Jan Ekström wrote: > Requires some extraneous top side and bottom front channels to be > defined. > > According to STD-B59v2, the defined channel layout is: > - FL > - FR > - FC > - LFE1 > - BL > - BR > - FLc > - FRc > - BC > - LFE2 > - SiL > - SiR > - TpFL > - TpFR > - TpFC > - TpC >

[FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-07-29 Thread Jan Ekström
Requires some extraneous top side and bottom front channels to be defined. According to STD-B59v2, the defined channel layout is: - FL - FR - FC - LFE1 - BL - BR - FLc - FRc - BC - LFE2 - SiL - SiR - TpFL - TpFR - TpFC - TpC - TpBL - TpBR - TpSiL - TpSiR - TpBC - BtFC - BtFL - BtFR --- doc/APIcha

[FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-06-16 Thread Jan Ekström
Requires some extraneous top side and bottom front channels to be defined. According to STD-B59v2, the defined channel layout is: - FL - FR - FC - LFE1 - BL - BR - FLc - FRc - BC - LFE2 - SiL - SiR - TpFL - TpFR - TpFC - TpC - TpBL - TpBR - TpSiL - TpSiR - TpBC - BtFC - BtFL - BtFR --- doc/APIcha