diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2018-04-03 12:14:54 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2018-04-03 12:14:54 -0700 |
commit | dabe51840e2790aa7e66c53990d15c663b0d628a (patch) | |
tree | 064f250b09f56feecd567e8c4c9a575b8790900c | |
parent | 3ac684b88148bf132fbf02a133cba9dcfb2e7aa2 (diff) | |
parent | e3d7fbab062b877b96c09ffb43733502cdc1bd0a (diff) | |
download | linux-dabe51840e2790aa7e66c53990d15c663b0d628a.tar.bz2 |
Merge tag 'hsi-for-4.17' of git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-hsi
Pull HSI updates from Sebastian Reichel:
- spelling/typo fixes
- remove extra error printing for -ENOMEM
* tag 'hsi-for-4.17' of git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-hsi:
HSI: hsi_char: Delete an error message for a failed memory allocation in hsc_probe()
HSI: ssi_protocol: fix spelling mistake: "trigerred" -> "triggered"
HSI: ssi_protocol: Delete an error message for a failed memory allocation in ssi_protocol_probe()
HSI: ssi_protocol: Fix a typo in two comment lines
-rw-r--r-- | drivers/hsi/clients/hsi_char.c | 5 | ||||
-rw-r--r-- | drivers/hsi/clients/ssi_protocol.c | 12 |
2 files changed, 7 insertions, 10 deletions
diff --git a/drivers/hsi/clients/hsi_char.c b/drivers/hsi/clients/hsi_char.c index cf9c2a332ad8..9d2a6d64dfb8 100644 --- a/drivers/hsi/clients/hsi_char.c +++ b/drivers/hsi/clients/hsi_char.c @@ -697,10 +697,9 @@ static int hsc_probe(struct device *dev) int i; cl_data = kzalloc(sizeof(*cl_data), GFP_KERNEL); - if (!cl_data) { - dev_err(dev, "Could not allocate hsc_client_data\n"); + if (!cl_data) return -ENOMEM; - } + hsc_baseminor = HSC_BASEMINOR(hsi_id(cl), hsi_port_id(cl)); if (!hsc_major) { ret = alloc_chrdev_region(&hsc_dev, hsc_baseminor, diff --git a/drivers/hsi/clients/ssi_protocol.c b/drivers/hsi/clients/ssi_protocol.c index 9b167bc6eee4..561abf7bdf1f 100644 --- a/drivers/hsi/clients/ssi_protocol.c +++ b/drivers/hsi/clients/ssi_protocol.c @@ -495,7 +495,7 @@ static void ssip_rx_wd(struct timer_list *t) struct ssi_protocol *ssi = from_timer(ssi, t, rx_wd); struct hsi_client *cl = ssi->cl; - dev_err(&cl->device, "Watchdog trigerred\n"); + dev_err(&cl->device, "Watchdog triggered\n"); ssip_error(cl); } @@ -504,7 +504,7 @@ static void ssip_tx_wd(struct timer_list *t) struct ssi_protocol *ssi = from_timer(ssi, t, tx_wd); struct hsi_client *cl = ssi->cl; - dev_err(&cl->device, "Watchdog trigerred\n"); + dev_err(&cl->device, "Watchdog triggered\n"); ssip_error(cl); } @@ -993,8 +993,8 @@ static int ssip_pn_xmit(struct sk_buff *skb, struct net_device *dev) goto inc_dropped; /* - * Modem sends Phonet messages over SSI with its own endianess... - * Assume that modem has the same endianess as we do. + * Modem sends Phonet messages over SSI with its own endianness. + * Assume that modem has the same endianness as we do. */ if (skb_cow_head(skb, 0)) goto drop; @@ -1088,10 +1088,8 @@ static int ssi_protocol_probe(struct device *dev) int err; ssi = kzalloc(sizeof(*ssi), GFP_KERNEL); - if (!ssi) { - dev_err(dev, "No memory for ssi protocol\n"); + if (!ssi) return -ENOMEM; - } spin_lock_init(&ssi->lock); timer_setup(&ssi->rx_wd, ssip_rx_wd, TIMER_DEFERRABLE); |