public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: "Pali Rohár" <pali@kernel.org>
To: Simon Glass <sjg@chromium.org>
Cc: Merlijn Wajer <merlijn@wizzup.org>,
	maemo-leste@lists.dyne.org,
	Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>,
	U-Boot Mailing List <u-boot@lists.denx.de>,
	Anatolij Gustschin <agust@denx.de>,
	Jagan Teki <jagan@amarulasolutions.com>,
	Andre Przywara <andre.przywara@arm.com>
Subject: Re: [PATCH 02/14] video: nokia_rx51: Drop obsolete video code
Date: Sun, 23 Jan 2022 15:57:45 +0100	[thread overview]
Message-ID: <20220123145745.72bmbb2stvg5d7kg@pali> (raw)
In-Reply-To: <CAPnjgZ2uzP2VnKcTphs+UguMbu=bAZ3CbZBOsktZrPLchKYSYQ@mail.gmail.com>

On Sunday 23 January 2022 07:36:22 Simon Glass wrote:
> Hi Pali,
> 
> On Sun, 23 Jan 2022 at 07:08, Pali Rohár <pali@kernel.org> wrote:
> >
> > + Maemo
> >
> > On Sunday 23 January 2022 07:04:03 Simon Glass wrote:
> > > Drop this code which uses a header that is about to be deleted.
> >
> > And what / where is the replacement?
> 
> This is DM_VIDEO. There are quite a few example drivers in
> drivers/video - perhaps the mxsfb.c one is a reasonable example. See
> the top of video_uclass.c for how frame-buffer allocation works.

I have already WIP patches for usage of video-uclass.c but because
reviewing of N900 patches is slow, I have not sent them yet.

So could you please do NOT remove N900 support? I would really
appreciative for reviewing pending patches instead of sending patches
with board removal.

Note that there is some issue with video_post_bind(), it throws
false-positive error "Video device '%s' cannot allocate frame buffer
memory" with "return -ENOSPC". If I remove that "return -ENOSPC" it is
working fine.

  reply	other threads:[~2022-01-23 14:57 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-23 14:04 [PATCH 00/14] video: Drop old CFB code Simon Glass
2022-01-23 14:04 ` [PATCH 01/14] video: Drop cfg_console Simon Glass
2022-01-23 14:04 ` [PATCH 02/14] video: nokia_rx51: Drop obsolete video code Simon Glass
2022-01-23 14:08   ` Pali Rohár
2022-01-23 14:36     ` Simon Glass
2022-01-23 14:57       ` Pali Rohár [this message]
2022-01-23 15:54         ` Simon Glass
2022-01-23 16:01           ` Pali Rohár
2022-01-23 20:13             ` Simon Glass
2022-01-23 14:04 ` [PATCH 03/14] video: siemens: Drop unused " Simon Glass
2022-01-23 14:04 ` [PATCH 04/14] video: nexell: Drop unused and invalid code Simon Glass
2022-01-31 16:57   ` Stefan Bosch
2022-02-11 15:05     ` Simon Glass
2022-01-23 14:04 ` [PATCH 05/14] video: Drop video_fb header Simon Glass
2022-01-23 14:04 ` [PATCH 06/14] video: Drop CONFIG_VIDEO_BMP_LOGO Simon Glass
2022-01-23 14:04 ` [PATCH 07/14] video: Drop references to CONFIG_VIDEO et al Simon Glass
2022-01-26  7:30   ` Jason Liu
2022-01-23 14:04 ` [PATCH 08/14] video: Clean up the uclass header Simon Glass
2022-01-23 14:04 ` [PATCH 09/14] video: Drop da8xx-fb Simon Glass
2022-01-23 14:04 ` [PATCH 10/14] video: fsl: colibri_vf: Drop FSL DCU driver Simon Glass
2022-01-23 14:04 ` [PATCH 11/14] video: Drop FSL DIU driver Simon Glass
2022-01-23 14:04 ` [PATCH 12/14] video: mxs: Drop old video code Simon Glass
2022-01-23 14:04 ` [PATCH 13/14] video: Convert CONFIG_VIDEO_BCM2835 to Kconfig Simon Glass
2022-01-24 12:10   ` Matthias Brugger
2022-01-23 14:04 ` [PATCH 14/14] video: Drop formike driver Simon Glass
2022-03-24 19:54 ` [PATCH 00/14] video: Drop old CFB code Simon Glass
2022-03-24 19:56   ` Pali Rohár
2022-03-24 20:05     ` Simon Glass
2022-03-24 22:16   ` Anatolij Gustschin
2022-03-24 22:41     ` Simon Glass
2022-03-28  7:50       ` Anatolij Gustschin
2022-03-29  6:37 ` Anatolij Gustschin

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=20220123145745.72bmbb2stvg5d7kg@pali \
    --to=pali@kernel.org \
    --cc=agust@denx.de \
    --cc=andre.przywara@arm.com \
    --cc=ivo.g.dimitrov.75@gmail.com \
    --cc=jagan@amarulasolutions.com \
    --cc=maemo-leste@lists.dyne.org \
    --cc=merlijn@wizzup.org \
    --cc=sjg@chromium.org \
    --cc=u-boot@lists.denx.de \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox