diff options
| author | Pavan Savoy <pavan_savoy@ti.com> | 2010-10-12 16:27:38 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-10-14 15:41:10 -0400 |
| commit | 73f12e8d3d94828b9efe2b8b8a34b4ad6d14ee47 (patch) | |
| tree | 42b4dad70d74feccd5c6c802eef6837606096ffe /include/linux | |
| parent | da6830ac3969c9480e05ed2b21eee64e21a101d2 (diff) | |
drivers: misc: ti-st: fix review comments
Based on comments from Jiri Slaby, drop the register
storage specifier, remove the unused code, cleanup
the const to non-const type casting.
Also make the line discipline ops structure static, since
its a singleton, unmodified structure which need not be
in heap.
Reported-by: Jiri Slaby <jirislaby@gmail.com>
Signed-off-by: Pavan Savoy <pavan_savoy@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/ti_wilink_st.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/include/linux/ti_wilink_st.h b/include/linux/ti_wilink_st.h index 2a5acf599598..4c7be2263011 100644 --- a/include/linux/ti_wilink_st.h +++ b/include/linux/ti_wilink_st.h | |||
| @@ -101,7 +101,6 @@ extern long st_unregister(enum proto_type); | |||
| 101 | * can occur , where as during other times other events CH8, CH9 | 101 | * can occur , where as during other times other events CH8, CH9 |
| 102 | * can occur. | 102 | * can occur. |
| 103 | * @tty: tty provided by the TTY core for line disciplines. | 103 | * @tty: tty provided by the TTY core for line disciplines. |
| 104 | * @ldisc_ops: the procedures that this line discipline registers with TTY. | ||
| 105 | * @tx_skb: If for some reason the tty's write returns lesser bytes written | 104 | * @tx_skb: If for some reason the tty's write returns lesser bytes written |
| 106 | * then to maintain the rest of data to be written on next instance. | 105 | * then to maintain the rest of data to be written on next instance. |
| 107 | * This needs to be protected, hence the lock inside wakeup func. | 106 | * This needs to be protected, hence the lock inside wakeup func. |
| @@ -132,7 +131,6 @@ extern long st_unregister(enum proto_type); | |||
| 132 | struct st_data_s { | 131 | struct st_data_s { |
| 133 | unsigned long st_state; | 132 | unsigned long st_state; |
| 134 | struct tty_struct *tty; | 133 | struct tty_struct *tty; |
| 135 | struct tty_ldisc_ops *ldisc_ops; | ||
| 136 | struct sk_buff *tx_skb; | 134 | struct sk_buff *tx_skb; |
| 137 | #define ST_TX_SENDING 1 | 135 | #define ST_TX_SENDING 1 |
| 138 | #define ST_TX_WAKEUP 2 | 136 | #define ST_TX_WAKEUP 2 |
