On Tue, Mar 27, 2018 at 09:48:23AM +0200, Arnd Bergmann wrote: > On Tue, Mar 27, 2018 at 4:21 AM, Guo Ren <ren_guo@xxxxxxxxx> wrote: > > On Mon, Mar 26, 2018 at 03:16:31PM +0200, Arnd Bergmann wrote: > > >> > +CONFIG_CFG80211=y > >> > +CONFIG_CFG80211_DEBUGFS=y > >> > +CONFIG_CFG80211_WEXT=y > >> > >> I would guess you want IPV6 but not WEXT here. > > Gx6605s devlepment board support usb-wifi. > > Perhaps WEXT is needed by iwconfig or iwlist? So I just enable it. > > However, I'll consider to remove it. > > I think it's only for backwards compatibility with old versions of > those tools. There are some older drivers that use WEXT as the > native interface, but CFG80211 based drivers have a netlink > interface. Thx for remind. I'll check. > > >> > +CONFIG_FB=y > >> > +CONFIG_FB_TILEBLITTING=y > >> > +CONFIG_FB_SIMPLE=y > >> > +CONFIG_BACKLIGHT_LCD_SUPPORT=y > >> > +# CONFIG_LCD_CLASS_DEVICE is not set > >> > +# CONFIG_BACKLIGHT_CLASS_DEVICE is not set > >> > +# CONFIG_VGA_CONSOLE is not set > >> > +CONFIG_FRAMEBUFFER_CONSOLE=y > >> > +CONFIG_FRAMEBUFFER_CONSOLE_DETECT_PRIMARY=y > >> > +CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y > >> > +CONFIG_LOGO=y > >> > +# CONFIG_LOGO_LINUX_MONO is not set > >> > +# CONFIG_LOGO_LINUX_VGA16 is not set > >> > >> For new platforms, using the DRM subsystem is the recommend > >> way to do graphics, it mostly replaces the framebuffer subsystem > >> here. > > These used by: > > https://github.com/c-sky/addons-linux/tree/master/addons/drivers/video/fbdev/nationalchip > > > > I'll consider the DRM subsystem. > > It used to be rather hard to write a new DRM driver, but that has > improved a lot recently. You might be able to fit your driver into > drivers/gpu/drm/tinydrm/, or maybe do something a bit more complex > like drivers/gpu/drm/pl111/ or drivers/gpu/drm/stm/. For this patchset, I won't upstream the fbdev driver. I'll remove the CONFIG_FB in defconfig first. Thx for remind about the develop-tips, and it's very helpful. Best Regards Guo Ren