[Bug 730349] New: gst-omx aac encoder on msm89x6

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=730349
  GStreamer | gst-omx | 1.x

           Summary: gst-omx aac encoder on msm89x6
    Classification: Platform
           Product: GStreamer
           Version: 1.x
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: critical
          Priority: Normal
         Component: gst-omx
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: jun.ji at lge.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


I just found that slomo recently updated gst-omx and I couldn't see yet if the
patch is to solve issues that I was able to workaround by my patch that I'm
posting now.

However, this is true that aacencoder didn't work as itself at least the moment
that I made this patch.

My device is qcom msm89x6 and the below is what my patch tries to fix:

1. qcom chipset needs not to disable any ports in
gst_omx_audio_enc_set_format()
-> this is what slomo noticed me a long time ago and probably what he just
updated.

2. gstomxaacenc.c doesn't seem to work with the general gstreamer negotiation
routine.
-> this encoder tries to use undefined caps yet by negotiation feature and dies
making severe error. (not-negotiated...) so I had to put some workaround to
gstomxaacenc.c


This is my first time to upload to bugzilla so please inform me if I missed
something.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.