Messages in this thread | | | Date | Tue, 26 Nov 2013 09:22:05 +0100 | From | Michal Simek <> | Subject | Re: [PATCH v3] video: add OpenCores VGA/LCD framebuffer driver |
| |
On 11/26/2013 09:19 AM, Geert Uytterhoeven wrote: > On Tue, Nov 26, 2013 at 8:10 AM, Michal Simek <monstr@monstr.eu> wrote: >> On 11/22/2013 05:34 AM, Stefan Kristiansson wrote: >> This construction seems to me too complicated and can be simpler. >> >> What about this? >> res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > > Don't forget to check for res being non-NULL here.
yes. you are right - you have to do it.
Thanks, Michal
-- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
[unhandled content-type:application/pgp-signature] | |