Hi Maxime, On 15 May 2017 at 00:11, Maxime Ripard <maxime.rip...@free-electrons.com> wrote: > On Sun, May 14, 2017 at 09:03:19PM -0600, Simon Glass wrote: >> Hi, >> >> On 12 May 2017 at 10:06, Maxime Ripard <maxime.rip...@free-electrons.com> >> wrote: >> > Hi Jernej, >> > >> > The patch content looks fine, but there's a few things that would need >> > to be addressed. >> > >> > On Wed, May 10, 2017 at 06:46:30PM +0200, Jernej Skrabec wrote: >> >> This commit adds support for TV (composite) output. >> >> >> >> Signed-off-by: Jernej Skrabec <jernej.skra...@siol.net> >> >> --- >> >> >> >> arch/arm/include/asm/arch-sunxi/cpu_sun4i.h | 10 ++ >> >> arch/arm/include/asm/arch-sunxi/display2.h | 17 +++ >> >> arch/arm/include/asm/arch-sunxi/tve.h | 17 ++- >> >> drivers/video/sunxi/Makefile | 2 +- >> >> drivers/video/sunxi/sunxi_de2.c | 60 ++++++++--- >> >> drivers/video/sunxi/sunxi_tve.c | 156 >> >> ++++++++++++++++++++++++++++ >> >> drivers/video/sunxi/tve.c | 6 +- >> > >> > The difference between sunxi_tve and tve is not really obvious. What >> > about calling sunxi_tve tve-uclass, or something like that? >> >> That name is reserved for actual uclasses. > > Ok. How are the driver-part usually called?
Well, something without the word 'uclass', and ideally using underscores instead of hyphens. Regards, Simon -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.