Skip to contentSkip to conversation

usb: serial: upd78f0730: Make constants static

PATCH In review Maksim Salau 2 participants 2 messages Apr 24, 2017 – Jun 1, 2017

Subsystems (2)
USB USB serial
lore
Share & tools
lore

Maksim Salau

author
#0 lore raw

PATCH 1/1 usb: serial: upd78f0730: Make constants static

Some local constants don't change from call to call and are good candidates to become static. This will prevent copying of these constants to stack during runtime.

Signed-off-by Maksim Salau <[email protected]>
---
 drivers/usb/serial/upd78f0730.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
drivers/usb/serial/upd78f0730.c +3 -3
index a028dd2..6819a34 100644
@@ -288,7 +288,7 @@ static void upd78f0730_dtr_rts(struct usb_serial_port *port, int on)
 static speed_t upd78f0730_get_baud_rate(struct tty_struct *tty)
 {
 	const speed_t baud_rate = tty_get_baud_rate(tty);
-	const speed_t supported[] = {
+	static const speed_t supported[] = {
 		0, 2400, 4800, 9600, 19200, 38400, 57600, 115200, 153600
 	};
 	int i;

@@ -384,7 +384,7 @@ static void upd78f0730_set_termios(struct tty_struct *tty,
 
 static int upd78f0730_open(struct tty_struct *tty, struct usb_serial_port *port)
 {
-	struct upd78f0730_open_close request = {
+	static const struct upd78f0730_open_close request = {
 		.opcode = UPD78F0730_CMD_OPEN_CLOSE,
 		.state = UPD78F0730_PORT_OPEN
 	};

@@ -402,7 +402,7 @@ static int upd78f0730_open(struct tty_struct *tty, struct usb_serial_port *port)
 
 static void upd78f0730_close(struct usb_serial_port *port)
 {
-	struct upd78f0730_open_close request = {
+	static const struct upd78f0730_open_close request = {
 		.opcode = UPD78F0730_CMD_OPEN_CLOSE,
 		.state = UPD78F0730_PORT_CLOSE
 	};
Signature · 1 line
2.9.3
maintainer
#1 lore raw
Quoted 5 lines · Maksim Salau

Some local constants don't change from call to call and are good candidates to become static. This will prevent copying of these constants to stack during runtime.

Signed-off-by Maksim Salau <[email protected]>

Now applied, thanks.

Johan