Thread (46 messages) 46 messages, 6 authors, 2017-05-10

[PATCH v2 02/30] pinctrl: mediatek: reuse pinctrl driver for mt7623

From: sean.wang@mediatek.com (Sean Wang)
Date: 2017-05-01 06:44:19
Also in: linux-devicetree, linux-gpio, linux-mediatek, lkml

On Fri, 2017-04-28 at 10:01 +0200, Linus Walleij wrote:
On Wed, Apr 26, 2017 at 11:25 AM,  [off-list ref] wrote:
quoted
From: Sean Wang <sean.wang@mediatek.com>

mt7623 pinctrl driver can be compatible with mt2701 one,
so the patch reuses the driver and deletes those redundant
ones.

Cc: John Crispin <john@phrozen.org>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
Partly correct.
quoted
        "mediatek,mt6397-pinctrl", compatible with mt6397 pinctrl.
-       "mediatek,mt7623-pinctrl", compatible with mt7623 pinctrl.
NO don't do this.

"compatible" means exactly this: this hardware is compatible with
this driver. That is why we have it!

So instead of mt7623 pretending to be mt2701, let the mt2701 driver
list that it is compatible with mt7623, simple.

So patch pinctrl-mt2701.c mt2701_pctrl_match[] instead.
Hi Linus,

really appreciate your clear guidance and reviewing on this

I will fix it up in the next version
	
	Sean
Yours,
Linus Walleij
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help