Re: [PATCH] [0904_13] Siano: move DVB_API and remove redundant code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 20 Apr 2009 11:24:11 -0700 (PDT)
Uri Shkolnik <urishk@xxxxxxxxx> wrote:

> 
> 
> 
> --- On Mon, 4/20/09, Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> wrote:
> 
> > From: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx>
> > Subject: Re: [PATCH] [0904_13] Siano: move DVB_API and remove redundant code
> > To: "Uri Shkolnik" <urishk@xxxxxxxxx>
> > Cc: "LinuxML" <linux-media@xxxxxxxxxxxxxxx>
> > Date: Monday, April 20, 2009, 9:19 PM
> > On Mon, 20 Apr 2009 11:07:57 -0700
> > (PDT)
> > Uri Shkolnik <urishk@xxxxxxxxx>
> > wrote:
> > 
> > > 
> > > 
> > > 
> > > --- On Mon, 4/20/09, Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx>
> > wrote:
> > > 
> > > > From: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx>
> > > > Subject: Re: [PATCH] [0904_13] Siano: move
> > DVB_API and remove redundant code
> > > > To: "Uri Shkolnik" <urishk@xxxxxxxxx>
> > > > Cc: "LinuxML" <linux-media@xxxxxxxxxxxxxxx>
> > > > Date: Monday, April 20, 2009, 9:02 PM
> > > > On Sun, 5 Apr 2009 03:31:32 -0700
> > > > (PDT)
> > > > Uri Shkolnik <urishk@xxxxxxxxx>
> > > > wrote:
> > > > 
> > > > > 
> > > > > # HG changeset patch
> > > > > # User Uri Shkolnik <uris@xxxxxxxxxxxx>
> > > > > # Date 1238755204 -10800
> > > > > # Node ID
> > f65a29f0f9a66f82a91525ae0085a15f00ac91c2
> > > > > # Parent 
> > > > 897669fdeb3be75a2bde978557b5398a4a7d8914
> > > > > [PATCH] [0904_13] Siano: move DVB_API and
> > remove
> > > > redundant code
> > > > > 
> > > > > From: Uri Shkolnik <uris@xxxxxxxxxxxx>
> > > > > 
> > > > > The DVB-API related information has been
> > moved from
> > > > the core header
> > > > > to the smsdvb, and the redundant code has
> > been removed
> > > > from the
> > > > > core header.
> > > > > 
> > > > > This code has been moved since it is used
> > only by
> > > > > the smsdvb client component.
> > > > 
> > > > This patch depends on the previous patches that I
> > asked
> > > > some changes. Please
> > > > re-submit it together with the other patches that
> > weren't
> > > > committed. It is
> > > > probably not much valuable to commit the later
> > patches, so
> > > > I'll stop analysing
> > > > the code here.
> > > > 
> > > > The patch itself looks sane to my eyes.
> > > > > 
> > > > > Priority: normal
> > > > > 
> > > > > Signed-off-by: Uri Shkolnik <uris@xxxxxxxxxxxx>
> > > > > 
> > > > > diff -r 897669fdeb3b -r f65a29f0f9a6
> > > > linux/drivers/media/dvb/siano/smscoreapi.h
> > > > > ---
> > > >
> > a/linux/drivers/media/dvb/siano/smscoreapi.h   
> > > > Fri Apr 03 13:31:13 2009 +0300
> > > > > +++
> > > >
> > b/linux/drivers/media/dvb/siano/smscoreapi.h   
> > > > Fri Apr 03 13:40:04 2009 +0300
> > > > > @@ -36,15 +36,6 @@ along with this
> > program.  If
> > > > not, see <h
> > > > >  #include <asm/page.h>
> > > > >  
> > > > >  /* #include "smsir.h" */
> > > > > -
> > > > > -#define SMS_DVB3_SUBSYS
> > > > > -#ifdef SMS_DVB3_SUBSYS
> > > > > -#include "dmxdev.h"
> > > > > -#include "dvbdev.h"
> > > > > -#include "dvb_demux.h"
> > > > > -#include "dvb_frontend.h"
> > > > > -
> > > > > -#endif
> > > > >  
> > > > >  #define kmutex_init(_p_) mutex_init(_p_)
> > > > >  #define kmutex_lock(_p_) mutex_lock(_p_)
> > > > > diff -r 897669fdeb3b -r f65a29f0f9a6
> > > > linux/drivers/media/dvb/siano/smsdvb.c
> > > > > ---
> > > > a/linux/drivers/media/dvb/siano/smsdvb.c   
> > > > Fri Apr 03 13:31:13 2009 +0300
> > > > > +++
> > > > b/linux/drivers/media/dvb/siano/smsdvb.c   
> > > > Fri Apr 03 13:40:04 2009 +0300
> > > > > @@ -22,6 +22,11 @@ along with this
> > program.  If
> > > > not, see <h
> > > > >  #include <linux/module.h>
> > > > >  #include <linux/init.h>
> > > > >  #include <asm/byteorder.h>
> > > > > +
> > > > > +#include "dmxdev.h"
> > > > > +#include "dvbdev.h"
> > > > > +#include "dvb_demux.h"
> > > > > +#include "dvb_frontend.h"
> > > > >  
> > > > >  #include "smscoreapi.h"
> > > > >  /*#include "smsendian.h"*/
> > > > > @@ -52,7 +57,7 @@ struct smsdvb_client_t {
> > > > >      fe_status_t fe_status;
> > > > >      int fe_ber, fe_snr, fe_unc,
> > > > fe_signal_strength;
> > > > >  
> > > > > -    struct completion tune_done,
> > > > stat_done;
> > > > > +    struct completion tune_done;
> > > > >  
> > > > >      /* todo: save freq/band
> > > > instead whole struct */
> > > > >      struct
> > > > dvb_frontend_parameters fe_params;
> > > > > 
> > > > > 
> > > > > 
> > > > >       
> > > > > --
> > > > > To unsubscribe from this list: send the
> > line
> > > > "unsubscribe linux-media" in
> > > > > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > > > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > > > 
> > > > 
> > > > 
> > > > 
> > > > Cheers,
> > > > Mauro
> > > > 
> > > 
> > > OK
> > > 
> > > I'll submit patches to fix the various rejects on the
> > coming Wednesday (I'm ooo tomorrow).
> > > 
> > > BTW - is it possible for me to clone the current tree
> > you currently have? (after applying the approved patches),
> > it will help me for future patches.
> > 
> > Sure. The better is to apply your patches over the fresh
> > clone. You can use
> > the ./hgimport script to help you to pick the patches from
> > your old tree.
> > 
> > Cheers,
> > Mauro
> > 
> 
> Regarding the ./hgimport script - how do I tell that script to refer to your tree and not to any other hg tree? Isn't the default look-up is the dvb-api trunk?

It has no defaults. You can point it to a remote URL or to a local path.
> 
> 
> Regards,
> 
> Uri
> 
> 
>       




Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux