On Wed, 2018-10-03 at 21:31 -0300, Leonardo Bras wrote: > On Fri, Sep 28, 2018 at 4:15 AM James Bottomley > <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> wrote: > > > > On Thu, 2018-09-27 at 23:08 -0300, Leonardo Brás wrote: > > > Avoids building driver if 'make drivers/parisc/' is called and > > > CONFIG_PARISC is disabled. > > > > Is that really a problem? The drivers/Makefile has this: > > > > obj-$(CONFIG_PARISC) += parisc/ > > And you just overrode that by forcing the build. It's not even > > clear we should refuse the build in that case; how would we know > > you don't have a legitimate reason for the override? > > > > Sorry I did not explained my reasons earlier. I sent everybody > involved an e-mail explaining the full reason of this change. > (For reference it's here: https://lkml.org/lkml/2018/10/3/707) Well it's not really that persuasive. Most people simply let the build run to completion, but if you have a problem with a job control 3h timelimit, then create a job that kills itself at 2:59 and then resubmits itself. That will produce a complete build in 3h chunks without any need to call sub Makefiles. All of our Makefiles are coded assuming the upper level can prevent descent into the lower ones. You're proposing to change that assumption, requiring a fairly large patch set, which doesn't really seem to provide a huge benefit. James > > Signed-off-by: Leonardo Brás <leobras.c@xxxxxxxxx> > > > --- > > > drivers/parisc/Makefile | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile > > > index 3cd5e6cb8478..80049d763aa0 100644 > > > --- a/drivers/parisc/Makefile > > > +++ b/drivers/parisc/Makefile > > > @@ -24,5 +24,5 @@ obj-$(CONFIG_EISA) += eisa.o > > > eisa_enumerator.o eisa_eeprom.o > > > obj-$(CONFIG_SUPERIO) += superio.o > > > obj-$(CONFIG_CHASSIS_LCD_LED) += led.o > > > obj-$(CONFIG_PDC_STABLE) += pdc_stable.o > > > -obj-y += power.o > > > +obj-$(CONFIG_PARISC) += power.o > > > > If we conclude the use case is legitimate, that's not enough: the > > two > > inner symbols are PARISC only but CONFIG_EISA isn't. > > You are right. > It worked for my needs because I am only building the drivers, and > not linking them. But i believe doing something like I did in > zorro/Makefile would fix this all. (For reference, > https://lkml.org/lkml/2018/9/28/150 ) > > If you agree, I will send the next patchset with this change. > > Thanks for your help! > > Leonardo Bras >
