From 6cd2432e309d999258d7bb152f5e27ddb2739f5a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Robert=20J=C3=B6rdens?= <rj@m-labs.hk>
Date: Tue, 7 Aug 2018 16:13:21 +0200
Subject: [PATCH] grabber: log all resolution changes

close #1120
---
 artiq/firmware/libboard_artiq/grabber.rs | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/artiq/firmware/libboard_artiq/grabber.rs b/artiq/firmware/libboard_artiq/grabber.rs
index fd9a0bf00..3e6e89a83 100644
--- a/artiq/firmware/libboard_artiq/grabber.rs
+++ b/artiq/firmware/libboard_artiq/grabber.rs
@@ -8,6 +8,7 @@ enum State {
 }
 
 static mut GRABBER_STATE: [State; csr::GRABBER_LEN] = [State::Down; csr::GRABBER_LEN];
+static mut GRABBER_RESOLUTION: [(u16, u16); csr::GRABBER_LEN] = [(0, 0); csr::GRABBER_LEN];
 
 fn get_pll_reset(g: usize) -> bool {
     unsafe { (csr::GRABBER[g].pll_reset_read)() != 0 }
@@ -92,11 +93,19 @@ pub fn tick() {
                 info!("grabber{} is down", g);
             }
             if unsafe { GRABBER_STATE[g] == State::WaitResolution } {
-                let (last_x, last_y) = get_last_pixels(g);
+                let last_xy = get_last_pixels(g);
+                unsafe { GRABBER_RESOLUTION[g] = last_xy; }
                 info!("grabber{} frame size: {}x{}",
-                    g, last_x + 1, last_y + 1);
+                    g, last_xy.0 + 1, last_xy.1 + 1);
                 info!("grabber{} video clock: {}MHz", g, get_video_clock(g));
                 unsafe { GRABBER_STATE[g] = State::Up; }
+            } else {
+                let last_xy = get_last_pixels(g);
+                if unsafe { last_xy != GRABBER_RESOLUTION[g] } {
+                    info!("grabber{} frame size: {}x{}",
+                        g, last_xy.0 + 1, last_xy.1 + 1);
+                    unsafe { GRABBER_RESOLUTION[g] = last_xy; }
+                }
             }
         } else {
             if get_pll_reset(g) {