Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1600035 > unrolled thread
| Started by | Vinay Simha BN <simhavcs@gmail.com> |
|---|---|
| First post | 2017-03-14 06:30 +0100 |
| Last post | 2017-03-22 12:20 +0100 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] drm/msm/hdmi: redefinitions of macros not required Vinay Simha BN <simhavcs@gmail.com> - 2017-03-14 06:30 +0100
Re: [PATCH] drm/msm/hdmi: redefinitions of macros not required Vinay Simha B N <simhavcs@gmail.com> - 2017-03-22 06:20 +0100
Re: [PATCH] drm/msm/hdmi: redefinitions of macros not required Rob Clark <robdclark@gmail.com> - 2017-03-22 12:20 +0100
| From | Vinay Simha BN <simhavcs@gmail.com> |
|---|---|
| Date | 2017-03-14 06:30 +0100 |
| Subject | [PATCH] drm/msm/hdmi: redefinitions of macros not required |
| Message-ID | <tkNbY-3N3-5@gated-at.bofh.it> |
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
[toc] | [next] | [standalone]
| From | Vinay Simha B N <simhavcs@gmail.com> |
|---|---|
| Date | 2017-03-22 06:20 +0100 |
| Message-ID | <tnGQF-5L3-3@gated-at.bofh.it> |
| In reply to | #1600035 |
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
[toc] | [prev] | [next] | [standalone]
| From | Rob Clark <robdclark@gmail.com> |
|---|---|
| Date | 2017-03-22 12:20 +0100 |
| Message-ID | <tnMt4-1zh-21@gated-at.bofh.it> |
| In reply to | #1606211 |
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
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web