Re: [PATCH v3 01/10] dt-bindings: pinctrl: Add bindings for Actions S900 SoC

[Manivannan Sadhasivam]

Hi Daniel,

On Thu, Mar 01, 2018 at 10:23:00AM +0000, Daniel Thompson wrote:
> On Thu, Mar 01, 2018 at 02:55:56PM +0530, Manivannan Sadhasivam wrote:
> > Hi Linus,
> > On Thu, Mar 01, 2018 at 10:20:29AM +0100, Linus Walleij wrote:
> > > On Wed, Feb 28, 2018 at 7:14 PM, Manivannan Sadhasivam
> > > <manivannan.sadhasivam@xxxxxxxxxx> wrote:
> > > 
> > > > Add pinctrl bindings for Actions Semi S900 SoC
> > > >
> > > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>
> > > 
> > > The previous (v2) version had Rob's Review tag, did you just forget
> > > to add it or has something changed?
> > > 
> > 
> > My bad. Slipped through the cracks.
> > 
> > > If nothing is changed I can add his tag when applying.
> > >
> > 
> > I will send out a new version of the whole series, incorporating Andy's
> > pinctrl driver review.
> > 
> > Also, FYI this pacthseries depends on Clock driver patchset I have submitted
> > earlier. So far no reply from Stephen yet. He said that he is working on setting
> > up his new environment and will begin reviewing shortly.
> 
> Is the dependancy at runtime (driver is not functional without clocks) or compile
> time (driver cannot build without other patchset landing)?
>

There is no build time dependency only runtime dependency. Driver is pretty
much useless without clock driver.

Also, there will be merge conflicts when this series gets applied independently.
Since there is no build time dependency, if Linus decides to apply it now, I
can generate a new independent patchset.

Thanks,
Mani 
> 
> Daniel.