Re: [PATCH 0/4] media: si4713: minor updates

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

 



Hi Hans,

My last email didn't reach the list, so re-sending.

On Wed, Mar 20, 2013 at 5:04 AM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote:
>
> Hi Eduardo!
>
> On Tue 19 March 2013 16:41:30 Eduardo Valentin wrote:
> > Hello Mauro and Hans,
> >
> > Here are a couple of minor changes for si4713 FM transmitter driver.
>
> Thanks!

No problem!

>
> Patches 2-4 are fine, but I don't really see the point of the first patch
> (except for the last chunk which is a real improvement).
>
> The Codingstyle doesn't require such alignment, and in fact it says:
>
> "Descendants are always substantially shorter than the parent and
> are placed substantially to the right. The same applies to function
> headers
> with a long argument list."
>
> Unless Mauro thinks otherwise, I would leave all the alignment stuff alone
> and just post a version with the last chunk.
>

OK. The chunks on patch 01 are mostly to get rid of these outputs out
of checkpatch.pl --strict -f drivers/media/radio/radio-si4713.c:
CHECK: Alignment should match open parenthesis
#97: FILE: media/radio/radio-si4713.c:97:
+       strlcpy(capability->card, "Silicon Labs Si4713 Modulator",
+                               sizeof(capability->card));


> For patches 2-4:
>
> Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>
>


Nice! I will add your Acked-by.


> Are you still able to test the si4713 driver? Because I have patches



I see. In fact that is my next step on my todo list for si4713. I
still have an n900 that I can fetch from my drobe, so just a matter of
booting it with newer kernel.

> outstanding that I would love for someone to test for me:
>
> http://git.linuxtv.org/hverkuil/media_tree.git/shortlog/refs/heads/si4713
>
> In particular, run the latest v4l2-compliance test over it.
>


OK. I will check your branch once I get my setup done and let you know.

> Regards,
>
>         Hans
>
> >
> > These changes are also available here:
> > https://git.gitorious.org/si4713/si4713.git
> >
> > All best,
> >
> > Eduardo Valentin (4):
> >   media: radio: CodingStyle changes on si4713
> >   media: radio: correct module license (==> GPL v2)
> >   media: radio: add driver owner entry for radio-si4713
> >   media: radio: add module alias entry for radio-si4713
> >
> >  drivers/media/radio/radio-si4713.c |   57
> > ++++++++++++++++++-----------------
> >  1 files changed, 29 insertions(+), 28 deletions(-)
> >
> >




--
Eduardo Bezerra Valentin
--
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