From mboxrd@z Thu Jan 1 00:00:00 1970 From: andi.shyti@samsung.com (Andi Shyti) Date: Tue, 20 Oct 2015 17:20:47 +0900 Subject: [PATCH v4 6/8] drivers: soc: add support for exynos SROM driver In-Reply-To: <014301d10b01$488873f0$d9995bd0$@samsung.com> References: <1445255206-11148-1-git-send-email-pankaj.dubey@samsung.com> <1445255206-11148-7-git-send-email-pankaj.dubey@samsung.com> <56258683.7070204@samsung.com> <5625B92C.10505@samsung.com> <5625C07D.4030706@samsung.com> <014301d10b01$488873f0$d9995bd0$@samsung.com> Message-ID: <20151020082047.GA923@samsunx.samsung> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Pavel, > > AFAIR Fedin was talking about missing functionality, not about adding > > the contribution by himself. So he might add it or he might not. I did > > not receive any commitments from him. > > I am waiting for the driver to be integrated, because i see it's constantly redesigned. Then i'll post my patches. By the way, they > will be useful only if pin controller driver for 5410 is accepted upstream, several authors have done it but i still didn't see it > in upstream. > Pin controller is needed in order to configure multi-functional pins correctly. > > > The driver should be "proper" for > > the time being (which could mean !PM dependency). If there is a need, > > then the dependency will be removed. > > I can do it later if you prefer. can we add the "depends on ... && PM" now, later, once you'll extend it, you remove it again? Personally I'd prefer this way rather than having a driver that does nothing in case of !PM. Andi