[PATCH 0/1] ST DDC I2C U300 bus driver v2

[PATCH 0/1] ST DDC I2C U300 bus driver v2

[Linus Walleij]

Updated patch for this driver, please tell me if it's good enough
enough for mainlining now...

Changelog v1->v2:

* Renamed from i2c-stddci2c to i2c-stu300, also in function
  prefixes and module name (so as not to confuse when tracing
  kernel calls).

* Removed all unnecessary #include <> directives.

* Replaced 8bit accessor macros with inline functions.

* Replaced all pr_debug() with proper dev_dbg() calls.

* Moved clock settings over to a table lookup.

* Introduced more documentation especially of state machine
  events. Described as far as the documentation goes.

* Clarified that zero-length messages are unsupported.

* Introduced more linespacing in terse sections to ease
  readability.

Yours,
Linus Walleij

Re: [PATCH 0/1] ST DDC I2C U300 bus driver v2

[Linus Walleij]

2009/5/6 Linus Walleij <linus.ml.walleij-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>:

> Updated patch for this driver, please tell me if it's good enough
> enough for mainlining now...

No reply means yes I assume :-)

Do I send this directly to Jean for integration?

Linus

Re: [PATCH 0/1] ST DDC I2C U300 bus driver v2

[Jean Delvare]

On Fri, 8 May 2009 10:17:58 +0200, Linus Walleij wrote:
> 2009/5/6 Linus Walleij <linus.ml.walleij-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>:
> 
> > Updated patch for this driver, please tell me if it's good enough
> > enough for mainlining now...
> 
> No reply means yes I assume :-)

Wrong assumption. No reply means we are busy. In other words: be
patient.

> Do I send this directly to Jean for integration?

Doesn't the platform this driver is aimed at qualify as embedded? I
doubt this driver is something for me.

-- 
Jean Delvare

Re: [PATCH 0/1] ST DDC I2C U300 bus driver v2

[Linus Walleij]

2009/5/8 Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>:

>> No reply means yes I assume :-)
>
> Wrong assumption. No reply means we are busy. In other words: be
> patient.

Aw sorry man. I still need to learn patience, I know.

> Doesn't the platform this driver is aimed at qualify as embedded? I
> doubt this driver is something for me.

It does, so Ben is the integrator then I assume.

Linus

Re: [PATCH 0/1] ST DDC I2C U300 bus driver v2

[Jean Delvare]

On Fri, 8 May 2009 14:36:30 +0200, Linus Walleij wrote:
> 2009/5/8 Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>:
> 
> >> No reply means yes I assume :-)
> >
> > Wrong assumption. No reply means we are busy. In other words: be
> > patient.
> 
> Aw sorry man. I still need to learn patience, I know.
> 
> > Doesn't the platform this driver is aimed at qualify as embedded? I
> > doubt this driver is something for me.
> 
> It does, so Ben is the integrator then I assume.

Correct. Or possibly Wolfram if Ben is too busy.

-- 
Jean Delvare

Re: [PATCH 0/1] ST DDC I2C U300 bus driver v2

[Ben Dooks]

On Fri, May 08, 2009 at 02:36:30PM +0200, Linus Walleij wrote:
> 2009/5/8 Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>:
> 
> >> No reply means yes I assume :-)
> >
> > Wrong assumption. No reply means we are busy. In other words: be
> > patient.
> 
> Aw sorry man. I still need to learn patience, I know.
> 
> > Doesn't the platform this driver is aimed at qualify as embedded? I
> > doubt this driver is something for me.
> 
> It does, so Ben is the integrator then I assume.

Always post the patch you want integrating to the
relevant mailing list. In my case, a direct CC doesn't
hurt.

I'll be reviewing patches for the next kernel closer to
the end of the week as I've still got some other arch
fixing to do.

-- 
Ben (ben-elnMNo+KYs3YtjvyW6yDsg@public.gmane.org, http://www.fluff.org/)

  'a smiley only costs 4 bytes'

Re: [PATCH 0/1] ST DDC I2C U300 bus driver v2

[Linus Walleij]

2009/5/12 Ben Dooks <ben-linux-elnMNo+KYs3YtjvyW6yDsg@public.gmane.org>:

> I'll be reviewing patches for the next kernel closer to
> the end of the week as I've still got some other arch
> fixing to do.

I know you're busy, sorry add to that burden... Can Wolfram
look at this instead perhaps?

Linus