Thread (5 messages) 5 messages, 3 authors, 2015-12-10

[PATCH 2/3] pinctrl: nsp: add pinmux driver support for Broadcom NSP SoC

From: rjui@broadcom.com (Ray Jui)
Date: 2015-12-10 22:58:38
Also in: linux-gpio, lkml

Hi Linus,

On 12/10/2015 8:50 AM, Linus Walleij wrote:
On Mon, Nov 30, 2015 at 6:05 PM, Ray Jui [off-list ref] wrote:
quoted
Hi Linus,

On 11/30/2015 4:49 AM, Linus Walleij wrote:
quoted
On Fri, Nov 20, 2015 at 6:57 PM, Yendapally Reddy Dhananjaya Reddy
[off-list ref] wrote:
quoted
This adds the initial support of the Broadcom NSP pinmux driver.

Signed-off-by: Yendapally Reddy Dhananjaya Reddy <redacted>

So what does Ray and Stephen say about this?

Is the hardware unique enough to warrant its own driver?

Or should it utilize/extend one of the existing Broadcom
drivers?

Currently, the IOMUX configurations in Cygnus, NSP, and NS2 are completely
different and require a separate driver to each of these SoCs. We now
realize the shortcomings and have been communicating with our ASIC team.
We'd like to fix this for the next-gen of our SoC. We are driving towards
using a register layout that will allow us to use just "pinctrl-single" and
"pinconf-single."
OK can you provide your reviews for the NSP driver?

Yours,
Linus Walleij
Yes, I just replied to the email thread with my "Reviewed-by". In fact, 
both Scott and I reviewed the patches internally in Broadcom before they 
went out to public.

Thanks,

Ray
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help