From: Sudip Mukherjee <sudipm.mukherjee@gmail.com> To: Yifeng Li <tomli@tomli.me> Cc: Teddy Wang <teddy.wang@siliconmotion.com>, linux-kernel@vger.kernel.org, Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH v2 6/7] fbdev: sm712fb: Kconfig: add information about docs. Date: Sun, 31 Mar 2019 20:01:18 +0100 [thread overview] Message-ID: <20190331190118.3kaf7erdbk25yd5n@debian> (raw) In-Reply-To: <20190322051759.15007-7-tomli@tomli.me> On Fri, Mar 22, 2019 at 01:17:58PM +0800, Yifeng Li wrote: > This commits update the Kconfig description of sm712fb, > and inform the user about the existence of documentation. > > Signed-off-by: Yifeng Li <tomli@tomli.me> > --- > Documentation/fb/sm712fb.txt | 44 ++++++++++++++++++++---------------- > drivers/video/fbdev/Kconfig | 4 ++++ > 2 files changed, 29 insertions(+), 19 deletions(-) > > diff --git a/Documentation/fb/sm712fb.txt b/Documentation/fb/sm712fb.txt > index 906b48aa40e4..c9a8fe059ac7 100644 > --- a/Documentation/fb/sm712fb.txt > +++ b/Documentation/fb/sm712fb.txt > @@ -18,9 +18,10 @@ and it was also an inexpensive platform for non-x86 hobbyists to explore. > How to use it? > ============== > > -You should not compile-in vesafb, since SM7xx can be used in a VGA > -compatible mode, resulting conflicts with this driver. In addition, > -the VGA compatible mode was never tested by the maintainers. > +SM7xx can be used in a VGA compatible mode and a native framebuffer mode, > +this driver is a native driver. By default, it has no problem taking over > +the VGA driver automatically, but you should not attempt to use both at > +the same time. You added this in the documentation with just the previous patch. > <snip> > > 2D acceleration > ============== > @@ -97,7 +103,7 @@ Missing Features > > The following features are not implemented in this driver, > > - * 2D acceleration on SM720 and Big-Endian CPUs. > + * 2D acceleration on Big-Endian CPUs. > * More VGA modes. > * dual-head support > * hardware cursor support You might want to squash the two patches for documentation instead of having a separate patch to fix what you added in a previous patch. -- Regards Sudip
WARNING: multiple messages have this Message-ID (diff)
From: Sudip Mukherjee <sudipm.mukherjee@gmail.com> To: Yifeng Li <tomli@tomli.me> Cc: Teddy Wang <teddy.wang@siliconmotion.com>, linux-kernel@vger.kernel.org, Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH v2 6/7] fbdev: sm712fb: Kconfig: add information about docs. Date: Sun, 31 Mar 2019 19:01:18 +0000 [thread overview] Message-ID: <20190331190118.3kaf7erdbk25yd5n@debian> (raw) In-Reply-To: <20190322051759.15007-7-tomli@tomli.me> On Fri, Mar 22, 2019 at 01:17:58PM +0800, Yifeng Li wrote: > This commits update the Kconfig description of sm712fb, > and inform the user about the existence of documentation. > > Signed-off-by: Yifeng Li <tomli@tomli.me> > --- > Documentation/fb/sm712fb.txt | 44 ++++++++++++++++++++---------------- > drivers/video/fbdev/Kconfig | 4 ++++ > 2 files changed, 29 insertions(+), 19 deletions(-) > > diff --git a/Documentation/fb/sm712fb.txt b/Documentation/fb/sm712fb.txt > index 906b48aa40e4..c9a8fe059ac7 100644 > --- a/Documentation/fb/sm712fb.txt > +++ b/Documentation/fb/sm712fb.txt > @@ -18,9 +18,10 @@ and it was also an inexpensive platform for non-x86 hobbyists to explore. > How to use it? > ======= > > -You should not compile-in vesafb, since SM7xx can be used in a VGA > -compatible mode, resulting conflicts with this driver. In addition, > -the VGA compatible mode was never tested by the maintainers. > +SM7xx can be used in a VGA compatible mode and a native framebuffer mode, > +this driver is a native driver. By default, it has no problem taking over > +the VGA driver automatically, but you should not attempt to use both at > +the same time. You added this in the documentation with just the previous patch. > <snip> > > 2D acceleration > ======= > @@ -97,7 +103,7 @@ Missing Features > > The following features are not implemented in this driver, > > - * 2D acceleration on SM720 and Big-Endian CPUs. > + * 2D acceleration on Big-Endian CPUs. > * More VGA modes. > * dual-head support > * hardware cursor support You might want to squash the two patches for documentation instead of having a separate patch to fix what you added in a previous patch. -- Regards Sudip
next prev parent reply other threads:[~2019-03-31 19:01 UTC|newest] Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-03-22 5:17 [PATCH v2 0/7] implement 2D acceleration, minor cleanups, doc updates Yifeng Li 2019-03-22 5:17 ` Yifeng Li 2019-03-22 5:17 ` [PATCH v2 1/7] fbdev: sm712fb: use type "u8" for 8-bit I/O Yifeng Li 2019-03-22 5:17 ` Yifeng Li 2019-03-31 17:16 ` Sudip Mukherjee 2019-03-31 17:16 ` Sudip Mukherjee 2019-03-31 17:16 ` Sudip Mukherjee 2019-03-22 5:17 ` [PATCH v2 2/7] fbdev: sm712fb: add 2D-related I/O headers and functions Yifeng Li 2019-03-22 5:17 ` Yifeng Li 2019-03-31 17:25 ` Sudip Mukherjee 2019-03-31 17:25 ` Sudip Mukherjee 2019-04-01 16:04 ` Tom Li 2019-04-01 16:04 ` Tom Li 2019-03-22 5:17 ` [PATCH v2 3/7] fbdev: sm712fb: support 2D acceleration on SM712 w/ Little-Endian CPU Yifeng Li 2019-03-22 5:17 ` Yifeng Li 2019-03-31 18:09 ` Sudip Mukherjee 2019-03-31 18:09 ` Sudip Mukherjee 2019-04-01 16:26 ` Tom Li 2019-04-01 16:26 ` Tom Li 2019-04-02 20:53 ` Sudip Mukherjee 2019-04-02 20:53 ` Sudip Mukherjee 2019-03-22 5:17 ` [PATCH v2 4/7] fbdev: sm712fb: add 32-bit color modes, drops some other modes Yifeng Li 2019-03-22 5:17 ` Yifeng Li 2019-03-31 18:33 ` Sudip Mukherjee 2019-03-31 18:33 ` Sudip Mukherjee 2019-04-01 16:41 ` Tom Li 2019-04-01 16:41 ` Tom Li 2019-04-02 20:59 ` Sudip Mukherjee 2019-04-02 20:59 ` Sudip Mukherjee 2019-03-22 5:17 ` [PATCH v2 5/7] Documentation: fb: sm712fb: add information mainly about 2D Yifeng Li 2019-03-22 5:17 ` Yifeng Li 2019-03-31 18:54 ` Sudip Mukherjee 2019-03-31 18:54 ` Sudip Mukherjee 2019-03-31 18:54 ` Sudip Mukherjee 2019-04-01 17:30 ` Tom Li 2019-04-01 17:30 ` Tom Li 2019-04-02 21:03 ` Sudip Mukherjee 2019-04-02 21:03 ` Sudip Mukherjee 2019-04-02 21:03 ` Sudip Mukherjee 2019-03-22 5:17 ` [PATCH v2 6/7] fbdev: sm712fb: Kconfig: add information about docs Yifeng Li 2019-03-22 5:17 ` Yifeng Li 2019-03-31 19:01 ` Sudip Mukherjee [this message] 2019-03-31 19:01 ` Sudip Mukherjee 2019-04-01 17:33 ` Tom Li 2019-04-01 17:33 ` Tom Li 2019-03-22 5:17 ` [PATCH v2 7/7] MAINTAINERS: sm712fb: list myself as one maintainer Yifeng Li 2019-03-22 5:17 ` Yifeng Li 2019-03-31 19:08 ` Sudip Mukherjee 2019-03-31 19:08 ` Sudip Mukherjee 2019-03-31 19:08 ` Sudip Mukherjee 2019-04-01 17:41 ` Tom Li 2019-04-01 17:41 ` Tom Li 2019-04-02 21:09 ` Sudip Mukherjee 2019-04-02 21:09 ` Sudip Mukherjee 2019-04-02 21:09 ` Sudip Mukherjee 2019-04-03 13:53 ` Bartlomiej Zolnierkiewicz 2019-04-03 13:53 ` Bartlomiej Zolnierkiewicz 2019-04-05 22:11 ` Sudip Mukherjee 2019-04-05 22:11 ` Sudip Mukherjee 2019-04-05 22:11 ` Sudip Mukherjee
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20190331190118.3kaf7erdbk25yd5n@debian \ --to=sudipm.mukherjee@gmail.com \ --cc=b.zolnierkie@samsung.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=linux-fbdev@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=teddy.wang@siliconmotion.com \ --cc=tomli@tomli.me \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.