greybus: fix manifest parsing size bug

The type-specific "create" routines that get called while parsing
the descriptor entries in the module manifest assume the size they
are provided is the size of their data portion only--not including
the descriptor header.

Compute this value in greybus_new_module(), and pass it to those
functions rather than the full descriptor size.  Move a few
declarations to the innermost block that uses them.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

greybus: validate descriptor sizes

When interpreting a manifest descriptor header, don't assume there
is enough space in the buffer to hold a descriptor header.  Also,
verify the remaining buffer is at least as big as the reported
descriptor size.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

greybus: interpret descriptor type properly

The type field in a manifest descriptor header is in little endian
format.  Make sure we interpret it that way.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

greybus: call put_device() on error

As soon as we've called device_initialize() we're required to call
put_device() in order to drop our reference to the device structure.
This was missed in the error path in greybus_new_module().  Fix that.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

greybus: define struct greybus_manifest

Define a structure that describes the entire greybus manifest.
Adjust greybus_new_module() to use that, making it explicit that
it's not just a header that's being provided to that function.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

greybus: switch to the term "manifest"

We agreed to rename a few things to improve clarity.  This patch
implements one of those changes.  The blob of data that describes
what's relevant to Greybus within an Ara module will now be called
the "module manifest."  In addition, in the context of Greybus we'll
also be calling what's in an Ara module a "module" or "Greybus module."

So this patch renames some structures and updates some comments.  It
also renames "greybus_desc.h" to be "greybus_manifest.h", and renames
greybus_new_device() to be greybus_new_module().

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

greybus: es1: forgot to free our urb on disconnect

greybus: es1: finialized USB device structure

Set up device properly and start up the SVC interrupt in endpoint for
processing data

greybus: es1_ap_desc.c: updated ES1 USB device descriptor

Interrupt IN endpoint added.

greybus: minor whitespace cleanups to make checkpatch.pl happy

greybus: battery FIXME added

greybus: add battery module

greybus: first cut at parsing svc messages sent to the AP

greybus: more hd work

greybus: host controller additions

Also some gbuf functions starting to get fleshed out.

greybus: add es1_ap_desc.c to describe the ES1 USB device descriptors

greybus: uart-gb: remove unneeded THIS_MODULE setting

greybus: uart-gb.c: dynamically allocate device numbers

greybus: uart-gb.c: replace alloc_tty_driver with tty_alloc_driver

alloc_tty_driver() is deprecated.

Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>

greybus: add proper packing to all greybus message types

greybus: minor checkpatch cleanups

greybus: split sysfs functions out to separate file.

greybus: greybus_string()

greybus: module id attributes

greybus: serial number attribute added

greybus: sysfs attributes for functions and more driver core integration.

greybus: start parsing descriptor fields

greybus: start parsing descriptor structures

greybus: export gb_new_ap_msg so that the es1 module can use it

greybus: turn off warnings for es1-ap-usb.c to make it easier to build for now...

greybus: get field names right for descriptors

greybus: s/greybus_device_id/greybus_module_id/g

greybus: more changes due to name changes in the greybus document

greybus: greybus.h: tiny movement around

greybus: header file s/u8/__u8/g

greybus: greybus_desc.h created

greybus: split svc msg out into separate header file

greybus: more structure definitions added

greybus: structures added

greybus: ap message loop added.

greybus: register the bus with the driver core and add framework for debugfs files.

greybus: es1-ap-usb: more init framework added.

greybus: first framework for the es1 ap controller

greybus: actually get the devm() change to build...

greybus: can't use devm anymore, we aren't tieing into the driver model lifecycle :(

greybus: initial framework for ES1 usb AP driver

greybus: uart: tie into gb core properly

greybus: sdio: tie into gb core properly

greybus: gpio: tie into gb core properly

greybus: i2c: use same naming convention everywhere

greybus: i2c: tie to the proper place on the greybus_device

greybus: hook up sdio, gpio, and tty into the greybus core.

greybus: static module_init/exit functions

greybus: start moving the function types into the greybus core

greybus: uart-gb: a few minor bug fixes

Here are a few small bug fixes in uart-gb.c:
    - In wait_serial_change():
        - Return -EINVAL if *none* of the relevant flags are set in
          the "arg" parameter.
        - Balance the spin_lock_irq() with an unlock call (not
          another lock).
        - Rearrange a nested if structure (not a bug fix).
    - In tty_gb_probe():
        - Reset the greybus_device driver data in case of error.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

greybus: uart-gb: improve minor device number error checking

When alloc_minor() finds an available minor device number it
does not constrain the highest number desired.  Instead, it
relies on its caller, tty_gb_probe() to see if the returned
number indicates all minor numbers have been exhausted.

There are a couple problems with this--or rather with this
code.

First, if an allocation is attempted *after* GB_NUM_MINORS
is returned, a new number greater than (but not equal to)
GB_NUM_MINORS will be allocated, and that won't produce
any error condition.

Second, alloc_minor() can return an error code (like -ENOMEM).  And
its caller is only checking for GB_NUM_MINORS.  If an error code
is returned, tty_gb_probe() simply uses it.

Change alloc_minor() so it requests minor device numbers in the
range 0..(GB_NUM_MINORS-1), and use an error return to detect
when the minor device numbers have been exhausted.

If alloc_minor() returns -ENOSPC (from idr_alloc()), translate that
to -ENODEV.  The only other error we might see is -ENOMEM, and if
we get that, return it.

Finally, zero gb_tty->minor when it's released.  (If this is
actually important a reserved value like GB_NUM_MINORS should
be used instead to signify a gb_tty with no minor assigned.)

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

greybus: uart-gb: now builds, more framework added

greybus: more uart work

greybus: uart-gb: more work on tty functions

greybus: uart framework added, doesn't build

greybus: gpio-gb.c: it now builds properly

greybus: gpio driver

greybus: add framework for 'struct gbuf'

This is the equlivant of sk_buf or urbs for Greybus.

greybus: Greybus SD/MMC host driver

Need to add specifics, but this should be enough to hook up to the mmc
framework.

greybus: greybus.h: tiny coding style cleanups

greybus: i2c-gb: actually add the i2c adapter properly...

greybus: update README with info on how to build and contact me.

greybus: README and .gitignore updates

greybus: Import most recent greybus code to new repo.

greybus: Initial commit