Merge tag 'hsi-for-4.17' of git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-hsi
authorLinus Torvalds <torvalds@linux-foundation.org>
Tue, 3 Apr 2018 19:14:54 +0000 (12:14 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 3 Apr 2018 19:14:54 +0000 (12:14 -0700)
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

drivers/hsi/clients/hsi_char.c
drivers/hsi/clients/ssi_protocol.c

index cf9c2a332ad8c1c6e5721733783172549d2e55f6..9d2a6d64dfb80c0dedbd472a1df9d4f38d540539 100644 (file)
@@ -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,
index 9b167bc6eee48ca498a070ca46cc09836f02d8c1..561abf7bdf1f5d36688e58bc6489f69e987eeda4 100644 (file)
@@ -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);