Re: [PATCH] Add GS1662 driver (a SPI video serializer)

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

 



On 04/04/2016 02:35 PM, Jean-Michel Hautbois wrote:
>>> Next, you should add a complete description to your commit. Just
>>> having an object and a signed-off-by line is not enough.
>> Oh, I'm sorry, I don't have any idea to explicit more details. I will
>> find something for that.
> 
> Just get the description from the datasheet as a start ;-).
> 
>>> You also have to use the scripts/checkpatch.pl script to verify that
>>> everything is ok with it.
>> I have executed this script before to send it. And it noticed nothing about that.
>>
>>> Last thing, I can't see anything related to V4L2 in your patch. It is
>>> just used to initialize the chip and the spi bus, that's all.
>>> Adding a subdev is a start, and some operations if it can do something
>>> else than just serializing.
>>
>> Maybe I'm in the wrong list for that in fact. I didn't know this list was about V4L2 and related topics.
>> This driver is only to configure the component to manage the video stream in electronic card, it is not to capture video stream via V4L.
>>
>> I should improve my driver to be configurable by userspace. But maybe I should submit my future patch in another ML.
> 
> Well, I am not really sure about that. I added Hans in cc as he may
> have a better view.
> From my point of view, it can be a V4L2 subdev, even a simple one, as
> you can configure outputs, etc.

I think the media subsystem is definitely the right place for this.

I would use the cs3308.c driver as a starting point. This is also a minimal driver
(and you can remove the code under CONFIG_VIDEO_ADV_DEBUG for your driver), but it
uses v4l2_subdev and that makes it ready to be extended in the future, which you
will likely need to do eventually.

Regards,

	Hans
--
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