Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH v2 4/4] rtc: ds1307: add freq_test sysfs attribute to
 check tick on m41txx
To:     a.zummo@towertech.it, alexandre.belloni@bootlin.com
Cc:     robh+dt@kernel.org, mark.rutland@arm.com,
        linux-rtc@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20180509182812.17646-1-giulio.benetti@micronovasrl.com>
 <20180509182812.17646-4-giulio.benetti@micronovasrl.com>
From:   Giulio Benetti <giulio.benetti@micronovasrl.com>
Message-ID: <de96728f-c5c6-0d2e-4814-21105c0683ed@micronovasrl.com>
Date:   Wed, 9 May 2018 20:46:09 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
 Thunderbird/52.7.0
MIME-Version: 1.0
In-Reply-To: <20180509182812.17646-4-giulio.benetti@micronovasrl.com>
Content-Type: text/plain; charset=iso-8859-15; format=flowed
Content-Language: it
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Sorry everybody,

Il 09/05/2018 20:28, Giulio Benetti ha scritto:
>   static int ds1307_probe(struct i2c_client *client,
>   			const struct i2c_device_id *id)
>   {
> @@ -1783,6 +1851,24 @@ static int ds1307_probe(struct i2c_client *client,
>   	if (err)
>   		return err;
>   
> +	/* Export sysfs entries */
> +	err = sysfs_create_group(&(&client->dev)->kobj, &rtc_calib_attr_group);
> +	if (err) {
> +		dev_err(&client->dev, "Failed to create sysfs group: %d\n",
> +			err);
> +		return err;
> +	}
> +
> +	err = devm_add_action_or_reset(&client->dev,
> +				       rtc_calib_remove_sysfs_group,
> +				       &client->dev);
> +	if (err) {
> +		dev_err(&client->dev,
> +			"Failed to add sysfs cleanup action: %d\n",
> +			err);
> +		return err;
> +	}
> +
>   	if (chip->nvram_size) {
>   		struct nvmem_config nvmem_cfg = {
>   			.name = "ds1307_nvram",
> 

Adding sysfs must be done only for mt41xx series.
I correct and submit V3 patchest.
Sorry again.

-- 
Giulio Benetti
CTO

MICRONOVA SRL
Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
Tel. 049/8931563 - Fax 049/8931346
Cod.Fiscale - P.IVA 02663420285
Capitale Sociale ? 26.000 i.v.
Iscritta al Reg. Imprese di Padova N. 02663420285
Numero R.E.A. 258642