linux-kernel - Re: [PATCH 1/6] mfd: altera-a10sr: make it explicitly non-modular

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAP=VYLoH04Z1fY3YBc4i2SMi=x4y+mrd3CSNfi_FGH7Ss4Gkow@mail.gmail.com>
Date:   Tue, 23 Aug 2016 16:56:32 -0400
From:   Paul Gortmaker <paul.gortmaker@...driver.com>
To:     Lee Jones <lee.jones@...aro.org>
Cc:     LKML <linux-kernel@...r.kernel.org>,
        Thor Thayer <tthayer@...nsource.altera.com>
Subject: Re: [PATCH 1/6] mfd: altera-a10sr: make it explicitly non-modular

On Fri, Aug 5, 2016 at 8:57 AM, Lee Jones <lee.jones@...aro.org> wrote:
> On Mon, 04 Jul 2016, Paul Gortmaker wrote:
>
>> The Kconfig currently controlling compilation of this code is:
>>
>> drivers/mfd/Kconfig:config MFD_ALTERA_A10SR
>> drivers/mfd/Kconfig:    bool "Altera Arria10 DevKit System Resource chip"
>>
>> ...meaning that it currently is not being built as a module by anyone.
>>
>> Lets remove the modular code that is essentially orphaned, so that
>> when reading the driver there is no doubt it is builtin-only.
>>

[...]

>>
>> Cc: Thor Thayer <tthayer@...nsource.altera.com>
>> Cc: Lee Jones <lee.jones@...aro.org>
>> Signed-off-by: Paul Gortmaker <paul.gortmaker@...driver.com>
>> ---
>>  drivers/mfd/altera-a10sr.c | 14 ++++++--------
>>  1 file changed, 6 insertions(+), 8 deletions(-)
>
> Applied, thanks.

Hi Lee,

I never did see this applied copy of the patch loop around via linux-next
and conflict with my local copy of the same patch.  Same for 3,4,5,6 of
the same series (patch #2 was not applied; it was dropped in favour of
a redo as a tristate conversion.)

Just wanting to double check I didn't mess something up that caused them
to get silently dropped after the fact, before I prepare to send any additional
mfd patches to be considered for your 4.9 content queue.

Thanks,
Paul.
--

>
>> diff --git a/drivers/mfd/altera-a10sr.c b/drivers/mfd/altera-a10sr.c
>> index c05aa4ff57fd..78c2fb7d7f33 100644
>> --- a/drivers/mfd/altera-a10sr.c
>> +++ b/drivers/mfd/altera-a10sr.c
>> @@ -1,4 +1,8 @@
>>  /*
>> + * Altera Arria10 DevKit System Resource MFD Driver
>> + *
>> + * Author: Thor Thayer <tthayer@...nsource.altera.com>
>> + *
>>   * Copyright Intel Corporation (C) 2014-2016. All Rights Reserved
>>   *
>>   * This program is free software; you can redistribute it and/or modify it
>> @@ -20,7 +24,7 @@
>>
>>  #include <linux/mfd/altera-a10sr.h>
>>  #include <linux/mfd/core.h>
>> -#include <linux/module.h>
>> +#include <linux/init.h>
>>  #include <linux/of.h>
>>  #include <linux/spi/spi.h>
>>
>> @@ -152,7 +156,6 @@ static const struct of_device_id altr_a10sr_spi_of_match[] = {
>>       { .compatible = "altr,a10sr" },
>>       { },
>>  };
>> -MODULE_DEVICE_TABLE(of, altr_a10sr_spi_of_match);
>>
>>  static struct spi_driver altr_a10sr_spi_driver = {
>>       .probe = altr_a10sr_spi_probe,
>> @@ -161,9 +164,4 @@ static struct spi_driver altr_a10sr_spi_driver = {
>>               .of_match_table = of_match_ptr(altr_a10sr_spi_of_match),
>>       },
>>  };
>> -
>> -module_spi_driver(altr_a10sr_spi_driver);
>> -
>> -MODULE_LICENSE("GPL v2");
>> -MODULE_AUTHOR("Thor Thayer <tthayer@...nsource.altera.com>");
>> -MODULE_DESCRIPTION("Altera Arria10 DevKit System Resource MFD Driver");
>> +builtin_driver(altr_a10sr_spi_driver, spi_register_driver)
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog