Re: [PATCH] rtc: pcf85063: Add support for specifying the clkout frequency from device tree node.

[Alexandre Belloni]

Hello,

On 04/07/2019 14:24:39+1200, Michael McCormick wrote:
> Primarily this allows the clkout signal to be disabled and save some
> power when running off battery backup. However, all hardware implemented
> values are implemented. Uses default value of 32768Hz if node is not
> specified.
> 

the proper way of doing that is to register the clkout signal in the
common clock framework. You can hava a look at rtc-pcf8563.c or rtc-m41t80.c

> Signed-off-by: Michael McCormick <michael.mccormick@xxxxxxxxxx>
> ---
>  drivers/rtc/rtc-pcf85063.c | 52 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 52 insertions(+)
> 
> diff --git a/drivers/rtc/rtc-pcf85063.c b/drivers/rtc/rtc-pcf85063.c
> index 1afa6d9fa9fb..5c19381899ed 100644
> --- a/drivers/rtc/rtc-pcf85063.c
> +++ b/drivers/rtc/rtc-pcf85063.c
> @@ -37,6 +37,9 @@
>  #define PCF85063_REG_CTRL2             0x01
>  #define PCF85063_CTRL2_AF              BIT(6)
>  #define PCF85063_CTRL2_AIE             BIT(7)
> +#define PCF85063_CTRL2_COF2            BIT(2)
> +#define PCF85063_CTRL2_COF1            BIT(1)
> +#define PCF85063_CTRL2_COF0            BIT(0)
> 
>  #define PCF85063_REG_OFFSET            0x02
>  #define PCF85063_OFFSET_SIGN_BIT       6       /* 2's complement sign bit */
> @@ -369,6 +372,51 @@ static int pcf85063_load_capacitance(struct pcf85063 *pcf85063,
>                                   PCF85063_REG_CTRL1_CAP_SEL, reg);
>  }
> 
> +static int pcf85063_set_clkout_mode(struct pcf85063 *pcf85063,
> +                                   const struct device_node *np)
> +{
> +       u32 load = 32768;
> +       u8 reg = 0;
> +
> +       of_property_read_u32(np, "clockout-frequency", &load);
> +       switch (load) {
> +       case 0:
> +               reg = PCF85063_CTRL2_COF2 | PCF85063_CTRL2_COF1 |
> +                     PCF85063_CTRL2_COF0;
> +               break;
> +       case 1:
> +               reg = PCF85063_CTRL2_COF2 | PCF85063_CTRL2_COF1;
> +               break;
> +       case 1024:
> +               reg = PCF85063_CTRL2_COF2 | PCF85063_CTRL2_COF0;
> +               break;
> +       case 2048:
> +               reg = PCF85063_CTRL2_COF2;
> +               break;
> +       case 4096:
> +               reg = PCF85063_CTRL2_COF1 | PCF85063_CTRL2_COF0;
> +               break;
> +       case 8192:
> +               reg = PCF85063_CTRL2_COF1;
> +               break;
> +       case 16384:
> +               reg = PCF85063_CTRL2_COF0;
> +               break;
> +       case 32768:
> +               reg = 0;
> +               break;
> +       default:
> +               dev_warn(&pcf85063->rtc->dev,
> +                       "Unknown clockout-frequency: %d. Assuming 32768", load);
> +               reg = 0;
> +               break;
> +       }
> +
> +       return regmap_update_bits(pcf85063->regmap, PCF85063_REG_CTRL2,
> +                                 PCF85063_CTRL2_COF2 | PCF85063_CTRL2_COF1 |
> +                                 PCF85063_CTRL2_COF0, reg);
> +}
> +
>  static const struct pcf85063_config pcf85063a_config = {
>         .regmap = {
>                 .reg_bits = 8,
> @@ -443,6 +491,10 @@ static int pcf85063_probe(struct i2c_client *client)
>                 dev_warn(&client->dev, "failed to set xtal load capacitance: %d",
>                          err);
> 
> +       err = pcf85063_set_clkout_mode(pcf85063, client->dev.of_node);
> +       if (err < 0)
> +               dev_warn(&client->dev, "failed to set clock out mode: %d", err);
> +
>         pcf85063->rtc->ops = &pcf85063_rtc_ops;
>         pcf85063->rtc->range_min = RTC_TIMESTAMP_BEGIN_2000;
>         pcf85063->rtc->range_max = RTC_TIMESTAMP_END_2099;
> --
> 2.17.1
> 
> 
> 
> **CONFIDENTIALITY STATEMENT**
> This message is intended for the sole use of the individual(s) and/or entity to whom it is addressed, and may contain information that is legally privileged, confidential, and exempt from disclosure under applicable law. If you are not the intended addressee, nor authorized to receive for the intended addressee, you are hereby notified that dissemination, distribution, copying or disclosure of this message is strictly prohibited. If you have received this message in error please immediately advise the sender by reply email, and delete the message.

-- 
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com