1. 09 5月, 2019 1 次提交
  2. 01 3月, 2019 1 次提交
  3. 08 2月, 2019 1 次提交
  4. 12 9月, 2018 1 次提交
  5. 17 4月, 2018 2 次提交
    • M
      media: platform: fix some 64-bits warnings · 34b2ea1c
      Mauro Carvalho Chehab 提交于
      The omap/omap3 and viu drivers are for 32 bit platforms only.
      There, a pointer has 32 bits. Now that those drivers build
      for 64 bits with COMPILE_TEST, they produce the following
      warnings:
      
      drivers/media/platform/omap/omap_vout_vrfb.c: In function 'omap_vout_allocate_vrfb_buffers':
      drivers/media/platform/omap/omap_vout_vrfb.c:57:10: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
         memset((void *) vout->smsshado_virt_addr[i], 0,
                ^
      drivers/media/platform/fsl-viu.c: In function 'viu_setup_preview':
      drivers/media/platform/fsl-viu.c:753:28: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
        reg_val.field_base_addr = (u32)dev->ovbuf.base;
                                  ^
      drivers/media/platform/omap/omap_vout.c: In function 'omap_vout_get_userptr':
      drivers/media/platform/omap/omap_vout.c:209:25: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
         *physp = virt_to_phys((void *)virtp);
                               ^
      drivers/media/platform/omap3isp/ispccdc.c: In function 'ccdc_config':
      drivers/media/platform/omap3isp/ispccdc.c:738:9: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
               (__force void __user *)fpc.fpcaddr,
               ^
      
      Add some typecasts to remove those warnings when building for
      64 bits.
      Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>
      34b2ea1c
    • M
      media: omap_vout: fix wrong identing · b1011a33
      Mauro Carvalho Chehab 提交于
      As warned:
      	drivers/media/platform/omap/omap_vout.c:711 omap_vout_buffer_setup() warn: inconsistent indenting
      Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>
      b1011a33
  6. 05 1月, 2018 1 次提交
  7. 28 11月, 2017 1 次提交
  8. 31 10月, 2017 1 次提交
  9. 21 10月, 2016 1 次提交
    • M
      [media] omap: don't break long lines · bc39030b
      Mauro Carvalho Chehab 提交于
      Due to the 80-cols restrictions, and latter due to checkpatch
      warnings, several strings were broken into multiple lines. This
      is not considered a good practice anymore, as it makes harder
      to grep for strings at the source code.
      
      As we're right now fixing other drivers due to KERN_CONT, we need
      to be able to identify what printk strings don't end with a "\n".
      It is a way easier to detect those if we don't break long lines.
      
      So, join those continuation lines.
      
      The patch was generated via the script below, and manually
      adjusted if needed.
      
      </script>
      use Text::Tabs;
      while (<>) {
      	if ($next ne "") {
      		$c=$_;
      		if ($c =~ /^\s+\"(.*)/) {
      			$c2=$1;
      			$next =~ s/\"\n$//;
      			$n = expand($next);
      			$funpos = index($n, '(');
      			$pos = index($c2, '",');
      			if ($funpos && $pos > 0) {
      				$s1 = substr $c2, 0, $pos + 2;
      				$s2 = ' ' x ($funpos + 1) . substr $c2, $pos + 2;
      				$s2 =~ s/^\s+//;
      
      				$s2 = ' ' x ($funpos + 1) . $s2 if ($s2 ne "");
      
      				print unexpand("$next$s1\n");
      				print unexpand("$s2\n") if ($s2 ne "");
      			} else {
      				print "$next$c2\n";
      			}
      			$next="";
      			next;
      		} else {
      			print $next;
      		}
      		$next="";
      	} else {
      		if (m/\"$/) {
      			if (!m/\\n\"$/) {
      				$next=$_;
      				next;
      			}
      		}
      	}
      	print $_;
      }
      </script>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@s-opensource.com>
      bc39030b
  10. 19 10月, 2016 1 次提交
  11. 24 8月, 2016 1 次提交
  12. 12 7月, 2016 1 次提交
  13. 03 6月, 2016 1 次提交
  14. 11 9月, 2015 1 次提交
  15. 17 8月, 2015 1 次提交
  16. 06 6月, 2015 1 次提交
  17. 03 3月, 2015 1 次提交
  18. 02 12月, 2014 2 次提交
    • H
      [media] omap_vout: fix compile warnings · e8bd888a
      Hans Verkuil 提交于
      When compiling under COMPILE_TEST on a x86_64 the following warnings
      appear:
      
      drivers/media/platform/omap/omap_vout.c: In function 'omap_vout_uservirt_to_phys':
      drivers/media/platform/omap/omap_vout.c:209:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
         return virt_to_phys((void *) virtp);
                             ^
      drivers/media/platform/omap/omap_vout.c: In function 'omapvid_setup_overlay':
      drivers/media/platform/omap/omap_vout.c:420:2: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'dma_addr_t' [-Wformat=]
        v4l2_dbg(1, debug, &vout->vid_dev->v4l2_dev,
        ^
      drivers/media/platform/omap/omap_vout.c: In function 'omap_vout_buffer_prepare':
      drivers/media/platform/omap/omap_vout.c:794:34: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
         vout->queued_buf_addr[vb->i] = (u8 *)
                                        ^
      In file included from arch/x86/include/asm/dma-mapping.h:44:0,
                       from include/linux/dma-mapping.h:82,
                       from drivers/media/platform/omap/omap_vout.c:40:
      drivers/media/platform/omap/omap_vout.c:803:58: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
         dma_addr = dma_map_single(vout->vid_dev->v4l2_dev.dev, (void *) addr,
                                                                ^
      include/asm-generic/dma-mapping-common.h:174:60: note: in definition of macro 'dma_map_single'
       #define dma_map_single(d, a, s, r) dma_map_single_attrs(d, a, s, r, NULL)
                                                                  ^
      
      These are fixed by this patch.
      Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@osg.samsung.com>
      e8bd888a
    • H
      [media] media/platform: fix querycap · a020c747
      Hans Verkuil 提交于
      Querycap shouldn't set the version field (the core does that for you),
      but it should set the device_caps field.
      Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
      Cc: Scott Jiang <scott.jiang.linux@gmail.com>
      Cc: Gerhard Sittig <gsi@denx.de>
      Cc: Jonathan Corbet <corbet@lwn.net>
      Cc: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@osg.samsung.com>
      a020c747
  19. 04 9月, 2014 1 次提交
  20. 27 8月, 2014 1 次提交
    • M
      [media] omap_vout: Get rid of a few warnings · 52334bb3
      Mauro Carvalho Chehab 提交于
      drivers/media/platform/omap/omap_vout.c: In function 'omapvid_setup_overlay':
      drivers/media/platform/omap/omap_vout.c:372:29: warning: variable 'pixheight' set but not used [-Wunused-but-set-variable]
        int cropheight, cropwidth, pixheight, pixwidth;
                                   ^
      drivers/media/platform/omap/omap_vout.c: In function 'vidioc_s_ctrl':
      drivers/media/platform/omap/omap_vout.c:1454:24: warning: variable 'ovl' set but not used [-Wunused-but-set-variable]
         struct omap_overlay *ovl;
                              ^
      drivers/media/platform/omap/omap_vout.c: In function 'vidioc_reqbufs':
      drivers/media/platform/omap/omap_vout.c:1492:55: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
        if ((req->type != V4L2_BUF_TYPE_VIDEO_OUTPUT) || (req->count < 0))
                                                             ^
      Signed-off-by: NMauro Carvalho Chehab <m.chehab@samsung.com>
      52334bb3
  21. 17 7月, 2014 1 次提交
    • L
      [media] v4l: Support extending the v4l2_pix_format structure · d52e2381
      Laurent Pinchart 提交于
      The v4l2_pix_format structure has no reserved field. It is embedded in
      the v4l2_framebuffer structure which has no reserved fields either, and
      in the v4l2_format structure which has reserved fields that were not
      previously required to be zeroed out by applications.
      
      To allow extending v4l2_pix_format, inline it in the v4l2_framebuffer
      structure, and use the priv field as a magic value to indicate that the
      application has set all v4l2_pix_format extended fields and zeroed all
      reserved fields following the v4l2_pix_format field in the v4l2_format
      structure.
      
      The availability of this API extension is reported to userspace through
      the new V4L2_CAP_EXT_PIX_FORMAT capability flag. Just checking that the
      priv field is still set to the magic value at [GS]_FMT return wouldn't
      be enough, as older kernels don't zero the priv field on return.
      
      To simplify the internal API towards drivers zero the extended fields
      and set the priv field to the magic value for applications not aware of
      the extensions.
      Signed-off-by: NLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
      Signed-off-by: NMauro Carvalho Chehab <m.chehab@samsung.com>
      d52e2381
  22. 11 3月, 2014 1 次提交
  23. 29 6月, 2013 1 次提交
  24. 23 5月, 2013 1 次提交
    • T
      OMAPDSS: Fix crash with DT boot · 591a0ac7
      Tomi Valkeinen 提交于
      When booting with DT, there's a crash when omapfb is probed. This is
      caused by the fact that omapdss+DT is not yet supported, and thus
      omapdss is not probed at all. On the other hand, omapfb is always
      probed. When omapfb tries to use omapdss, there's a NULL pointer
      dereference crash. The same error should most likely happen with omapdrm
      and omap_vout also.
      
      To fix this, add an "initialized" state to omapdss. When omapdss has
      been probed, it's marked as initialized. omapfb, omapdrm and omap_vout
      check this state when they are probed to see that omapdss is actually
      there.
      Signed-off-by: NTomi Valkeinen <tomi.valkeinen@ti.com>
      Tested-by: NPeter Ujfalusi <peter.ujfalusi@ti.com>
      591a0ac7
  25. 22 3月, 2013 1 次提交
    • F
      [media] Corrected Oops on omap_vout when no manager is connected · 15170025
      Federico Fuga 提交于
      If no manager is connected to the vout device, the omapvid_init() function
      fails. No error condition is checked, and the device is started. Later on,
      when irq is serviced, a NULL pointer dereference occurs.
      Also, the isr routine must be registered only if no error occurs, otherwise
      the isr triggers without the proper setup, and the kernel oops again.
      To prevent this, the error condition is checked, and the streamon function
      exits with error. Also the isr registration call is moved after the setup
      procedure is completed.
      Reviewed-by: NPrabhakar Lad <prabhakar.lad@ti.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      15170025
  26. 06 1月, 2013 1 次提交
    • A
      [media] omap_vout: find_vma() needs ->mmap_sem held · 55ee64b3
      Al Viro 提交于
      Walking rbtree while it's modified is a Bad Idea(tm); besides,
      the result of find_vma() can be freed just as it's getting returned
      to caller.  Fortunately, it's easy to fix - just take ->mmap_sem a bit
      earlier (and don't bother with find_vma() at all if virtp >= PAGE_OFFSET -
      in that case we don't even look at its result).
      
      While we are at it, what prevents VIDIOC_PREPARE_BUF calling
      v4l_prepare_buf() -> (e.g) vb2_ioctl_prepare_buf() -> vb2_prepare_buf() ->
      __buf_prepare() -> __qbuf_userptr() -> vb2_vmalloc_get_userptr() -> find_vma(),
      AFAICS without having taken ->mmap_sem anywhere in process?  The code flow
      is bloody convoluted and depends on a bunch of things done by initialization,
      so I certainly might've missed something...
      
      Cc: stable@vger.kernel.org [2.6.35]
      Signed-off-by: NAl Viro <viro@zeniv.linux.org.uk>
      Cc: Sakari Ailus <sakari.ailus@iki.fi>
      Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
      Cc: Archit Taneja <archit@ti.com>
      Cc: Prabhakar Lad <prabhakar.lad@ti.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      55ee64b3
  27. 21 12月, 2012 1 次提交
  28. 19 12月, 2012 2 次提交
  29. 07 12月, 2012 1 次提交
  30. 01 12月, 2012 1 次提交
    • T
      ARM: OMAP: Move plat-omap/dma-omap.h to include/linux/omap-dma.h · 45c3eb7d
      Tony Lindgren 提交于
      Based on earlier discussions[1] we attempted to find a suitable
      location for the omap DMA header in commit 2b6c4e73 (ARM: OMAP:
      DMA: Move plat/dma.h to plat-omap/dma-omap.h) until the conversion
      to dmaengine is complete.
      
      Unfortunately that was before I was able to try to test compile
      of the ARM multiplatform builds for omap2+, and the end result
      was not very good.
      
      So I'm creating yet another all over the place patch to cut the
      last dependency for building omap2+ for ARM multiplatform. After
      this, we have finally removed the driver dependencies to the
      arch/arm code, except for few drivers that are being worked on.
      
      The other option was to make the <plat-omap/dma-omap.h> path
      to work, but we'd have to add some new header directory to for
      multiplatform builds.
      
      Or we would have to manually include arch/arm/plat-omap/include
      again from arch/arm/Makefile for omap2+.
      
      Neither of these alternatives sound appealing as they will
      likely lead addition of various other headers exposed to the
      drivers, which we want to avoid for the multiplatform kernels.
      
      Since we already have a minimal include/linux/omap-dma.h,
      let's just use that instead and add a note to it to not
      use the custom omap DMA functions any longer where possible.
      
      Note that converting omap DMA to dmaengine depends on
      dmaengine supporting automatically incrementing the FIFO
      address at the device end, and converting all the remaining
      legacy drivers. So it's going to be few more merge windows.
      
      [1] https://patchwork.kernel.org/patch/1519591/#
      
      cc: Russell King <linux@arm.linux.org.uk>
      cc: Kevin Hilman <khilman@ti.com>
      cc: "Benoît Cousson" <b-cousson@ti.com>
      cc: Herbert Xu <herbert@gondor.apana.org.au>
      cc: "David S. Miller" <davem@davemloft.net>
      cc: Vinod Koul <vinod.koul@intel.com>
      cc: Dan Williams <djbw@fb.com>
      cc: Mauro Carvalho Chehab <mchehab@infradead.org>
      cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
      cc: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
      cc: David Woodhouse <dwmw2@infradead.org>
      cc: Kyungmin Park <kyungmin.park@samsung.com>
      cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
      cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
      cc: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>
      cc: Hans Verkuil <hans.verkuil@cisco.com>
      cc: Vaibhav Hiremath <hvaibhav@ti.com>
      cc: Lokesh Vutla <lokeshvutla@ti.com>
      cc: Rusty Russell <rusty@rustcorp.com.au>
      cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
      cc: Afzal Mohammed <afzal@ti.com>
      cc: linux-crypto@vger.kernel.org
      cc: linux-media@vger.kernel.org
      cc: linux-mtd@lists.infradead.org
      cc: linux-usb@vger.kernel.org
      cc: linux-fbdev@vger.kernel.org
      Acked-by: NFelipe Balbi <balbi@ti.com>
      Signed-off-by: NTony Lindgren <tony@atomide.com>
      45c3eb7d
  31. 30 11月, 2012 2 次提交
  32. 27 10月, 2012 1 次提交
    • A
      [media] omap_vout: Set DSS overlay_info only if paddr is non zero · 32d640de
      Archit Taneja 提交于
      The omap_vout driver tries to set the DSS overlay_info using set_overlay_info()
      when the physical address for the overlay is still not configured. This happens
      in omap_vout_probe() and vidioc_s_fmt_vid_out().
      The calls to omapvid_init(which internally calls set_overlay_info()) are removed
      from these functions. They don't need to be called as the omap_vout_device
      struct anyway maintains the overlay related changes made. Also, remove the
      explicit call to set_overlay_info() in vidioc_streamon(), this was used to set
      the paddr, this isn't needed as omapvid_init() does the same thing later.
      These changes are required as the DSS2 driver since 3.3 kernel doesn't let you
      set the overlay info with paddr as 0.
      Signed-off-by: NArchit Taneja <archit@ti.com>
      Tested-by: NLaurent Pinchart <laurent.pinchart@ideasonboard.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      32d640de
  33. 17 10月, 2012 1 次提交
  34. 16 10月, 2012 1 次提交
  35. 09 10月, 2012 1 次提交
    • K
      mm: kill vma flag VM_RESERVED and mm->reserved_vm counter · 314e51b9
      Konstantin Khlebnikov 提交于
      A long time ago, in v2.4, VM_RESERVED kept swapout process off VMA,
      currently it lost original meaning but still has some effects:
      
       | effect                 | alternative flags
      -+------------------------+---------------------------------------------
      1| account as reserved_vm | VM_IO
      2| skip in core dump      | VM_IO, VM_DONTDUMP
      3| do not merge or expand | VM_IO, VM_DONTEXPAND, VM_HUGETLB, VM_PFNMAP
      4| do not mlock           | VM_IO, VM_DONTEXPAND, VM_HUGETLB, VM_PFNMAP
      
      This patch removes reserved_vm counter from mm_struct.  Seems like nobody
      cares about it, it does not exported into userspace directly, it only
      reduces total_vm showed in proc.
      
      Thus VM_RESERVED can be replaced with VM_IO or pair VM_DONTEXPAND | VM_DONTDUMP.
      
      remap_pfn_range() and io_remap_pfn_range() set VM_IO|VM_DONTEXPAND|VM_DONTDUMP.
      remap_vmalloc_range() set VM_DONTEXPAND | VM_DONTDUMP.
      
      [akpm@linux-foundation.org: drivers/vfio/pci/vfio_pci.c fixup]
      Signed-off-by: NKonstantin Khlebnikov <khlebnikov@openvz.org>
      Cc: Alexander Viro <viro@zeniv.linux.org.uk>
      Cc: Carsten Otte <cotte@de.ibm.com>
      Cc: Chris Metcalf <cmetcalf@tilera.com>
      Cc: Cyrill Gorcunov <gorcunov@openvz.org>
      Cc: Eric Paris <eparis@redhat.com>
      Cc: H. Peter Anvin <hpa@zytor.com>
      Cc: Hugh Dickins <hughd@google.com>
      Cc: Ingo Molnar <mingo@redhat.com>
      Cc: James Morris <james.l.morris@oracle.com>
      Cc: Jason Baron <jbaron@redhat.com>
      Cc: Kentaro Takeda <takedakn@nttdata.co.jp>
      Cc: Matt Helsley <matthltc@us.ibm.com>
      Cc: Nick Piggin <npiggin@kernel.dk>
      Cc: Oleg Nesterov <oleg@redhat.com>
      Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
      Cc: Robert Richter <robert.richter@amd.com>
      Cc: Suresh Siddha <suresh.b.siddha@intel.com>
      Cc: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
      Cc: Venkatesh Pallipadi <venki@google.com>
      Acked-by: NLinus Torvalds <torvalds@linux-foundation.org>
      Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
      Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
      314e51b9
  36. 26 9月, 2012 1 次提交