| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 15 Nov 2016 19:06:32 +0530
From: Kishon Vijay Abraham I <kishon@...com>
To: Vivek Gautam <vivek.gautam@...eaurora.org>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-samsung-soc@...r.kernel.org"
<linux-samsung-soc@...r.kernel.org>,
<linux-rockchip@...ts.infradead.org>,
linux-tegra <linux-tegra@...r.kernel.org>
Subject: Re: [PATCH 1/3] phy: fix semicolon.cocci warnings
On Tuesday 08 November 2016 04:23 PM, Vivek Gautam wrote:
> Hi Kishon,
>
> On Thu, Oct 20, 2016 at 12:23 PM, Vivek Gautam
> <vivek.gautam@...eaurora.org> wrote:
>> Remove unneeded semicolon.
>>
>> Generated by: coccinellery/semicolon/semicolon.cocci
>>
>> Signed-off-by: Vivek Gautam <vivek.gautam@...eaurora.org>
>> Cc: Kishon Vijay Abraham I <kishon@...com>
>> ---
>
> Gentle ping.
> Do you plan to pull in 3 patches in this series ?
merged now.
Thanks
Kishon
>
>> drivers/phy/phy-brcm-sata.c | 6 +++---
>> drivers/phy/phy-exynos4210-usb2.c | 4 ++--
>> drivers/phy/phy-exynos4x12-usb2.c | 4 ++--
>> drivers/phy/phy-exynos5250-usb2.c | 2 +-
>> drivers/phy/phy-rockchip-emmc.c | 2 +-
>> drivers/phy/phy-s5pv210-usb2.c | 4 ++--
>> 6 files changed, 11 insertions(+), 11 deletions(-)
>>
>> diff --git a/drivers/phy/phy-brcm-sata.c b/drivers/phy/phy-brcm-sata.c
>> index 8ffc44a..ccbc3d9 100644
>> --- a/drivers/phy/phy-brcm-sata.c
>> +++ b/drivers/phy/phy-brcm-sata.c
>> @@ -140,7 +140,7 @@ static inline void __iomem *brcm_sata_pcb_base(struct brcm_sata_port *port)
>> default:
>> dev_err(priv->dev, "invalid phy version\n");
>> break;
>> - };
>> + }
>>
>> return priv->phy_base + (port->portnum * size);
>> }
>> @@ -157,7 +157,7 @@ static inline void __iomem *brcm_sata_ctrl_base(struct brcm_sata_port *port)
>> default:
>> dev_err(priv->dev, "invalid phy version\n");
>> break;
>> - };
>> + }
>>
>> return priv->ctrl_base + (port->portnum * size);
>> }
>> @@ -365,7 +365,7 @@ static int brcm_sata_phy_init(struct phy *phy)
>> break;
>> default:
>> rc = -ENODEV;
>> - };
>> + }
>>
>> return rc;
>> }
>> diff --git a/drivers/phy/phy-exynos4210-usb2.c b/drivers/phy/phy-exynos4210-usb2.c
>> index f30bbb0..1f50e10 100644
>> --- a/drivers/phy/phy-exynos4210-usb2.c
>> +++ b/drivers/phy/phy-exynos4210-usb2.c
>> @@ -141,7 +141,7 @@ static void exynos4210_isol(struct samsung_usb2_phy_instance *inst, bool on)
>> break;
>> default:
>> return;
>> - };
>> + }
>>
>> regmap_update_bits(drv->reg_pmu, offset, mask, on ? 0 : mask);
>> }
>> @@ -179,7 +179,7 @@ static void exynos4210_phy_pwr(struct samsung_usb2_phy_instance *inst, bool on)
>> rstbits = EXYNOS_4210_URSTCON_PHY1_P1P2 |
>> EXYNOS_4210_URSTCON_HOST_LINK_P2;
>> break;
>> - };
>> + }
>>
>> if (on) {
>> clk = readl(drv->reg_phy + EXYNOS_4210_UPHYCLK);
>> diff --git a/drivers/phy/phy-exynos4x12-usb2.c b/drivers/phy/phy-exynos4x12-usb2.c
>> index 765da90..7f27a91 100644
>> --- a/drivers/phy/phy-exynos4x12-usb2.c
>> +++ b/drivers/phy/phy-exynos4x12-usb2.c
>> @@ -187,7 +187,7 @@ static void exynos4x12_isol(struct samsung_usb2_phy_instance *inst, bool on)
>> break;
>> default:
>> return;
>> - };
>> + }
>>
>> regmap_update_bits(drv->reg_pmu, offset, mask, on ? 0 : mask);
>> }
>> @@ -237,7 +237,7 @@ static void exynos4x12_phy_pwr(struct samsung_usb2_phy_instance *inst, bool on)
>> rstbits = EXYNOS_4x12_URSTCON_HSIC1 |
>> EXYNOS_4x12_URSTCON_HOST_LINK_P1;
>> break;
>> - };
>> + }
>>
>> if (on) {
>> pwr = readl(drv->reg_phy + EXYNOS_4x12_UPHYPWR);
>> diff --git a/drivers/phy/phy-exynos5250-usb2.c b/drivers/phy/phy-exynos5250-usb2.c
>> index 2ed1735..aad8062 100644
>> --- a/drivers/phy/phy-exynos5250-usb2.c
>> +++ b/drivers/phy/phy-exynos5250-usb2.c
>> @@ -192,7 +192,7 @@ static void exynos5250_isol(struct samsung_usb2_phy_instance *inst, bool on)
>> break;
>> default:
>> return;
>> - };
>> + }
>>
>> regmap_update_bits(drv->reg_pmu, offset, mask, on ? 0 : mask);
>> }
>> diff --git a/drivers/phy/phy-rockchip-emmc.c b/drivers/phy/phy-rockchip-emmc.c
>> index fd57345..f1b24f1 100644
>> --- a/drivers/phy/phy-rockchip-emmc.c
>> +++ b/drivers/phy/phy-rockchip-emmc.c
>> @@ -132,7 +132,7 @@ static int rockchip_emmc_phy_power(struct phy *phy, bool on_off)
>> default:
>> ideal_rate = 200000000;
>> break;
>> - };
>> + }
>>
>> diff = (rate > ideal_rate) ?
>> rate - ideal_rate : ideal_rate - rate;
>> diff --git a/drivers/phy/phy-s5pv210-usb2.c b/drivers/phy/phy-s5pv210-usb2.c
>> index 004d320..f6f7233 100644
>> --- a/drivers/phy/phy-s5pv210-usb2.c
>> +++ b/drivers/phy/phy-s5pv210-usb2.c
>> @@ -103,7 +103,7 @@ static void s5pv210_isol(struct samsung_usb2_phy_instance *inst, bool on)
>> break;
>> default:
>> return;
>> - };
>> + }
>>
>> regmap_update_bits(drv->reg_pmu, S5PV210_USB_ISOL_OFFSET,
>> mask, on ? 0 : mask);
>> @@ -127,7 +127,7 @@ static void s5pv210_phy_pwr(struct samsung_usb2_phy_instance *inst, bool on)
>> rstbits = S5PV210_URSTCON_PHY1_ALL |
>> S5PV210_URSTCON_HOST_LINK_ALL;
>> break;
>> - };
>> + }
>>
>> if (on) {
>> writel(drv->ref_reg_val, drv->reg_phy + S5PV210_UPHYCLK);
>> --
>> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
>> a Linux Foundation Collaborative Project
>>
>
>
>
Powered by blists - more mailing lists