linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Steve Twiss <stwiss.opensource@diasemi.com>
To: Paul Gortmaker <paul.gortmaker@windriver.com>
Cc: Support Opensource <Support.Opensource@diasemi.com>,
	Lee Jones <lee.jones@linaro.org>,
	Eric Miao <eric.miao@marvell.com>,
	Mike Rapoport <mike@compulab.co.il>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 1/4] mfd: da903x: Make it explicitly non-modular
Date: Tue, 6 Jun 2017 15:27:50 +0000	[thread overview]
Message-ID: <6ED8E3B22081A4459DAC7699F3695FB7018CD92218@SW-EX-MBX02.diasemi.com> (raw)
In-Reply-To: <20170605192944.GI29293@windriver.com>

Hi Paul,

On 05 June 2017 20:30, Paul Gortmaker wrote:

> Subject: Re: [PATCH 1/4] mfd: da903x: Make it explicitly non-modular
> 
> [RE: [PATCH 1/4] mfd: da903x: Make it explicitly non-modular]
> On 05/06/2017 (Mon 10:30) Steve Twiss wrote:
> 
> > On 03 June 2017 14:04 Paul Gortmaker wrote:
> > > The Kconfig currently controlling compilation of this code is:
> > >
> > > drivers/mfd/Kconfig:config PMIC_DA903X
> > > drivers/mfd/Kconfig:    bool "Dialog Semiconductor DA9030/DA9034 PMIC
> > > Support"
> > >
> > > ...meaning that it currently is not being built as a module by anyone.
> >
> > With regards to your four patches,
> >
> >  [4/4] mfd: da9055-i2c: Make it explicitly non-modular	- - -	2017-06-03
> >  [3/4] mfd: da9055-core: make it explicitly non-modular	- - -	2017-06-03
> >  [2/4] mfd: da9052-*: Make it explicitly non-modular	- - -	2017-06-03
> >  [1/4] mfd: da903x: Make it explicitly non-modular	- - -	2017-06-03
> >
> > Is there an reason not to make the Kconfig tristate?
> > Would that not be simpler?
> 
> I explicitly covered that in the 0/4 e-mail:

Apologies, I don't seem to have been CC'ed on the 0/4 e-mail, although I
got the others. Dialog's developer e-mail addresses come as part of the
Support Opensource e-mail distribution list.

>    As always, the option exists for someone with the hardware and the desire
>    to extend the functionality to make any given driver tristate.  But given
>    the number of these tree wide and the fact that I can't test that new
>    extended functionality in all cases, I just make the code consistent with
>    existing Kconfig/Makefile settings that currently restrict them to "bool".

I see your point, however we have many customers and it is unclear whether
they are using modules for any of these drivers. 
Even if that feature in Kconfig is not enabled, it is possible a tristate change has
been made and is being used, but has not been pushed back into the Linux
mainline.

So I would recommend against removing this feature.

The driver code is being supported by Dialog Semiconductor, where possible.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/MAINTAINERS?h=v4.12-rc4#n3984
If there is a question about supporting modules in these drivers, we have the
ability to test on target for da9055/52.

Regards,
Steve

  reply	other threads:[~2017-06-06 15:28 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-03 13:03 [PATCH 0/4] mfd: make da90xx drivers explicitly non-modular Paul Gortmaker
2017-06-03 13:03 ` [PATCH 1/4] mfd: da903x: Make it " Paul Gortmaker
2017-06-05 10:30   ` Steve Twiss
2017-06-05 19:29     ` Paul Gortmaker
2017-06-06 15:27       ` Steve Twiss [this message]
2017-06-06 20:22         ` Paul Gortmaker
2017-06-07 14:36           ` Steve Twiss
2017-06-05 10:36   ` Lee Jones
2017-06-06 14:22     ` Paul Gortmaker
2017-06-03 13:03 ` [PATCH 2/4] mfd: da9052-*: " Paul Gortmaker
2017-06-03 13:03 ` [PATCH 3/4] mfd: da9055-core: make " Paul Gortmaker
2017-06-04  5:51   ` kbuild test robot
2017-06-06 14:26     ` Paul Gortmaker
2017-06-04  5:54   ` kbuild test robot
2017-06-03 13:03 ` [PATCH 4/4] mfd: da9055-i2c: Make " Paul Gortmaker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6ED8E3B22081A4459DAC7699F3695FB7018CD92218@SW-EX-MBX02.diasemi.com \
    --to=stwiss.opensource@diasemi.com \
    --cc=Support.Opensource@diasemi.com \
    --cc=eric.miao@marvell.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mike@compulab.co.il \
    --cc=paul.gortmaker@windriver.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).