* [PATCH] drm/msm/hdmi: redefinitions of macros not required
@ 2017-03-14 5:25 Vinay Simha BN
2017-03-22 5:11 ` Vinay Simha B N
0 siblings, 1 reply; 3+ messages in thread
From: Vinay Simha BN @ 2017-03-14 5:25 UTC (permalink / raw)
Cc: Vinay Simha BN, Rob Clark, David Airlie,
open list:DRM DRIVER FOR MSM ADRENO GPU,
open list:DRM DRIVER FOR MSM ADRENO GPU,
open list:DRM DRIVER FOR MSM ADRENO GPU, open list
4 macros already defined in hdmi.h,
which is not required to redefine in hdmi_audio.c
Signed-off-by: Vinay Simha BN <simhavcs@gmail.com>
---
drivers/gpu/drm/msm/hdmi/hdmi_audio.c | 7 -------
1 file changed, 7 deletions(-)
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_audio.c b/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
index a54d3bb..8177e85 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
@@ -18,13 +18,6 @@
#include <linux/hdmi.h>
#include "hdmi.h"
-
-/* Supported HDMI Audio channels */
-#define MSM_HDMI_AUDIO_CHANNEL_2 0
-#define MSM_HDMI_AUDIO_CHANNEL_4 1
-#define MSM_HDMI_AUDIO_CHANNEL_6 2
-#define MSM_HDMI_AUDIO_CHANNEL_8 3
-
/* maps MSM_HDMI_AUDIO_CHANNEL_n consts used by audio driver to # of channels: */
static int nchannels[] = { 2, 4, 6, 8 };
--
2.7.4
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] drm/msm/hdmi: redefinitions of macros not required
2017-03-14 5:25 [PATCH] drm/msm/hdmi: redefinitions of macros not required Vinay Simha BN
@ 2017-03-22 5:11 ` Vinay Simha B N
2017-03-22 11:19 ` Rob Clark
0 siblings, 1 reply; 3+ messages in thread
From: Vinay Simha B N @ 2017-03-22 5:11 UTC (permalink / raw)
Cc: Vinay Simha BN, Rob Clark, David Airlie,
open list:DRM DRIVER FOR MSM ADRENO GPU,
open list:DRM DRIVER FOR MSM ADRENO GPU,
open list:DRM DRIVER FOR MSM ADRENO GPU, open list
hi,
Could you please give some feedback or review comments for this patch
On 3/14/17, Vinay Simha BN <simhavcs@gmail.com> wrote:
> 4 macros already defined in hdmi.h,
> which is not required to redefine in hdmi_audio.c
>
> Signed-off-by: Vinay Simha BN <simhavcs@gmail.com>
> ---
> drivers/gpu/drm/msm/hdmi/hdmi_audio.c | 7 -------
> 1 file changed, 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
> b/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
> index a54d3bb..8177e85 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
> @@ -18,13 +18,6 @@
> #include <linux/hdmi.h>
> #include "hdmi.h"
>
> -
> -/* Supported HDMI Audio channels */
> -#define MSM_HDMI_AUDIO_CHANNEL_2 0
> -#define MSM_HDMI_AUDIO_CHANNEL_4 1
> -#define MSM_HDMI_AUDIO_CHANNEL_6 2
> -#define MSM_HDMI_AUDIO_CHANNEL_8 3
> -
> /* maps MSM_HDMI_AUDIO_CHANNEL_n consts used by audio driver to # of
> channels: */
> static int nchannels[] = { 2, 4, 6, 8 };
>
> --
> 2.7.4
>
>
--
regards,
vinaysimha
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] drm/msm/hdmi: redefinitions of macros not required
2017-03-22 5:11 ` Vinay Simha B N
@ 2017-03-22 11:19 ` Rob Clark
0 siblings, 0 replies; 3+ messages in thread
From: Rob Clark @ 2017-03-22 11:19 UTC (permalink / raw)
To: Vinay Simha B N
Cc: David Airlie, open list:DRM DRIVER FOR MSM ADRENO GPU,
open list:DRM DRIVER FOR MSM ADRENO GPU,
open list:DRM DRIVER FOR MSM ADRENO GPU, open list
On Wed, Mar 22, 2017 at 1:11 AM, Vinay Simha B N <simhavcs@gmail.com> wrote:
> hi,
>
> Could you please give some feedback or review comments for this patch
looks good, I've pulled it into msm-next
BR,
-R
> On 3/14/17, Vinay Simha BN <simhavcs@gmail.com> wrote:
>> 4 macros already defined in hdmi.h,
>> which is not required to redefine in hdmi_audio.c
>>
>> Signed-off-by: Vinay Simha BN <simhavcs@gmail.com>
>> ---
>> drivers/gpu/drm/msm/hdmi/hdmi_audio.c | 7 -------
>> 1 file changed, 7 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
>> b/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
>> index a54d3bb..8177e85 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_audio.c
>> @@ -18,13 +18,6 @@
>> #include <linux/hdmi.h>
>> #include "hdmi.h"
>>
>> -
>> -/* Supported HDMI Audio channels */
>> -#define MSM_HDMI_AUDIO_CHANNEL_2 0
>> -#define MSM_HDMI_AUDIO_CHANNEL_4 1
>> -#define MSM_HDMI_AUDIO_CHANNEL_6 2
>> -#define MSM_HDMI_AUDIO_CHANNEL_8 3
>> -
>> /* maps MSM_HDMI_AUDIO_CHANNEL_n consts used by audio driver to # of
>> channels: */
>> static int nchannels[] = { 2, 4, 6, 8 };
>>
>> --
>> 2.7.4
>>
>>
>
>
> --
> regards,
> vinaysimha
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-03-22 11:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-14 5:25 [PATCH] drm/msm/hdmi: redefinitions of macros not required Vinay Simha BN
2017-03-22 5:11 ` Vinay Simha B N
2017-03-22 11:19 ` Rob Clark
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®