[i2c] [PATCH 2/2] I2C: ISP1301_OMAP: New-style i2c driver updates, part 2
Felipe Balbi
felipebalbi at users.sourceforge.net
Tue Feb 26 10:22:12 CET 2008
On Tue, Feb 26, 2008 at 7:38 AM, David Brownell <david-b at pacbell.net> wrote:
> On Monday 25 February 2008, Felipe Balbi wrote:
> > Jean, I'm thinking about introducing include/asm/arch/isp1301.h
>
> Or maybe isp1301_omap.h to match the driver name.
>
>
>
> > so I
> > can introduce struct isp1301_platform_data to pass irq_flags to the
> > driver.
>
> Passing irq_flags??? That sounds like nonsense. Please explain ...
request_irq was using different irq flags depending on the board, so
instead of keeping if (machine_is_*) in the driver just because of irq
flags we could bring it from a platform_data.
>
>
> By the way, I see why there's no board-h2.c update. One of
> the patches in December stripped out most of the tps65010
> i2c_board info from mainline, replacing some of it with
> isp1301 stuff ... that can't possibly work in mainline.
>
> So there are going to be some H2 and H3 regression-fix
> patches upcoming...
that's true...
--
Best Regards,
Felipe Balbi
felipebalbi at users.sourceforge.net
More information about the i2c
mailing list