Re: [linux-sunxi] Cedrus driver

Hi Thomas, On Wed, Nov 29, 2017 at 04:36:01PM +0100, Thomas van Kleef wrote: > > C) I'm not sure what you tried to do with the application of the > >request API patches (such as e1ca861c168f) but we want to have the > >whole commits in there, and not a patch adding all of them. This > >

Re: [linux-sunxi] Cedrus driver

Hi Maxime, > > So there's a couple of issues with those patches (the pull request > itself is fine though :)) > > I'll try to break them down as much as possible. > > A) If you want to have proper commit logs, you will usually do two >things: first create a commit title, which is what

Re: [linux-sunxi] Cedrus driver

Hi, On Tue, Nov 28, 2017 at 03:51:14PM +0100, Thomas van Kleef wrote: > On 28-11-17 13:26, Maxime Ripard wrote: > > On Tue, Nov 28, 2017 at 12:20:59PM +0100, Thomas van Kleef wrote: > >>> So, I have been rebasing to 4.14.0 and have the cedrus driver working. > >> I have pulled linux-mainline

Re: [linux-sunxi] Cedrus driver

Il 28/11/2017 16:17, Maxime Ripard ha scritto: On Tue, Nov 28, 2017 at 02:12:31PM +0100, Giulio Benetti wrote: And really, just develop against 4.14. sunxi-next is rebased, and it's just not something you can base some work on. Where do we can work on then? Should Thomas setup his own github

Re: [linux-sunxi] Cedrus driver

On Tue, Nov 28, 2017 at 02:12:31PM +0100, Giulio Benetti wrote: > > > > And really, just develop against 4.14. sunxi-next is rebased, and it's > > > > just not something you can base some work on. > > > > > > Where do we can work on then? > > > Should Thomas setup his own github repo? > > > What

Re: [linux-sunxi] Cedrus driver

On 28-11-17 13:26, Maxime Ripard wrote: > On Tue, Nov 28, 2017 at 12:20:59PM +0100, Thomas van Kleef wrote: >>> So, I have been rebasing to 4.14.0 and have the cedrus driver working. >> I have pulled linux-mainline 4.14.0. Then pulled the requests2 branch from >> Hans >> Verkuil's media_tree. I

Re: [linux-sunxi] Cedrus driver

Il 28/11/2017 14:07, Maxime Ripard ha scritto: On Tue, Nov 28, 2017 at 02:03:43PM +0100, Giulio Benetti wrote: Hi, Il giorno 28 nov 2017, alle ore 13:52, Maxime Ripard ha scritto: On Tue, Nov 28, 2017 at 12:54:08PM +0100, Giulio Benetti wrote: Should I be

Re: [linux-sunxi] Cedrus driver

On Tue, Nov 28, 2017 at 02:03:43PM +0100, Giulio Benetti wrote: > Hi, > > > Il giorno 28 nov 2017, alle ore 13:52, Maxime Ripard > > ha scritto: > > > > On Tue, Nov 28, 2017 at 12:54:08PM +0100, Giulio Benetti wrote: > > Should I be working in sunxi-next I

Re: [linux-sunxi] Cedrus driver

Hi, > Il giorno 28 nov 2017, alle ore 13:52, Maxime Ripard > ha scritto: > > On Tue, Nov 28, 2017 at 12:54:08PM +0100, Giulio Benetti wrote: > Should I be working in sunxi-next I wonder? Yes, this is the best way, cedrus is very specific to

Re: [linux-sunxi] Cedrus driver

On Tue, Nov 28, 2017 at 12:54:08PM +0100, Giulio Benetti wrote: > > > > Should I be working in sunxi-next I wonder? > > > > > > Yes, this is the best way, cedrus is very specific to sunxi. > > > So before working on mainline, I think the best is to work un sunxi-next > > > branch. > > > > Is the

Re: [linux-sunxi] Cedrus driver

Hi Giulio On 28-11-17 12:54, Giulio Benetti wrote: > Hi Thomas, > > Il 28/11/2017 12:29, Thomas van Kleef ha scritto: >> Hi, >> >> On 28-11-17 12:26, Giulio Benetti wrote: >>> Hi Thomas, >>> >>> Il 28/11/2017 12:20, Thomas van Kleef ha scritto: On 28-11-17 10:50, Giulio Benetti wrote: >

Re: [linux-sunxi] Cedrus driver

On Tue, Nov 28, 2017 at 12:20:59PM +0100, Thomas van Kleef wrote: > > So, I have been rebasing to 4.14.0 and have the cedrus driver working. > I have pulled linux-mainline 4.14.0. Then pulled the requests2 branch from > Hans > Verkuil's media_tree. I have a patch available of the merge between

Re: [linux-sunxi] Cedrus driver

Hi Thomas, Il 28/11/2017 12:29, Thomas van Kleef ha scritto: Hi, On 28-11-17 12:26, Giulio Benetti wrote: Hi Thomas, Il 28/11/2017 12:20, Thomas van Kleef ha scritto: On 28-11-17 10:50, Giulio Benetti wrote: Hi Maxime, Il 28/11/2017 09:35, Maxime Ripard ha scritto: On Tue, Nov 28, 2017

Re: [linux-sunxi] Cedrus driver

Hi, On 28-11-17 12:26, Giulio Benetti wrote: > Hi Thomas, > > Il 28/11/2017 12:20, Thomas van Kleef ha scritto: >> On 28-11-17 10:50, Giulio Benetti wrote: >>> Hi Maxime, >>> >>> Il 28/11/2017 09:35, Maxime Ripard ha scritto: On Tue, Nov 28, 2017 at 01:03:59AM +0100, Giulio Benetti wrote:

Re: [linux-sunxi] Cedrus driver

On 28-11-17 10:50, Giulio Benetti wrote: > Hi Maxime, > > Il 28/11/2017 09:35, Maxime Ripard ha scritto: >> On Tue, Nov 28, 2017 at 01:03:59AM +0100, Giulio Benetti wrote: >>> Hi Maxime, >>> >>> Il 16/11/2017 14:42, Giulio Benetti ha scritto: Hi, Il 16/11/2017 14:39, Maxime Ripard

Re: [linux-sunxi] Cedrus driver

Hi Thomas, Il 28/11/2017 12:20, Thomas van Kleef ha scritto: On 28-11-17 10:50, Giulio Benetti wrote: Hi Maxime, Il 28/11/2017 09:35, Maxime Ripard ha scritto: On Tue, Nov 28, 2017 at 01:03:59AM +0100, Giulio Benetti wrote: Hi Maxime, Il 16/11/2017 14:42, Giulio Benetti ha scritto: Hi,

Re: [linux-sunxi] Cedrus driver

Hi Maxime, Il 28/11/2017 09:35, Maxime Ripard ha scritto: On Tue, Nov 28, 2017 at 01:03:59AM +0100, Giulio Benetti wrote: Hi Maxime, Il 16/11/2017 14:42, Giulio Benetti ha scritto: Hi, Il 16/11/2017 14:39, Maxime Ripard ha scritto: On Thu, Nov 16, 2017 at 02:17:08PM +0100, Giulio Benetti

Re: [linux-sunxi] Cedrus driver

On Tue, Nov 28, 2017 at 01:03:59AM +0100, Giulio Benetti wrote: > Hi Maxime, > > Il 16/11/2017 14:42, Giulio Benetti ha scritto: > > Hi, > > > > Il 16/11/2017 14:39, Maxime Ripard ha scritto: > > > On Thu, Nov 16, 2017 at 02:17:08PM +0100, Giulio Benetti wrote: > > > > Hi Hans, > > > > > > > >

Re: [linux-sunxi] Cedrus driver

Hi Maxime, Il 16/11/2017 14:42, Giulio Benetti ha scritto: Hi, Il 16/11/2017 14:39, Maxime Ripard ha scritto: On Thu, Nov 16, 2017 at 02:17:08PM +0100, Giulio Benetti wrote: Hi Hans, Il 16/11/2017 14:12, Hans Verkuil ha scritto: On 16/11/17 13:57, Giulio Benetti wrote: Il 16/11/2017

Re: [linux-sunxi] Cedrus driver

Hi Nicolas, On Thu, Nov 16, 2017 at 02:59:55PM -0500, Nicolas Dufresne wrote: > Le jeudi 16 novembre 2017 à 12:02 +0100, Maxime Ripard a écrit : > > Assuming that the request API is in, we'd need to: > > - Finish the MPEG4 support > > - Work on more useful codecs (H264 comes to my mind) > >

Re: [linux-sunxi] Cedrus driver

Le jeudi 16 novembre 2017 à 12:02 +0100, Maxime Ripard a écrit : > Assuming that the request API is in, we'd need to: > - Finish the MPEG4 support > - Work on more useful codecs (H264 comes to my mind) For which we will have to review the tables and make sure they match the spec (the easy

Re: [linux-sunxi] Cedrus driver

Hi, Il 16/11/2017 14:39, Maxime Ripard ha scritto: On Thu, Nov 16, 2017 at 02:17:08PM +0100, Giulio Benetti wrote: Hi Hans, Il 16/11/2017 14:12, Hans Verkuil ha scritto: On 16/11/17 13:57, Giulio Benetti wrote: Il 16/11/2017 13:53, Maxime Ripard ha scritto: On Thu, Nov 16, 2017 at

Re: [linux-sunxi] Cedrus driver

On 16/11/17 14:17, Giulio Benetti wrote: > Hi Hans, > > Il 16/11/2017 14:12, Hans Verkuil ha scritto: >> On 16/11/17 13:57, Giulio Benetti wrote: >>> Il 16/11/2017 13:53, Maxime Ripard ha scritto: On Thu, Nov 16, 2017 at 01:30:52PM +0100, Giulio Benetti wrote: >> On Thu, Nov 16, 2017 at

Re: [linux-sunxi] Cedrus driver

On Thu, Nov 16, 2017 at 02:17:08PM +0100, Giulio Benetti wrote: > Hi Hans, > > Il 16/11/2017 14:12, Hans Verkuil ha scritto: > > On 16/11/17 13:57, Giulio Benetti wrote: > > > Il 16/11/2017 13:53, Maxime Ripard ha scritto: > > > > On Thu, Nov 16, 2017 at 01:30:52PM +0100, Giulio Benetti wrote: >

Re: [linux-sunxi] Cedrus driver

Hi Hans, Il 16/11/2017 14:12, Hans Verkuil ha scritto: On 16/11/17 13:57, Giulio Benetti wrote: Il 16/11/2017 13:53, Maxime Ripard ha scritto: On Thu, Nov 16, 2017 at 01:30:52PM +0100, Giulio Benetti wrote: On Thu, Nov 16, 2017 at 11:37:30AM +0100, Giulio Benetti wrote: Il 16/11/2017 11:31,

Re: [linux-sunxi] Cedrus driver

On 16/11/17 13:57, Giulio Benetti wrote: > Il 16/11/2017 13:53, Maxime Ripard ha scritto: >> On Thu, Nov 16, 2017 at 01:30:52PM +0100, Giulio Benetti wrote: On Thu, Nov 16, 2017 at 11:37:30AM +0100, Giulio Benetti wrote: > Il 16/11/2017 11:31, Andreas Baierl ha scritto: >> Am

Re: [linux-sunxi] Cedrus driver

Il 16/11/2017 13:53, Maxime Ripard ha scritto: On Thu, Nov 16, 2017 at 01:30:52PM +0100, Giulio Benetti wrote: On Thu, Nov 16, 2017 at 11:37:30AM +0100, Giulio Benetti wrote: Il 16/11/2017 11:31, Andreas Baierl ha scritto: Am 16.11.2017 um 11:13 schrieb Giulio Benetti: Hello, Hello, I'm

Re: [linux-sunxi] Cedrus driver

On Thu, Nov 16, 2017 at 01:30:52PM +0100, Giulio Benetti wrote: > > On Thu, Nov 16, 2017 at 11:37:30AM +0100, Giulio Benetti wrote: > > > Il 16/11/2017 11:31, Andreas Baierl ha scritto: > > > > Am 16.11.2017 um 11:13 schrieb Giulio Benetti: > > > > > Hello, > > > > > > > > > Hello, > > > > > I'm

Re: [linux-sunxi] Cedrus driver

Hi, Il 16/11/2017 12:02, Maxime Ripard ha scritto: Hi, I'm not sure why there's so many recipients (Russell, Rob or Mark have a limited interest in this I assume), and why you're also missing some key ones (like the v4l2 list). added in cc linux-media mailing list that appears to be the

Re: [linux-sunxi] Cedrus driver

Hi, I'm not sure why there's so many recipients (Russell, Rob or Mark have a limited interest in this I assume), and why you're also missing some key ones (like the v4l2 list). On Thu, Nov 16, 2017 at 11:37:30AM +0100, Giulio Benetti wrote: > Il 16/11/2017 11:31, Andreas Baierl ha scritto: > >