Hello!

On 10.08.2020 3:59, YourName wrote:

From: Daniel <[email protected]>

   Full name needed here.

---
  drivers/ata/acard-ahci.c  |  6 +++---
  drivers/bluetooth/bfusb.c |  5 ++---
  drivers/clk/clk-asm9260.c | 12 ++++++------
  3 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/drivers/ata/acard-ahci.c b/drivers/ata/acard-ahci.c
index 2a04e8abd397..3ffb21f3e88b 100644
--- a/drivers/ata/acard-ahci.c
+++ b/drivers/ata/acard-ahci.c
@@ -79,10 +79,10 @@ static struct ata_port_operations acard_ops = {
#define AHCI_HFLAGS(flags) .private_data = (void *)(flags) -static const struct ata_port_info acard_ahci_port_info[] = {
+static const struct ata_port_info acard_ahci_port_info[] ={
        [board_acard_ahci] =
-       {
-               AHCI_HFLAGS     (AHCI_HFLAG_NO_NCQ),
+       
+        {       AHCI_HFLAGS    (AHCI_HFLAG_NO_NCQ),

   This does nothing except ruining the valid code formatting.

                .flags          = AHCI_FLAG_COMMON,
                .pio_mask       = ATA_PIO4,
                .udma_mask      = ATA_UDMA6,
diff --git a/drivers/bluetooth/bfusb.c b/drivers/bluetooth/bfusb.c
index 5a321b4076aa..dc6a62cb1941 100644
--- a/drivers/bluetooth/bfusb.c
+++ b/drivers/bluetooth/bfusb.c
@@ -355,15 +355,14 @@ static void bfusb_rx_complete(struct urb *urb)
        while (count) {
                hdr = buf[0] | (buf[1] << 8);
- if (hdr & 0x4000) {
+               if (hdr & 0x4000)
                        len = 0;
                        count -= 2;
                        buf   += 2;
-               } else {
+                else {
                        len = (buf[2] == 0) ? 256 : buf[2];
                        count -= 3;
                        buf   += 3;
-               }

   This just ruins the code.

if (count < len) {
                        bt_dev_err(data->hdev, "block extends over URB buffer 
ranges");
diff --git a/drivers/clk/clk-asm9260.c b/drivers/clk/clk-asm9260.c
index bacebd457e6f..4e608807a00a 100644
--- a/drivers/clk/clk-asm9260.c
+++ b/drivers/clk/clk-asm9260.c
@@ -92,8 +92,8 @@ static const struct asm9260_div_clk asm9260_div_clks[] 
__initconst = {
        { CLKID_SYS_CPU,        "cpu_div", "main_gate", HW_CPUCLKDIV },
        { CLKID_SYS_AHB,        "ahb_div", "cpu_div", HW_SYSAHBCLKDIV },
- /* i2s has two deviders: one for only external mclk and internal
-        * devider for all clks. */
+       //i2s has two deviders: one for only external mclk and internal
+       //devider for all clks.

   Divider. :-) This is not the preferred multi-line comment formatting anyway.

        { CLKID_SYS_I2S0M,      "i2s0m_div", "i2s0_mclk",  HW_I2S0MCLKDIV },
        { CLKID_SYS_I2S1M,      "i2s1m_div", "i2s1_mclk",  HW_I2S1MCLKDIV },
        { CLKID_SYS_I2S0S,      "i2s0s_div", "i2s0_gate",  HW_I2S0SCLKDIV },
[...]

MBR, Sergei

Reply via email to