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? Thanks, Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
