Re: Re: [PATCH] tdfxfb: move I2C functionality into the tdfxfb

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

 



Jean Delvare napisał(a):
> On 23 Mar 2009 12:01:33 +0100, krzysztof.h1@xxxxxxxxx wrote:
> > Jean Delvare napisał(a):
> > > On Sat, 21 Mar 2009 20:29:54 +0100, Krzysztof Helt wrote:
> > > > +	depends on FB_3DFX >> EXPERIMENTAL
> > > > +	select FB_DDC
> > > > +	default y
> > > > +	help
> > > > +	  Say Y here if you want DDC/I2C support for your 3dfx Voodoo3.
> > > 
> > > It might be worth adding a note that the driver itself doesn>#039;t
> yet take
> > > benefit of the DDC channel.
> > 
> > IHMO, it does not matter. A separate patch which make use of it is
> already
> > ready to post. If this patch is accepted, the second one will be posted.
> It 
> > is not worth to add a comment then remove it in the next patch.
> 
> Ah, very well then. I wasn't sure if you had already been working on it
> or if it was just a plan for the future. If you already have a patch
> implementing "full support" then I agree with you.
> 

It was work in progress when the first patch was posted. It is ready now.

> > > Next step is to deprecate i2c-voodoo3. This requires changes to
> > > drivers/i2c/busses/Kconfig and
> > > Documentation/feature-removal-schedule.txt. Are you going to do it or
> > > should I?
> > 
> > I don't care. First, this patch must be accepted.
> 
> It will be accepted, don't worry :)
> 
> I'll prepare the deprecation patch now, should be fairly trivial.
> 

Thanks.

Regards,
Krzysztof

----------------------------------------------------------------------
Szukasz pieniedzy? Wez podwojny limit zadluzenia w koncie direct.
>> http://link.interia.pl/f20a3

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux