* [PATCH] drm: exynos: Use proper macro definition for HDMI_I2S_PIN_SEL_1 [not found] <CGME20180205174229epcas1p1e15e6f8ef5248ee28fa2e649faa8a866@epcas1p1.samsung.com> @ 2018-02-05 17:42 ` Sylwester Nawrocki 0 siblings, 0 replies; 6+ messages in thread From: Sylwester Nawrocki @ 2018-02-05 17:42 UTC (permalink / raw) To: inki.dae Cc: jy0922.shim, sw0312.kim, linux-samsung-soc, dri-devel, linux-arm-kernel, b.zolnierkie, m.szyprowski, Sylwester Nawrocki Bit field [2:0] of HDMI_I2S_PIN_SEL_1 corresponds to SDATA_0, not SDATA_2. This patch removes redefinition of HDMI_I2S_SEL_DATA2 constant and adds missing HDMI_I2S_SEL_DATA0. There is no functional change. Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com> --- drivers/gpu/drm/exynos/exynos_hdmi.c | 7 +++++-- drivers/gpu/drm/exynos/regs-hdmi.h | 2 +- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c index a4b75a46f946..abd84cbcf1c2 100644 --- a/drivers/gpu/drm/exynos/exynos_hdmi.c +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c @@ -1068,10 +1068,13 @@ static void hdmi_audio_config(struct hdmi_context *hdata) /* Configuration I2S input ports. Configure I2S_PIN_SEL_0~4 */ hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_0, HDMI_I2S_SEL_SCLK(5) | HDMI_I2S_SEL_LRCK(6)); - hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(1) - | HDMI_I2S_SEL_SDATA2(4)); + + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(3) + | HDMI_I2S_SEL_SDATA0(4)); + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_2, HDMI_I2S_SEL_SDATA3(1) | HDMI_I2S_SEL_SDATA2(2)); + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_3, HDMI_I2S_SEL_DSD(0)); /* I2S_CON_1 & 2 */ diff --git a/drivers/gpu/drm/exynos/regs-hdmi.h b/drivers/gpu/drm/exynos/regs-hdmi.h index 04be0f7e8193..4420c203ac85 100644 --- a/drivers/gpu/drm/exynos/regs-hdmi.h +++ b/drivers/gpu/drm/exynos/regs-hdmi.h @@ -464,7 +464,7 @@ /* I2S_PIN_SEL_1 */ #define HDMI_I2S_SEL_SDATA1(x) (((x) & 0x7) << 4) -#define HDMI_I2S_SEL_SDATA2(x) ((x) & 0x7) +#define HDMI_I2S_SEL_SDATA0(x) ((x) & 0x7) /* I2S_PIN_SEL_2 */ #define HDMI_I2S_SEL_SDATA3(x) (((x) & 0x7) << 4) -- 2.14.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] drm: exynos: Use proper macro definition for HDMI_I2S_PIN_SEL_1 @ 2018-02-05 17:42 ` Sylwester Nawrocki 0 siblings, 0 replies; 6+ messages in thread From: Sylwester Nawrocki @ 2018-02-05 17:42 UTC (permalink / raw) To: linux-arm-kernel Bit field [2:0] of HDMI_I2S_PIN_SEL_1 corresponds to SDATA_0, not SDATA_2. This patch removes redefinition of HDMI_I2S_SEL_DATA2 constant and adds missing HDMI_I2S_SEL_DATA0. There is no functional change. Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com> --- drivers/gpu/drm/exynos/exynos_hdmi.c | 7 +++++-- drivers/gpu/drm/exynos/regs-hdmi.h | 2 +- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c index a4b75a46f946..abd84cbcf1c2 100644 --- a/drivers/gpu/drm/exynos/exynos_hdmi.c +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c @@ -1068,10 +1068,13 @@ static void hdmi_audio_config(struct hdmi_context *hdata) /* Configuration I2S input ports. Configure I2S_PIN_SEL_0~4 */ hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_0, HDMI_I2S_SEL_SCLK(5) | HDMI_I2S_SEL_LRCK(6)); - hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(1) - | HDMI_I2S_SEL_SDATA2(4)); + + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(3) + | HDMI_I2S_SEL_SDATA0(4)); + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_2, HDMI_I2S_SEL_SDATA3(1) | HDMI_I2S_SEL_SDATA2(2)); + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_3, HDMI_I2S_SEL_DSD(0)); /* I2S_CON_1 & 2 */ diff --git a/drivers/gpu/drm/exynos/regs-hdmi.h b/drivers/gpu/drm/exynos/regs-hdmi.h index 04be0f7e8193..4420c203ac85 100644 --- a/drivers/gpu/drm/exynos/regs-hdmi.h +++ b/drivers/gpu/drm/exynos/regs-hdmi.h @@ -464,7 +464,7 @@ /* I2S_PIN_SEL_1 */ #define HDMI_I2S_SEL_SDATA1(x) (((x) & 0x7) << 4) -#define HDMI_I2S_SEL_SDATA2(x) ((x) & 0x7) +#define HDMI_I2S_SEL_SDATA0(x) ((x) & 0x7) /* I2S_PIN_SEL_2 */ #define HDMI_I2S_SEL_SDATA3(x) (((x) & 0x7) << 4) -- 2.14.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] drm: exynos: Use proper macro definition for HDMI_I2S_PIN_SEL_1 2018-02-05 17:42 ` Sylwester Nawrocki @ 2018-02-14 5:57 ` Inki Dae -1 siblings, 0 replies; 6+ messages in thread From: Inki Dae @ 2018-02-14 5:57 UTC (permalink / raw) To: Sylwester Nawrocki Cc: jy0922.shim, sw0312.kim, linux-samsung-soc, dri-devel, linux-arm-kernel, b.zolnierkie, m.szyprowski Hi Sylwester, 2018년 02월 06일 02:42에 Sylwester Nawrocki 이(가) 쓴 글: > Bit field [2:0] of HDMI_I2S_PIN_SEL_1 corresponds to SDATA_0, > not SDATA_2. This patch removes redefinition of HDMI_I2S_SEL_DATA2 > constant and adds missing HDMI_I2S_SEL_DATA0. There is no functional > change. > > Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com> > --- > drivers/gpu/drm/exynos/exynos_hdmi.c | 7 +++++-- > drivers/gpu/drm/exynos/regs-hdmi.h | 2 +- > 2 files changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c > index a4b75a46f946..abd84cbcf1c2 100644 > --- a/drivers/gpu/drm/exynos/exynos_hdmi.c > +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c > @@ -1068,10 +1068,13 @@ static void hdmi_audio_config(struct hdmi_context *hdata) > /* Configuration I2S input ports. Configure I2S_PIN_SEL_0~4 */ > hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_0, HDMI_I2S_SEL_SCLK(5) > | HDMI_I2S_SEL_LRCK(6)); > - hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(1) > - | HDMI_I2S_SEL_SDATA2(4)); > + > + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(3) Seems you fixed pin_sel_3 field value of I2S_PIN_SEL_1 register. According to datasheet, 0x3 should be set to this field to select SDATA_1 not 0x1. So could you update the description of this patch? Thanks, Inki Dae > + | HDMI_I2S_SEL_SDATA0(4)); > + > hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_2, HDMI_I2S_SEL_SDATA3(1) > | HDMI_I2S_SEL_SDATA2(2)); > + > hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_3, HDMI_I2S_SEL_DSD(0)); > > /* I2S_CON_1 & 2 */ > diff --git a/drivers/gpu/drm/exynos/regs-hdmi.h b/drivers/gpu/drm/exynos/regs-hdmi.h > index 04be0f7e8193..4420c203ac85 100644 > --- a/drivers/gpu/drm/exynos/regs-hdmi.h > +++ b/drivers/gpu/drm/exynos/regs-hdmi.h > @@ -464,7 +464,7 @@ > > /* I2S_PIN_SEL_1 */ > #define HDMI_I2S_SEL_SDATA1(x) (((x) & 0x7) << 4) > -#define HDMI_I2S_SEL_SDATA2(x) ((x) & 0x7) > +#define HDMI_I2S_SEL_SDATA0(x) ((x) & 0x7) > > /* I2S_PIN_SEL_2 */ > #define HDMI_I2S_SEL_SDATA3(x) (((x) & 0x7) << 4) > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] drm: exynos: Use proper macro definition for HDMI_I2S_PIN_SEL_1 @ 2018-02-14 5:57 ` Inki Dae 0 siblings, 0 replies; 6+ messages in thread From: Inki Dae @ 2018-02-14 5:57 UTC (permalink / raw) To: linux-arm-kernel Hi Sylwester, 2018? 02? 06? 02:42? Sylwester Nawrocki ?(?) ? ?: > Bit field [2:0] of HDMI_I2S_PIN_SEL_1 corresponds to SDATA_0, > not SDATA_2. This patch removes redefinition of HDMI_I2S_SEL_DATA2 > constant and adds missing HDMI_I2S_SEL_DATA0. There is no functional > change. > > Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com> > --- > drivers/gpu/drm/exynos/exynos_hdmi.c | 7 +++++-- > drivers/gpu/drm/exynos/regs-hdmi.h | 2 +- > 2 files changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c > index a4b75a46f946..abd84cbcf1c2 100644 > --- a/drivers/gpu/drm/exynos/exynos_hdmi.c > +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c > @@ -1068,10 +1068,13 @@ static void hdmi_audio_config(struct hdmi_context *hdata) > /* Configuration I2S input ports. Configure I2S_PIN_SEL_0~4 */ > hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_0, HDMI_I2S_SEL_SCLK(5) > | HDMI_I2S_SEL_LRCK(6)); > - hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(1) > - | HDMI_I2S_SEL_SDATA2(4)); > + > + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(3) Seems you fixed pin_sel_3 field value of I2S_PIN_SEL_1 register. According to datasheet, 0x3 should be set to this field to select SDATA_1 not 0x1. So could you update the description of this patch? Thanks, Inki Dae > + | HDMI_I2S_SEL_SDATA0(4)); > + > hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_2, HDMI_I2S_SEL_SDATA3(1) > | HDMI_I2S_SEL_SDATA2(2)); > + > hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_3, HDMI_I2S_SEL_DSD(0)); > > /* I2S_CON_1 & 2 */ > diff --git a/drivers/gpu/drm/exynos/regs-hdmi.h b/drivers/gpu/drm/exynos/regs-hdmi.h > index 04be0f7e8193..4420c203ac85 100644 > --- a/drivers/gpu/drm/exynos/regs-hdmi.h > +++ b/drivers/gpu/drm/exynos/regs-hdmi.h > @@ -464,7 +464,7 @@ > > /* I2S_PIN_SEL_1 */ > #define HDMI_I2S_SEL_SDATA1(x) (((x) & 0x7) << 4) > -#define HDMI_I2S_SEL_SDATA2(x) ((x) & 0x7) > +#define HDMI_I2S_SEL_SDATA0(x) ((x) & 0x7) > > /* I2S_PIN_SEL_2 */ > #define HDMI_I2S_SEL_SDATA3(x) (((x) & 0x7) << 4) > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm: exynos: Use proper macro definition for HDMI_I2S_PIN_SEL_1 2018-02-14 5:57 ` Inki Dae @ 2018-02-14 17:28 ` Sylwester Nawrocki -1 siblings, 0 replies; 6+ messages in thread From: Sylwester Nawrocki @ 2018-02-14 17:28 UTC (permalink / raw) To: Inki Dae Cc: jy0922.shim, sw0312.kim, linux-samsung-soc, dri-devel, linux-arm-kernel, b.zolnierkie, m.szyprowski Hi Inki, On 02/14/2018 06:57 AM, Inki Dae wrote: >> diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c >> index a4b75a46f946..abd84cbcf1c2 100644 >> --- a/drivers/gpu/drm/exynos/exynos_hdmi.c >> +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c >> @@ -1068,10 +1068,13 @@ static void hdmi_audio_config(struct hdmi_context *hdata) >> /* Configuration I2S input ports. Configure I2S_PIN_SEL_0~4 */ >> hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_0, HDMI_I2S_SEL_SCLK(5) >> | HDMI_I2S_SEL_LRCK(6)); >> - hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(1) >> - | HDMI_I2S_SEL_SDATA2(4)); >> + >> + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(3) > > Seems you fixed pin_sel_3 field value of I2S_PIN_SEL_1 register. According > to datasheet, 0x3 should be set to this field to select SDATA_1 not 0x1. > So could you update the description of this patch? Indeed, I finally missed that when I was writing the commit message. I've posted v2 already. -- Regards, Sylwester ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] drm: exynos: Use proper macro definition for HDMI_I2S_PIN_SEL_1 @ 2018-02-14 17:28 ` Sylwester Nawrocki 0 siblings, 0 replies; 6+ messages in thread From: Sylwester Nawrocki @ 2018-02-14 17:28 UTC (permalink / raw) To: linux-arm-kernel Hi Inki, On 02/14/2018 06:57 AM, Inki Dae wrote: >> diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c >> index a4b75a46f946..abd84cbcf1c2 100644 >> --- a/drivers/gpu/drm/exynos/exynos_hdmi.c >> +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c >> @@ -1068,10 +1068,13 @@ static void hdmi_audio_config(struct hdmi_context *hdata) >> /* Configuration I2S input ports. Configure I2S_PIN_SEL_0~4 */ >> hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_0, HDMI_I2S_SEL_SCLK(5) >> | HDMI_I2S_SEL_LRCK(6)); >> - hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(1) >> - | HDMI_I2S_SEL_SDATA2(4)); >> + >> + hdmi_reg_writeb(hdata, HDMI_I2S_PIN_SEL_1, HDMI_I2S_SEL_SDATA1(3) > > Seems you fixed pin_sel_3 field value of I2S_PIN_SEL_1 register. According > to datasheet, 0x3 should be set to this field to select SDATA_1 not 0x1. > So could you update the description of this patch? Indeed, I finally missed that when I was writing the commit message. I've posted v2 already. -- Regards, Sylwester ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-02-14 17:28 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20180205174229epcas1p1e15e6f8ef5248ee28fa2e649faa8a866@epcas1p1.samsung.com>
2018-02-05 17:42 ` [PATCH] drm: exynos: Use proper macro definition for HDMI_I2S_PIN_SEL_1 Sylwester Nawrocki
2018-02-05 17:42 ` Sylwester Nawrocki
2018-02-14 5:57 ` Inki Dae
2018-02-14 5:57 ` Inki Dae
2018-02-14 17:28 ` Sylwester Nawrocki
2018-02-14 17:28 ` Sylwester Nawrocki
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.