[Acked/cmt] [SRU][RFC][Utopic] Pull Request: powerpc: backport AST graphic driver updates for OpenPower

[Joseph Salisbury]

On 03/18/2015 05:31 AM, Andy Whitcroft wrote:
> On Mon, Mar 16, 2015 at 12:56:35PM -0400, Joseph Salisbury wrote:
>> On 03/05/2015 02:08 PM, Joseph Salisbury wrote:
>>> A test build was performed for powerpc without any failures.  However,
>>> testing on actual hardware was not performed yet, but testing was requested in the bug.  
>>>
>>> A test kernel was posted for testing here:
>>> http://kernel.ubuntu.com/~jsalisbury/lp1420627/
>>>
>>> Requesting review and feedback on these patches.
>>>
>>>
>>> The following changes since commit e093858a13f30d8b981b5e9a750bf3b1ff60d819:
>>>
>>>   UBUNTU: Ubuntu-3.16.0-32.42 (2015-03-03 13:01:54 -0600)
>>>
>>> are available in the git repository at:
>>>
>>>   zinc.canonical.com:/srv/kernel.ubuntu.com/git/jsalisbury/bugs/lp1420627/ubuntu-utopic.git 
>>>
>>> for you to fetch changes up to 13a15d170a08023f9f16ed6ab3086b08e59f9b30:
>>>
>>>   [PATCH 8/8] drm/ast: Cleanup analog init code path (2015-03-05 13:04:49 -0500)
>>>
>>> ----------------------------------------------------------------
>>> Benjamin Herrenschmidt (5):
>>>       drm/ast: Try to use MMIO registers when PIO isn't supported
>>>       drm/ast: POST chip at probe time if VGA not enabled
>>>       drm/ast: Properly initialize P2A base before using it in ast_init_3rdtx()
>>>       drm/ast: Don't assume DVO enabled means SIL164 on uninitialized chips
>>>       drm/ast: Cleanup analog init code path
>>>
>>> Dave Airlie (1):
>>>       drm/ast: switch to using CACHED by default for sysram
>>>
>>> Y.C. Chen (2):
>>>       drm/ast: Add missing entry to dclk_table[]
>>>       drm/ast: Add reduced blanking modes for wide screen mode
>>>
>>>  drivers/gpu/drm/ast/ast_dp501.c  | 38 +++++++++++++++----
>>>  drivers/gpu/drm/ast/ast_drv.h    |  8 +++-
>>>  drivers/gpu/drm/ast/ast_main.c   | 79 ++++++++++++++++++++++++++++++++++++----
>>>  drivers/gpu/drm/ast/ast_mode.c   | 42 ++++++++++++++++-----
>>>  drivers/gpu/drm/ast/ast_post.c   | 23 +++++++-----
>>>  drivers/gpu/drm/ast/ast_tables.h | 39 ++++++++++++--------
>>>  drivers/gpu/drm/ast/ast_ttm.c    |  4 +-
>>>  7 files changed, 182 insertions(+), 51 deletions(-)
>>>
>> We now have some testing results with this patch set:
>> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1420627/comments/9
> Ugg, this is a pretty large patch set.  That said reading the patches
> they are pretty small individually, and all self contained to this one
> driver and it is that driver that they aim to fix.  It is easily
> testable with h/w and has indeed been so.
>
> I will note that all the patches have been committed with "[PATCH N/M]"
> on them which should be cleaned off, and one of them is meant to have
> needed minor porting so it would be good to figure out which one and
> make sure its marked as backported from.  You should be albe to repeat
> the cherry-picks till they fail.
Thanks for reviewing, Andy.  I removed "[PATCH N/M]" from all the commit
messages.  I also identified this patch as the one backported and
changed the "cherry-picked from" to "Backported from":
drm/ast: switch to using CACHED by default for sysram


> Overall:
>
> Acked-by: Andy Whitcroft <apw at canonical.com>
>
> -apw