Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1475255 > unrolled thread

Re: [RESEND PATCH] ARM: multi_v7_defconfig: Build Atmel maXTouch driver as a module

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [RESEND PATCH] ARM: multi_v7_defconfig: Build Atmel maXTouch driver as a module Arnd Bergmann <arnd@arndb.de> - 2016-09-02 17:40 +0200
    Re: [RESEND PATCH] ARM: multi_v7_defconfig: Build Atmel maXTouch  driver as a module Javier Martinez Canillas <javier@osg.samsung.com> - 2016-09-05 01:40 +0200

#1475255 — Re: [RESEND PATCH] ARM: multi_v7_defconfig: Build Atmel maXTouch driver as a module

On Tuesday, August 16, 2016 11:08:14 AM CEST Javier Martinez Canillas wrote:
> The driver is for a trackpad device so is not needed for booting and
> makes more sense to have it as module to reduce the kernel image size.
> 
> It was probably enabled as built-in because module autoload was not
> working when the I2C device was registered by OF but this got fixed
> in commit b7d21058b40b ("Input: atmel_mxt_ts - add maxtouch to I2C
> table for module autoload") so it's safe to enable as a module now.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> 
> ---

Applied to next/defconfig.

My preferred way is to get defconfig updates forwarded by the platform
maintainers that are interested, but this seems to apply to multiple
platforms, so I'm not asking you to pick one to resend it to.

	Arnd

[toc] | [next] | [standalone]

#1476053 — Re: [RESEND PATCH] ARM: multi_v7_defconfig: Build Atmel maXTouch driver as a module

Hello Arnd,

On 09/02/2016 05:38 PM, Arnd Bergmann wrote:
> On Tuesday, August 16, 2016 11:08:14 AM CEST Javier Martinez Canillas wrote:
>> The driver is for a trackpad device so is not needed for booting and
>> makes more sense to have it as module to reduce the kernel image size.
>>
>> It was probably enabled as built-in because module autoload was not
>> working when the I2C device was registered by OF but this got fixed
>> in commit b7d21058b40b ("Input: atmel_mxt_ts - add maxtouch to I2C
>> table for module autoload") so it's safe to enable as a module now.
>>
>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
>>
>> ---
> 
> Applied to next/defconfig.
> 

Thanks.

> My preferred way is to get defconfig updates forwarded by the platform
> maintainers that are interested, but this seems to apply to multiple
> platforms, so I'm not asking you to pick one to resend it to.
> 

Yes. I usually route defconfig patches through a platform maintainer, but as
you said, this driver is used by different platforms so I decided to send it
to you directly since I didn't know which platform to choose.

Sorry if that caused inconveniences.

> 	Arnd
> 

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web