[PATCH 5/6] staging: bcm2835-firmware-gpio: Add a build system for the driver

[Michael Zoran]

On Mon, 2017-03-20 at 13:43 +0300, Dan Carpenter wrote:
> On Sat, Mar 18, 2017 at 10:40:06PM -0700, Michael Zoran wrote:
> > On Sun, 2017-03-19 at 13:27 +0800, kbuild test robot wrote:
> > > Hi Michael,
> > > 
> > > [auto build test WARNING on staging/staging-testing]
> > > [also build test WARNING on next-20170310]
> > > [cannot apply to linux-rpi/for-rpi-next v4.11-rc2]
> > > [if your patch is applied to the wrong git tree, please drop us a
> > > note to help improve the system]
> > > 
> > > url:    https://github.com/0day-ci/linux/commits/Michael-Zoran/st
> > > agin
> > > g-bcm2835-firmware-gpio-Initial-staging-commit/20170319-111114
> > > 
> > > 
> > > coccinelle warnings: (new ones prefixed by >>)
> > > 
> > > > > drivers/staging/vc04_services/bcm2835-firmware-gpio/gpio-bcm-
> > > > > exp.c:271:3-8: No need to set .owner here. The core will do
> > > > > it.
> > > Please review and possibly fold the followup patch.
> > > 
> > > ---
> > > 0-DAY kernel test infrastructure                Open Source
> > > Technology Center
> > > https://lists.01.org/pipermail/kbuild-all                   Intel
> > > Corporation
> > 
> > OK, Some I'm completely confused here.  Has this patch been
> > applied,
> > and through which tree has it been applied to?
> > 
> > I think I need to send a V2.  So who should I be sending V2 to?
> 
> kbuild grabs patches off the mailing list and automatically tests
> them.
> 
> They haven't been applied.  Just resend a normal v2 series.
> 
> regards,
> dan carpenter
> 

Actually, Stefan and Eric I think this driver has more of your
interest.  I really don't know the ins and out of the DT world, and
that's most of what it's going to take to make this happen.  Perhaps it
would be better for either of you to be the primary drivers of making
this happen?

I'm not sure this driver fits well in with VideoCore which is more my
interest anyway which I think is becoming clearer that it's about vchiq
with all the faults that implies.   Things that are half DT don't seem
to fit in well.

It sounds like you guys want to fast track this driver out of staging
as soon as possible.  Or perhaps it shouldn't go through staging at all
since the original code is rather clear to begin with.

Everyone know where the original source of the driver is now, so no
reason anybody even needs to start with my changes.

Thanks.