Hi,
On Friday 22 May 2015 05:08 PM, Joachim Eastwood wrote:
On 22 May 2015 at 13:34, Joachim Eastwood <manabian@xxxxxxxxx> wrote:
On 22 May 2015 at 13:18, Kishon Vijay Abraham I <kishon@xxxxxx> wrote:
Hi,
On Thursday 21 May 2015 09:29 PM, Joachim Eastwood wrote:
Add PHY driver for the internal USB OTG PHY found on NXP
LPC18xx and LPC43xx devices. This driver takes care of
enabling the PHY in CREG (syscon) and setting the required
clock frequency.
Signed-off-by: Joachim Eastwood <manabian@xxxxxxxxx>
---
drivers/phy/Kconfig | 11 +++
drivers/phy/Makefile | 1 +
drivers/phy/phy-lpc18xx-usb-otg.c | 143
++++++++++++++++++++++++++++++++++++++
3 files changed, 155 insertions(+)
create mode 100644 drivers/phy/phy-lpc18xx-usb-otg.c
diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
index a53bd5b52df9..e11fb9e3c760 100644
--- a/drivers/phy/Kconfig
+++ b/drivers/phy/Kconfig
@@ -52,6 +52,17 @@ config PHY_EXYNOS_MIPI_VIDEO
Support for MIPI CSI-2 and MIPI DSI DPHY found on Samsung S5P
and EXYNOS SoCs.
+config PHY_LPC18XX_USB_OTG
+ tristate "NXP LPC18xx/43xx SoC USB OTG PHY driver"
+ depends on OF && (ARCH_LPC18XX || COMPILE_TEST)
+ select GENERIC_PHY
+ select MFD_SYSCON
using select on visible symbols should be avoided. see [1]
[1] ->
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/kbuild/kconfig-language.txt#n111
For these two symbols using select is the right thing to do.
Look at the users in kernel:
$ git grep -E 'depend (MFD_SYSCON|GENERIC_PHY)' | wc -l
0
$ git grep -E 'select MFD_SYSCON' | wc -l
42
Sorry, screwed up the regex for depends.
For depens we have:
$ git grep -E 'depends on (MFD_SYSCON|GENERIC_PHY)' | wc -l
10
select for GENERIC_PHY is okay but I'd prefer to have depends on for MFD_SYSCON.
Thanks
Kishon
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html