Bug#808830: QNAP LCD support in installer

[Martin Michlmayr]

* Ian Campbell  [2015-12-23 14:35]:
> Package: flash-kernel-installer

Note that the patch is against network-console.

> Thanks, looks good on first glance, stashing in the BTS for safekeeping
> until I get around to it.

Thanks!

-- 
Martin Michlmayr
http://www.cyrius.com/

Bug#808830: QNAP LCD support in installer

[Ian Campbell]

Package: flash-kernel-installer
Tags: patch
Severity: wishlist
Control: submitter Martin Michlmayr 

On Tue, 2015-12-22 at 11:23 -0800, Martin Michlmayr wrote:
> d-i doesn't show the IP address of the installer on the LCD at the
> moment.  I believe the attached patch should work.  Maybe you can test
> it on your TS-419 some time (maybe once with the serial attached, and
> once without, so you can test both scenarios).

Thanks, looks good on first glance, stashing in the BTS for safekeeping
until I get around to it.

Ian.
diff --git a/debian/network-console.postinst b/debian/network-console.postinst
index f164446..9646c56 100755
--- a/debian/network-console.postinst
+++ b/debian/network-console.postinst
@@ -123,6 +123,10 @@ case "$ARCHDETECT" in
 	if type qcommand >/dev/null 2>&1; then
 		qcommand statusled greenon
 		qcommand buzzer short
+		# Only some devices have an LCD, so don't fail if the
+		# commands don't work
+		qcommand lcd-line0 "SSH to installer" || true
+		qcommand lcd-line1 "@$IPADDR" || true
 	fi
 	# LaCie Network Space v2 (and parents) and d2 Network v2
 	for led_dev in ns_v2 d2net_v2; do

Bug#808830: QNAP LCD support in installer

[Ian Campbell]

control: reassign -1 network-console
control: owner -1 !


On Wed, 2015-12-23 at 08:22 -0800, Martin Michlmayr wrote:
> * Ian Campbell  [2015-12-23 14:35]:
> > Package: flash-kernel-installer
> 
> Note that the patch is against network-console.

Gah, of course. And I even meant qcontrol not flash-kernel, which is
still wrong but at least closer!

Ian.