public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
To: One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>
Cc: Greg KH <gregkh@linuxfoundation.org>,
	Dan Carpenter <dan.carpenter@oracle.com>,
	jdelvare@suse.de, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 WIP 3/3] i2c-parport: modify driver to use new parport device model
Date: Sun, 26 Apr 2015 10:34:24 +0530	[thread overview]
Message-ID: <20150426050424.GA5943@sudip-PC> (raw)
In-Reply-To: <20150425233247.3fe61f57@lxorguk.ukuu.org.uk>

On Sat, Apr 25, 2015 at 11:32:47PM +0100, One Thousand Gnomes wrote:
> > +static int parport;
> > +module_param(parport, int, 0);
> > +MODULE_PARM_DESC(parport, "The parport to be used, default parport0\n");
> 
> Minor suggested enhancement: this ought to be an array so you can attach
> multiple instances to a system.
ok. infact, i was looking at paride yesterday night, and it is also
having option to register maximum of 4 instances. the current patchset
allows registration of device to only one port. I need to change that
part.

regards
sudip


> 
> Alan

  reply	other threads:[~2015-04-26  5:04 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-25 13:07 [PATCH v3 WIP 1/3] parport: add device-model to parport subsystem Sudip Mukherjee
2015-04-25 13:07 ` [PATCH v3 WIP 2/3] staging: panel: modify driver to use new parport device model Sudip Mukherjee
2015-04-25 13:07 ` [PATCH v3 WIP 3/3] i2c-parport: " Sudip Mukherjee
2015-04-25 22:32   ` One Thousand Gnomes
2015-04-26  5:04     ` Sudip Mukherjee [this message]
2015-04-25 22:35 ` [PATCH v3 WIP 1/3] parport: add device-model to parport subsystem One Thousand Gnomes

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=20150426050424.GA5943@sudip-PC \
    --to=sudipm.mukherjee@gmail.com \
    --cc=dan.carpenter@oracle.com \
    --cc=gnomes@lxorguk.ukuu.org.uk \
    --cc=gregkh@linuxfoundation.org \
    --cc=jdelvare@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    /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