From cc5c81050119cda6caedf68c2758ea583f72fa6e Mon Sep 17 00:00:00 2001 From: Paulus Schoutsen Date: Sat, 28 Feb 2026 02:47:08 -0500 Subject: [PATCH] Remove unnecessary volume_up/volume_down overrides from NADtcp media player (#164434) Co-authored-by: Claude --- homeassistant/components/nad/media_player.py | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/homeassistant/components/nad/media_player.py b/homeassistant/components/nad/media_player.py index c1efa18f72b..2af8c607610 100644 --- a/homeassistant/components/nad/media_player.py +++ b/homeassistant/components/nad/media_player.py @@ -198,8 +198,10 @@ class NADtcp(MediaPlayerEntity): self._nad_receiver = NADReceiverTCP(config.get(CONF_HOST)) self._min_vol = (config[CONF_MIN_VOLUME] + 90) * 2 # from dB to nad vol (0-200) self._max_vol = (config[CONF_MAX_VOLUME] + 90) * 2 # from dB to nad vol (0-200) - self._volume_step = config[CONF_VOLUME_STEP] self._nad_volume = None + vol_range = self._max_vol - self._min_vol + if vol_range: + self._attr_volume_step = 2 * config[CONF_VOLUME_STEP] / vol_range self._source_list = self._nad_receiver.available_sources() def turn_off(self) -> None: @@ -210,14 +212,6 @@ class NADtcp(MediaPlayerEntity): """Turn the media player on.""" self._nad_receiver.power_on() - def volume_up(self) -> None: - """Step volume up in the configured increments.""" - self._nad_receiver.set_volume(self._nad_volume + 2 * self._volume_step) - - def volume_down(self) -> None: - """Step volume down in the configured increments.""" - self._nad_receiver.set_volume(self._nad_volume - 2 * self._volume_step) - def set_volume_level(self, volume: float) -> None: """Set volume level, range 0..1.""" nad_volume_to_set = int(