Skip to content

Commit 1672f76

Browse files
committed
Fix printf argument and formatting
1 parent f95062f commit 1672f76

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

src/app/clusters/level-control/level-control.cpp

+1-2
Original file line numberDiff line numberDiff line change
@@ -311,8 +311,7 @@ static void setOnOffValue(EndpointId endpoint, bool onOff)
311311
if (emberAfContainsServer(endpoint, OnOff::Id))
312312
{
313313
emberAfLevelControlClusterPrintln("Setting on/off to %p due to level change", onOff ? "ON" : "OFF");
314-
OnOffServer::Instance().setOnOffValue(
315-
endpoint, (onOff ? OnOff::Commands::On::Id : OnOff::Commands::Off::Id), true);
314+
OnOffServer::Instance().setOnOffValue(endpoint, (onOff ? OnOff::Commands::On::Id : OnOff::Commands::Off::Id), true);
316315
}
317316
#endif // EMBER_AF_PLUGIN_ON_OFF
318317
}

src/app/clusters/on-off-server/on-off-server.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ EmberAfStatus OnOffServer::setOnOffValue(chip::EndpointId endpoint, chip::Comman
114114
EmberAfStatus status;
115115
bool currentValue, newValue;
116116

117-
emberAfOnOffClusterPrintln("On/Off set value: %x %lx", endpoint, command);
117+
emberAfOnOffClusterPrintln("On/Off set value: %x %x", endpoint, static_cast<uint8_t>(command));
118118

119119
// read current on/off value
120120
status = Attributes::OnOff::Get(endpoint, &currentValue);

0 commit comments

Comments
 (0)