implementing users remarks

homeassistant/components/hdmi_cec.py

@@ -22,16 +22,18 @@
                                  EVENT_HOMEASSISTANT_STOP, STATE_ON,
                                  STATE_OFF, CONF_DEVICES, CONF_PLATFORM,
                                  CONF_CUSTOMIZE, STATE_PLAYING, STATE_IDLE,
-                                 STATE_PAUSED, CONF_HOST)
+                                 STATE_PAUSED, CONF_HOST, CONF_DISPLAY_NAME)
 from homeassistant.core import HomeAssistant, callback
 from homeassistant.helpers.entity import Entity
 
-REQUIREMENTS = ['pyCEC==0.4.6']
+REQUIREMENTS = ['pyCEC==0.4.9']
 
 DOMAIN = 'hdmi_cec'
 
 _LOGGER = logging.getLogger(__name__)
 
+DEFAULT_DISPLAY_NAME = "HomeAssistant"
+
 ICON_UNKNOWN = 'mdi:help'
 ICON_AUDIO = 'mdi:speaker'
 ICON_PLAYER = 'mdi:play'
@@ -76,6 +78,9 @@
 ATTR_DIR = 'dir'
 ATTR_ABT = 'abt'
 ATTR_NEW = 'new'
+ATTR_ON = 'on'
+ATTR_OFF = 'off'
+ATTR_TOGGLE = 'toggle'
 
 _VOL_HEX = vol.Any(vol.Coerce(int), lambda x: int(x, 16))
 
@@ -92,9 +97,7 @@
 SERVICE_VOLUME_SCHEMA = vol.Schema({
     vol.Optional(CMD_UP): vol.Any(CMD_PRESS, CMD_RELEASE, vol.Coerce(int)),
     vol.Optional(CMD_DOWN): vol.Any(CMD_PRESS, CMD_RELEASE, vol.Coerce(int)),
-    vol.Optional(CMD_MUTE): None,
-    vol.Optional(CMD_UNMUTE): None,
-    vol.Optional(CMD_MUTE_TOGGLE): None
+    vol.Optional(CMD_MUTE): vol.Any(ATTR_ON, ATTR_OFF, ATTR_TOGGLE),
 }, extra=vol.PREVENT_EXTRA)
 
 SERVICE_UPDATE_DEVICES = 'update'
@@ -127,6 +130,7 @@
                                             })),
         vol.Optional(CONF_PLATFORM): vol.Any(SWITCH, MEDIA_PLAYER),
         vol.Optional(CONF_HOST): cv.string,
+        vol.Optional(CONF_DISPLAY_NAME): cv.string,
     })
 }, extra=vol.ALLOW_EXTRA)
 
@@ -156,8 +160,9 @@ def setup(hass: HomeAssistant, base_config):
     """Setup CEC capability."""
     from pycec.network import HDMINetwork
     from pycec.commands import CecCommand, KeyReleaseCommand, KeyPressCommand
-    from pycec.const import KEY_VOLUME_UP, KEY_VOLUME_DOWN, KEY_MUTE, \
-        ADDR_AUDIOSYSTEM, ADDR_BROADCAST, ADDR_UNREGISTERED
+    from pycec.const import KEY_VOLUME_UP, KEY_VOLUME_DOWN, KEY_MUTE_ON, \
+        KEY_MUTE_OFF, KEY_MUTE_TOGGLE, ADDR_AUDIOSYSTEM, ADDR_BROADCAST, \
+        ADDR_UNREGISTERED
     from pycec.cec import CecAdapter
     from pycec.tcp import TcpAdapter
 
@@ -175,22 +180,27 @@ def setup(hass: HomeAssistant, base_config):
         # Create own thread if more than 1 CPU
         hass.loop if multiprocessing.cpu_count() < 2 else None)
     host = base_config[DOMAIN].get(CONF_HOST, None)
+    display_name = base_config[DOMAIN].get(CONF_DISPLAY_NAME,
+                                           DEFAULT_DISPLAY_NAME)
     if host:
-        adapter = TcpAdapter(host, name="HASS", activate_source=False)
+        adapter = TcpAdapter(host, name=display_name, activate_source=False)
     else:
-        adapter = CecAdapter(name="HASS", activate_source=False)
+        adapter = CecAdapter(name=display_name, activate_source=False)
     hdmi_network = HDMINetwork(adapter, loop=loop)
 
     def _volume(call):
         """Increase/decrease volume and mute/unmute system."""
+        mute_key_mapping = {ATTR_TOGGLE: KEY_MUTE_TOGGLE, ATTR_ON: KEY_MUTE_ON,
+                            ATTR_OFF: KEY_MUTE_OFF}
         for cmd, att in call.data.items():
             if cmd == CMD_UP:
                 _process_volume(KEY_VOLUME_UP, att)
             elif cmd == CMD_DOWN:
                 _process_volume(KEY_VOLUME_DOWN, att)
             elif cmd == CMD_MUTE:
                 hdmi_network.send_command(
-                    KeyPressCommand(KEY_MUTE, dst=ADDR_AUDIOSYSTEM))
+                    KeyPressCommand(mute_key_mapping[att],
+                                    dst=ADDR_AUDIOSYSTEM))
                 hdmi_network.send_command(
                     KeyReleaseCommand(dst=ADDR_AUDIOSYSTEM))
                 _LOGGER.info("Audio muted")
@@ -207,7 +217,7 @@ def _process_volume(cmd, att):
             hdmi_network.send_command(KeyReleaseCommand(dst=ADDR_AUDIOSYSTEM))
         else:
             att = 1 if att == "" else int(att)
-            for _ in range(1, att):
+            for _ in range(0, att):
                 hdmi_network.send_command(
                     KeyPressCommand(cmd, dst=ADDR_AUDIOSYSTEM))
                 hdmi_network.send_command(

homeassistant/components/media_player/hdmi_cec.py

@@ -59,8 +59,8 @@ def send_playback(self, key):
 
     def mute_volume(self, mute):
         """Mute volume."""
-        from pycec.const import KEY_MUTE
-        self.send_keypress(KEY_MUTE)
+        from pycec.const import KEY_MUTE_TOGGLE
+        self.send_keypress(KEY_MUTE_TOGGLE)
 
     def media_previous_track(self):
         """Go to previous track."""

homeassistant/components/services.yaml

@@ -192,9 +192,9 @@ hdmi_cec:
       down:
         description: Decreases volume x levels.
         example: 3
-      mute: Mutes audio system. Value is ignored.
-      unmute: Unmutes audio system. Value is ignored.
-      toggle mute: Toggles mute of audio system. Value is ignored.
+      mute:
+        description: Mutes audio system. Value should be on, off or toggle.
+        example: "toggle"
 
   select_device:
     description: Select HDMI device.

homeassistant/const.py

@@ -95,6 +95,7 @@
 CONF_FRIENDLY_NAME = 'friendly_name'
 CONF_HEADERS = 'headers'
 CONF_HOST = 'host'
+CONF_DISPLAY_NAME = 'osd_name'
 CONF_HOSTS = 'hosts'
 CONF_ICON = 'icon'
 CONF_INCLUDE = 'include'

requirements_all.txt

@@ -378,7 +378,7 @@ pwaqi==1.3
 py-cpuinfo==0.2.3
 
 # homeassistant.components.hdmi_cec
-pyCEC==0.4.6
+pyCEC==0.4.9
 
 # homeassistant.components.switch.tplink
 pyHS100==0.2.3