Re: [PATCH 7/8] USB: gadget: g_multi: more configurable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Added Kconfig options for each
function used by g_multi so that
one can customize the gadget to a greater extend.

Note that it will be wise to change vendor and product ID

On Thu, 20 May 2010 18:22:54 +0200, David Brownell <david-b@xxxxxxxxxxx> wrote:
For the record .... I would *MUCH* rather see new gadgets than
pursue this path of tweaking/customizing existing ones and thus
removing/changing behaviors folk may rely on.

The problem with this approach is that if you have 4 USB functions you
can came up with 16 different gadgets (and that does not take into
consideration order of functions but lets ignore that).

A fully customizable gadget may let one tweak it and when he finally
settles for a set of functions develop a custom gadget (for instance by
hacking off parts of the g_multi).

I don't really expect g_multi will be widely used in its base form.  Most likely,
it will be hacked and slashed for one's needs so why not make it simple to do?

Tweaking is OK during certain development stages, but when it comes
to production, there should be a single version of the gadget driver
which isn't especially dependant on Kconfig options.  It should act the
same at all times.

On Thu, 20 May 2010 18:30:47 +0200, David Brownell <david-b@xxxxxxxxxxx> wrote:
Note that g_multi was intended to be a a fairly simple example,
so another reason I don't like seeing all those config options
is that it would no longer be a simple example ....

To be honest it was never my intend. :)  As a matter of fact, I'm
working on other features to add to g_multi, most notably at the
moment an "Install Mode" or "NoCD" mode.

In the end, I'd like to turn g_multi into a full flagged run-time
customizable composite gadget similar to Android's composite (except
better ;) ) so that one would be able to build her own composite
gadget in a matter of seconds without the need of calling all the
setup, cleanup, bind, etc. functions herself.

I see you did a bunch of good work on MS-Windows interop (yay!),
but that can be done without complexifying this driver.  Let some
other (new) driver take that role of "complex example"), but keep
the simple one so there's an easy approach to the easy concepts.

All in all, if I understand your point correctly, you have only
issue with the 7th patch which introduces Kconfig options.  All the
other patches seem OK, or do you have some comments on those as well?

--
Best regards,                                        _     _
| Humble Liege of Serenely Enlightened Majesty of  o' \,=./ `o
| Computer Science,  Michał "mina86" Nazarewicz       (o o)
+----[mina86*mina86.com]---[mina86*jabber.org]----ooO--(_)--Ooo--
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux