RE: [PATCH - v3 4/4] DaVinci - vpfe-capture-converting ccdc drivers to platform driver

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

 



> -----Original Message-----
> From: davinci-linux-open-source-bounces@xxxxxxxxxxxxxxxxxxxx
> [mailto:davinci-linux-open-source-bounces@xxxxxxxxxxxxxxxxxxxx] On
> Behalf Of Karicheri, Muralidharan
> Sent: Friday, January 08, 2010 4:55 AM
> To: Kevin Hilman
> Cc: davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx; linux-
> media@xxxxxxxxxxxxxxx
> Subject: RE: [PATCH - v3 4/4] DaVinci - vpfe-capture-converting ccdc
> drivers to platform driver
> 
> Arch patches are not usually merged in Hans tree.
> 
[Hiremath, Vaibhav] Hi Kevin and Murali,

Sorry for jumping into this discussion so late, 

Can we use clk_add_alias() function exported by clkdev.c file here? With this board specific file can define aliases for all required platform_data keeping CLK() entry generic.

Thanks,
Vaibhav

> Murali Karicheri
> Software Design Engineer
> Texas Instruments Inc.
> Germantown, MD 20874
> phone: 301-407-9583
> email: m-karicheri2@xxxxxx
> 
> >-----Original Message-----
> >From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx]
> >Sent: Thursday, January 07, 2010 4:50 PM
> >To: Karicheri, Muralidharan
> >Cc: linux-media@xxxxxxxxxxxxxxx; hverkuil@xxxxxxxxx; davinci-linux-
> open-
> >source@xxxxxxxxxxxxxxxxxxxx
> >Subject: Re: [PATCH - v3 4/4] DaVinci - vpfe-capture-converting
> ccdc
> >drivers to platform driver
> >
> >"Karicheri, Muralidharan" <m-karicheri2@xxxxxx> writes:
> >
> >> Can I remove it through a separate patch? This patch is already
> merged in
> >Hans tree.
> >
> >Hmm, arch patches should not be merged yet as I have not ack'd
> them.
> >
> >Kevin
> >
> >
> >>>-----Original Message-----
> >>>From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx]
> >>>Sent: Thursday, January 07, 2010 2:44 PM
> >>>To: Karicheri, Muralidharan
> >>>Cc: linux-media@xxxxxxxxxxxxxxx; hverkuil@xxxxxxxxx; davinci-
> linux-open-
> >>>source@xxxxxxxxxxxxxxxxxxxx
> >>>Subject: Re: [PATCH - v3 4/4] DaVinci - vpfe-capture-converting
> ccdc
> >>>drivers to platform driver
> >>>
> >>>"Karicheri, Muralidharan" <m-karicheri2@xxxxxx> writes:
> >>>
> >>>> Kevin,
> >>>>
> >>>>>
> >>>>>OK, I'm not extremely familar with the whole video architecture
> here,
> >>>>>but are all of these drivers expected to be doing clk_get() and
> >>>>>clk_enable()?
> >>>>>
> >>>>
> >>>> [MK]Many IPs on DaVinci VPFE would require vpss master clock.
> So
> >>>> it is better to do the way I have done in my patch. So it is
> expected
> >>>> that clk_get, clk_enable etc are called from other drivers as
> well.
> >>>
> >>>OK, then you are expecting to add clkdev nodes for the other
> devices
> >>>as well.  That's ok.
> >>>
> >>>However, you still haven't answered my original question.
> AFAICT,
> >>>there are no users of the clkdev nodes "vpss_master" and
> "vpss_slave".
> >>>Why not remove those and replace them with your new nodes instead
> of
> >>>leaving them and adding new ones?
> >>>
> >>>Kevin
> _______________________________________________
> Davinci-linux-open-source mailing list
> Davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx
> http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-
> source
--
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