Re: [PATCH 0/3] staging: sm750fb: remove sii164 DVI controller support

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

 



On Thu, Oct 08, 2015 at 09:11:07PM +0100, Mike Rapoport wrote:
> Hi Sudip,
> 
> On Thu, Oct 08, 2015 at 11:46:28AM +0530, Sudip Mukherjee wrote:
> > On Tue, Oct 06, 2015 at 04:49:13PM +0100, Mike Rapoport wrote:
> > > Currently the sii164 DVI controller support is unused anywhere in the rest of
> > > the driver, and, anyway it does not belong to framebuffer device driver.
> > > These patches remove the sii164 DVI controller support.
> > I am not sure if we can remove sii164. Sil164 has two configuration
> > method. One is from i2c R/W and the other is hardware gpio setting.
> > 
> > Generally, we use hardware GPIO setting. So we don't need any driver for
> > sil164. And that is why you see it is unused here.
> > 
> > But SM750LE will always require swi2c for its dvi chip. And SM750LE is a
> > special version of the hardware which only Huawei uses.
> > 
> > And if i remember correctly sii164 code will be used if USE_DVICHIP and
> > USE_HW_I2C are not defined.
> 
> The code from ddk750_dvi and ddk750_sii164 is never called from the rest
> of the driver, the USE_DVICHIP is defined and used only by ddk750_dvi
> and ddk750_sii164 themself, and USE_HW_I2C is used only by
> ddk750_sii164.

Hmmm, I checked up the old code (v4.2) and the current one in staging.
sii164 will be initialized and would have been used by dviInit() and
dviInit() was being used by ddk750_initDVIDisp(). But ddk750_initDVIDisp()
has been removed by 86b5f4ed3525 ("staging: sm750fb: remove unused
ddk750_initDVIDisp function").
And the call to ddk750_initDVIDisp() was disabled by #ifdef 0 and that
was removed by 1282bade3b8e ("staging: sm750fb: remove dead code").

I am not sure what should be done in this situation. Greg, can you
please suggest something... I think we can add another config option for
SM750LE and reintroduce ddk750_initDVIDisp() (with a better name) and
enable building them if CONFIG_SM750LE is enabled.

I have also not checked the code on hardware for many days now as there
had been no hardware related changes. I will check it today and confirm
that dvi and vga both are working.

regards
sudip
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux