Re: [PATCH v2 1/3] video: clps711x: Add new Cirrus Logic CLPS711X framebuffer driver

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

 



Fri, 16 May 2014 15:56:26 -0700 от Olof Johansson <olof@xxxxxxxxx>:
> On Thu, May 08, 2014 at 01:14:40PM +0300, Tomi Valkeinen wrote:
> > On 08/05/14 11:27, Alexander Shiyan wrote:
> > 
> > >>> At this time the driver has three user.
> > >>> Only one of them should theoretically work.
> > >>> clps711x-autcpu12 should not work in the absence of memblock_reserve().
> > >>> clps711x-p720t should not work due to physical address limitation as i
> > >>> noticed before. Board means to use SRAM instead of SDRAM.
> > >>> Only clps711x-edb7211 should work fine (in theory).
> > >>> Is this a good reason to replace the driver? I think yes.
> > >>
> > >> Ok, if the situation is that bad, maybe we can just switch to the new
> > >> driver. Have you verified that those boards do not work from anyone? Or
> > >> asked someone to test the new driver with those boards?
> > > 
> > > I'm not familiar with other users of this platform .
> > > I am do not have these boards, all that I have written before that it's just a theory.
> > > Firm in which I work, uses its own board with CLPS711X CPU , this board is the
> > > only way to check for changes on real hardware .
> > 
> > Ok. That makes me a bit nervous... You're removing a driver, which may
> > (or may not) have been working for other users. And adding a new one,
> > which may not (or may) work for the other users.
> 
> Keep the old one around under another Kconfig name, mark it BROKEN,
> and if nobody speaks up in a couple of releases, remove it?

I like this variant, Tomi are you agree with this?

---

��.n��������+%������w��{.n�����{����n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�


[Index of Archives]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Tourism]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux