From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Date: Wed, 26 Jun 2013 12:00:36 +0000 Subject: Re: [PATCH 2/2] video:da8xx-fb: Convert to devm_* api Message-Id: <51CAD7E4.2000403@ti.com> MIME-Version: 1 Content-Type: multipart/mixed; boundary="----enig2IEJDLSRPTTVQWJLOXTCJ" List-Id: References: <1368688257-18705-1-git-send-email-prabhakar.csengg@gmail.com> <1368688257-18705-3-git-send-email-prabhakar.csengg@gmail.com> <51CAD305.30407@ti.com> In-Reply-To: To: Prabhakar Lad Cc: "Etheridge, Darren" , DLOS , LFBDEV , Florian Tobias Schandinat , LKML ------enig2IEJDLSRPTTVQWJLOXTCJ Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 26/06/13 14:56, Prabhakar Lad wrote: > Hi Tomi, >=20 > On Wed, Jun 26, 2013 at 5:09 PM, Tomi Valkeinen = wrote: >> On 16/05/13 10:10, Lad Prabhakar wrote: >>> From: Lad, Prabhakar >>> >>> Use devm_ioremap_resource instead of reques_mem_region()/ioremap() an= d >>> devm_request_irq() instead of request_irq(). >>> >>> This ensures more consistent error values and simplifies error paths.= >>> >>> Signed-off-by: Lad, Prabhakar >>> --- >>> drivers/video/da8xx-fb.c | 39 +++++++-----------------------------= --- >>> 1 files changed, 7 insertions(+), 32 deletions(-) >> >> There's something similar in Darren's da8xx-fb series. Can you check i= f >> there are differences? >> > I don't see similar changes in his patch series. [PATCH 14/23] video: da8xx-fb: use devres Tomi ------enig2IEJDLSRPTTVQWJLOXTCJ Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJRytfkAAoJEPo9qoy8lh71AsEP/1sQaYeYGmwiI+kmVmjKJS0W YNDzHBtcZw+oB6on32pd+16+XPw+r2eLSRjz4L41VAbeQ5Dnq3sKSikCTomdRVKt +OtcsSgrrORQf6Le2PhXECaUa663HG2m24di6ijDrnGAeHfsIVZUwtU/LaldD/Ea TXWtv/GTm4QcZee6dgriTQACEHCjTgaXND+NHz3SxE4uFvqcardMe/2Gjutub602 L7Rj7l3M9hWMShR9npkJXeT6CUl1jzHly4T6u3LAZHYwZ1umEkNDe4XfZwOIKsNj KHdsWvOu2Kh8rjAqzstogrLVyUjjDMG12E/Bfhs814s9ctAlLey9VnoAraHbNkM1 uLum/DX4MiG6ylmWHuynUP68rMjkfW4hWUjI/t2xIrucP+SHhH0VXqv0PnYC+pqf VCuT2bCJr3YtP06wzmUcSjgJemTGDvftDoJflvpY1wqTpA+lzQ712S7u2lVjh3K+ BQzXWa3Y6pxEIMn0V7bWoVK5AiFPNpTZhCUcpVlfPaBsc9myJ7yE4ZYKfZx1mVqV +9Bh16YFedXVqRvwD9UdVvqPFYKo1EiTiWbXu6BDpXDjcrP+68XL/Dtq9jarUDQ1 x8FJYv6RJC65pHUllX0yMr1oOTJzSMsgfn5sVBJbD00XY4Zq+5MwQ+9vmjPSnYzu oh3y0EgfA+x90WaUbWRY =oA6T -----END PGP SIGNATURE----- ------enig2IEJDLSRPTTVQWJLOXTCJ--