Hi,
On Tue, Dec 17, 2013 at 10:55:31PM +0100, Laurent Pinchart wrote:
> Hi Wolfram,
>
> Thank you for the patch.
>
> On Tuesday 17 December 2013 22:44:36 Wolfram Sang wrote:
> > From: Wolfram Sang <wsa@xxxxxxxxxxxxxxxxxxxx>
> >
> > I decided to put the pinmuxing into the dtsi file since there is only
> > one pinmux posiibility which one probably wants to have when using the
> > bus.
I tried to explain here why the pin-groups are in the dtsi.
> > + riic0_pins: i2c0 {
> > + renesas,groups = "riic0_scl_p1_0", "riic0_sda_p1_1";
> > + renesas,function = "riic0";
> > + };
> > +
> > + riic1_pins: i2c1 {
> > + renesas,groups = "riic1_scl_p1_2", "riic1_sda_p1_3";
> > + renesas,function = "riic1";
> > + };
> > +
> > + riic2_pins: i2c2 {
> > + renesas,groups = "riic2_scl_p1_4", "riic2_sda_p1_5";
> > + renesas,function = "riic2";
> > + };
> > +
> > + riic3_pins: i2c3 {
> > + renesas,groups = "riic3_scl_p1_6", "riic3_sda_p1_7";
> > + renesas,function = "riic3";
> > + };
>
> The SoC allows other options for the I2C pin groups. Instead of declaring all
> possible groups here, I think we should thus only add the groups that are
> really used to the board .dts files.
Really? Couldn't find any. This is why I included it here: since it is
the only option, people probably want to use it when they activate the
bus.
> > + clock-frequency = <100000>;
>
> Isn't that a board-specific property ?
It is meant as a sane default. Can be overridden.
Attachment:
signature.asc
Description: Digital signature
