Re: [PATCH RESEND] display: panel: Add KOE tx14d24vm1bpa display support (320x240)

[Lukasz Majewski]

Hi Thierry,

> On Tue, May 29, 2018 at 05:01:48PM +0200, Lukasz Majewski wrote:
> > Hi Thierry,
> >   
> > > On Mon, May 28, 2018 at 09:55:19AM +0200, Lukasz Majewski wrote:  
> > > > Hi,
> > > >     
> > > > > Hi Thierry,
> > > > >     
> > > > > > This commit adds support for KOE's 5.7" display.
> > > > > >       
> > > > > 
> > > > > Thierry, shall I perform some more work on this code, or is it
> > > > > eligible for applying to your tree?    
> > > > 
> > > > Gentle ping. If Thierry is overworked - maybe there is a
> > > > co-maintainer so he/she could apply this patch?    
> > > 
> > > Please use the proper prefix for the commit subject to increase
> > > the chances of this being noticed.  
> > 
> > Ok. Is there any list of prefixes in the kernel repository, so I
> > could look for them (like get_prefix.py - similar to get_maintainer
> > script)?  
> 
> I don't think there is. A good rule of thumb that I use is to go over
> the git log for the last couple of commits and see if there's a clear
> pattern. This doesn't work for every subsystem, but drm/panel is very
> consistent in this regard, on purpose.

I see.

Is the DRM/panel tree hosted on git.kernel.org?

The tree maintained by you there seems to be:
kernel/git/thierry.reding/linux-pwm.git

IIRC it is hosted elsewhere. Am I right?

> 
> > I've used "display: panel" prefix, but I should have used
> > "drm/panel" 
> > > 
> > > This is also still missing a Reviewed-by or Acked-by from Rob.  
> > 
> > Rob has already reviewed this patch. I will send v2
> > with Rob's Reviewed-by tag.  
> 
> Okay, I'll apply that v2 then.

Thanks.

> 
> Thanks,
> Thierry




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@xxxxxxx

Attachment: pgpWoKen0syge.pgp
Description: OpenPGP digital signature