Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-10-06 Thread Sekhar Nori
On Wednesday 05 October 2016 01:52 PM, Peter Ujfalusi wrote: > On 10/04/16 16:02, Kevin Hilman wrote: >> Peter Ujfalusi writes: >> >>> On 10/01/16 12:24, Sekhar Nori wrote: >> >> [...] >> In any case, to configure the PBBR, you will have to introduce a driver for it in drivers/memory. Th

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-10-05 Thread Peter Ujfalusi
On 10/04/16 16:02, Kevin Hilman wrote: > Peter Ujfalusi writes: > >> On 10/01/16 12:24, Sekhar Nori wrote: > > [...] > >>> In any case, to configure the PBBR, you will have to introduce a driver >>> for it in drivers/memory. Then you can set it up per board using a DT >>> parameter. >> >> and w

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-10-05 Thread Peter Ujfalusi
On 10/04/16 12:20, Bartosz Golaszewski wrote: > 2016-09-30 21:19 GMT+02:00 Peter Ujfalusi : >> On 09/30/2016 06:06 PM, Bartosz Golaszewski wrote: >>> >>> Just ran a quick test with speaker-test -c2 -twav. Besides the fact >>> that the left and right channels are inverted (I'm looking into that), >>

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-10-04 Thread Kevin Hilman
Peter Ujfalusi writes: > On 10/01/16 12:24, Sekhar Nori wrote: [...] >> In any case, to configure the PBBR, you will have to introduce a driver >> for it in drivers/memory. Then you can set it up per board using a DT >> parameter. > > and we can reuse the introduced bindings for am335x and OMAP

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-10-04 Thread Bartosz Golaszewski
2016-09-30 21:19 GMT+02:00 Peter Ujfalusi : > On 09/30/2016 06:06 PM, Bartosz Golaszewski wrote: >> >> Just ran a quick test with speaker-test -c2 -twav. Besides the fact >> that the left and right channels are inverted (I'm looking into that), >> I didn't notice any problems. Even at 1024x768 reso

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-10-03 Thread Peter Ujfalusi
On 10/01/16 12:24, Sekhar Nori wrote: >> That's good to hear, but I think the priorities should be set: >> LCDC and EDMA30TC1 to highest priority >> EDMA30TC0 to priority 2 >> >> The 0TC0 is used by MMC and if you want to play a video you might need the >> servicing TC to be higher priority then ot

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-10-01 Thread Sekhar Nori
On Saturday 01 October 2016 12:49 AM, Peter Ujfalusi wrote: > On 09/30/2016 06:06 PM, Bartosz Golaszewski wrote: >> 2016-09-30 14:59 GMT+02:00 Peter Ujfalusi : >>> On 09/29/16 19:31, Bartosz Golaszewski wrote: Default memory settings of da850 do not meet the throughput/latency requirement

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-09-30 Thread Peter Ujfalusi
On 09/30/2016 06:06 PM, Bartosz Golaszewski wrote: > 2016-09-30 14:59 GMT+02:00 Peter Ujfalusi : >> On 09/29/16 19:31, Bartosz Golaszewski wrote: >>> Default memory settings of da850 do not meet the throughput/latency >>> requirements of tilcdc. This results in the image displayed being >>> incorre

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-09-30 Thread Bartosz Golaszewski
2016-09-30 14:59 GMT+02:00 Peter Ujfalusi : > On 09/29/16 19:31, Bartosz Golaszewski wrote: >> Default memory settings of da850 do not meet the throughput/latency >> requirements of tilcdc. This results in the image displayed being >> incorrect and the following warning being displayed by the LCDC

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-09-30 Thread Peter Ujfalusi
On 09/29/16 19:31, Bartosz Golaszewski wrote: > Default memory settings of da850 do not meet the throughput/latency > requirements of tilcdc. This results in the image displayed being > incorrect and the following warning being displayed by the LCDC > drm driver: > > tilcdc da8xx_lcdc.0: tilcdc_

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-09-30 Thread Karl Beldan
On Fri, Sep 30, 2016 at 9:31 AM, Bartosz Golaszewski wrote: > 2016-09-29 21:07 GMT+02:00 Karl Beldan : >> Hi, >> >> On Thu, Sep 29, 2016 at 06:31:55PM +0200, Bartosz Golaszewski wrote: >>> Default memory settings of da850 do not meet the throughput/latency >>> requirements of tilcdc. This results

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-09-30 Thread Bartosz Golaszewski
2016-09-29 21:07 GMT+02:00 Karl Beldan : > Hi, > > On Thu, Sep 29, 2016 at 06:31:55PM +0200, Bartosz Golaszewski wrote: >> Default memory settings of da850 do not meet the throughput/latency >> requirements of tilcdc. This results in the image displayed being >> incorrect and the following warning

Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc

2016-09-29 Thread Karl Beldan
Hi, On Thu, Sep 29, 2016 at 06:31:55PM +0200, Bartosz Golaszewski wrote: > Default memory settings of da850 do not meet the throughput/latency > requirements of tilcdc. This results in the image displayed being > incorrect and the following warning being displayed by the LCDC > drm driver: > >