Retroplayer: RenderBufferPoolGBM: remove bpp

gbm does not need to know the bpp of the format
xbmc · Aug 9, 2018 · af3ee37 · af3ee37
1 parent 13b8f5b
commit af3ee37
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 11 deletions.
diff --git a/xbmc/cores/RetroPlayer/buffers/RenderBufferGBM.cpp b/xbmc/cores/RetroPlayer/buffers/RenderBufferGBM.cpp
@@ -16,11 +16,9 @@ using namespace KODI;
 using namespace RETRO;
 
 CRenderBufferGBM::CRenderBufferGBM(CRenderContext &context,
-                                   int fourcc,
-                                   int bpp) :
+                                   int fourcc) :
   m_context(context),
   m_fourcc(fourcc),
-  m_bpp(bpp),
   m_egl(new CEGLImage(static_cast<CWinSystemGbmEGLContext*>(CServiceBroker::GetWinSystem())->GetEGLDisplay())),
   m_bo(new CGBMBufferObject(fourcc))
 {

diff --git a/xbmc/cores/RetroPlayer/buffers/RenderBufferGBM.h b/xbmc/cores/RetroPlayer/buffers/RenderBufferGBM.h
@@ -27,8 +27,7 @@ namespace RETRO
   {
   public:
     CRenderBufferGBM(CRenderContext &context,
-                     int fourcc,
-                     int bpp);
+                     int fourcc);
     ~CRenderBufferGBM() override;
 
     // implementation of IRenderBuffer via CRenderBufferSysMem
@@ -46,7 +45,6 @@ namespace RETRO
     // Construction parameters
     CRenderContext &m_context;
     const int m_fourcc = 0;
-    const int m_bpp;
 
     const GLenum m_textureTarget = GL_TEXTURE_EXTERNAL_OES;
     GLuint m_textureId = 0;

diff --git a/xbmc/cores/RetroPlayer/buffers/RenderBufferPoolGBM.cpp b/xbmc/cores/RetroPlayer/buffers/RenderBufferPoolGBM.cpp
@@ -32,8 +32,7 @@ bool CRenderBufferPoolGBM::IsCompatible(const CRenderVideoSettings &renderSettin
 IRenderBuffer *CRenderBufferPoolGBM::CreateRenderBuffer(void *header /* = nullptr */)
 {
   return new CRenderBufferGBM(m_context,
-                              m_fourcc,
-                              m_bpp);
+                              m_fourcc);
 }
 
 bool CRenderBufferPoolGBM::ConfigureInternal()
@@ -43,14 +42,12 @@ bool CRenderBufferPoolGBM::ConfigureInternal()
     case AV_PIX_FMT_0RGB32:
     {
       m_fourcc = DRM_FORMAT_ARGB8888;
-      m_bpp = sizeof(uint32_t);
       return true;
     }
     case AV_PIX_FMT_RGB555:
     case AV_PIX_FMT_RGB565:
     {
       m_fourcc = DRM_FORMAT_RGB565;
-      m_bpp = sizeof(uint16_t);
       return true;
     }
     default:

diff --git a/xbmc/cores/RetroPlayer/buffers/RenderBufferPoolGBM.h b/xbmc/cores/RetroPlayer/buffers/RenderBufferPoolGBM.h
@@ -35,7 +35,6 @@ namespace RETRO
 
     // Configuration parameters
     int m_fourcc = 0;
-    int m_bpp = 0;
   };
 }
 }