linux-kernel - RE: [RFC] [PATCH] phy: Add new Exynos USB PHY driver

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-id: <001701ceb92b$dad5f3b0$9081db10$%debski@samsung.com>
Date:	Tue, 24 Sep 2013 15:42:03 +0200
From:	Kamil Debski <k.debski@...sung.com>
To:	'Vivek Gautam' <gautamvivek1987@...il.com>
Cc:	linux-kernel@...r.kernel.org,
	'Kyungmin Park' <kyungmin.park@...sung.com>,
	'Kishon Vijay Abraham I' <kishon@...com>,
	Tomasz Figa <t.figa@...sung.com>,
	Sylwester Nawrocki <s.nawrocki@...sung.com>,
	Marek Szyprowski <m.szyprowski@...sung.com>
Subject: RE: [RFC] [PATCH] phy: Add new Exynos USB PHY driver

Hi,

> From: Vivek Gautam [mailto:gautamvivek1987@...il.com]
> Sent: Friday, September 20, 2013 1:44 PM
> 
> On Fri, Sep 20, 2013 at 5:11 PM, Vivek Gautam
> <gautamvivek1987@...il.com> wrote:
> > On Wed, Sep 11, 2013 at 4:57 PM, Kamil Debski <k.debski@...sung.com>
> wrote:
> >> Add a new driver for the Exynos USB PHY. The new driver uses the
> >> generic PHY framework.
> >>
> >> Signed-off-by: Kamil Debski <k.debski@...sung.com>
> >> Signed-off-by: Kyungmin Park <kyungmin.park@...sung.com>
> >> Cc: Kishon Vijay Abraham I <kishon@...com>
> >> Cc: Tomasz Figa <t.figa@...sung.com>
> >> Cc: Sylwester Nawrocki <s.nawrocki@...sung.com>
> >> Cc: Marek Szyprowski <m.szyprowski@...sung.com>
> >> ---
> >>
> >> Hi,
> >>
> >> This patch adds a new drive for USB PHYs for Samsung SoCs. The
> driver
> >> is using the Generic PHY Framework created by Kishon Vijay Abrahan I.
> >> It can be found here https://lkml.org/lkml/2013/8/21/29.
> >> This patch adds support to Exynos4 family of SoCs. Support for
> >> Exynos3 and
> >> Exynos5 is planned to be added in the near future.
> >>
> >> I welcome your comments.
> >
> > How do you plan to put support for exynos5 series. I can see you have
> > made separate files for exynos4210 and 4212.
> > Now for exynos5250 and above, we should be able to re-use some of the
> > functions like "4212_rate_to_clk()" and "4212_iso()".

I see two options. One - moving the functions that could be reused to
separate file.
Two - exporting these functions in let's say 4212 and using them in 5250.
What do you think?

I will also support for S5PC110 in the Exynos 4210 file, as they are very
similar.

> >
> > Let me know if i can help with something for exynos5 stuff.

Now, I am trying to run the USB device on Exynos5. The board I am using is
the Arndale board. Unfortunately I am having trouble as the USB lines from
SoC
are connected both to the device port and the USB/Ethernet chip (AX88796).
Did you use the Arndale board when working with Exynos 5?

> >
> 
> Also please add linux-usb and linux-samsung mailing list too.

I will.

[snip]

Best wishes,
-- 
Kamil Debski
Linux Kernel Developer
Samsung R&D Institute Poland

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/