* [PATCH] ARM: pxa168/gplugd: add the correct SSP device @ 2011-11-11 12:19 Eric Miao 2011-11-11 16:02 ` Tanmay Upadhyay 0 siblings, 1 reply; 7+ messages in thread From: Eric Miao @ 2011-11-11 12:19 UTC (permalink / raw) To: linux-arm-kernel There is no SSP0, but SSP2 is used on gplugd. Cc: Tanmay Upadhyay <tanmay.upadhyay@einfochips.com> Signed-off-by: Eric Miao <eric.y.miao@gmail.com> --- arch/arm/mach-mmp/gplugd.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c index 6915656..0469c74 100644 --- a/arch/arm/mach-mmp/gplugd.c +++ b/arch/arm/mach-mmp/gplugd.c @@ -182,7 +182,7 @@ static void __init gplugd_init(void) /* on-chip devices */ pxa168_add_uart(3); - pxa168_add_ssp(0); + pxa168_add_ssp(2); pxa168_add_twsi(0, NULL, ARRAY_AND_SIZE(gplugd_i2c_board_info)); pxa168_add_eth(&gplugd_eth_platform_data); -- 1.7.5.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH] ARM: pxa168/gplugd: add the correct SSP device 2011-11-11 12:19 [PATCH] ARM: pxa168/gplugd: add the correct SSP device Eric Miao @ 2011-11-11 16:02 ` Tanmay Upadhyay 2011-11-14 4:21 ` Eric Miao 0 siblings, 1 reply; 7+ messages in thread From: Tanmay Upadhyay @ 2011-11-11 16:02 UTC (permalink / raw) To: linux-arm-kernel On Friday 11 November 2011 05:49 PM, Eric Miao wrote: > There is no SSP0, but SSP2 is used on gplugd. > > Cc: Tanmay Upadhyay<tanmay.upadhyay@einfochips.com> > Signed-off-by: Eric Miao<eric.y.miao@gmail.com> > --- > arch/arm/mach-mmp/gplugd.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c > index 6915656..0469c74 100644 > --- a/arch/arm/mach-mmp/gplugd.c > +++ b/arch/arm/mach-mmp/gplugd.c > @@ -182,7 +182,7 @@ static void __init gplugd_init(void) > > /* on-chip devices */ > pxa168_add_uart(3); > - pxa168_add_ssp(0); > + pxa168_add_ssp(2); > pxa168_add_twsi(0, NULL, ARRAY_AND_SIZE(gplugd_i2c_board_info)); > > pxa168_add_eth(&gplugd_eth_platform_data); Hi Eric, HDMI chip (TDA9981A) on gplugD gets audio data from SSP0 - used as I2S. Page # 7 of [1] & page # 1 & 3 of [2] indicates this. SSP2 is also used on gplugD as SPI to talk to the NOR flash. Thanks, Tanmay [1] http://www.globalscaletechnologies.com/download/gplugD-MBV2R1-web.pdf [2] http://www.globalscaletechnologies.com/download/gplugD-IOV2R1-web.pdf ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: pxa168/gplugd: add the correct SSP device 2011-11-11 16:02 ` Tanmay Upadhyay @ 2011-11-14 4:21 ` Eric Miao 2011-11-14 5:05 ` Tanmay Upadhyay 0 siblings, 1 reply; 7+ messages in thread From: Eric Miao @ 2011-11-14 4:21 UTC (permalink / raw) To: linux-arm-kernel On Sat, Nov 12, 2011 at 12:02 AM, Tanmay Upadhyay <tanmay.upadhyay@einfochips.com> wrote: > > > On Friday 11 November 2011 05:49 PM, Eric Miao wrote: >> >> There is no SSP0, but SSP2 is used on gplugd. >> >> Cc: Tanmay Upadhyay<tanmay.upadhyay@einfochips.com> >> Signed-off-by: Eric Miao<eric.y.miao@gmail.com> >> --- >> ?arch/arm/mach-mmp/gplugd.c | ? ?2 +- >> ?1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c >> index 6915656..0469c74 100644 >> --- a/arch/arm/mach-mmp/gplugd.c >> +++ b/arch/arm/mach-mmp/gplugd.c >> @@ -182,7 +182,7 @@ static void __init gplugd_init(void) >> >> ? ? ? ?/* on-chip devices */ >> ? ? ? ?pxa168_add_uart(3); >> - ? ? ? pxa168_add_ssp(0); >> + ? ? ? pxa168_add_ssp(2); >> ? ? ? ?pxa168_add_twsi(0, NULL, ARRAY_AND_SIZE(gplugd_i2c_board_info)); >> >> ? ? ? ?pxa168_add_eth(&gplugd_eth_platform_data); > > Hi Eric, > > HDMI chip (TDA9981A) on gplugD gets audio data from SSP0 - used as I2S. Page > # 7 of [1] & page # 1 & 3 of [2] indicates this. Shouldn't this be SSP1 instead, if following the public PXA168 software manual (there is no SSP0, and the existing code doesn't suppport it) > SSP2 is also used on gplugD > as SPI to talk to the NOR flash. If ignoring this port at this moment, how about the patch below? ARM: pxa168/gplugd: add the correct SSP device There is no SSP0, but SSP1 is used on gplugd as an I2S port. Cc: Tanmay Upadhyay <tanmay.upadhyay@einfochips.com> Signed-off-by: Eric Miao <eric.y.miao@gmail.com> diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c index 6915656..4665767 100644 --- a/arch/arm/mach-mmp/gplugd.c +++ b/arch/arm/mach-mmp/gplugd.c @@ -182,7 +182,7 @@ static void __init gplugd_init(void) /* on-chip devices */ pxa168_add_uart(3); - pxa168_add_ssp(0); + pxa168_add_ssp(1); pxa168_add_twsi(0, NULL, ARRAY_AND_SIZE(gplugd_i2c_board_info)); pxa168_add_eth(&gplugd_eth_platform_data); ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH] ARM: pxa168/gplugd: add the correct SSP device 2011-11-14 4:21 ` Eric Miao @ 2011-11-14 5:05 ` Tanmay Upadhyay 2011-11-14 5:07 ` Eric Miao 0 siblings, 1 reply; 7+ messages in thread From: Tanmay Upadhyay @ 2011-11-14 5:05 UTC (permalink / raw) To: linux-arm-kernel On Monday 14 November 2011 09:51 AM, Eric Miao wrote: > On Sat, Nov 12, 2011 at 12:02 AM, Tanmay Upadhyay > <tanmay.upadhyay@einfochips.com> wrote: >> >> On Friday 11 November 2011 05:49 PM, Eric Miao wrote: >>> There is no SSP0, but SSP2 is used on gplugd. >>> >>> Cc: Tanmay Upadhyay<tanmay.upadhyay@einfochips.com> >>> Signed-off-by: Eric Miao<eric.y.miao@gmail.com> >>> --- >>> arch/arm/mach-mmp/gplugd.c | 2 +- >>> 1 files changed, 1 insertions(+), 1 deletions(-) >>> >>> diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c >>> index 6915656..0469c74 100644 >>> --- a/arch/arm/mach-mmp/gplugd.c >>> +++ b/arch/arm/mach-mmp/gplugd.c >>> @@ -182,7 +182,7 @@ static void __init gplugd_init(void) >>> >>> /* on-chip devices */ >>> pxa168_add_uart(3); >>> - pxa168_add_ssp(0); >>> + pxa168_add_ssp(2); >>> pxa168_add_twsi(0, NULL, ARRAY_AND_SIZE(gplugd_i2c_board_info)); >>> >>> pxa168_add_eth(&gplugd_eth_platform_data); >> Hi Eric, >> >> HDMI chip (TDA9981A) on gplugD gets audio data from SSP0 - used as I2S. Page >> # 7 of [1]& page # 1& 3 of [2] indicates this. > Shouldn't this be SSP1 instead, if following the public PXA168 software > manual (there is no SSP0, and the existing code doesn't suppport it) Ah! My bad! You are right. Current code doesn't support SSP0. It just starts from 1! Sorry I just missed it. :( >> SSP2 is also used on gplugD >> as SPI to talk to the NOR flash. > If ignoring this port at this moment, how about the patch below? It looks good. We should go with it. Thanks, Tanmay > ARM: pxa168/gplugd: add the correct SSP device > > There is no SSP0, but SSP1 is used on gplugd as an I2S port. > > Cc: Tanmay Upadhyay<tanmay.upadhyay@einfochips.com> > Signed-off-by: Eric Miao<eric.y.miao@gmail.com> > > diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c > index 6915656..4665767 100644 > --- a/arch/arm/mach-mmp/gplugd.c > +++ b/arch/arm/mach-mmp/gplugd.c > @@ -182,7 +182,7 @@ static void __init gplugd_init(void) > > /* on-chip devices */ > pxa168_add_uart(3); > - pxa168_add_ssp(0); > + pxa168_add_ssp(1); > pxa168_add_twsi(0, NULL, ARRAY_AND_SIZE(gplugd_i2c_board_info)); > > pxa168_add_eth(&gplugd_eth_platform_data); > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: pxa168/gplugd: add the correct SSP device 2011-11-14 5:05 ` Tanmay Upadhyay @ 2011-11-14 5:07 ` Eric Miao 2011-11-14 5:17 ` Tanmay Upadhyay 0 siblings, 1 reply; 7+ messages in thread From: Eric Miao @ 2011-11-14 5:07 UTC (permalink / raw) To: linux-arm-kernel On Mon, Nov 14, 2011 at 1:05 PM, Tanmay Upadhyay <tanmay.upadhyay@einfochips.com> wrote: > > > On Monday 14 November 2011 09:51 AM, Eric Miao wrote: >> >> On Sat, Nov 12, 2011 at 12:02 AM, Tanmay Upadhyay >> <tanmay.upadhyay@einfochips.com> ?wrote: >>> >>> On Friday 11 November 2011 05:49 PM, Eric Miao wrote: >>>> >>>> There is no SSP0, but SSP2 is used on gplugd. >>>> >>>> Cc: Tanmay Upadhyay<tanmay.upadhyay@einfochips.com> >>>> Signed-off-by: Eric Miao<eric.y.miao@gmail.com> >>>> --- >>>> ?arch/arm/mach-mmp/gplugd.c | ? ?2 +- >>>> ?1 files changed, 1 insertions(+), 1 deletions(-) >>>> >>>> diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c >>>> index 6915656..0469c74 100644 >>>> --- a/arch/arm/mach-mmp/gplugd.c >>>> +++ b/arch/arm/mach-mmp/gplugd.c >>>> @@ -182,7 +182,7 @@ static void __init gplugd_init(void) >>>> >>>> ? ? ? ?/* on-chip devices */ >>>> ? ? ? ?pxa168_add_uart(3); >>>> - ? ? ? pxa168_add_ssp(0); >>>> + ? ? ? pxa168_add_ssp(2); >>>> ? ? ? ?pxa168_add_twsi(0, NULL, ARRAY_AND_SIZE(gplugd_i2c_board_info)); >>>> >>>> ? ? ? ?pxa168_add_eth(&gplugd_eth_platform_data); >>> >>> Hi Eric, >>> >>> HDMI chip (TDA9981A) on gplugD gets audio data from SSP0 - used as I2S. >>> Page >>> # 7 of [1]& ?page # 1& ?3 of [2] indicates this. >> >> Shouldn't this be SSP1 instead, if following the public PXA168 software >> manual (there is no SSP0, and the existing code doesn't suppport it) > > Ah! My bad! You are right. Current code doesn't support SSP0. It just starts > from 1! Sorry I just missed it. :( > >>> SSP2 is also used on gplugD >>> as SPI to talk to the NOR flash. >> >> If ignoring this port at this moment, how about the patch below? > > It looks good. We should go with it. Can I take your Acked-by? ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: pxa168/gplugd: add the correct SSP device 2011-11-14 5:07 ` Eric Miao @ 2011-11-14 5:17 ` Tanmay Upadhyay 2011-11-14 5:25 ` Eric Miao 0 siblings, 1 reply; 7+ messages in thread From: Tanmay Upadhyay @ 2011-11-14 5:17 UTC (permalink / raw) To: linux-arm-kernel On Monday 14 November 2011 10:37 AM, Eric Miao wrote: > On Mon, Nov 14, 2011 at 1:05 PM, Tanmay Upadhyay > <tanmay.upadhyay@einfochips.com> wrote: >> >> On Monday 14 November 2011 09:51 AM, Eric Miao wrote: >>> On Sat, Nov 12, 2011 at 12:02 AM, Tanmay Upadhyay >>> <tanmay.upadhyay@einfochips.com> wrote: >>>> On Friday 11 November 2011 05:49 PM, Eric Miao wrote: >>>>> There is no SSP0, but SSP2 is used on gplugd. >>>>> >>>>> Cc: Tanmay Upadhyay<tanmay.upadhyay@einfochips.com> >>>>> Signed-off-by: Eric Miao<eric.y.miao@gmail.com> >>>>> --- >>>>> arch/arm/mach-mmp/gplugd.c | 2 +- >>>>> 1 files changed, 1 insertions(+), 1 deletions(-) >>>>> >>>>> diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c >>>>> index 6915656..0469c74 100644 >>>>> --- a/arch/arm/mach-mmp/gplugd.c >>>>> +++ b/arch/arm/mach-mmp/gplugd.c >>>>> @@ -182,7 +182,7 @@ static void __init gplugd_init(void) >>>>> >>>>> /* on-chip devices */ >>>>> pxa168_add_uart(3); >>>>> - pxa168_add_ssp(0); >>>>> + pxa168_add_ssp(2); >>>>> pxa168_add_twsi(0, NULL, ARRAY_AND_SIZE(gplugd_i2c_board_info)); >>>>> >>>>> pxa168_add_eth(&gplugd_eth_platform_data); >>>> Hi Eric, >>>> >>>> HDMI chip (TDA9981A) on gplugD gets audio data from SSP0 - used as I2S. >>>> Page >>>> # 7 of [1]& page # 1& 3 of [2] indicates this. >>> Shouldn't this be SSP1 instead, if following the public PXA168 software >>> manual (there is no SSP0, and the existing code doesn't suppport it) >> Ah! My bad! You are right. Current code doesn't support SSP0. It just starts >> from 1! Sorry I just missed it. :( >> >>>> SSP2 is also used on gplugD >>>> as SPI to talk to the NOR flash. >>> If ignoring this port at this moment, how about the patch below? >> It looks good. We should go with it. > Can I take your Acked-by? > Yes. Thanks for taking care! Could you please also provide your feedback for a pending patch for gplugD USB [1]? Thanks, Tanmay [1] http://lists.arm.linux.org.uk/lurker/message/20110718.065211.c3b28958.en.html ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: pxa168/gplugd: add the correct SSP device 2011-11-14 5:17 ` Tanmay Upadhyay @ 2011-11-14 5:25 ` Eric Miao 0 siblings, 0 replies; 7+ messages in thread From: Eric Miao @ 2011-11-14 5:25 UTC (permalink / raw) To: linux-arm-kernel On Mon, Nov 14, 2011 at 1:17 PM, Tanmay Upadhyay <tanmay.upadhyay@einfochips.com> wrote: > > > On Monday 14 November 2011 10:37 AM, Eric Miao wrote: >> >> On Mon, Nov 14, 2011 at 1:05 PM, Tanmay Upadhyay >> <tanmay.upadhyay@einfochips.com> ?wrote: >>> >>> On Monday 14 November 2011 09:51 AM, Eric Miao wrote: >>>> >>>> On Sat, Nov 12, 2011 at 12:02 AM, Tanmay Upadhyay >>>> <tanmay.upadhyay@einfochips.com> ? ?wrote: >>>>> >>>>> On Friday 11 November 2011 05:49 PM, Eric Miao wrote: >>>>>> >>>>>> There is no SSP0, but SSP2 is used on gplugd. >>>>>> >>>>>> Cc: Tanmay Upadhyay<tanmay.upadhyay@einfochips.com> >>>>>> Signed-off-by: Eric Miao<eric.y.miao@gmail.com> >>>>>> --- >>>>>> ?arch/arm/mach-mmp/gplugd.c | ? ?2 +- >>>>>> ?1 files changed, 1 insertions(+), 1 deletions(-) >>>>>> >>>>>> diff --git a/arch/arm/mach-mmp/gplugd.c b/arch/arm/mach-mmp/gplugd.c >>>>>> index 6915656..0469c74 100644 >>>>>> --- a/arch/arm/mach-mmp/gplugd.c >>>>>> +++ b/arch/arm/mach-mmp/gplugd.c >>>>>> @@ -182,7 +182,7 @@ static void __init gplugd_init(void) >>>>>> >>>>>> ? ? ? ?/* on-chip devices */ >>>>>> ? ? ? ?pxa168_add_uart(3); >>>>>> - ? ? ? pxa168_add_ssp(0); >>>>>> + ? ? ? pxa168_add_ssp(2); >>>>>> ? ? ? ?pxa168_add_twsi(0, NULL, >>>>>> ARRAY_AND_SIZE(gplugd_i2c_board_info)); >>>>>> >>>>>> ? ? ? ?pxa168_add_eth(&gplugd_eth_platform_data); >>>>> >>>>> Hi Eric, >>>>> >>>>> HDMI chip (TDA9981A) on gplugD gets audio data from SSP0 - used as I2S. >>>>> Page >>>>> # 7 of [1]& ? ?page # 1& ? ?3 of [2] indicates this. >>>> >>>> Shouldn't this be SSP1 instead, if following the public PXA168 software >>>> manual (there is no SSP0, and the existing code doesn't suppport it) >>> >>> Ah! My bad! You are right. Current code doesn't support SSP0. It just >>> starts >>> from 1! Sorry I just missed it. :( >>> >>>>> SSP2 is also used on gplugD >>>>> as SPI to talk to the NOR flash. >>>> >>>> If ignoring this port at this moment, how about the patch below? >>> >>> It looks good. We should go with it. >> >> Can I take your Acked-by? >> > > Yes. Thanks for taking care! Could you please also provide your feedback for > a pending patch for gplugD USB [1]? Thanks. I will. > > Thanks, > > Tanmay > > [1] > http://lists.arm.linux.org.uk/lurker/message/20110718.065211.c3b28958.en.html > ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-11-14 5:25 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-11-11 12:19 [PATCH] ARM: pxa168/gplugd: add the correct SSP device Eric Miao 2011-11-11 16:02 ` Tanmay Upadhyay 2011-11-14 4:21 ` Eric Miao 2011-11-14 5:05 ` Tanmay Upadhyay 2011-11-14 5:07 ` Eric Miao 2011-11-14 5:17 ` Tanmay Upadhyay 2011-11-14 5:25 ` Eric Miao
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox