GNU Linux-libre 4.14.290-gnu1
[releases.git] / drivers / phy / broadcom / phy-brcm-sata.c
1 /*
2  * Broadcom SATA3 AHCI Controller PHY Driver
3  *
4  * Copyright (C) 2016 Broadcom
5  *
6  * This program is free software; you can redistribute it and/or modify
7  * it under the terms of the GNU General Public License as published by
8  * the Free Software Foundation; either version 2, or (at your option)
9  * any later version.
10  *
11  * This program is distributed in the hope that it will be useful,
12  * but WITHOUT ANY WARRANTY; without even the implied warranty of
13  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
14  * GNU General Public License for more details.
15  */
16
17 #include <linux/delay.h>
18 #include <linux/device.h>
19 #include <linux/init.h>
20 #include <linux/interrupt.h>
21 #include <linux/io.h>
22 #include <linux/kernel.h>
23 #include <linux/module.h>
24 #include <linux/of.h>
25 #include <linux/phy/phy.h>
26 #include <linux/platform_device.h>
27
28 #define SATA_PCB_BANK_OFFSET                            0x23c
29 #define SATA_PCB_REG_OFFSET(ofs)                        ((ofs) * 4)
30
31 #define MAX_PORTS                                       2
32
33 /* Register offset between PHYs in PCB space */
34 #define SATA_PCB_REG_28NM_SPACE_SIZE                    0x1000
35
36 /* The older SATA PHY registers duplicated per port registers within the map,
37  * rather than having a separate map per port.
38  */
39 #define SATA_PCB_REG_40NM_SPACE_SIZE                    0x10
40
41 /* Register offset between PHYs in PHY control space */
42 #define SATA_PHY_CTRL_REG_28NM_SPACE_SIZE               0x8
43
44 enum brcm_sata_phy_version {
45         BRCM_SATA_PHY_STB_28NM,
46         BRCM_SATA_PHY_STB_40NM,
47         BRCM_SATA_PHY_IPROC_NS2,
48         BRCM_SATA_PHY_IPROC_NSP,
49         BRCM_SATA_PHY_IPROC_SR,
50 };
51
52 struct brcm_sata_port {
53         int portnum;
54         struct phy *phy;
55         struct brcm_sata_phy *phy_priv;
56         bool ssc_en;
57 };
58
59 struct brcm_sata_phy {
60         struct device *dev;
61         void __iomem *phy_base;
62         void __iomem *ctrl_base;
63         enum brcm_sata_phy_version version;
64
65         struct brcm_sata_port phys[MAX_PORTS];
66 };
67
68 enum sata_phy_regs {
69         BLOCK0_REG_BANK                         = 0x000,
70         BLOCK0_XGXSSTATUS                       = 0x81,
71         BLOCK0_XGXSSTATUS_PLL_LOCK              = BIT(12),
72         BLOCK0_SPARE                            = 0x8d,
73         BLOCK0_SPARE_OOB_CLK_SEL_MASK           = 0x3,
74         BLOCK0_SPARE_OOB_CLK_SEL_REFBY2         = 0x1,
75
76         PLL_REG_BANK_0                          = 0x050,
77         PLL_REG_BANK_0_PLLCONTROL_0             = 0x81,
78         PLLCONTROL_0_FREQ_DET_RESTART           = BIT(13),
79         PLLCONTROL_0_FREQ_MONITOR               = BIT(12),
80         PLLCONTROL_0_SEQ_START                  = BIT(15),
81         PLL_CAP_CONTROL                         = 0x85,
82         PLL_ACTRL2                              = 0x8b,
83         PLL_ACTRL2_SELDIV_MASK                  = 0x1f,
84         PLL_ACTRL2_SELDIV_SHIFT                 = 9,
85         PLL_ACTRL6                              = 0x86,
86
87         PLL1_REG_BANK                           = 0x060,
88         PLL1_ACTRL2                             = 0x82,
89         PLL1_ACTRL3                             = 0x83,
90         PLL1_ACTRL4                             = 0x84,
91
92         TX_REG_BANK                             = 0x070,
93         TX_ACTRL0                               = 0x80,
94         TX_ACTRL0_TXPOL_FLIP                    = BIT(6),
95
96         OOB_REG_BANK                            = 0x150,
97         OOB1_REG_BANK                           = 0x160,
98         OOB_CTRL1                               = 0x80,
99         OOB_CTRL1_BURST_MAX_MASK                = 0xf,
100         OOB_CTRL1_BURST_MAX_SHIFT               = 12,
101         OOB_CTRL1_BURST_MIN_MASK                = 0xf,
102         OOB_CTRL1_BURST_MIN_SHIFT               = 8,
103         OOB_CTRL1_WAKE_IDLE_MAX_MASK            = 0xf,
104         OOB_CTRL1_WAKE_IDLE_MAX_SHIFT           = 4,
105         OOB_CTRL1_WAKE_IDLE_MIN_MASK            = 0xf,
106         OOB_CTRL1_WAKE_IDLE_MIN_SHIFT           = 0,
107         OOB_CTRL2                               = 0x81,
108         OOB_CTRL2_SEL_ENA_SHIFT                 = 15,
109         OOB_CTRL2_SEL_ENA_RC_SHIFT              = 14,
110         OOB_CTRL2_RESET_IDLE_MAX_MASK           = 0x3f,
111         OOB_CTRL2_RESET_IDLE_MAX_SHIFT          = 8,
112         OOB_CTRL2_BURST_CNT_MASK                = 0x3,
113         OOB_CTRL2_BURST_CNT_SHIFT               = 6,
114         OOB_CTRL2_RESET_IDLE_MIN_MASK           = 0x3f,
115         OOB_CTRL2_RESET_IDLE_MIN_SHIFT          = 0,
116
117         TXPMD_REG_BANK                          = 0x1a0,
118         TXPMD_CONTROL1                          = 0x81,
119         TXPMD_CONTROL1_TX_SSC_EN_FRC            = BIT(0),
120         TXPMD_CONTROL1_TX_SSC_EN_FRC_VAL        = BIT(1),
121         TXPMD_TX_FREQ_CTRL_CONTROL1             = 0x82,
122         TXPMD_TX_FREQ_CTRL_CONTROL2             = 0x83,
123         TXPMD_TX_FREQ_CTRL_CONTROL2_FMIN_MASK   = 0x3ff,
124         TXPMD_TX_FREQ_CTRL_CONTROL3             = 0x84,
125         TXPMD_TX_FREQ_CTRL_CONTROL3_FMAX_MASK   = 0x3ff,
126 };
127
128 enum sata_phy_ctrl_regs {
129         PHY_CTRL_1                              = 0x0,
130         PHY_CTRL_1_RESET                        = BIT(0),
131 };
132
133 static inline void __iomem *brcm_sata_pcb_base(struct brcm_sata_port *port)
134 {
135         struct brcm_sata_phy *priv = port->phy_priv;
136         u32 size = 0;
137
138         switch (priv->version) {
139         case BRCM_SATA_PHY_STB_28NM:
140         case BRCM_SATA_PHY_IPROC_NS2:
141                 size = SATA_PCB_REG_28NM_SPACE_SIZE;
142                 break;
143         case BRCM_SATA_PHY_STB_40NM:
144                 size = SATA_PCB_REG_40NM_SPACE_SIZE;
145                 break;
146         default:
147                 dev_err(priv->dev, "invalid phy version\n");
148                 break;
149         }
150
151         return priv->phy_base + (port->portnum * size);
152 }
153
154 static inline void __iomem *brcm_sata_ctrl_base(struct brcm_sata_port *port)
155 {
156         struct brcm_sata_phy *priv = port->phy_priv;
157         u32 size = 0;
158
159         switch (priv->version) {
160         case BRCM_SATA_PHY_IPROC_NS2:
161                 size = SATA_PHY_CTRL_REG_28NM_SPACE_SIZE;
162                 break;
163         default:
164                 dev_err(priv->dev, "invalid phy version\n");
165                 break;
166         }
167
168         return priv->ctrl_base + (port->portnum * size);
169 }
170
171 static void brcm_sata_phy_wr(void __iomem *pcb_base, u32 bank,
172                              u32 ofs, u32 msk, u32 value)
173 {
174         u32 tmp;
175
176         writel(bank, pcb_base + SATA_PCB_BANK_OFFSET);
177         tmp = readl(pcb_base + SATA_PCB_REG_OFFSET(ofs));
178         tmp = (tmp & msk) | value;
179         writel(tmp, pcb_base + SATA_PCB_REG_OFFSET(ofs));
180 }
181
182 static u32 brcm_sata_phy_rd(void __iomem *pcb_base, u32 bank, u32 ofs)
183 {
184         writel(bank, pcb_base + SATA_PCB_BANK_OFFSET);
185         return readl(pcb_base + SATA_PCB_REG_OFFSET(ofs));
186 }
187
188 /* These defaults were characterized by H/W group */
189 #define STB_FMIN_VAL_DEFAULT    0x3df
190 #define STB_FMAX_VAL_DEFAULT    0x3df
191 #define STB_FMAX_VAL_SSC        0x83
192
193 static int brcm_stb_sata_init(struct brcm_sata_port *port)
194 {
195         void __iomem *base = brcm_sata_pcb_base(port);
196         struct brcm_sata_phy *priv = port->phy_priv;
197         u32 tmp;
198
199         /* override the TX spread spectrum setting */
200         tmp = TXPMD_CONTROL1_TX_SSC_EN_FRC_VAL | TXPMD_CONTROL1_TX_SSC_EN_FRC;
201         brcm_sata_phy_wr(base, TXPMD_REG_BANK, TXPMD_CONTROL1, ~tmp, tmp);
202
203         /* set fixed min freq */
204         brcm_sata_phy_wr(base, TXPMD_REG_BANK, TXPMD_TX_FREQ_CTRL_CONTROL2,
205                          ~TXPMD_TX_FREQ_CTRL_CONTROL2_FMIN_MASK,
206                          STB_FMIN_VAL_DEFAULT);
207
208         /* set fixed max freq depending on SSC config */
209         if (port->ssc_en) {
210                 dev_info(priv->dev, "enabling SSC on port%d\n", port->portnum);
211                 tmp = STB_FMAX_VAL_SSC;
212         } else {
213                 tmp = STB_FMAX_VAL_DEFAULT;
214         }
215
216         brcm_sata_phy_wr(base, TXPMD_REG_BANK, TXPMD_TX_FREQ_CTRL_CONTROL3,
217                           ~TXPMD_TX_FREQ_CTRL_CONTROL3_FMAX_MASK, tmp);
218
219         return 0;
220 }
221
222 /* NS2 SATA PLL1 defaults were characterized by H/W group */
223 #define NS2_PLL1_ACTRL2_MAGIC   0x1df8
224 #define NS2_PLL1_ACTRL3_MAGIC   0x2b00
225 #define NS2_PLL1_ACTRL4_MAGIC   0x8824
226
227 static int brcm_ns2_sata_init(struct brcm_sata_port *port)
228 {
229         int try;
230         unsigned int val;
231         void __iomem *base = brcm_sata_pcb_base(port);
232         void __iomem *ctrl_base = brcm_sata_ctrl_base(port);
233         struct device *dev = port->phy_priv->dev;
234
235         /* Configure OOB control */
236         val = 0x0;
237         val |= (0xc << OOB_CTRL1_BURST_MAX_SHIFT);
238         val |= (0x4 << OOB_CTRL1_BURST_MIN_SHIFT);
239         val |= (0x9 << OOB_CTRL1_WAKE_IDLE_MAX_SHIFT);
240         val |= (0x3 << OOB_CTRL1_WAKE_IDLE_MIN_SHIFT);
241         brcm_sata_phy_wr(base, OOB_REG_BANK, OOB_CTRL1, 0x0, val);
242         val = 0x0;
243         val |= (0x1b << OOB_CTRL2_RESET_IDLE_MAX_SHIFT);
244         val |= (0x2 << OOB_CTRL2_BURST_CNT_SHIFT);
245         val |= (0x9 << OOB_CTRL2_RESET_IDLE_MIN_SHIFT);
246         brcm_sata_phy_wr(base, OOB_REG_BANK, OOB_CTRL2, 0x0, val);
247
248         /* Configure PHY PLL register bank 1 */
249         val = NS2_PLL1_ACTRL2_MAGIC;
250         brcm_sata_phy_wr(base, PLL1_REG_BANK, PLL1_ACTRL2, 0x0, val);
251         val = NS2_PLL1_ACTRL3_MAGIC;
252         brcm_sata_phy_wr(base, PLL1_REG_BANK, PLL1_ACTRL3, 0x0, val);
253         val = NS2_PLL1_ACTRL4_MAGIC;
254         brcm_sata_phy_wr(base, PLL1_REG_BANK, PLL1_ACTRL4, 0x0, val);
255
256         /* Configure PHY BLOCK0 register bank */
257         /* Set oob_clk_sel to refclk/2 */
258         brcm_sata_phy_wr(base, BLOCK0_REG_BANK, BLOCK0_SPARE,
259                          ~BLOCK0_SPARE_OOB_CLK_SEL_MASK,
260                          BLOCK0_SPARE_OOB_CLK_SEL_REFBY2);
261
262         /* Strobe PHY reset using PHY control register */
263         writel(PHY_CTRL_1_RESET, ctrl_base + PHY_CTRL_1);
264         mdelay(1);
265         writel(0x0, ctrl_base + PHY_CTRL_1);
266         mdelay(1);
267
268         /* Wait for PHY PLL lock by polling pll_lock bit */
269         try = 50;
270         while (try) {
271                 val = brcm_sata_phy_rd(base, BLOCK0_REG_BANK,
272                                         BLOCK0_XGXSSTATUS);
273                 if (val & BLOCK0_XGXSSTATUS_PLL_LOCK)
274                         break;
275                 msleep(20);
276                 try--;
277         }
278         if (!try) {
279                 /* PLL did not lock; give up */
280                 dev_err(dev, "port%d PLL did not lock\n", port->portnum);
281                 return -ETIMEDOUT;
282         }
283
284         dev_dbg(dev, "port%d initialized\n", port->portnum);
285
286         return 0;
287 }
288
289 static int brcm_nsp_sata_init(struct brcm_sata_port *port)
290 {
291         struct brcm_sata_phy *priv = port->phy_priv;
292         struct device *dev = port->phy_priv->dev;
293         void __iomem *base = priv->phy_base;
294         unsigned int oob_bank;
295         unsigned int val, try;
296
297         /* Configure OOB control */
298         if (port->portnum == 0)
299                 oob_bank = OOB_REG_BANK;
300         else if (port->portnum == 1)
301                 oob_bank = OOB1_REG_BANK;
302         else
303                 return -EINVAL;
304
305         val = 0x0;
306         val |= (0x0f << OOB_CTRL1_BURST_MAX_SHIFT);
307         val |= (0x06 << OOB_CTRL1_BURST_MIN_SHIFT);
308         val |= (0x0f << OOB_CTRL1_WAKE_IDLE_MAX_SHIFT);
309         val |= (0x06 << OOB_CTRL1_WAKE_IDLE_MIN_SHIFT);
310         brcm_sata_phy_wr(base, oob_bank, OOB_CTRL1, 0x0, val);
311
312         val = 0x0;
313         val |= (0x2e << OOB_CTRL2_RESET_IDLE_MAX_SHIFT);
314         val |= (0x02 << OOB_CTRL2_BURST_CNT_SHIFT);
315         val |= (0x16 << OOB_CTRL2_RESET_IDLE_MIN_SHIFT);
316         brcm_sata_phy_wr(base, oob_bank, OOB_CTRL2, 0x0, val);
317
318
319         brcm_sata_phy_wr(base, PLL_REG_BANK_0, PLL_ACTRL2,
320                 ~(PLL_ACTRL2_SELDIV_MASK << PLL_ACTRL2_SELDIV_SHIFT),
321                 0x0c << PLL_ACTRL2_SELDIV_SHIFT);
322
323         brcm_sata_phy_wr(base, PLL_REG_BANK_0, PLL_CAP_CONTROL,
324                                                 0xff0, 0x4f0);
325
326         val = PLLCONTROL_0_FREQ_DET_RESTART | PLLCONTROL_0_FREQ_MONITOR;
327         brcm_sata_phy_wr(base, PLL_REG_BANK_0, PLL_REG_BANK_0_PLLCONTROL_0,
328                                                                 ~val, val);
329         val = PLLCONTROL_0_SEQ_START;
330         brcm_sata_phy_wr(base, PLL_REG_BANK_0, PLL_REG_BANK_0_PLLCONTROL_0,
331                                                                 ~val, 0);
332         mdelay(10);
333         brcm_sata_phy_wr(base, PLL_REG_BANK_0, PLL_REG_BANK_0_PLLCONTROL_0,
334                                                                 ~val, val);
335
336         /* Wait for pll_seq_done bit */
337         try = 50;
338         while (--try) {
339                 val = brcm_sata_phy_rd(base, BLOCK0_REG_BANK,
340                                         BLOCK0_XGXSSTATUS);
341                 if (val & BLOCK0_XGXSSTATUS_PLL_LOCK)
342                         break;
343                 msleep(20);
344         }
345         if (!try) {
346                 /* PLL did not lock; give up */
347                 dev_err(dev, "port%d PLL did not lock\n", port->portnum);
348                 return -ETIMEDOUT;
349         }
350
351         dev_dbg(dev, "port%d initialized\n", port->portnum);
352
353         return 0;
354 }
355
356 /* SR PHY PLL0 registers */
357 #define SR_PLL0_ACTRL6_MAGIC                    0xa
358
359 /* SR PHY PLL1 registers */
360 #define SR_PLL1_ACTRL2_MAGIC                    0x32
361 #define SR_PLL1_ACTRL3_MAGIC                    0x2
362 #define SR_PLL1_ACTRL4_MAGIC                    0x3e8
363
364 static int brcm_sr_sata_init(struct brcm_sata_port *port)
365 {
366         struct brcm_sata_phy *priv = port->phy_priv;
367         struct device *dev = port->phy_priv->dev;
368         void __iomem *base = priv->phy_base;
369         unsigned int val, try;
370
371         /* Configure PHY PLL register bank 1 */
372         val = SR_PLL1_ACTRL2_MAGIC;
373         brcm_sata_phy_wr(base, PLL1_REG_BANK, PLL1_ACTRL2, 0x0, val);
374         val = SR_PLL1_ACTRL3_MAGIC;
375         brcm_sata_phy_wr(base, PLL1_REG_BANK, PLL1_ACTRL3, 0x0, val);
376         val = SR_PLL1_ACTRL4_MAGIC;
377         brcm_sata_phy_wr(base, PLL1_REG_BANK, PLL1_ACTRL4, 0x0, val);
378
379         /* Configure PHY PLL register bank 0 */
380         val = SR_PLL0_ACTRL6_MAGIC;
381         brcm_sata_phy_wr(base, PLL_REG_BANK_0, PLL_ACTRL6, 0x0, val);
382
383         /* Wait for PHY PLL lock by polling pll_lock bit */
384         try = 50;
385         do {
386                 val = brcm_sata_phy_rd(base, BLOCK0_REG_BANK,
387                                         BLOCK0_XGXSSTATUS);
388                 if (val & BLOCK0_XGXSSTATUS_PLL_LOCK)
389                         break;
390                 msleep(20);
391                 try--;
392         } while (try);
393
394         if ((val & BLOCK0_XGXSSTATUS_PLL_LOCK) == 0) {
395                 /* PLL did not lock; give up */
396                 dev_err(dev, "port%d PLL did not lock\n", port->portnum);
397                 return -ETIMEDOUT;
398         }
399
400         /* Invert Tx polarity */
401         brcm_sata_phy_wr(base, TX_REG_BANK, TX_ACTRL0,
402                          ~TX_ACTRL0_TXPOL_FLIP, TX_ACTRL0_TXPOL_FLIP);
403
404         /* Configure OOB control to handle 100MHz reference clock */
405         val = ((0xc << OOB_CTRL1_BURST_MAX_SHIFT) |
406                 (0x4 << OOB_CTRL1_BURST_MIN_SHIFT) |
407                 (0x8 << OOB_CTRL1_WAKE_IDLE_MAX_SHIFT) |
408                 (0x3 << OOB_CTRL1_WAKE_IDLE_MIN_SHIFT));
409         brcm_sata_phy_wr(base, OOB_REG_BANK, OOB_CTRL1, 0x0, val);
410         val = ((0x1b << OOB_CTRL2_RESET_IDLE_MAX_SHIFT) |
411                 (0x2 << OOB_CTRL2_BURST_CNT_SHIFT) |
412                 (0x9 << OOB_CTRL2_RESET_IDLE_MIN_SHIFT));
413         brcm_sata_phy_wr(base, OOB_REG_BANK, OOB_CTRL2, 0x0, val);
414
415         return 0;
416 }
417
418 static int brcm_sata_phy_init(struct phy *phy)
419 {
420         int rc;
421         struct brcm_sata_port *port = phy_get_drvdata(phy);
422
423         switch (port->phy_priv->version) {
424         case BRCM_SATA_PHY_STB_28NM:
425         case BRCM_SATA_PHY_STB_40NM:
426                 rc = brcm_stb_sata_init(port);
427                 break;
428         case BRCM_SATA_PHY_IPROC_NS2:
429                 rc = brcm_ns2_sata_init(port);
430                 break;
431         case BRCM_SATA_PHY_IPROC_NSP:
432                 rc = brcm_nsp_sata_init(port);
433                 break;
434         case BRCM_SATA_PHY_IPROC_SR:
435                 rc = brcm_sr_sata_init(port);
436                 break;
437         default:
438                 rc = -ENODEV;
439         }
440
441         return rc;
442 }
443
444 static const struct phy_ops phy_ops = {
445         .init           = brcm_sata_phy_init,
446         .owner          = THIS_MODULE,
447 };
448
449 static const struct of_device_id brcm_sata_phy_of_match[] = {
450         { .compatible   = "brcm,bcm7445-sata-phy",
451           .data = (void *)BRCM_SATA_PHY_STB_28NM },
452         { .compatible   = "brcm,bcm7425-sata-phy",
453           .data = (void *)BRCM_SATA_PHY_STB_40NM },
454         { .compatible   = "brcm,iproc-ns2-sata-phy",
455           .data = (void *)BRCM_SATA_PHY_IPROC_NS2 },
456         { .compatible = "brcm,iproc-nsp-sata-phy",
457           .data = (void *)BRCM_SATA_PHY_IPROC_NSP },
458         { .compatible   = "brcm,iproc-sr-sata-phy",
459           .data = (void *)BRCM_SATA_PHY_IPROC_SR },
460         {},
461 };
462 MODULE_DEVICE_TABLE(of, brcm_sata_phy_of_match);
463
464 static int brcm_sata_phy_probe(struct platform_device *pdev)
465 {
466         struct device *dev = &pdev->dev;
467         struct device_node *dn = dev->of_node, *child;
468         const struct of_device_id *of_id;
469         struct brcm_sata_phy *priv;
470         struct resource *res;
471         struct phy_provider *provider;
472         int ret, count = 0;
473
474         if (of_get_child_count(dn) == 0)
475                 return -ENODEV;
476
477         priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
478         if (!priv)
479                 return -ENOMEM;
480         dev_set_drvdata(dev, priv);
481         priv->dev = dev;
482
483         res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "phy");
484         priv->phy_base = devm_ioremap_resource(dev, res);
485         if (IS_ERR(priv->phy_base))
486                 return PTR_ERR(priv->phy_base);
487
488         of_id = of_match_node(brcm_sata_phy_of_match, dn);
489         if (of_id)
490                 priv->version = (enum brcm_sata_phy_version)of_id->data;
491         else
492                 priv->version = BRCM_SATA_PHY_STB_28NM;
493
494         if (priv->version == BRCM_SATA_PHY_IPROC_NS2) {
495                 res = platform_get_resource_byname(pdev, IORESOURCE_MEM,
496                                                    "phy-ctrl");
497                 priv->ctrl_base = devm_ioremap_resource(dev, res);
498                 if (IS_ERR(priv->ctrl_base))
499                         return PTR_ERR(priv->ctrl_base);
500         }
501
502         for_each_available_child_of_node(dn, child) {
503                 unsigned int id;
504                 struct brcm_sata_port *port;
505
506                 if (of_property_read_u32(child, "reg", &id)) {
507                         dev_err(dev, "missing reg property in node %s\n",
508                                         child->name);
509                         ret = -EINVAL;
510                         goto put_child;
511                 }
512
513                 if (id >= MAX_PORTS) {
514                         dev_err(dev, "invalid reg: %u\n", id);
515                         ret = -EINVAL;
516                         goto put_child;
517                 }
518                 if (priv->phys[id].phy) {
519                         dev_err(dev, "already registered port %u\n", id);
520                         ret = -EINVAL;
521                         goto put_child;
522                 }
523
524                 port = &priv->phys[id];
525                 port->portnum = id;
526                 port->phy_priv = priv;
527                 port->phy = devm_phy_create(dev, child, &phy_ops);
528                 port->ssc_en = of_property_read_bool(child, "brcm,enable-ssc");
529                 if (IS_ERR(port->phy)) {
530                         dev_err(dev, "failed to create PHY\n");
531                         ret = PTR_ERR(port->phy);
532                         goto put_child;
533                 }
534
535                 phy_set_drvdata(port->phy, port);
536                 count++;
537         }
538
539         provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
540         if (IS_ERR(provider)) {
541                 dev_err(dev, "could not register PHY provider\n");
542                 return PTR_ERR(provider);
543         }
544
545         dev_info(dev, "registered %d port(s)\n", count);
546
547         return 0;
548 put_child:
549         of_node_put(child);
550         return ret;
551 }
552
553 static struct platform_driver brcm_sata_phy_driver = {
554         .probe  = brcm_sata_phy_probe,
555         .driver = {
556                 .of_match_table = brcm_sata_phy_of_match,
557                 .name           = "brcm-sata-phy",
558         }
559 };
560 module_platform_driver(brcm_sata_phy_driver);
561
562 MODULE_DESCRIPTION("Broadcom SATA PHY driver");
563 MODULE_LICENSE("GPL");
564 MODULE_AUTHOR("Marc Carino");
565 MODULE_AUTHOR("Brian Norris");
566 MODULE_ALIAS("platform:phy-brcm-sata");