From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guennadi Liakhovetski Date: Thu, 15 Dec 2011 22:29:29 +0000 Subject: Re: [PATCH 34/57] fbdev: sh_mobile_lcdc: Store the format in struct Message-Id: List-Id: References: <1323784972-24205-35-git-send-email-laurent.pinchart@ideasonboard.com> In-Reply-To: <1323784972-24205-35-git-send-email-laurent.pinchart@ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-fbdev@vger.kernel.org On Tue, 13 Dec 2011, Laurent Pinchart wrote: > Store the active format in the channel structure, and use it instead of > parsing info->var all over the place when the format is needed. Right, this is what I was wondering about, while looking at the previous patch:-) But: > > Signed-off-by: Laurent Pinchart > --- > drivers/video/sh_mobile_lcdcfb.c | 21 ++++++++++----------- > drivers/video/sh_mobile_lcdcfb.h | 4 +++- > 2 files changed, 13 insertions(+), 12 deletions(-) > > diff --git a/drivers/video/sh_mobile_lcdcfb.c b/drivers/video/sh_mobile_lcdcfb.c > index c6b6b9d..9829e01 100644 > --- a/drivers/video/sh_mobile_lcdcfb.c > +++ b/drivers/video/sh_mobile_lcdcfb.c [snip] > @@ -1350,6 +1345,8 @@ static int sh_mobile_set_par(struct fb_info *info) > info->fix.line_length = info->var.xres > * info->var.bits_per_pixel / 8; > > + ch->format = sh_mobile_format_info(sh_mobile_format_fourcc(&info->var)); Cannot this be NULL? As far as I could trace it back, I'm not sure with hotplug bits_per_pixel would be initialised correctly along the lines of sh_mobile_fb_reconfig(). Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/