[U-Boot] [PATCH v2 00/25] exynos: video: Convert exynos LCD driver to use driver model

Simon Glass
2016-05-23T20:18:57Z

Hi Minkyu,

On 16 May 2016 at 22:38, Minkyu Kang <mk7.kang@samsung.com> wrote:
> Hi,
>
> On 17/05/16 13:22, Jaehoon Chung wrote:
>> Hi Simon.
>>
>> On 05/15/2016 06:22 AM, Simon Glass wrote:
>>> Hi Jaehoon,
>>>
>>> On 13 May 2016 at 07:09, Jaehoon Chung <jh80.chung@samsung.com> wrote:
>>>> Hi Simon,
>>>>
>>>> On 05/09/2016 08:00 PM, Minkyu Kang wrote:
>>>>> Hi Jaehoon,
>>>>>
>>>>> On 08/05/16 06:25, Simon Glass wrote:
>>>>>> Hi,
>>>>>>
>>>>>> On 1 March 2016 at 18:51, Minkyu Kang <mk7.kang@samsung.com> wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> On 29/02/16 16:18, Jaehoon Chung wrote:
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> On 02/29/2016 01:23 PM, Simon Glass wrote:
>>>>>>>>> Hi,
>>>>>>>>>
>>>>>>>>> On 23 February 2016 at 05:04, Minkyu Kang <mk7.kang@samsung.com> wrote:
>>>>>>>>>> Dear Simon Glass,
>>>>>>>>>>
>>>>>>>>>> On 23/02/16 15:42, Simon Glass wrote:
>>>>>>>>>>> Hi Minkyu,
>>>>>>>>>>>
>>>>>>>>>>> On 21 February 2016 at 21:08, Simon Glass <sjg@chromium.org> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>> This series converts the exynos LCD driver to work with driver model. Only
>>>>>>>>>>>> the eDP display is converted - the MIPI display is left mostly alone as I
>>>>>>>>>>>> do not have a device to test with.
>>>>>>>>>>>>
>>>>>>>>>>>> The conversion involves some refactoring to simplify the code, reduce the
>>>>>>>>>>>> amount of global/static data and allow boards to be built without LCD
>>>>>>>>>>>> support.
>>>>>>>>>>>>
>>>>>>>>>>>> The series is tested with snow, spring, pit and pi. It disables LCD on other
>>>>>>>>>>>> exynos boards so that they will still build with these changes. Further work
>>>>>>>>>>>> will be needed (by someone with a board) to convert the others. I am not
>>>>>>>>>>>> sure when this series can be applied.
>>>>>>>>>>>>
>>>>>>>>>>>> The series also brings in device tree features from Linux v4.4. It would be
>>>>>>>>>>>> useful to do a full sync with the kernel on these files, but this is really
>>>>>>>>>>>> a job for the Samsung maintainer :-)
>>>>>>>>>>>>
>>>>>>>>>>>> The series is available at u-boot-dm/rkg-working.
>>>>>>>>>>>>
>>>>>>>>>>>> Changes in v2:
>>>>>>>>>>>> - Rebase to master
>>>>>>>>>>>> - Rebase to master
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> What do you think we should do about applying this series?
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Although we have to do remaining work, I think applying is possible without next work.
>>>>>>>>>>
>>>>>>>>>> Board maintainers, what do you think?
>>>>>>>>>>
>>>>>>>>>> Remaining Work:
>>>>>>>>>> - MIPI display driver
>>>>>>>>>> - s5pc210_universal
>>>>>>>>>> - smdk5420
>>>>>>>>>> - smdk5250
>>>>>>>>>> - trats
>>>>>>>>>> - trats2
>>>>>>>>>>
>>>>>>>>>> Thanks,
>>>>>>>>>> Minkyu Kang.
>>>>>>>>>
>>>>>>>>> Well, no replies as yet. The main work requires is MIPI, but as I
>>>>>>>>> don't have the hardware, I can't help with that. At least I have made
>>>>>>>>> sure that everything builds OK.
>>>>>>>>
>>>>>>>> Sorry for late..I will check these patches on this week..Tomorrow is holiday..
>>>>>>>> So I can check from Wed.
>>>>>>>
>>>>>>> Thank you, Jaehoon.
>>>>>>
>>>>>> It has been a few months now - if you like I can apply these via the -dm tree.
>>>>
>>>> Yep, I checked these..It needs to remain the future work.
>>>> But it seems that it's not problem to apply at -dm repository.
>>>> (I have checked with your patch on other boadrs.)
>>>>
>>>> I needs to check one more things..Could you wait until next Tuesday?
>>>>
>>>
>>> Sure, it's been 3 months so another week won't matter :-)
>>
>> Sorry. I have checked more with these patches..We have to work too many things in future for remaining work.
>> But it seems those task are our role..If you and Minkyu are ok, you can apply these at dm repository.
>
> I'm OK.
> If you want, those can be merged to samsung tree.

Thanks, please go ahead!

>
>>
>> I will make a plan to work. (Based on your repository.)
>>
>> Actually, i didn't follow a few month, so i'm looking for u-boot history. Sorry. :(
>> If you want to add my Reviewed-by tag, then you can add.
>>
>> In future work,
>> Change to driver model for above targets. (trats2/trats...)
>> MIPI display driver..(I will assign to my colleague for this task.)

Regards,
Simon